Re: [csswg-drafts] [css-color-adjust] Animation type of forced-color-adjust (#5342)

The CSS Working Group just discussed `[css-color-adjust] Animation type of forced-color-adjust`, and agreed to the following:

* `RESOLVED: make forced-color-adjust not animatable`

<details><summary>The full IRC log of that discussion</summary>
&lt;dael> Topic:  [css-color-adjust] Animation type of forced-color-adjust<br>
&lt;dael> github: https://github.com/w3c/csswg-drafts/issues/5342<br>
&lt;dael> Rossen_: Raised by anders, not sure if he's on<br>
&lt;dael> iank_: Middle of the night for them<br>
&lt;dael> AmeliaBR: I think it's straight forward<br>
&lt;dael> Rossen_: Curious if we needed to ask their input<br>
&lt;dael> fantasai: Summary: In general we allow pretty much any prop to be animated. A couple we don't b/c have particular inpacts on cascade or animations. Direction and writing mode are not b/c cascade-time interactions.<br>
&lt;dael> fantasai: forced-color-adjust also has cascade implications so proposal is mark as not animatable. I don't know of a use case for animatable so should be fine<br>
&lt;dael> AmeliaBR: Agree, not worth complexity<br>
&lt;florian> +1<br>
&lt;dael> Rossen_: Yeah. We've never seen use case for it<br>
&lt;dael> Rossen_: Any reasons why we should make it animatable? If not objections?<br>
&lt;dael> myles: Suprised at reasoning for it being more complicated to not animate. THis is fit for switch halfway through<br>
&lt;dael> AmeliaBR: Getting special rules for lots of other properties like bg color is doing thing based on if you have f-c-a on. Not a simple dependency like currentColor, lots of rules for other properties<br>
&lt;dael> fantasai: If there's an impl that thinks this should be not animatable we should make it animatable. If all impl think it should animate than leave it.<br>
&lt;fantasai> s/animatable/not animatable/<br>
&lt;fantasai> s/than/then/<br>
&lt;dael> Rossen_: Dependency change is pretty complex here. Since we have had this feature previously in high contrast adjust name this has never been requested for animation. no signals historically and complexity high. That is the reason not to add additional complexity intot he system.<br>
&lt;dael> Rossen_: Does that satisfy myles or do you have other things to consider?<br>
&lt;dael> myles: Nothing more to say<br>
&lt;dael> Rossen_: Objections?<br>
&lt;dael> RESOLVED: make forced-color-adjust not animatable<br>
&lt;dael> AmeliaBR: Follow up. not sure how it works for animating other properties like color and having a transition on it based on if f-c-a turns on it changes computed value of other properties. DOn't know what to expect but need to define<br>
&lt;dael> florian: Some other properties that might not want to animate when f-c-a on?<br>
&lt;dael> AmeliaBR: If f-c-a turns on it changes computed value of many other properties. If those other properties have transitions should they trigger?<br>
&lt;dael> florian: Understood.<br>
&lt;dael> fantasai: I don't know who impl. I don't think matters for authors it's what easiest for impl<br>
&lt;dael> Rossen_: Transitions between forced and not forced colors requires system reset of user shell. If we run or trigger animations after shell reset is questionable<br>
&lt;dael> AmeliaBR: I wasn't thinking about changing if forced-color mode is on but if f-c-a mode applied within css. lots of complex interactions so I'm happy to define as simple as possible but it needs to be defined<br>
&lt;dael> Rossen_: If we have a conditional color rule that applies under a f-c-a none and that rule is evaluated wither or not color change triggers animation. Is that fair?<br>
&lt;dael> AmeliaBR: Yeah<br>
&lt;myles> I don't understand the complexity argument because because script can change the value of this property at any time<br>
&lt;dael> Rossen_: I think we should open a new issue on if that will trigger transitions<br>
&lt;dael> AmeliaBR: I'll file an issue<br>
</details>


-- 
GitHub Notification of comment by css-meeting-bot
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/5342#issuecomment-669593075 using your GitHub account


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

Received on Wednesday, 5 August 2020 23:17:38 UTC