-
Notifications
You must be signed in to change notification settings - Fork 746
Description
In 5.4.2 The effect stack, the associated animation of an animation effect is defined as
Let the associated animation of an animation effect be the animation associated with the animation effect that affecting the property with which this effect stack is associated.
"That affecting" doesn't seem grammatical, and I'm not sure what is the intended meaning. Possibly "that is affecting the property" or "that affects the property", but that reading of the clause doesn't add any actual information to the sentence; an animation effect is associated with at most one animation, so there's no need for any such predicate. It's not clear why or how the definition would be more complicated than "the associated animation of an animation effect is the animation with which the animation effect is associated, if any".
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. (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.
- 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.