Skip to content

chore(css): Update group field for css selectors - #965

Merged
pepelsbey merged 6 commits into
mdn:mainfrom
skyclouds2001:css-selector-group
Sep 30, 2025
Merged

chore(css): Update group field for css selectors#965
pepelsbey merged 6 commits into
mdn:mainfrom
skyclouds2001:css-selector-group

Conversation

@skyclouds2001

@skyclouds2001 skyclouds2001 commented Apr 3, 2025

Copy link
Copy Markdown
Contributor

Description

to make the group field really represents which module the feature is defined

Motivation

Additional details

Related issues and pull requests

@skyclouds2001
skyclouds2001 marked this pull request as ready for review April 3, 2025 18:27
@skyclouds2001
skyclouds2001 requested review from a team and bsmth and removed request for a team April 3, 2025 18:27
Comment thread css/selectors.json
@@ -146,25 +130,20 @@
":active-view-transition": {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@bsmth

bsmth commented Apr 7, 2025

Copy link
Copy Markdown
Member

Thanks a lot. In general, this is technically correct, so I would be curious to get another opinion again on whether:

make the group field really represents which module the feature is defined

is what consumers expect as opposed to these returned when someone is looking for a list of pseudo-classes or pseudo-elements. "Defined in" versus "can be classed as" is a good distinction to double-check.

@github-actions github-actions Bot added the idle Issues and pull requests with no activity for three months. label May 9, 2025
@bsmth
bsmth self-requested a review August 25, 2025 09:03
@github-actions github-actions Bot removed the idle Issues and pull requests with no activity for three months. label Aug 29, 2025

@bsmth bsmth left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think this is good. @pepelsbey do you agree?

@bsmth
bsmth requested a review from pepelsbey September 25, 2025 13:40

@pepelsbey pepelsbey left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yep, makes sense to me! Thank you :)

@pepelsbey
pepelsbey merged commit 632fd10 into mdn:main Sep 30, 2025
1 check passed
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.

4 participants