Closed
Bug 1483808
Opened 6 years ago
Closed 6 years ago
-webkit-device-pixel-ratio should be an alias for resolution
Categories
(Core :: CSS Parsing and Computation, enhancement, P2)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla63
Tracking | Status | |
---|---|---|
firefox63 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
Details
Attachments
(1 file)
According to the spec: https://compat.spec.whatwg.org/#css-media-queries-webkit-device-pixel-ratio And to the Chromium implementation: https://cs.chromium.org/chromium/src/third_party/blink/renderer/core/css/media_query_evaluator.cc?l=366&rcl=1d7328865bcf06a687aafc18ff95d55317030672 They're no different than resolution. In our implementation `resolution` does slightly different stuff. I think we should make them match.
Assignee | ||
Comment 1•6 years ago
|
||
According to the spec: https://compat.spec.whatwg.org/#css-media-queries-webkit-device-pixel-ratio And to the Chromium implementation: https://cs.chromium.org/chromium/src/third_party/blink/renderer/core/css/media_query_evaluator.cc?l=366&rcl=1d7328865bcf06a687aafc18ff95d55317030672 They're no different than resolution. In our implementation `resolution` does slightly different stuff. Given we still haven't shipped -webkit-device-pixel-ratio, making this match resolution looks better than the opposite.
Comment 2•6 years ago
|
||
Comment on attachment 9001876 [details]
Make webkit device-pixel-ratio media queries a proper alias to resolution.
Daniel Holbert [:dholbert] has approved the revision.
Attachment #9001876 -
Flags: review+
Pushed by emilio@crisal.io: https://hg.mozilla.org/integration/autoland/rev/b107fee86cba Make webkit device-pixel-ratio media queries a proper alias to resolution. r=dholbert
Comment 4•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b107fee86cba
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Pushed by emilio@crisal.io: https://hg.mozilla.org/integration/autoland/rev/0c82bae4916e No bug - Remove comment that is outdated since I fixed it in bug 1483808.
Comment 6•5 years ago
|
||
bugherder |
You need to log in
before you can comment on or make changes to this bug.
Description
•