Closed
Bug 1834388
Opened 1 year ago
Closed 1 year ago
Make dynamic-range media features eval to unknown in single context.
Categories
(Core :: CSS Parsing and Computation, task)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
115 Branch
Tracking | Status | |
---|---|---|
firefox115 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
Details
Attachments
(1 file, 1 obsolete file)
No description provided.
Assignee | ||
Comment 1•1 year ago
|
||
As I wrote this I realized that it doesn't match the spec.
Assignee | ||
Comment 2•1 year ago
|
||
https://drafts.csswg.org/mediaqueries-5/#mq-boolean-context is pretty
clear:
If the feature would be true for any value other than the number 0, a
<dimension> with the value 0, the keyword none, or a value explicitly
defined by that media feature to evaluate as false in a boolean
context, the media feature evaluates to true.
Otherwise, it evaluates to false.
These hit the "otherwise". The unknown behavior might make sense, but if
someone wants it the spec needs to change (for all media features
consistently).
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/10ef626b7f82
Fix dynamic-range WPT tests for the boolean context. r=CanadaHonk
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/40146 for changes under testing/web-platform/tests
Comment 5•1 year ago
|
||
Backed out for causing wpt failures on dynamic-range.html
Flags: needinfo?(emilio)
Upstream PR was closed without merging
Assignee | ||
Updated•1 year ago
|
Flags: needinfo?(emilio)
Updated•1 year ago
|
Attachment #9335311 -
Attachment is obsolete: true
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7d1342b9fef6
Fix dynamic-range WPT tests for the boolean context. r=CanadaHonk
Comment 8•1 year ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 1 year ago
status-firefox115:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 115 Branch
Upstream PR merged by moz-wptsync-bot
You need to log in
before you can comment on or make changes to this bug.
Description
•