w3c / fxtf-drafts

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

[css-masking-1] Is mask-type a longhand of mask? #528

Closed nt1m closed 9 months ago

nt1m commented 10 months ago

Should mask set the mask-type or should it not?

css-meeting-bot commented 9 months ago

The CSS Working Group just discussed [css-masking-1] Is mask-type a longhand of mask?, and agreed to the following:

The full IRC log of that discussion <fantasai> ntim: I noticed that mask-type has a prefix mask-
<fantasai> ntim: it looks like a longhand of mask, but idk if it should be or not
<TabAtkins> actually "Animation type" is already a required line in Bikeshed
<TabAtkins> fantasai: Looking at the dfn now I think it shouldn't be
<astearns> ack fantasai
<TabAtkins> fantasai: The property is set on a mask *element* and says whether the mask is based on luminance or alpha
<TabAtkins> fantasai: I think that's not an appropraite longhand for the mask shorthand
<TabAtkins> ntim: I wonder if th ename is appropriate, then?
<TabAtkins> fantasai: It is - we dont' always have the situation where the prefix indicates a shorthand.
<TabAtkins> fantasai: I can't think of a different name
<TabAtkins> fantasai: Also this is an SVG property so unsure we can even change it at this point
<miriam> q+
<astearns> ack miriam
<fantasai> miriam: I agree on intuition, mask- feels like a longhand of mask, but there are several mask- properties that are not part of the shorthand
<fantasai> miriam: so it's not a lone standout
<fantasai> ntim: even mask-border is part of mask
<TabAtkins> fantasai: Yeah all the rest are on the element using the mask. This is the only one that's not that.
<TabAtkins> fantasai: It defines the mask, not applies it
<fantasai> ntim: would prefer to rename if it's compatible, but seems maybe it's not?
<fantasai> astearns: Seems resolution is no, mask-type is not a longhand of mask
<fantasai> https://developer.mozilla.org/en-US/docs/Web/CSS/mask-type#browser_compatibility
<TabAtkins> fantasai: I think mask-type is shipping in all browsers
<fantasai> astearns: could leave it open if people want to look into renaming, but doesn't seem likely
<fantasai> RESOLVED: mask-type is not a longhand of mask
tabatkins commented 9 months ago

We don't have the same set of labels here as in the CSS repo, so I didn't mark it as "question answered" :(