Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[css-values-4] mix() can not represent retargeted transitions #9341

Closed
andruud opened this issue Sep 12, 2023 · 5 comments
Closed

[css-values-4] mix() can not represent retargeted transitions #9341

andruud opened this issue Sep 12, 2023 · 5 comments

Comments

@andruud
Copy link
Member

andruud commented Sep 12, 2023

Say you are transitioning from A to B, and then in the middle of that transition, you retarget to C. If the interpolated values can only be represented with mix(), then we'd have mix(mix(A; B; x%); C; y%), which isn't valid.

Can we drop the weird "must be the sole value" and actually give mix() a type?

Note: See w3c/css-houdini-drafts#425 for a real example of something that can be transitioned, but not represented.

@Loirooriol
Copy link
Contributor

I think that these functions which accept a list of values and behave as one of them should allow the individual values to be things that would normally require to be the sole value.

@tabatkins
Copy link
Member

Yes, I haven't gotten around to specifying this correctly, but it really should be written as something like <top-level-value>, which is only allowed in places that explicitly allow it (such as in mix() or as the sole value of a property.

@tabatkins
Copy link
Member

Okay, defined <whole-value> and switched mix() and toggle() to using it.

@Loirooriol
Copy link
Contributor

@tabatkins You forgot to replace some <declaration-value>:

interpolates between the first <<declaration-value>> (its [=mix start value=])

and the second <<declaration-value>> (its [=mix end value=]).

if the two <<declaration-value>>s being interpolated by ''mix()''

and its <<declaration-value>>s (if provided)

@tabatkins
Copy link
Member

Ah thanks, fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants