Skip to content

Split custom elements into two features #1089

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

Merged
merged 4 commits into from
May 15, 2024

Conversation

foolip
Copy link
Collaborator

@foolip foolip commented May 15, 2024

Fixes #1088.

@github-actions github-actions bot added the feature definition Creating or defining new features or groups of features. label May 15, 2024
Copy link
Collaborator

@ddbeck ddbeck left a comment

Choose a reason for hiding this comment

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

What do you think of a custom-elements group to put these together?

@foolip
Copy link
Collaborator Author

foolip commented May 15, 2024

What do you think of a custom-elements group to put these together?

I did consider that, but I hesitated because maybe we'll want a composite feature later to match the caniuse feature. But let me add the group and deal with that problem if it appears later. Some of the CSS pseudo-classes that aren't part of either feature now should probably go in the same group.

@foolip foolip requested a review from ddbeck May 15, 2024 19:30
Copy link
Collaborator

@ddbeck ddbeck left a comment

Choose a reason for hiding this comment

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

Thanks, Philip!

(For what it's worth, I also tried to find a more concrete example tag name but also found that it narrowed the apparent utility of the feature.)

@ddbeck ddbeck merged commit 552c029 into web-platform-dx:main May 15, 2024
@foolip foolip deleted the custom-elements branch May 16, 2024 11:53
foolip added a commit to foolip/browser-compat-data that referenced this pull request May 16, 2024
foolip added a commit to foolip/wpt that referenced this pull request May 16, 2024
This should match web-platform-dx/web-features#1089.

The test results on wpt.fyi were used as a guide for where there might
be tests involving customized built-in elements, and all tests with
failures were given a quick review before deciding where to map it.
Mistakes are still possible.

This is the most complicated mapping to web-features so far, and shows
some limitations of the WEB_FEATURES.yml format. This could be
simplified with exclusion patterns in the format, but would still not be
simple.

Two files were renamed in the process of preparing this:
web-platform-tests#46305
web-platform-tests#46306
@foolip
Copy link
Collaborator Author

foolip commented May 16, 2024

The WPT side of this is done in web-platform-tests/wpt#46309 and was quite hairy.

ddbeck pushed a commit to mdn/browser-compat-data that referenced this pull request May 16, 2024
foolip added a commit to foolip/wpt that referenced this pull request May 17, 2024
This should match web-platform-dx/web-features#1089.

The test results on wpt.fyi were used as a guide for where there might
be tests involving customized built-in elements, and all tests with
failures were given a quick review before deciding where to map it.
Mistakes are still possible.

This is the most complicated mapping to web-features so far, and shows
some limitations of the WEB_FEATURES.yml format. This could be
simplified with exclusion patterns in the format, but would still not be
simple.

Two files were renamed in the process of preparing this:
web-platform-tests#46305
web-platform-tests#46306
foolip added a commit to web-platform-tests/wpt that referenced this pull request May 21, 2024
This should match web-platform-dx/web-features#1089.

The test results on wpt.fyi were used as a guide for where there might
be tests involving customized built-in elements, and all tests with
failures were given a quick review before deciding where to map it.
Mistakes are still possible.

This is the most complicated mapping to web-features so far, and shows
some limitations of the WEB_FEATURES.yml format. This could be
simplified with exclusion patterns in the format, but would still not be
simple.

Two files were renamed in the process of preparing this:
#46305
#46306
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request May 30, 2024
…tures, a=testonly

Automatic update from web-platform-tests
Map custom elements to two different features (#46309)

This should match web-platform-dx/web-features#1089.

The test results on wpt.fyi were used as a guide for where there might
be tests involving customized built-in elements, and all tests with
failures were given a quick review before deciding where to map it.
Mistakes are still possible.

This is the most complicated mapping to web-features so far, and shows
some limitations of the WEB_FEATURES.yml format. This could be
simplified with exclusion patterns in the format, but would still not be
simple.

Two files were renamed in the process of preparing this:
web-platform-tests/wpt#46305
web-platform-tests/wpt#46306
--

wpt-commits: d437c3d7a1bc22ac2cd0cc902ff5d891ae448e2d
wpt-pr: 46309
jamienicol pushed a commit to jamienicol/gecko that referenced this pull request May 31, 2024
…tures, a=testonly

Automatic update from web-platform-tests
Map custom elements to two different features (#46309)

This should match web-platform-dx/web-features#1089.

The test results on wpt.fyi were used as a guide for where there might
be tests involving customized built-in elements, and all tests with
failures were given a quick review before deciding where to map it.
Mistakes are still possible.

This is the most complicated mapping to web-features so far, and shows
some limitations of the WEB_FEATURES.yml format. This could be
simplified with exclusion patterns in the format, but would still not be
simple.

Two files were renamed in the process of preparing this:
web-platform-tests/wpt#46305
web-platform-tests/wpt#46306
--

wpt-commits: d437c3d7a1bc22ac2cd0cc902ff5d891ae448e2d
wpt-pr: 46309
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature definition Creating or defining new features or groups of features.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Split custom elements
2 participants