Re: [csswg-drafts] [css-color-adjust] viewport propagation of forced-color-adjust (#6307)

The CSS Working Group just discussed `[css-color-adjust] viewport propagation of forced-color-adjust`.

<details><summary>The full IRC log of that discussion</summary>
&lt;dael> Topic: [css-color-adjust] viewport propagation of forced-color-adjust<br>
&lt;dael> github: https://github.com/w3c/csswg-drafts/issues/6307<br>
&lt;dael> alisonmaher: Currently prop from root and body to viewport. Since we force colors at used value time we can get wrong if don't prop forced-color-adjust to viewport<br>
&lt;dael> alisonmaher: We previously resolved not to prop any new properties from body to viewport but wondering if we should have an acception here so forced color does prop when set on body<br>
&lt;dael> TabAtkins: I see argument for it.<br>
&lt;dael> TabAtkins: As not a direct implementer I can't say if it's cool to add one more to the list, but I can see why it's confusing to figure out when color comes off body<br>
&lt;dael> Rossen_: Effect of not doing it? You may have bg of viewport that's different than forced?<br>
&lt;dael> alisonmaher: Set forced-color:adjust to none and want bg to be another color we would end up forcing the viewport bg color b/c it's not none at viewport. Wouldn't get bg color at the viewport<br>
&lt;futhark> q+<br>
&lt;dael> fantasai: Two comments. Seems bad practice to set forced-color-adjust none on body. Seems a bit user hostile to say I don't care what you want<br>
&lt;dael> fantasai: If concern is tweak color of bg we have similar problem with color scheme. If we prop one we should prop both. Not sure we should; we should encourage people to set in html<br>
&lt;dael> alisonmaher: If we were to do it for forced-color-adjust doing it for color-scheme would make sense as well<br>
&lt;Rossen_> ack futhark<br>
&lt;dael> futhark: I was thinking that is it really we should prop the property to viewport and not we should take into acocunt when prop. When try and prop bg we look at display value and if it's display:none it's not prop. Maybe similar<br>
&lt;dael> alisonmaher: Yeah, when look at f-c-s at used value time we could end up forcing hte prop value of viewport no matter what.<br>
&lt;dael> s/f-c-s/forced-color-adjust<br>
&lt;dael> Rossen_: Are options to leave as-is and use this as a soft mechanism to discourage such usage patterns by authors? otoh we can still add that same question applies to do we add back color scheme to the list<br>
&lt;dael> alisonmaher: Those are two options. One use case from author to set a bg color for svg image which are similar to root and viewport prop. To fix that we would need to prop from root to viewport. Hoping can resolve on prop from root. If doing for root might makes sense to do from body as well<br>
&lt;dael> Rossen_: Other opinions?<br>
&lt;dael> fantasai: If we're doing from root make sense to do from body as well statement doesn't make sense. Have lot of properties that prop from root but not body so don't think that holds<br>
&lt;dael> alisonmaher: If feel we shouldn't do from body I'm okay with that. Root piece is major thing looking for. Can see author confusion but wouldn't object<br>
&lt;emilio> +1 for doing it just for the root<br>
&lt;dael> fantasai: A bunch of scrolling properties that don't prop from body. We locked down to some css 2 properties<br>
&lt;fantasai> s/from body/from body even though overflow does/<br>
&lt;dael> Rossen_: Not hearing disagreement about root. Sounds reasonable. Convo seems to support adding to root. For body we've been making steady attempts to min exposure that's prop.<br>
&lt;dael> Rossen_: Sounds like current consensus is around adding to root but not body.<br>
&lt;dael> alisonmaher: That works<br>
&lt;dael> Rossen_: Other thoughts?<br>
&lt;dael> Rossen_: Obj to adding forced-color-adjust propagation to apply to root<br>
&lt;dael> RESOLvED: Add forced-color-adjust propagation to apply to root<br>
&lt;oriol> Note in https://github.com/w3c/csswg-drafts/issues/6079#issuecomment-816307011 we resolved "No future properties should propagate from &lt;body> to the ICB"<br>
</details>


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


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

Received on Wednesday, 9 June 2021 16:31:14 UTC