Re: [csswg-drafts] [web-animations-2] Specify Animation progress API (#9937)

> I've reviewed this PR and I think it looks good. Thanks @DavMila for your work on this.
> 
> My only outstanding question is [from here](https://github.com/w3c/csswg-drafts/pull/9937#discussion_r1491825782):
> 
> > We _could_ mimic the behaviour of `progress` on `ComputedEffectTiming` and use the fill mode and phase of the root animation effect to make `progress` be `unresolved` when the root effect is not in effect but that might be a bit of a layering violation?
> > ...
> > I guess it comes back to reviewing these issues and seeing what would work best in each of those cases (quoting #8799):
> > > In #8669, #8765 and #8201 there seems to be a common desire to have more convenient APIs for getting some representation of progress.
> 
> Does anyone know any specific intended uses of this API that would help determine if the progress should be unresolved when the animation is in the before/after phase and not filling?

 (not exactly a use case but just a thought that might be worth considering): `currentTime` seems to remain resolved regardless of the fill mode.
I made a [codepen](https://codepen.io/awogbemila/pen/eYXwvdw) where you can run the animation and display `currentTime` afterwards by clicking "Paste CurrentTime". This example shows that `currentTime` remains resolved even with `fill: "none"`.

-- 
GitHub Notification of comment by DavMila
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/pull/9937#issuecomment-1961783321 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Friday, 23 February 2024 18:15:03 UTC