Re: [csswg-drafts] [web-animations-1] Unclear, possibly mislocated definition of "associated animation of an animation effect" (#8497)

Thanks for the report! I think I've fixed up the definitions now.

> Other issues with this sentence:
> 
>     * it's only meaningful in the context of this enumerated procedure for establishing composite order, since it refers to "this effect stack", but nevertheless it's also referenced from [another place in the spec](https://w3c.github.io/csswg-drafts/web-animations-1/#ref-for-associated-animation-of-an-animation-effect). (Admittedly the other reference is also discussing event stacks, but "this event stack" was defined by closure rather than parameter, so there's still a problem.) Possibly the definition could be hoisted up to [4.5.1 Relationship between animation effects and animations](https://w3c.github.io/csswg-drafts/web-animations-1/#animation-effects-and-animations).

I think the definition itself now is independent of any effect stack, but I agree we possibly should shift where it lives. Perhaps we might do that in level 2 since it needs to rework these definitions anyway.

>     * The associated animation of one or both of the animation effects _A_ and _B_ being compared could be undefined (I think), but the procedure doesn't account for that in any way.

Ah, yes, that's an interesting point. In practice that never happens in the context of establishing composite order since effects not attached to animations don't contribute to the animation result (they have no input time so they produce no output) but in terms of making the definition more useful it might be helpful to call out that case and sort such effects first (i.e. lowest composite order so that everything else overrides them).

-- 
GitHub Notification of comment by birtles
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/8497#issuecomment-1451411517 using your GitHub account


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

Received on Thursday, 2 March 2023 07:27:54 UTC