-
Notifications
You must be signed in to change notification settings - Fork 22.6k
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
enhance(sidebars/cssref): group CSS pseudo-classes #38019
Conversation
That's pretty neat. Question: is it possible to give a title to these or insert something before to indicate what the proceeding details element contains. Specifically - Vendor-prefixed
- :-a
- :-b or - Non-standard
- :-a
- :-b What do you think? |
This looks like a bug in Rari, because |
Yes, it's interesting that rari isn't greedy and make I would suggest to merge this as is, because it's already a sidebar UX improvement as is, and file a rari bug to enhance the grouping. |
I'm pretty sure it's not possible with rari today, because complex includes and excludes in sidebar sections aren't supported, and my gut feeling is it's not worth the extra complexity at this point in time. If we enhance the list grouping to properly group |
Filed issue: mdn/rari#127 |
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.
LGTM. Thanks! 🎉
Description
Groups the CSS Pseudo-classes in the sidebar, similar to the CSS Properties.
Motivation
Ensures especially that all the non-standard pseudo-classes are grouped together.
Additional details
Related issues and pull requests
Fixes #38018.