Closed Bug 1251106 Opened 9 years ago Closed 8 years ago

'mask' shorthand should require 'mask-mode' (if present) to be right after 'mask-image'

Categories

(Core :: CSS Parsing and Computation, defect)

Product:
Core
Component:
CSS Parsing and Computation
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED INVALID
Tracking Flags:
Tracking Status
firefox47 --- affected

People

(Reporter: dbaron, Unassigned)

References

(Blocks 1 open bug,
URL
)

Details

I just noticed that the spec defines the 'mask' shorthand as having the mask-mode part required (if present) to be immediately after the mask-image part. This isn't what CSSParserImpl::ParseImageLayersItem does, although I'm not sure if it's correct. I asked: https://lists.w3.org/Archives/Public/public-fx/2016JanMar/0058.html and depending on responses we should probably fix this before shipping.
Given no response to that post since February, I copied it to a github issue: https://github.com/w3c/fxtf-drafts/issues/13
See Also: → https://github.com/w3c/fxtf-drafts/issues/13
I just commented in the issue that I think the spec should change to allow the mask-mode part anywhere. At this point I don't think it's worth it holding up shipping.
The issue was resolved (and spec updated) to allow the mask-mode value anywhere.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.