You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When a given workletAnimation’s play state changes to pending, running, or paused, run the procedure to associate animator instance of worklet animation given workletAnimation.
When a given workletAnimation’s play state changes to idle or finished, run the procedure to disassociate animator instance of worklet animation given workletAnimation.
When the procedure to set the target effect of an animation for a given workletAnimation is called, then set animator instance of worklet animation given workletAnimation.
When the procedure to set the timeline of an animation for a given workletAnimation is called, then set animator instance of worklet animation given workletAnimation.
It is difficult-to-impossible for a naive reader to realize that this step actually also sets the animator instance, as it is not mentioned in either the text or the link.
My suggestions would be to:
Clearly mark the area where we override Web Animations concepts as 'extensions to Web Animations APIs' or similar, and
From the AnimationWorklet spec at least, link to that section rather than the Web Animations API where appropriate (e.g. in the WorkletAnimation creation steps).
The text was updated successfully, but these errors were encountered:
* Improve wording on how worklet animation interacts with web-animations (issue #870)
* Update web-animations spec link to be https://drafts.csswg.org which is more recent
* Add section to deal with addition of "replace state" in Web animations.
* Add issue to indicate our current shortcoming for async operations.
* lots of small fixes
It is quite hard to find the "override" behavior specified in the 'Interaction with Animator Instances' section:
For example, when reading Creating a Worklet Animation, step 4:
It is difficult-to-impossible for a naive reader to realize that this step actually also sets the animator instance, as it is not mentioned in either the text or the link.
My suggestions would be to:
The text was updated successfully, but these errors were encountered: