web-platform-tests / web-platform-tests/wpt
[scroll-animations] multiple issues with `animation-timeline` and the `animation` shorthand
Nobody has claimed this yet.
- Dominant language
- HTML
- Stars
- 6.2k
- Forks
- 3.9k
- Avg merge
- 1d 6h
- Merged PRs (30d)
- 586
Description
While looking at adding support for the various CSS properties relevant to Scroll-linked Animations, I came across these tests which look incorrect to me:
This test does not match the CSS Animations Level 2 grammar for animation-timeline since the final value <single-animation-timeline> is not serialized. I realize that this test is perfectly valid for implementations that support CSS Animations Level 1, but how should we deal with the expansion of the animation shorthand?
This test assumes scroll() and view() are not valid values for animation-timeline but they are. This seems like a straight up bug that I'd be happy to file a PR for.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with css/css-animations/parsing/animation-computed.html and scroll-animations/css/animation-shorthand.html at the referenced lines, then compare their expectations with the CSS Animations Level 2 grammar. Update the tests so animation-timeline serialization and scroll()/view() validity match the specification, and run the relevant WPT tests to confirm the corrected behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- css, html
- Domain
- testing, web-dev
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100