Re: [csswg-drafts] [css-animations-2] Set `animation-trigger-type` as the coordinating list base property (#11653)

> > And notice that animation-range is also a coordinating list property that cannot be specified in the same shorthand as its own group.
> 
> I do not see this specified.

OK, I think that's simply an oversight, but let's clear first what is the intent behind [coordinating list properties](https://www.w3.org/TR/css-values-4/#linked-properties). It means that each longhand (or sub-property) can also be specified as a _**coordinated value list**_, meaning, it applies its effect on the _N_ th item in the list based on its position in the list, and the order of that list is defined by the coordinating property.
So in `animation-trigger-*`'s case (and `animation-range`'s), the coordinating property should still be `animation-name`, and not `animation-trigger-type`, or some other property. Otherwise you'll have a new sub-list inside that coordinated list.

Example:
```css
.target {
  animation: flip 1s, rotate 1s;
  animation-trigger-timeline: view(), view();
}
```

The values of `animation-trigger-timeline` correspond to each animation based on the list created in `animation`.

> I only see that its longhands are reset-only sub-properties of `animation`.

Yes, that's a separate concern that  _was_ specified.

-----------------------

> Here, it is the opposite. I see `animation-trigger` specified as a coordinating list property with `animation-name` as its base property, but not as reset-only sub-property of animation.

I checked the text again:

> Animation Triggers are defined using the animation-trigger-* properties. These list-valued properties, which are all [longhands](https://drafts.csswg.org/css-cascade-5/#longhand) of the [animation-trigger](https://drafts.csswg.org/css-animations-2/#propdef-animation-trigger) [shorthand](https://drafts.csswg.org/css-cascade-5/#shorthand-property), form a [coordinating list property group](https://drafts.csswg.org/css-values-4/#coordinating-list-property) with [animation-name](https://drafts.csswg.org/css-animations-1/#propdef-animation-name) as the [coordinating list base property](https://drafts.csswg.org/css-values-4/#coordinating-list-base-property) and each item in the [coordinated value list](https://drafts.csswg.org/css-values-4/#coordinated-value-list) defining the properties of a single animation trigger.

I think it's correct. It says `animation-trigger-*` properties form a group with `animation-name` as the coordinating property.
I think it says what I intended.

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


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

Received on Friday, 14 February 2025 14:51:53 UTC