data icon indicating copy to clipboard operation
data copied to clipboard

[WIP] fix(css): update `mask-repeat` and `mask-position` according to spec

Open sigveio opened this issue 4 years ago • 1 comments

This pull request aims to update the initial value of mask-repeat and mask-position to be in line with the current spec.

Specs: https://drafts.fxtf.org/css-masking/#the-mask-repeat https://drafts.fxtf.org/css-masking/#the-mask-position

Additional context: w3c/csswg-drafts/issues/599 w3c/fxtf-drafts/issues/111

sigveio avatar Jul 25 '21 18:07 sigveio

Going over the spec for this again, I noticed that some of the other JSON properties for these two could also use updating. So I'm marking it as a WIP for now.

This is the definition of appliesto in the latest spec for these elements:

All elements. In SVG, it applies to container elements excluding the defs element, all graphics elements and the use element

Which deviates from the allElementsSVGContainerElements used in the JSON currently.

What would be a suitable replacement?

Attempting to break it down:

All elements. -> allElements In SVG, it applies to container elements -> SVGContainerElements excluding the defs element, all graphics elements and the use element -> ExcludingDefsGraphicsAndUse

Result:

allElementsSVGContainerElementsExcludingDefsGraphicsAndUse

or (perhaps slightly clearer)

allElementsAndSVGContainerElementsExcludingDefsGraphicsAndUse

Thoughts?

Excluding (more in line with text in spec) or Except (more in line with existing schema entries)?

sigveio avatar Jul 28 '21 13:07 sigveio