-
Notifications
You must be signed in to change notification settings - Fork 719
[mediaqueries-4][css-cascade-5] Restrict 'layer' from the production?
#7225
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
Comments
Agenda+ bc it should be a tiny discussion, and we'll need a resolution if we go with my suggestion to fix MQ4. |
The CSS Working Group just discussed
The full IRC log of that discussion |
frivoal
added a commit
to web-platform-tests/wpt
that referenced
this issue
May 5, 2022
moz-v2v-gh
pushed a commit
to mozilla/gecko-dev
that referenced
this issue
May 16, 2022
…edia type, a=testonly Automatic update from web-platform-tests tests for error handling of layer as a media type Based on w3c/csswg-drafts#7225 -- wpt-commits: bed0261ca3f7826bf56b149be069a38288102082 wpt-pr: 33940
jamienicol
pushed a commit
to jamienicol/gecko
that referenced
this issue
May 25, 2022
…edia type, a=testonly Automatic update from web-platform-tests tests for error handling of layer as a media type Based on w3c/csswg-drafts#7225 -- wpt-commits: bed0261ca3f7826bf56b149be069a38288102082 wpt-pr: 33940
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
@nex3 brought up that the grammar for @import is technically ambiguous at the moment:
@import url(foo) layer
could parse as thelayer
keyword to trigger a Cascade Layer, or as an unknown
, since
is just
, with a few restricted keywords.We could fix this either at the
@import
location, specifying thatlayer
cannot match
there, or directly in MQ, restrictinglayer
from ever being a
.The only effect of restricting it at the
level, fwiw, is thatnot layer
will change from matching (because it's negating an unknown media type) to not matching (because it's an invalid keyword, so the whole MQ turns intonot all
). Any other case will be equivalent in matching behavior, but will serialize differently (becomingnot all
rather than preserving the original MQ). I think it's, uh, exceedingly unlikely anyone has ever writtenlayer
for a media type in the first place, tho, so we should be good.The text was updated successfully, but these errors were encountered: