-
Notifications
You must be signed in to change notification settings - Fork 108
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
Generate lots more .dist.yml files #801
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.
Self review A-C. I will back out anything that needs more careful review.
description: The `abs()` and `sign()` CSS functions compute the absolute value or the sign of the input. | ||
spec: https://drafts.csswg.org/css-values-4/#sign-funcs | ||
status: | ||
baseline: false |
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.
This is correct, it's in progress in Chrome: https://issues.chromium.org/issues/40253181
spec: https://drafts.csswg.org/css-ui-4/#widget-accent | ||
usage_stats: https://chromestatus.com/metrics/css/timeline/popularity/695 | ||
status: | ||
baseline: low |
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.
spec: https://drafts.csswg.org/css-animations-2/#animation-composition | ||
usage_stats: https://chromestatus.com/metrics/css/timeline/popularity/743 | ||
status: | ||
baseline: low |
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.
spec: https://drafts.csswg.org/css-sizing-4/#aspect-ratio | ||
usage_stats: https://chromestatus.com/metrics/css/timeline/popularity/657 | ||
status: | ||
baseline: high |
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.
Yes: https://developer.mozilla.org/en-US/docs/Web/CSS/aspect-ratio#browser_compatibility
The aspect_ratio_computed_from_attributes
bits were supported slightly earlier, but I don't think that's worth treating as a separate feature now.
description: "The `:autofill` pseudo-class matches `<input>` elements that have been filled in automatically by the browser." | ||
spec: https://html.spec.whatwg.org/multipage/semantics-other.html#selector-autofill | ||
status: | ||
baseline: low |
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.
Yes: https://developer.mozilla.org/en-US/docs/Web/CSS/:autofill#browser_compatibility
This was supported with a prefix earlier, but that doesn't count as Baseline.
spec: https://www.iso.org/standard/87621.html | ||
caniuse: colr-v1 | ||
status: | ||
baseline: false |
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.
https://caniuse.com/colr-v1 agrees.
baseline: low | ||
baseline_low_date: 2023-05-09 | ||
support: | ||
chrome: "103" |
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.
Large parts were supported much earlier, need to review if we should split or override:
https://developer.mozilla.org/en-US/docs/Web/API/CompressionStream#browser_compatibility
- https://drafts.csswg.org/cssom-1/#dom-documentorshadowroot-adoptedstylesheets | ||
usage_stats: https://chromestatus.com/metrics/feature/timeline/popularity/2846 | ||
status: | ||
baseline: false |
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.
This is probably caused by the note in https://developer.mozilla.org/en-US/docs/Web/API/CSSStyleSheet/CSSStyleSheet#browser_compatibility
Need to review and do an editorial override.
baseline: low | ||
baseline_low_date: 2023-09-18 | ||
support: | ||
chrome: "117" |
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 important parts were supported much earlier. Need to review:
https://developer.mozilla.org/en-US/docs/Web/CSS/contain-intrinsic-size#browser_compatibility
status: | ||
baseline: false | ||
support: | ||
chrome: "111" |
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.
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.
https://caniuse.com/css-container-queries-style agrees too, only difference is that it calls out that it's limited to custom properties. I suspect it will remain like that ~indefinitely and that'll just be what the feature is.
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.
I put a suggestion relating to this here: https://github.com/web-platform-dx/web-features/pull/751/files#r1557476030
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.
Self-review of the rest. I'll back out what needs a closer look to leave only the verified bits.
caniuse: declarative-shadow-dom | ||
usage_stats: https://chromestatus.com/metrics/feature/timeline/popularity/4421 | ||
status: | ||
baseline: low |
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.
name: Mutually exclusive <details> elements | ||
spec: https://html.spec.whatwg.org/multipage/interactive-elements.html#attr-details-name | ||
status: | ||
baseline: false |
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.
spec: https://html.spec.whatwg.org/multipage/interactive-elements.html#the-details-element | ||
status: | ||
baseline: high | ||
baseline_low_date: 2020-01-15 |
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 toggle event is a newer addition and I think we should do an override to ignore it:
https://developer.mozilla.org/en-US/docs/Web/API/HTMLDetailsElement#browser_compatibility
spec: https://drafts.csswg.org/css-values-4/#exponent-funcs | ||
status: | ||
baseline: low | ||
baseline_low_date: 2023-12-07 |
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.
spec: https://drafts.csswg.org/css-fonts-4/#font-optical-sizing-def | ||
usage_stats: https://chromestatus.com/metrics/css/timeline/popularity/647 | ||
status: | ||
baseline: high |
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.
baseline: low | ||
baseline_low_date: 2022-03-14 | ||
support: | ||
chrome: "83" |
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.
https://caniuse.com/transforms3d disagrees.
status: | ||
baseline: false | ||
support: | ||
chrome: "117" |
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.
spec: https://drafts.csswg.org/css-values-4/#trig-funcs | ||
status: | ||
baseline: low | ||
baseline_low_date: 2023-03-13 |
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.
spec: https://drafts.csswg.org/selectors-4/#user-pseudos | ||
status: | ||
baseline: low | ||
baseline_low_date: 2023-11-02 |
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.
baseline_low_date: 2021-08-10 | ||
baseline_high_date: 2024-02-10 | ||
support: | ||
chrome: "62" |
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.
This is 62 instead of 61 due to the events in https://developer.mozilla.org/en-US/docs/Web/API/VisualViewport#browser_compatibility, but that was just missing onresize
and onscroll
properties. I think this should be an editorial override.
@ddbeck this should now be quite easy to review. For everything that remains, I've linked to the entry point to the API and confirmed the versions match, and when there's a caniuse entry I've checked that. I'll submit separate PRs for the hard cases. |
support: | ||
chrome: "76" | ||
chrome_android: "76" | ||
edge: "17" |
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.
Oops, https://caniuse.com/css-backdrop-filter disagrees about this. Will back out to look at separately.
baseline: low | ||
baseline_low_date: 2024-02-20 | ||
support: | ||
chrome: "111" |
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.
https://caniuse.com/declarative-shadow-dom disagrees. Will back out.
baseline_low_date: 2015-07-29 | ||
baseline_high_date: 2018-01-29 | ||
support: | ||
chrome: "5" |
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.
https://caniuse.com/input-search has slightly different data. I'll back this out.
These are cases that were excluded from web-platform-dx#801 due to some complication, small or large.
These are cases that were excluded from web-platform-dx#801 due to some complication, small or large.
chrome_android: "121" | ||
edge: "121" | ||
compat_features: | ||
- css.properties.font-palette.palette-mix_function |
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.
Hold up, are we using the right key? Should this be css.properties.font-palette.animation_computed
?
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 palette-mix()
function can in principle be used without animations, but the only reason it was implemented was for animations. So I think it makes sense in this entry. But I'll also add css.properties.font-palette.animation_computed
in a separate PR, to be landed after this.
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.
OK, great. Thank you—approving and merging now.
These are cases that were excluded from web-platform-dx#801 due to some complication, small or large.
These are cases that were excluded from web-platform-dx#801 due to some complication, small or large.
These are cases that were excluded from web-platform-dx#801 due to some complication, small or large.
No description provided.