You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I noticed that we don't have relative_luminance for AlphaColor and I assume that's because this formulation isn't as useful when there's alpha involved. But for DynamicColor, we just discard the alpha. But we also don't have this function for PremulColor ...
Should we have it for AlphaColor and discarding the Alpha?
Should we have it for PremulColor?
I don't have good arguments in favor of having it, so I'm just asking so we can have a decision.
The text was updated successfully, but these errors were encountered:
Those are the reasons I didn't add it. My feeling is we don't add the methods unless there's specific call for them.
The motivation for it in DynamicColor is to support CSS Color 5 contrast-color adequate contrast logic. Of course, if you think about it, the idea is pretty broken when alpha is involved, so it might end up being one of those poorly thought through things the web is full of.
Not having it in the static color alpha types may encourage people to resolve to opaque color, which is sensible.
I noticed that we don't have
relative_luminance
forAlphaColor
and I assume that's because this formulation isn't as useful when there's alpha involved. But forDynamicColor
, we just discard the alpha. But we also don't have this function forPremulColor
...Should we have it for
AlphaColor
and discarding the Alpha?Should we have it for
PremulColor
?I don't have good arguments in favor of having it, so I'm just asking so we can have a decision.
The text was updated successfully, but these errors were encountered: