Closed
Bug 1224424
Opened 9 years ago
Closed 9 years ago
mask-mode must not have a value called 'auto'
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla45
Tracking | Status | |
---|---|---|
firefox47 | --- | fixed |
People
(Reporter: u459114, Unassigned)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-needed)
Attachments
(1 file)
Report by dbaron
https://lists.w3.org/Archives/Public/public-fx/2015OctDec/0021.html
In short, mask-mode should not have a value called 'auto' since it makes parsing the mask shorthand ambiguous.
For example,
mask: url(xxx) auto
You have no idea that this auto means mask-mode:auto or mask-size:auto
The CSS WG just (a few minutes ago) resolved to rename 'auto' to 'match-source'.
(We also noticed that the definition for SVG images should really say to honor mask-type, not to just do luminance -- but there was discussion about maybe removing mask-type.)
Review commit: https://reviewboard.mozilla.org/r/36117/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/36117/
Change in this patch is very trivial. Waiting for test result.
Attachment #8722595 -
Flags: review?(dbaron)
Comment on attachment 8722595 [details]
MozReview Request: Bug 1224424 - Replace mask-mode:auto keyword by mask-mode:match-source
https://reviewboard.mozilla.org/r/36117/#review33059
Attachment #8722595 -
Flags: review?(dbaron) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/244ab4103b969298cc006d1459de0653344235dd
Bug 1224424 - Replace mask-mode:auto keyword by mask-mode:match-source; r=dbaron
Comment 6•9 years ago
|
||
bugherder |
Updated•9 years ago
|
Keywords: dev-doc-needed
You need to log in
before you can comment on or make changes to this bug.
Description
•