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

Generate lots more .dist.yml files #801

Merged
merged 4 commits into from
Apr 9, 2024
Merged

Conversation

foolip
Copy link
Collaborator

@foolip foolip commented Apr 6, 2024

No description provided.

Copy link
Collaborator Author

@foolip foolip left a 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
Copy link
Collaborator Author

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
Copy link
Collaborator Author

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
Copy link
Collaborator Author

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
Copy link
Collaborator Author

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
Copy link
Collaborator Author

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
Copy link
Collaborator Author

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: 2023-05-09
support:
chrome: "103"
Copy link
Collaborator Author

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
Copy link
Collaborator Author

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"
Copy link
Collaborator Author

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"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

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.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

@foolip foolip left a 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
Copy link
Collaborator Author

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
Copy link
Collaborator Author

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
Copy link
Collaborator Author

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
Copy link
Collaborator Author

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
Copy link
Collaborator Author

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"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

status:
baseline: false
support:
chrome: "117"
Copy link
Collaborator Author

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
Copy link
Collaborator Author

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
Copy link
Collaborator Author

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"
Copy link
Collaborator Author

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.

@foolip foolip marked this pull request as ready for review April 6, 2024 16:25
@foolip foolip requested a review from ddbeck April 6, 2024 16:25
@foolip
Copy link
Collaborator Author

foolip commented Apr 6, 2024

@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"
Copy link
Collaborator Author

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"
Copy link
Collaborator Author

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"
Copy link
Collaborator Author

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.

foolip added a commit to foolip/web-features that referenced this pull request Apr 6, 2024
These are cases that were excluded from
web-platform-dx#801 due to some
complication, small or large.
foolip added a commit to foolip/web-features that referenced this pull request Apr 9, 2024
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
Copy link
Collaborator

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?

Copy link
Collaborator Author

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.

Copy link
Collaborator

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.

@ddbeck ddbeck merged commit 38c2128 into web-platform-dx:main Apr 9, 2024
2 checks passed
foolip added a commit to foolip/web-features that referenced this pull request Apr 13, 2024
These are cases that were excluded from
web-platform-dx#801 due to some
complication, small or large.
foolip added a commit to foolip/web-features that referenced this pull request Apr 13, 2024
These are cases that were excluded from
web-platform-dx#801 due to some
complication, small or large.
@foolip foolip deleted the more-dist branch April 13, 2024 22:55
foolip added a commit to foolip/web-features that referenced this pull request Apr 23, 2024
These are cases that were excluded from
web-platform-dx#801 due to some
complication, small or large.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants