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

[css-nesting] Define how invalid nested rules should be handled #2880

Closed
upsuper opened this issue Jul 5, 2018 · 3 comments
Closed

[css-nesting] Define how invalid nested rules should be handled #2880

upsuper opened this issue Jul 5, 2018 · 3 comments
Labels
Closed Accepted by Editor Discretion Commenter Satisfied Commenter has indicated satisfaction with the resolution / edits. css-nesting-1 Current Work

Comments

@upsuper
Copy link
Member

upsuper commented Jul 5, 2018

There are several cases in the spec that a nested rule becomes invalid, but it doesn't mention how an invalid nested rule should be handled.

Should the specific sub-rule be dropped, or should its parent rule be treated invalid as well?

My intuition is that probably only the invalid sub-rule should be dropped, but it's not clear to me whether the other way around would be better.

@upsuper upsuper added the css-nesting-1 Current Work label Jul 5, 2018
@tabatkins
Copy link
Member

Only the invalid sub-rule should be dropped.

@jonathantneal
Copy link
Contributor

Should this section be updated to include invalid rules or invalid nesting rules (emphasis added):

Unexpected at-rules (which could be all of them, in a given context) are invalid and should be ignored by the consumer.
https://drafts.csswg.org/css-syntax/#parse-list-of-declarations

@tabatkins
Copy link
Member

tabatkins commented Jan 10, 2023

Went ahead and put in some explicit text just to make it 100% clear. (Assuming commenter satisfied, as question was answered.)

@tabatkins tabatkins added Commenter Satisfied Commenter has indicated satisfaction with the resolution / edits. Closed Accepted by Editor Discretion labels Jan 10, 2023
jakearchibald pushed a commit to jakearchibald/csswg-drafts that referenced this issue Jan 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Closed Accepted by Editor Discretion Commenter Satisfied Commenter has indicated satisfaction with the resolution / edits. css-nesting-1 Current Work
Projects
None yet
Development

No branches or pull requests

3 participants