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

[mediaqueries-4][mediaqueries-5] Is <mf-range> syntax correct? #2790

Closed
emilio opened this issue Jun 16, 2018 · 3 comments
Closed

[mediaqueries-4][mediaqueries-5] Is <mf-range> syntax correct? #2790

emilio opened this issue Jun 16, 2018 · 3 comments
Assignees
Labels

Comments

@emilio
Copy link
Collaborator

emilio commented Jun 16, 2018

https://drafts.csswg.org/mediaqueries-4/#mq-syntax says:

<mf-range> = <mf-name> [ '<' | '>' ]? '='? <mf-value>
           | <mf-value> [ '<' | '>' ]? '='? <mf-name>

That means that width 500px is a valid production, for example, without any operator, neither = or > / <.

Is this intentional? If so, is it worth a test / clarification?

@emilio
Copy link
Collaborator Author

emilio commented Jun 16, 2018

cc @zcorpan

@emilio
Copy link
Collaborator Author

emilio commented Jun 16, 2018

Err, sorry @zcorpan, meant to mention @frivoal :)

@tabatkins
Copy link
Member

Whoops, no, that's not intended. Should be

<mf-range> = <mf-name> [ [ '<' | '>' ]? '='? ]! <mf-value>
           | <mf-value> [ [ '<' | '>' ]? '='? ]! <mf-name>

Or perhaps just replace that middle bit with <mq-comparison>, defined as:

<mq-comparison> = [ '<' | '> ] '='? | '='

@frivoal frivoal self-assigned this Oct 3, 2018
frivoal added a commit to frivoal/wpt that referenced this issue Jul 3, 2020
frivoal added a commit to web-platform-tests/wpt that referenced this issue Jul 3, 2020
@frivoal frivoal added Tested Memory aid - issue has WPT tests and removed Needs Testcase (WPT) labels Jul 3, 2020
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this issue Jul 8, 2020
…ax is rejected, a=testonly

Automatic update from web-platform-tests
[mediaqueries] Confirm that invalid syntax is rejected (#24430)

Tests w3c/csswg-drafts#2790
--

wpt-commits: b1d24b1a98699bf65855c6ee5d6140b5a24a7bf6
wpt-pr: 24430
xeonchen pushed a commit to xeonchen/gecko that referenced this issue Jul 9, 2020
…ax is rejected, a=testonly

Automatic update from web-platform-tests
[mediaqueries] Confirm that invalid syntax is rejected (#24430)

Tests w3c/csswg-drafts#2790
--

wpt-commits: b1d24b1a98699bf65855c6ee5d6140b5a24a7bf6
wpt-pr: 24430
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants