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

Status: RESOLVED INVALID

(Core :: CSS Parsing and Computation, defect)

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

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.

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Given no response to that post since February, I copied it to a github issue:
https://github.com/w3c/fxtf-drafts/issues/13

Comment 2

[Cameron McCormack (:heycam)]

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.

Comment 3

[Cameron McCormack (:heycam)]

The issue was resolved (and spec updated) to allow the mask-mode value anywhere.