search

w3c / fxtf-drafts

Mirror of https://hg.fxtf.org/drafts
https://drafts.fxtf.org/
Other
68 stars 49 forks source link

[css-masking-1] Remove margin-box from description mask-clip #534

Closed alkorlos closed 5 months ago

alkorlos commented 9 months ago

In the specification CSS Masking Module Level 1 in the section Changes since last publication in the list “The following changes were made since the 26 August 2014 Candidate Recommendation.” has change:

Remove margin-box as possible value from mask-origin and mask-clip.

Value margin-box removed from mask-clip, but in the 7.5. Masking Area: the mask-clip property value margin-box left in description:

For SVG elements without associated CSS layout box, the used values for content-box and padding-box compute to fill-box and for border-box and margin-box compute to stroke-box.

FormularSumo commented 5 months ago

This looks to be along the same lines as https://github.com/w3c/fxtf-drafts/issues/439, and likewise looks like it was just forgotten about. Also seems to be the only outdated use of margin-box that remains in the sepc - hopefully your PR will get merged soon! Pinging @dbaron who merged my related PR.

alkorlos commented 5 months ago

When I read this spec first I saw mask-clip and mask-origin have grammar <geometry-box>, however they have no value margin-box, found https://github.com/w3c/fxtf-drafts/issues/439, this was a known issue, it was fixed a couple of days after the current issue was created.

Later I saw margin-box in description mask-clip, similar error, but in a different place.

Thank you for pinging!