-
Notifications
You must be signed in to change notification settings - Fork 672
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-conditional-3]: Inconsistent phrasing around placement of @import rules #5697
Comments
That is because it is inside an example. The normative statement is the definition of <stylesheet>. The further explanation says that
There is then an example of something ( |
The normative definition restricting |
@jyasskin does that explanation make sense? If so we can close the issue; but if not, I have Agenda+ the issue for the next CSS telcon (9 Dec). Might you be able to join the call, in that case? It is 12 noon US East Coast. |
I hadn't read "Any I think my second paragraph, about https://drafts.csswg.org/css-conditional-3/#use, is less resolved. I'm happy to dial into the 9 Dec telecon. |
Thanks @jyasskin. @astearns @atanassov would it be possible to have this early on the agenda, so that @jyasskin doesn't need to call in for the whole hour? CSS Conditional 3 is now in the queue for updated CR so I would like to get any edits arising from this issue cleared up before it is published. |
I'm happy to dial in for the whole meeting. I should know more about the CSSWG anyway. :) |
This seems editorial, I'm not sure it warrants telecon time? I agree with both comments in #5697 (comment) and I think we can just fix it. That said, happy for @jyasskin to attend a CSSWG call either way. :) |
OK, rewrote https://drafts.csswg.org/css-conditional-3/#use @jyasskin Lmk if it seems acceptable! Wrt “style sheet” vs |
That looks good, thanks! Do you want to keep this issue open for the |
The CSS Working Group just discussed The full IRC log of that discussion<dael> Topic: [css-conditional-3]: Inconsistent phrasing around placement of @import rules<dael> github: https://github.com//issues/5697 <dael> jeffery: Looks fine <dael> fantasai: Remaining question is sthere is a stylesheet construct that isn't for entire style sheet. Confusing and should change. Should we file as separate issue? Concerns? <dael> astearns: Tracked as a separate issue <dael> fantasai: Should we resolve it should be changed? <dael> astearns: Why don't you raise the separate issue, discuss there, see if anyone disagrees, particularly TabAtkins. <dael> astearns: We'll close this as editorial? |
https://drafts.csswg.org/css-conditional-3/#contents-of talks about
@import
being disallowed inside conditional rules, but doesn't use the same must language as https://drafts.csswg.org/css-conditional-3/#use. Unless there's something I'm missing about CSS editorial style, these should probably be consistent.https://drafts.csswg.org/css-conditional-3/#use is phrased as restricting where
@import
rules can appear, but if that were actually its purpose, it ought to repeat or be unified with the restriction from https://drafts.csswg.org/css-conditional-3/#contents-of that@import
and friends can't appear inside a conditional rule. But I think it's actually specifying that conditional rules must not appear before@import
and friends. If that's your intent, it might be clearer to say it that way.The text was updated successfully, but these errors were encountered: