Skip to content

[css-animations-2] Add / separator before animation-trigger exit range #11948

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

Open
ydaniv opened this issue Mar 17, 2025 · 2 comments
Open

[css-animations-2] Add / separator before animation-trigger exit range #11948

ydaniv opened this issue Mar 17, 2025 · 2 comments

Comments

@ydaniv
Copy link
Contributor

ydaniv commented Mar 17, 2025

Animation Triggers allow specifying a default and exit ranges for the specified timeline which currently, in the animation-trigger shorthand, look like 4 (or less) subsequent range definitions.

It was previously suggested to allow separating these 2 ranges using a / to allow specifying the default range with 1 or even 0 values while still be able to unambiguously specify the exit range:

Should we add a / separator between the two ranges to distinguish them more explicitly?
So it would become:

#target {
 animation-trigger: repeat view() contain / cover;
}

And if there's no range on the lefthand side it can be computed to normal.

So, proposing to add a / in the syntax for the animation-trigger shorthand before the animation-trigger-exit-range value(s).

/cc @DavMila @flackr

@bramus
Copy link
Contributor

bramus commented Mar 25, 2025

Seems like a good idea or parsing the shorthand would otherwise be impossible.

@ydaniv ydaniv added the Agenda+ label Mar 25, 2025
@DavMila
Copy link
Contributor

DavMila commented Mar 26, 2025

+1 I like the idea

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Regular agenda
Development

No branches or pull requests

3 participants