Closed Bug 1224424 Opened 5 years ago Closed 4 years ago

mask-mode must not have a value called 'auto'

Categories

(Core :: Layout, defect)

defect
Not set
normal

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.)
Blocks: 1228354
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+
Blocks: mask-ship
https://hg.mozilla.org/integration/mozilla-inbound/rev/244ab4103b969298cc006d1459de0653344235dd
Bug 1224424 - Replace mask-mode:auto keyword by mask-mode:match-source; r=dbaron
https://hg.mozilla.org/mozilla-central/rev/244ab4103b96
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.