Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Linux custom css scrollbar #4884

Closed
Zolmeister opened this issue Aug 22, 2013 · 14 comments
Closed

Linux custom css scrollbar #4884

Zolmeister opened this issue Aug 22, 2013 · 14 comments
Assignees

Comments

@Zolmeister
Copy link

The default Linux scrollbar for content looks poor due to sandbox rendering.

Here is a proposed css default scrollbar replacement (http://jsfiddle.net/6kEx5/):

::-webkit-scrollbar {
    width: 12px;
    height: 12px;
}

::-webkit-scrollbar-track-piece {
    background: #aaa;
}

::-webkit-scrollbar-thumb {
    background: #7a7a7a;
    border-radius: 2px;
}

::-webkit-scrollbar-corner       {
    background: #999;
}

::-webkit-scrollbar-thumb:window-inactive {
    background: #888;
}
@ghost ghost assigned larz0 Aug 23, 2013
@redmunds
Copy link
Contributor

Thanks for the tip! Assigning to @larz0 to take a look.

@ghost
Copy link

ghost commented Sep 5, 2013

Thanks

@larz0
Copy link
Member

larz0 commented Sep 5, 2013

@Zolmeister @vinigordex could you check out the custom webkit scrollbar on this branch and let me know what you think? https://github.com/adobe/brackets/tree/larz/linux-scrollbar

Once we're happy with it we'll need to make the scrollbar CSS linux-only.

@larz0
Copy link
Member

larz0 commented Sep 5, 2013

Here's a screenshot:

screen shot 2013-09-05 at 11 35 29 am

@TomMalbran
Copy link
Contributor

Once we're happy with it we'll need to make the scrollbar CSS linux-only.

You can just replace the body wrapper with .platform-linux.

@larz0
Copy link
Member

larz0 commented Sep 5, 2013

Awesome. Thanks Tom I'll go ahead and do that.

@TomMalbran
Copy link
Contributor

NP. You can also change it to .platform-win or .platform-mac to test in other platforms.

@Zolmeister
Copy link
Author

I put it in a js-fiddle: http://jsfiddle.net/HrsxL/ for others to check out more easily. It looks fine for me, though I can't say I like the use of the base64 encoded png.

Edit: updated fiddle - http://jsfiddle.net/HrsxL/1/

@larz0
Copy link
Member

larz0 commented Sep 5, 2013

Cool. I'm not using base64 encoded png.

@larz0
Copy link
Member

larz0 commented Sep 5, 2013

@Zolmeister @vinigordex the pull request is here #5082

Let me know if there's any problem.

@Zolmeister
Copy link
Author

Oh, sorry, must have been looking at some other code then.
Here is the updated fiddle: http://jsfiddle.net/HrsxL/1/
And I must say, I do like it more now. Thanks for taking the time to implement it.

@larz0
Copy link
Member

larz0 commented Sep 5, 2013

Great. NP man.

@ghost
Copy link

ghost commented Sep 6, 2013

My screen Ubuntu

brackets

@jasonsanjose
Copy link
Member

Confirmed fixed. Closing. @Zolmeister if you have a dev env for brackets setup, you can sync to master to see the changes.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants