-
Notifications
You must be signed in to change notification settings - Fork 2.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dark theme scrollbar styles #956
Comments
you know how i feel about custom scrollbars 😒 but i hear the IE argument, and it's true that IE's custom scrollbar support is pretty great. more likely this is a simple CSS class |
I'd really like to avoid putting this IE-specific mess into core. Can we just provide folks with a snippet that makes IE scrollbars look nice? It's then up to them to integrate it however they see fit in their app |
It's not IE specific. It affects all Windows users, and a lot of those are on Chrome. My first screenshot was on Chrome |
Windows-specific, sorry. My comment still applies |
I don't really see a conflict in providing opt-in CSS classes to enable custom scrollbar styles on IE or WebKit browsers only (the only browsers that support custom scrollbars). |
I mean, sure, we could throw it in. In my view, this isn't a strong case to start providing OS/browser-specific things |
It's been almost a year, is there any progress on this? |
Not yet. If you are interested in contributing, feel free to write a proposal and send a PR. |
Have you guys tried this? https://github.com/KingSora/OverlayScrollbars |
I think we can make meaningful progress here by using the |
We've had a few discussions around custom scroll bar styling provided out of the box with Blueprint. I think this is a good idea. A number of our users are running Windows, which can often produce jarring visuals:
Chrome:
IE11:
So apps often end up rolling their own solution to make this situation look less ugly. It would be great if we could standardize a styling solution to address this problem.
Challenges
Proposals
from @crazytoucan:
The text was updated successfully, but these errors were encountered: