Skip to content
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

[css-scrollbars-1] Loosen requirement for scrollbar-width:thin #2992

Merged
merged 1 commit into from
Sep 15, 2018

Conversation

upsuper
Copy link
Member

@upsuper upsuper commented Aug 6, 2018

I think the main point of having thin scrollbars is to give content more room when the space is already tight, so it is probably better using the concept of "scrollbar gutter" for this value, which is also clearer when overlay scrollbar / auto-expand scrollbar involves.

Also when the scrollbar is already fully overlay (e.g. on macOS when trackpad is used, or on Android), I believe there is no point to create an even thinner scrollbar, so the value probably shouldn't use a "must".

@upsuper upsuper added the css-scrollbars-1 Current Work label Aug 6, 2018
@upsuper upsuper requested a review from tantek August 6, 2018 01:15
Copy link
Member

@tantek tantek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good except for the "In that case, this value should have no effect." I disagree. Even in that case the author intent is to visually provide something thinner and less noisy even when overlaid. The presence/absence of a gutter also doesn't clarify whether it is clickable or not. E.g. some overlay scrollbars are clickable, and thus making them thin absolutely should have the effect of narrowing the clickable area (typically because you don't want it obscure/prevent clicking the thing being scrolled!). This is especially important in narrow fields and lists.

@upsuper
Copy link
Member Author

upsuper commented Aug 30, 2018

Updated, and remove the gutter part. It now just mentions thinner scrollbar.

Copy link
Member

@tantek tantek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes this is an improvement. Approving this and we can iterate further with minor tweaks as needed, e.g. "implementations may treat"

@tantek tantek merged commit 71966ce into w3c:master Sep 15, 2018
@upsuper upsuper deleted the scrollbar-thin branch September 15, 2018 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
css-scrollbars-1 Current Work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants