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-animations-2] Accept 'auto' for animation-duration in shorthand #8570

Closed
wants to merge 2 commits into from

Conversation

cdoublev
Copy link
Collaborator

@cdoublev cdoublev commented Mar 13, 2023

2a7cc4b added auto to animation-duration but <single-animation> only accept <time>.

Name: animation-duration
Value: [ auto | <<time [0s,∞]>> ]#

<single-animation> = <time [0s,∞]> || <easing-function> || <time> || <single-animation-iteration-count> || <single-animation-direction> || <single-animation-fill-mode> || <single-animation-play-state> || [ none | <keyframes-name> ]

Note that order is important within each animation definition: the first value in each <single-animation> that can be parsed as a <time> is assigned to the animation-duration

https://drafts.csswg.org/css-animations-1/#typedef-single-animation

@fantasai
Copy link
Collaborator

Hi @cdoublev ! Because of its parsing complications, I think any changes or additions to the animation shorthand need to go through WG discussion, so maybe we need to open an issue first?

@cdoublev
Copy link
Collaborator Author

Superseded by 5feda7d.

@cdoublev cdoublev closed this Jul 21, 2023
@cdoublev cdoublev deleted the css-animations-2-issue branch July 21, 2023 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants