Skip to content

Update animation property documentation for clarity#43645

Merged
estelle merged 10 commits intomainfrom
estelle-patch-5
Apr 13, 2026
Merged

Update animation property documentation for clarity#43645
estelle merged 10 commits intomainfrom
estelle-patch-5

Conversation

@estelle
Copy link
Copy Markdown
Member

@estelle estelle commented Apr 2, 2026

Clarified the specification of the animation property, detailing the components and their order. Updated notes on animation-timeline and animation-fill-mode behavior.

animation-range-start and animation-range-end were listed as part of the shorthand, but they aren't, and they aren't reset by the shorthan.

Clarified the specification of the `animation` property, detailing the components and their order. Updated notes on `animation-timeline` and `animation-fill-mode` behavior.
@estelle estelle requested a review from a team as a code owner April 2, 2026 01:01
@estelle estelle requested review from chrisdavidmills and removed request for a team April 2, 2026 01:01
@github-actions github-actions bot added Content:CSS Cascading Style Sheets docs size/m [PR only] 51-500 LoC changed labels Apr 2, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

Preview URLs (1 page)

Flaws (1)

Found an unexpected or unresolvable flaw? Please report it here.

URL: /en-US/docs/Web/CSS/Reference/Properties/animation
Title: animation
Flaw count: 1

  • broken_links:
    • Link /en-US/docs/Web/CSS/Reference/Values/number-token doesn't resolve

(comment last updated: 2026-04-13 17:16:49)

chrisdavidmills and others added 3 commits April 2, 2026 08:56
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@chrisdavidmills chrisdavidmills left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@estelle looking much improved, nice work. I've got some comments for you.


One or more single `<animation>` declarations, separated by commas, with each `<animation>` including:

- `<single-animation-name>`
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should check the type names in the spec to make sure they match with the ones stated here. For example, <keyframes-name> now seems to be used in place of <single-animation-name>.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

matched the formal syntax

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The formal syntax says <keyframes-name>, and this uses <single-animation-name>?

estelle and others added 2 commits April 2, 2026 11:57
Co-authored-by: Chris Mills <chrisdavidmills@gmail.com>
@estelle estelle requested a review from chrisdavidmills April 9, 2026 16:22
Copy link
Copy Markdown
Contributor

@chrisdavidmills chrisdavidmills left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@estelle I reckon this is just about ready to merge now. I added one comment about the Datatypes under values still not matching the formal syntax, but I'm not sure how important that really is. I'll let you consider that one again, and then merge when you are done.

@estelle estelle merged commit 7972cd9 into main Apr 13, 2026
13 checks passed
@estelle estelle deleted the estelle-patch-5 branch April 13, 2026 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:CSS Cascading Style Sheets docs size/m [PR only] 51-500 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants