-
Notifications
You must be signed in to change notification settings - Fork 22.4k
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
fix(css): update at-rule page #34709
Conversation
Preview URLs (9 pages)
Flaws (1)Note! 8 documents with no flaws that don't need to be listed. 🎉 URL:
(comment last updated: 2024-07-11 04:49:57) |
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.
Let's talk about it at the meeting.
Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
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 new DL looks so much better than the original. Thank you!
let's get rid of the "use case" area that divides them by use case.
In a future PR, we should add all the missing at-rules and see if we can make the left sidebar have the at-rule section expanded. But that is not this PR. For this PR, we can get rid of the 3rd list to improve the page.
files/en-us/web/css/at-rule/index.md
Outdated
- {{cssxref("@supports")}} | ||
- : A conditional group rule that will apply its content if the browser meets the criteria of the given condition. | ||
|
||
## At-rule uses |
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.
Let's eliminate this entire section. The page has two other lists of at-rules. This third list may be more confusing than helpful for some.
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.
Done.
There is no official classification of at-rules
in the specs, so we don't have to come up with one forcefully.
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.
Thanks! 🎉
* fix(css): update at-rule page * Apply suggestions from code review Co-authored-by: Estelle Weyl <estelle@openwebdocs.org> * address remaining review comments * remove usage section --------- Co-authored-by: Estelle Weyl <estelle@openwebdocs.org>
The PR restructures the page as per the specification at-rule section .
The new classification is based on at-rule usage.