Does this intent deprecate or change behavior of existing APIs, such that it has potentially high risk for Android WebView-based applications?
No information providedNo milestones specified
Open questions about a feature may be a source of future web compat or interop issues. Please list open issues (e.g. links to known github issues in the project for the feature specification) whose resolution may introduce web compat/interop risk (e.g., changing to naming or structure of the API in a non-backward-compatible way).
No information providedIt’s stated here that “links” mode will be the default, but when I experiment with the example at https://developer.mozilla.org/en-US/docs/Web/CSS/Reference/Properties/scroll-marker-group it looks like “tab” mode is the scroll-marker-group behavior that’s shipped in Chromium today. Changing this will cause a difference in behavior for sites that already use scroll-marker-group without specifying the mode, e.g. sites will see the single tab stop on a scroll-marker-group become multiple tab stops. Is this compatibility risk something you’ve considered?
> Is this feature fully tested by web-platform-tests?
This looks like test coverage for the broader scroll-marker-group feature, but is there coverage for the new functionality being proposed here? The only of these tests with “mode”, “link”, or “tab” in the name are already passing in Chrome Stable.
> Web developers: No signals
It would be better if we could point to some evidence of developer interest. Who is asking for this feature?
> TAG review status
Not applicable
Can you say why TAG review isn’t applicable?
> Gecko: No signal (https://github.com/mozilla/standards-positions/issues/1161)
> WebKit: No signal (https://github.com/WebKit/standards-positions/issues/447)
I think it’s reasonable to reuse the Gecko and WebKit signals for carousel, but it’d be nice to post an update to those threads mentioning that you intend to ship this additional behavior.
Lastly, please request the other reviews in Chromestatus (Privacy, WP security, etc).
Thanks,
Dan
--
You received this message because you are subscribed to the Google Groups "blink-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to
blink-dev+...@chromium.org.
To view this discussion visit
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/6a57e20c.854c7482.198413.02a4.GAE%40google.com.
To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+unsubscribe@chromium.org.
It'd be good to get WPT coverage at least for the focus order changes. Several such tests already exist (for example scroll-marker-next-focus.html), but they still expect the default behavior to be "tabs". This might explain why several of the WPTs are failing in Chrome Experimental but passing in Stable: https://wpt.fyi/results/css/css-overflow/scroll-markers?label=master&product=chrome%5Bexperimental%… Seems to me like the existing set of tests should be updated to handle the new default, and there should be tests that validate focus order for both "links" and "tabs" mode.
The Carousels TAG review is still in progress, and the latest response from the TAG actually has some comments relevant to this change: https://github.com/w3ctag/design-reviews/issues/1037#issuecomment-3189080896, particularly the note on developers choosing between "tabs" and "links" semantics. Will there be documentation that helps developers understand that choice?
To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+...@chromium.org.
I want to dig into the compat question a bit more too. The 0.09% usage from https://chromestatus.com/metrics/css/timeline/popularity/793 is above the threshold we'd typically accept for outright breakage. To what degree do you expect sites assuming the old default to be impacted by this change? It seems like the scroll markers will become tab stops, and will have a different accessibility role. In your experience with sites using the feature, would this tend to be a breaking change or more of a small tweak to how users will interact with the component?
Does that same answer apply to users of assistive tech who could be impacted by the default role change?
> 3) These mods are direct response to CSS Carousels criticism from both devs and different WGs.Are there links to any of these conversations you could reference?
The compat risk goes away if we were to instead make tabs mode the default. I'd like to better understand the motivation for going with "links" as the default given the risk it entails.
To view this discussion visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/26f0c678-14d6-438e-9068-2fa148940188n%40chromium.org.