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] The initial keyword value for scrollbar color properties should probably be auto #2660

Closed
upsuper opened this Issue May 9, 2018 · 2 comments

Comments

Projects
None yet
2 participants
@upsuper
Copy link
Member

commented May 9, 2018

Currently the initial keyword value for scrollbar color properties is normal, while I propose that we use auto instead. There are three reasons:

  • We already have a precedent for using auto with <color>: caret-color, so reusing the same keyword likely allows implementations to have better code reuse.
  • Semantically, auto seems to be giving implementations more leeway on what color should be used. For example, on macOS, scrollbar colors have already been based on the background color on some cases.
  • In addition to the default case, when some colors are not specified, auto also makes more sense to allow implementations deriving them from specified ones, while normal feels like a fixed implementation-dependent value.

tantek added a commit that referenced this issue May 24, 2018

change keyword normal to auto per #2660
Per suggestion and reasoning in issue #2660, change keyword normal to auto.
@tantek

This comment has been minimized.

Copy link
Contributor

commented May 24, 2018

Thanks <https://github.com/upsuper>, this suggestion makes sense per your reasoning. I have updated the spec accordingly. Please take a look <https://drafts.csswg.org/css-scrollbars-1/#scrollbar-color-properties> and if the updates are sufficient to resolve this issue, go ahead and close.

(Originally published at: http://tantek.com/2018/143/t4/)

@upsuper

This comment has been minimized.

Copy link
Member Author

commented May 24, 2018

The change looks good, thanks.

@upsuper upsuper closed this May 24, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.