-
Notifications
You must be signed in to change notification settings - Fork 296
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
ENH: Make scroll bar style more accessible and noticeable #776
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good improvement 👍
Why not using the default sizes from the browser? It seems to be roughly the width of the pointer which seems good for grabbing.
Also I would dim the main one as well. I find it a bit distracting otherwise.
I made the scrollbars a little bit wider to be more noticeable / clickable. I'm hesitant to use the default widths because they are much wider and (IMO) more distracting. I'd prefer to keep this PR towards an iterative improvement rather than a reversion back to default behavior if possible. I tried to get the right scrollbar to have the same hover behavior, but I couldn't figure it out because it's attached to |
This looks much easier on first sight! I sometimes use epiphany-browser, what it does by default (maybe specific to epiphany), is to have a fairly narrow (maybe a bit wider then the current). But when you come close enough to grab, it expands (rather than hides) the scroll-bar to normal siz (with a bit of alpha, not that it matters). But I think this is good. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the "on-surface" color was IMO not super nice but that was a legacy color from the sidebars
directive. I'm happy to see it moving towards a more grey color.
I really like the color effect that make it much easier to grab
OK thanks for the feedback and reviews all! We have two 👍 so I'll merge this in and we can iterate if folks have more suggestions! |
This makes a few improvements to the scroll bar, to address some of the concerns in #775 . Here's a summary:
on-surface
color on light theme had a slight yellow tint to it, rather than just following the shading gradient from light to dark (which is what we do on the dark theme). So this changes that color's default value...happy to discuss that one if there's a rationale for wanting to use yellowPreview:
closes #775