Add data for Element.animate() options.timeline
#18776
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Checked Firefox, Chromium support, researched Safari, Chromium support. Mirrors are pure speculation.
github-actions
bot
added
the
data:api 🐇
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Adds support data for the
timelineproperty of theoptionsparameter ofElement.animate().Firefox does not support this option. Chromium does, likely since version 85. Safari likely does, since version 16.
Test results and supporting details
Manually tested in Firefox 111 and Chromium 109. A simple test case is:
Chromium support was added in d393163, which released in Chromium 85.
Safari support was added in WebKit/WebKit@1a2775d, included in WebKit since 614.1.6, which is Safari 16.
I don't have a setup to test Safari 16 or any additional browsers at the moment; the
"mirror"entries are pure speculation, since the linter disallowsnull. I apologize if that makes this PR not useful.