- From: CSS Meeting Bot via GitHub <sysbot+gh@w3.org>
- Date: Wed, 09 Jun 2021 16:35:21 +0000
- To: public-css-archive@w3.org
The CSS Working Group just discussed `[css-color-adjust-1] Spec currently breaks use of currentColor for SVG icons in WHCM`, and agreed to the following: * `RESOLVED: add color-only value as described in the issue and clarified by TabAtkins` <details><summary>The full IRC log of that discussion</summary> <dael> Topic: [css-color-adjust-1] Spec currently breaks use of currentColor for SVG icons in WHCM<br> <dael> github: https://github.com/w3c/csswg-drafts/issues/6310<br> <dael> alisonmaher: This is around handling currentColor. AmeliaBR noted that SVG icons will not inherit appropriate forced-color through currentColor.<br> <dael> alisonmaher: 2 solutions. 1 undo the resolution to use forced colors at used value time. 2) intoduce a color only value that only adjusted color. Make that default for SVGs. b/c color only effects svg through currentColor works well for icon<br> <TabAtkins> q+<br> <dael> alisonmaher: Only possible unexpected is an svg ancestor set forced-color to none the svg would still set to currentCOlor. This seems rare so in favor of color-only. Looking for other opinions<br> <Rossen_> ack TabAtkins<br> <dael> TabAtkins: Definitely need to fix this. Only consideration to fix issue you raised could the value act as none if inheritied value is none but act as color if it's auto.<br> <dael> TabAtkins: Then it still works as expected<br> <dael> alisonmaher: Good idea. Will need to look into how to impl but could be good way to handle<br> <dael> TabAtkins: Seems reasonable to add, I'm in value<br> <dael> s/value/favor<br> <dael> Rossen_: Prop to fix the issue through a spec clarification and not adding color-only value?<br> <dael> TabAtkins: NO, still add value. The value acts as alisonmaher and AmeliaBR spec in auto case. If inherited is none it would act as none. It would automatically opt-out<br> <dael> Rossen_: I see<br> <dael> Rossen_: Other opinions or suggestions?<br> <dael> Rossen_: alisonmaher do you want a resolution now? Or do you prefer to go back and figure out if implementable?<br> <dael> alisonmaher: Can resolve and come back if impl is tricky. Seems adding color-only value we can resolve on<br> <dael> Rossen_: Objections to add color-only value as described in the issue and clarified by TabAtkins<br> <dael> RESOLVED: add color-only value as described in the issue and clarified by TabAtkins<br> </details> -- GitHub Notification of comment by css-meeting-bot Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/6310#issuecomment-857854589 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:36:30 UTC