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] Upgrade to Recommendation? #9508

Open
lukewarlow opened this issue Oct 20, 2023 · 5 comments
Open

[css-scrollbars] Upgrade to Recommendation? #9508

lukewarlow opened this issue Oct 20, 2023 · 5 comments
Labels
a11y-tracker Group bringing to attention of a11y, or tracked by the a11y Group but not needing response. Administrative Tracker For external review / publication tracking issues. css-scrollbars-1 Current Work

Comments

@lukewarlow
Copy link
Member

Is the CSS Scrollbars Level 1 specification ready to published as a recommendation? Currently it's a candidate and has been for a while. The only changes from the current draft compared to the CR is a minor correction.

It has long been supported in Firefox, but will soon also be supported in Chromium (see my I2S), it also has a positive standards position from WebKit scrollbar-width position scrollbar-color position.

@svgeesus
Copy link
Contributor

Most of the 115 tests are passed and there is only 1 open issue. There have been only minor changes since CR.

@tantek @frivoal @atanassov WDYT?

@svgeesus svgeesus added the css-scrollbars-1 Current Work label Oct 23, 2023
@lukewarlow
Copy link
Member Author

lukewarlow commented Oct 23, 2023

There's also this #9169 but it doesn't block shipping imo was more just an observation I had.

Also just fyi the 1 test failing in Chrome on wpt.fyi is fixed in Canary

@svgeesus
Copy link
Contributor

svgeesus commented Jan 5, 2024

WPT results show full interop of 115/115 for Chrome (and Edge) and 115/115 for Firefox. Safari trails somewhat with 74/115

My +1 for requesting Proposed Recommendation

@svgeesus svgeesus added the Administrative Tracker For external review / publication tracking issues. label Jan 23, 2024
@css-meeting-bot
Copy link
Member

The CSS Working Group just discussed [css-scrollbars] Upgrade to Recommendation?.

The full IRC log of that discussion <florian> q+
<Rossen_> ack florian
<Frances> Chris L: css scrollbars have not changed much. 115/115 for chrome, edge, and Firefox, safari a little bit less. Accessibility concerns are scroll-bars do not need AA and scrollbars with people in cognitive disability. Both flagged.
<Frances> Florian: Some tweaks to the bike shed, some notes that we changed to draw a diagram. Should draw a diagram, we have a bunch of tests, but I think they are incomplete. Some tests are not according to the spec. Testing specified and out of scope elements.
<Frances> Florian: If you assume the feature works at all, does it work correctly in the test? Very few tests do it correctly currently.
<PaulG> q+
<Frances> Florian: If make track transparent, scrollbar might have up and down buttons or the spec might use color only and do a gradient if rounded or shaded. Some tests assume wrongly behaviors and implementations.
<fantasai> s/If you assume the feature works at all, does it work correctly in the test?/The tests assume the feature works at all, and if that's true, test other assumptions. But there's no test for whether the feature works at all./
<Frances> Florian: Not an issue with the spec, it is an issue with the site. Authors should not do this. Just because there is white text on a white background, would be bad to test possibly since it is not realistic.
<emilio> q+
<Frances> Paul: When a developer alters a user-agent component take on the onus of passing from most auditors, such as changing colors from 3 to 1, it would fail. If we don't accommodate 3:1 the target size would not work with auto. Transparent is a visual affordance.
<florian> q+
<Rossen_> ack PaulG
<Rossen_> ack emilio
<chris> sorry, promised this was a quick issue or take it back to github.
<Frances> Emilio: It is a good way to provide a scrollbar:hover.
<Frances> Florian: spec already contains a fair amount. If aspects that are missing, would be welcome to adding them.
<Frances> GitHub-bot #5426

@frivoal
Copy link
Collaborator

frivoal commented Jan 25, 2024

As a result of reading the spec and test suite to check if they were indeed ready for REC, I have a found a few issues, and filed a few spec bugs as well as a few test bugs accordingly.

We should try to reduce these to zero before pushing as PR.

Help welcome as well for making PRs that would solve the issues inline in the spec (they're about making diagrams and illustrations).

See also https://drafts.csswg.org/css-scrollbars-1/test-coverage for some information about the overall coverage of this spec.

All in all, I don't think we're that far off, but we're not there yet.


Side note:

Safari trails somewhat with 74/115

This says more about the quality of the test suite than about the status of interoperability, given that as of now, Safari does not implement either of the properties defined in the specification.

@frivoal frivoal removed the Agenda+ label Jan 26, 2024
@matatk matatk added the a11y-tracker Group bringing to attention of a11y, or tracked by the a11y Group but not needing response. label Jan 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y-tracker Group bringing to attention of a11y, or tracked by the a11y Group but not needing response. Administrative Tracker For external review / publication tracking issues. css-scrollbars-1 Current Work
Projects
None yet
Development

No branches or pull requests

6 participants
@frivoal @matatk @svgeesus @css-meeting-bot @lukewarlow and others