Re: [EXTERNAL] Re: [blink-dev] Intent to Ship: forced-color-adjust: preserve-parent-color

2021-11-05 Thread 'Sara Tang' via blink-dev 
Thanks Chris, I've filed a request for position here: preserve-parent-color 
value for forced-color-adjust CSS property · Issue #591 · 
mozilla/standards-positions 
(github.com)<https://github.com/mozilla/standards-positions/issues/591> and 
I've updated the ChromeStatus to reflect the change.

From: Chris Harrelson 
Sent: Thursday, November 4, 2021 12:33 PM
To: Fernando Serboncini 
Cc: Sara Tang ; blink-dev@chromium.org 
; Alison Maher ; Daniel 
Libby 
Subject: Re: [EXTERNAL] Re: [blink-dev] Intent to Ship: forced-color-adjust: 
preserve-parent-color



On Fri, Oct 29, 2021 at 1:13 PM Fernando Serboncini 
mailto:fs...@chromium.org>> wrote:

Great. It makes sense.

thank you :)

On Fri, Oct 29, 2021 at 4:11 PM Sara Tang 
mailto:sara.t...@microsoft.com>> wrote:
Hi Fernando,

This I2S does #3.

Thanks!
Sara

From: Fernando Serboncini mailto:fs...@chromium.org>>
Sent: Friday, October 29, 2021 8:21 AM
To: Sara Tang mailto:sara.t...@microsoft.com>>
Cc: blink-dev@chromium.org<mailto:blink-dev@chromium.org> 
mailto:blink-dev@chromium.org>>; Alison Maher 
mailto:alison.ma...@microsoft.com>>; Daniel Libby 
mailto:dli...@microsoft.com>>
Subject: [EXTERNAL] Re: [blink-dev] Intent to Ship: forced-color-adjust: 
preserve-parent-color

Could you clarify if this I2S:
1. just adds the 'preserve-parent-color' value to forced-color-adjust and 
doesn't apply it anywhere by default.
2. adds it and changes the default value of forced-color-adjust to be 
"preserve-parent-color" on all SVG elements
3. adds it and changes the default value of forced-color-adjust to be 
"preserve-parent-color" only on the SVG root element (that then gets propagated 
down)

it seems that #3 was 
decided<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fw3c%2Fcsswg-drafts%2Fissues%2F6310%23issuecomment-862517612&data=04%7C01%7CSara.Tang%40microsoft.com%7C62d286347825457df11408d99fc9f115%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637716512028514093%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=QT1poDe5TrNYtJgf5etEq87HIDY1uwk8SR1vGdbfkEY%3D&reserved=0>
 on the 
CSSWG<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fw3c%2Fcsswg-drafts%2Fissues%2F6310%23issuecomment-862517612&data=04%7C01%7CSara.Tang%40microsoft.com%7C62d286347825457df11408d99fc9f115%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637716512028524050%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=7JGgSyqt9IHbD0t5XbOCFBf8r9T7tPI3XzRLT4QUmU8%3D&reserved=0>,
 but your summary and crbug's CL looks more like #1, while your motivation 
seems like #2.


On Thu, Oct 28, 2021 at 5:45 PM 'Sara Tang' via blink-dev 
mailto:blink-dev@chromium.org>> wrote:
Contact emails
sart...@microsoft.com<mailto:sart...@microsoft.com>, 
alison.ma...@microsoft.com<mailto:alison.ma...@microsoft.com>

Explainer
https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/Accessibility/PreserveParentColor/explainer.md<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftEdge%2FMSEdgeExplainers%2Fblob%2Fmain%2FAccessibility%2FPreserveParentColor%2Fexplainer.md&data=04%7C01%7CSara.Tang%40microsoft.com%7C62d286347825457df11408d99fc9f115%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637716512028524050%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=wLDalF154UJpXmGbWqvaAn0c1O5uo8xlgLcEGEvQHaM%3D&reserved=0>

Specification
https://www.w3.org/TR/css-color-adjust-1/#forced-color-adjust-prop<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.w3.org%2FTR%2Fcss-color-adjust-1%2F%23forced-color-adjust-prop&data=04%7C01%7CSara.Tang%40microsoft.com%7C62d286347825457df11408d99fc9f115%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637716512028534007%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=LwA5QtYzsEN9Bt7DApE8o1UmQ3yP2Z%2BWQkJavM8KGNM%3D&reserved=0>

Summary

Adds the ‘preserve-parent-color' value to the ‘forced-color-adjust' CSS 
property. When Forced Colors Mode is enabled, the ‘color’ property is 
inherited, and we’ve set ‘forced-color-adjust: preserve-parent-color', the 
‘color’ property will compute to the used value of its parent. Otherwise, 
‘forced-color-adjust: preserve-parent-color' value behaves the same as 
‘forced-color-adjust: none’.

Contact emails
sart...@microsoft.com<mailto:sart...@microsoft.com>, 
alison.ma...@microsoft.com<mailto:alison.ma...@microsoft.com>

Explainer
https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/Accessibility/PreserveParentCo

Re: [EXTERNAL] Re: [blink-dev] Intent to Ship: forced-color-adjust: preserve-parent-color

2021-11-05 Thread Manuel Rego Casasnovas 
On a similar note of what I mentioned in the new "only" keyword for
color-scheme property, I'm having a hard time getting the whole picture
of that spec: https://drafts.csswg.org/css-color-adjust/

Do we have a full explainer of the whole spec? The spec is about 2
different things color schemes and forced colors, but they have some
similarities. Then the properties names and values are quite different.
E.g. "color-scheme: light only" vs "forced-color-adjust: none" vs
"color-adjust: exact". They are somehow about opting out of something,
but the properties names and syntax is pretty different.

Cheers,
  Rego

On 04/11/2021 20:33, Chris Harrelson wrote:
> 
> 
> On Fri, Oct 29, 2021 at 1:13 PM Fernando Serboncini  <mailto:fs...@chromium.org>> wrote:
> 
> 
> Great. It makes sense.
> 
> thank you :) 
> 
> On Fri, Oct 29, 2021 at 4:11 PM Sara Tang  <mailto:sara.t...@microsoft.com>> wrote:
> 
> Hi Fernando,
> 
> This I2S does #3.
> 
> Thanks!
> Sara
> 
> 
> *From:* Fernando Serboncini  <mailto:fs...@chromium.org>>
> *Sent:* Friday, October 29, 2021 8:21 AM
> *To:* Sara Tang  <mailto:sara.t...@microsoft.com>>
> *Cc:* blink-dev@chromium.org <mailto:blink-dev@chromium.org>
> mailto:blink-dev@chromium.org>>; Alison
>     Maher      <mailto:alison.ma...@microsoft.com>>; Daniel Libby
> mailto:dli...@microsoft.com>>
> *Subject:* [EXTERNAL] Re: [blink-dev] Intent to Ship:
> forced-color-adjust: preserve-parent-color
>  
> Could you clarify if thisI2S:
> 1. just adds the 'preserve-parent-color' value to
> forced-color-adjust and doesn't apply it anywhere by default.
> 2. adds it and changes the default value of forced-color-adjust
> to be "preserve-parent-color" on all SVG elements
> 3. adds it and changes the default value of forced-color-adjust
> to be "preserve-parent-color" only on the SVG root element(that
> then gets propagated down)
> 
> it seems that #3 was decided
> 
> <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fw3c%2Fcsswg-drafts%2Fissues%2F6310%23issuecomment-862517612&data=04%7C01%7CSara.Tang%40microsoft.com%7Cafe9d7d59d5d4f31cb2108d99aefdd45%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637711177339738955%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=DU66ohWIfaMOCWEffOH%2FZnWTEmITV7ZaSmeWT%2BmZIcY%3D&reserved=0>on
> the CSSWG
> 
> <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fw3c%2Fcsswg-drafts%2Fissues%2F6310%23issuecomment-862517612&data=04%7C01%7CSara.Tang%40microsoft.com%7Cafe9d7d59d5d4f31cb2108d99aefdd45%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637711177339748916%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=KS9HCv0cUqSOVfnn%2FPoGxNwQNv5S1hztDwF%2FVSvUmyI%3D&reserved=0>,
> but your summary and crbug's CL looks more like #1, while your
> motivation seems like #2.
>  
> 
> On Thu, Oct 28, 2021 at 5:45 PM 'Sara Tang' via blink-dev
> mailto:blink-dev@chromium.org>> wrote:
> 
> 
> Contact emails
> 
> sart...@microsoft.com
> <mailto:sart...@microsoft.com>, alison.ma...@microsoft.com
> <mailto:alison.ma...@microsoft.com>
> 
> 
> Explainer
> 
> 
> https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/Accessibility/PreserveParentColor/explainer.md
> 
> <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftEdge%2FMSEdgeExplainers%2Fblob%2Fmain%2FAccessibility%2FPreserveParentColor%2Fexplainer.md&data=04%7C01%7CSara.Tang%40microsoft.com%7Cafe9d7d59d5d4f31cb2108d99aefdd45%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637711177339748916%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=DWthG7wyL%2BhzWXHdx4oQs2PNg49AhRoV0vXyNz5gXAk%3D&reserved=0>
> 
> 
> Specification
> 
> https://www.w3.org/TR/css-color-adjust-1/#forced-color-adjust-prop
> 
> <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.w3.org%2FTR%2Fcss-color-adjust-1%2F%23forced-color-adjust-prop&data=04%7C01%7CSara.

Re: [EXTERNAL] Re: [blink-dev] Intent to Ship: forced-color-adjust: preserve-parent-color

2021-11-04 Thread Chris Harrelson 
On Fri, Oct 29, 2021 at 1:13 PM Fernando Serboncini 
wrote:

>
> Great. It makes sense.
>
> thank you :)
>
> On Fri, Oct 29, 2021 at 4:11 PM Sara Tang  wrote:
>
>> Hi Fernando,
>>
>> This I2S does #3.
>>
>> Thanks!
>> Sara
>> --
>> *From:* Fernando Serboncini 
>> *Sent:* Friday, October 29, 2021 8:21 AM
>> *To:* Sara Tang 
>> *Cc:* blink-dev@chromium.org ; Alison Maher <
>> alison.ma...@microsoft.com>; Daniel Libby 
>> *Subject:* [EXTERNAL] Re: [blink-dev] Intent to Ship:
>> forced-color-adjust: preserve-parent-color
>>
>> Could you clarify if this I2S:
>> 1. just adds the 'preserve-parent-color' value to forced-color-adjust and
>> doesn't apply it anywhere by default.
>> 2. adds it and changes the default value of forced-color-adjust to be
>> "preserve-parent-color" on all SVG elements
>> 3. adds it and changes the default value of forced-color-adjust to be "
>> preserve-parent-color" only on the SVG root element (that then gets
>> propagated down)
>>
>> it seems that #3 was decided
>> <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fw3c%2Fcsswg-drafts%2Fissues%2F6310%23issuecomment-862517612&data=04%7C01%7CSara.Tang%40microsoft.com%7Cafe9d7d59d5d4f31cb2108d99aefdd45%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637711177339738955%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=DU66ohWIfaMOCWEffOH%2FZnWTEmITV7ZaSmeWT%2BmZIcY%3D&reserved=0>
>> on the CSSWG
>> <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fw3c%2Fcsswg-drafts%2Fissues%2F6310%23issuecomment-862517612&data=04%7C01%7CSara.Tang%40microsoft.com%7Cafe9d7d59d5d4f31cb2108d99aefdd45%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637711177339748916%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=KS9HCv0cUqSOVfnn%2FPoGxNwQNv5S1hztDwF%2FVSvUmyI%3D&reserved=0>,
>> but your summary and crbug's CL looks more like #1, while your
>> motivation seems like #2.
>>
>>
>> On Thu, Oct 28, 2021 at 5:45 PM 'Sara Tang' via blink-dev <
>> blink-dev@chromium.org> wrote:
>>
>> Contact emails sart...@microsoft.com, alison.ma...@microsoft.com
>>
>> Explainer
>> https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/Accessibility/PreserveParentColor/explainer.md
>> <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftEdge%2FMSEdgeExplainers%2Fblob%2Fmain%2FAccessibility%2FPreserveParentColor%2Fexplainer.md&data=04%7C01%7CSara.Tang%40microsoft.com%7Cafe9d7d59d5d4f31cb2108d99aefdd45%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637711177339748916%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=DWthG7wyL%2BhzWXHdx4oQs2PNg49AhRoV0vXyNz5gXAk%3D&reserved=0>
>>
>> Specification
>> https://www.w3.org/TR/css-color-adjust-1/#forced-color-adjust-prop
>> <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.w3.org%2FTR%2Fcss-color-adjust-1%2F%23forced-color-adjust-prop&data=04%7C01%7CSara.Tang%40microsoft.com%7Cafe9d7d59d5d4f31cb2108d99aefdd45%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637711177339758868%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=20lJLa8c%2BxRDoldBHmd8Zro%2F%2FYtqwK%2BI%2BYwne0ES6Y8%3D&reserved=0>
>>
>> Summary
>>
>> Adds the ‘preserve-parent-color' value to the ‘forced-color-adjust' CSS
>> property. When Forced Colors Mode is enabled, the ‘color’ property is
>> inherited, and we’ve set ‘forced-color-adjust: preserve-parent-color', the
>> ‘color’ property will compute to the used value of its parent. Otherwise,
>> ‘forced-color-adjust: preserve-parent-color' value behaves the same as
>> ‘forced-color-adjust: none’.
>>
>> Contact emails sart...@microsoft.com, alison.ma...@microsoft.com
>>
>> Explainer
>> https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/Accessibility/PreserveParentColor/explainer.md
>> <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftEdge%2FMSEdgeExplainers%2Fblob%2Fmain%2FAccessibility%2FPreserveParentColor%2Fexplainer.md&data=04%7C01%7CSara.Tang%40microsoft.com%7Cafe9d7d59d5d4f31cb2108d99aefdd45%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637711177339768823%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=aRu0keSh5Rgj%2BVTIFQ9%2BLXLuWr0bdGWAgxk

Re: [EXTERNAL] Re: [blink-dev] Intent to Ship: forced-color-adjust: preserve-parent-color

2021-10-29 Thread Fernando Serboncini 
Great. It makes sense.

thank you :)

On Fri, Oct 29, 2021 at 4:11 PM Sara Tang  wrote:

> Hi Fernando,
>
> This I2S does #3.
>
> Thanks!
> Sara
> --
> *From:* Fernando Serboncini 
> *Sent:* Friday, October 29, 2021 8:21 AM
> *To:* Sara Tang 
> *Cc:* blink-dev@chromium.org ; Alison Maher <
> alison.ma...@microsoft.com>; Daniel Libby 
> *Subject:* [EXTERNAL] Re: [blink-dev] Intent to Ship:
> forced-color-adjust: preserve-parent-color
>
> Could you clarify if this I2S:
> 1. just adds the 'preserve-parent-color' value to forced-color-adjust and
> doesn't apply it anywhere by default.
> 2. adds it and changes the default value of forced-color-adjust to be
> "preserve-parent-color" on all SVG elements
> 3. adds it and changes the default value of forced-color-adjust to be "
> preserve-parent-color" only on the SVG root element (that then gets
> propagated down)
>
> it seems that #3 was decided
> <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fw3c%2Fcsswg-drafts%2Fissues%2F6310%23issuecomment-862517612&data=04%7C01%7CSara.Tang%40microsoft.com%7Cafe9d7d59d5d4f31cb2108d99aefdd45%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637711177339738955%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=DU66ohWIfaMOCWEffOH%2FZnWTEmITV7ZaSmeWT%2BmZIcY%3D&reserved=0>
> on the CSSWG
> <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fw3c%2Fcsswg-drafts%2Fissues%2F6310%23issuecomment-862517612&data=04%7C01%7CSara.Tang%40microsoft.com%7Cafe9d7d59d5d4f31cb2108d99aefdd45%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637711177339748916%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=KS9HCv0cUqSOVfnn%2FPoGxNwQNv5S1hztDwF%2FVSvUmyI%3D&reserved=0>,
> but your summary and crbug's CL looks more like #1, while your motivation
> seems like #2.
>
>
> On Thu, Oct 28, 2021 at 5:45 PM 'Sara Tang' via blink-dev <
> blink-dev@chromium.org> wrote:
>
> Contact emails sart...@microsoft.com, alison.ma...@microsoft.com
>
> Explainer
> https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/Accessibility/PreserveParentColor/explainer.md
> <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftEdge%2FMSEdgeExplainers%2Fblob%2Fmain%2FAccessibility%2FPreserveParentColor%2Fexplainer.md&data=04%7C01%7CSara.Tang%40microsoft.com%7Cafe9d7d59d5d4f31cb2108d99aefdd45%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637711177339748916%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=DWthG7wyL%2BhzWXHdx4oQs2PNg49AhRoV0vXyNz5gXAk%3D&reserved=0>
>
> Specification
> https://www.w3.org/TR/css-color-adjust-1/#forced-color-adjust-prop
> <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.w3.org%2FTR%2Fcss-color-adjust-1%2F%23forced-color-adjust-prop&data=04%7C01%7CSara.Tang%40microsoft.com%7Cafe9d7d59d5d4f31cb2108d99aefdd45%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637711177339758868%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=20lJLa8c%2BxRDoldBHmd8Zro%2F%2FYtqwK%2BI%2BYwne0ES6Y8%3D&reserved=0>
>
> Summary
>
> Adds the ‘preserve-parent-color' value to the ‘forced-color-adjust' CSS
> property. When Forced Colors Mode is enabled, the ‘color’ property is
> inherited, and we’ve set ‘forced-color-adjust: preserve-parent-color', the
> ‘color’ property will compute to the used value of its parent. Otherwise,
> ‘forced-color-adjust: preserve-parent-color' value behaves the same as
> ‘forced-color-adjust: none’.
>
> Contact emails sart...@microsoft.com, alison.ma...@microsoft.com
>
> Explainer
> https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/Accessibility/PreserveParentColor/explainer.md
> <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftEdge%2FMSEdgeExplainers%2Fblob%2Fmain%2FAccessibility%2FPreserveParentColor%2Fexplainer.md&data=04%7C01%7CSara.Tang%40microsoft.com%7Cafe9d7d59d5d4f31cb2108d99aefdd45%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637711177339768823%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=aRu0keSh5Rgj%2BVTIFQ9%2BLXLuWr0bdGWAgxk3niZPKkw%3D&reserved=0>
>
> Specification
> https://www.w3.org/TR/css-color-adjust-1/#forced-color-adjust-prop
> <https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.w3.org%2FTR%2Fcss-color-adjust-1%2F%23forced-color-adjust-prop&data=04%7C01%7CSara.Tang%40microsoft.com%7Cafe9d7d59d5d

Re: [EXTERNAL] Re: [blink-dev] Intent to Ship: forced-color-adjust: preserve-parent-color

2021-10-29 Thread 'Sara Tang' via blink-dev 
Hi Fernando,

This I2S does #3.

Thanks!
Sara

From: Fernando Serboncini 
Sent: Friday, October 29, 2021 8:21 AM
To: Sara Tang 
Cc: blink-dev@chromium.org ; Alison Maher 
; Daniel Libby 
Subject: [EXTERNAL] Re: [blink-dev] Intent to Ship: forced-color-adjust: 
preserve-parent-color

Could you clarify if this I2S:
1. just adds the 'preserve-parent-color' value to forced-color-adjust and 
doesn't apply it anywhere by default.
2. adds it and changes the default value of forced-color-adjust to be 
"preserve-parent-color" on all SVG elements
3. adds it and changes the default value of forced-color-adjust to be 
"preserve-parent-color" only on the SVG root element (that then gets propagated 
down)

it seems that #3 was 
decided<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fw3c%2Fcsswg-drafts%2Fissues%2F6310%23issuecomment-862517612&data=04%7C01%7CSara.Tang%40microsoft.com%7Cafe9d7d59d5d4f31cb2108d99aefdd45%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637711177339738955%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=DU66ohWIfaMOCWEffOH%2FZnWTEmITV7ZaSmeWT%2BmZIcY%3D&reserved=0>
 on the 
CSSWG<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fw3c%2Fcsswg-drafts%2Fissues%2F6310%23issuecomment-862517612&data=04%7C01%7CSara.Tang%40microsoft.com%7Cafe9d7d59d5d4f31cb2108d99aefdd45%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637711177339748916%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=KS9HCv0cUqSOVfnn%2FPoGxNwQNv5S1hztDwF%2FVSvUmyI%3D&reserved=0>,
 but your summary and crbug's CL looks more like #1, while your motivation 
seems like #2.


On Thu, Oct 28, 2021 at 5:45 PM 'Sara Tang' via blink-dev 
mailto:blink-dev@chromium.org>> wrote:
Contact emails
sart...@microsoft.com<mailto:sart...@microsoft.com>, 
alison.ma...@microsoft.com<mailto:alison.ma...@microsoft.com>

Explainer
https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/Accessibility/PreserveParentColor/explainer.md<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftEdge%2FMSEdgeExplainers%2Fblob%2Fmain%2FAccessibility%2FPreserveParentColor%2Fexplainer.md&data=04%7C01%7CSara.Tang%40microsoft.com%7Cafe9d7d59d5d4f31cb2108d99aefdd45%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637711177339748916%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=DWthG7wyL%2BhzWXHdx4oQs2PNg49AhRoV0vXyNz5gXAk%3D&reserved=0>

Specification
https://www.w3.org/TR/css-color-adjust-1/#forced-color-adjust-prop<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.w3.org%2FTR%2Fcss-color-adjust-1%2F%23forced-color-adjust-prop&data=04%7C01%7CSara.Tang%40microsoft.com%7Cafe9d7d59d5d4f31cb2108d99aefdd45%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637711177339758868%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=20lJLa8c%2BxRDoldBHmd8Zro%2F%2FYtqwK%2BI%2BYwne0ES6Y8%3D&reserved=0>

Summary

Adds the ‘preserve-parent-color' value to the ‘forced-color-adjust' CSS 
property. When Forced Colors Mode is enabled, the ‘color’ property is 
inherited, and we’ve set ‘forced-color-adjust: preserve-parent-color', the 
‘color’ property will compute to the used value of its parent. Otherwise, 
‘forced-color-adjust: preserve-parent-color' value behaves the same as 
‘forced-color-adjust: none’.

Contact emails
sart...@microsoft.com<mailto:sart...@microsoft.com>, 
alison.ma...@microsoft.com<mailto:alison.ma...@microsoft.com>

Explainer
https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/Accessibility/PreserveParentColor/explainer.md<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftEdge%2FMSEdgeExplainers%2Fblob%2Fmain%2FAccessibility%2FPreserveParentColor%2Fexplainer.md&data=04%7C01%7CSara.Tang%40microsoft.com%7Cafe9d7d59d5d4f31cb2108d99aefdd45%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637711177339768823%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=aRu0keSh5Rgj%2BVTIFQ9%2BLXLuWr0bdGWAgxk3niZPKkw%3D&reserved=0>

Specification
https://www.w3.org/TR/css-color-adjust-1/#forced-color-adjust-prop<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.w3.org%2FTR%2Fcss-color-adjust-1%2F%23forced-color-adjust-prop&data=04%7C01%7CSara.Tang%40microsoft.com%7Cafe9d7d59d5d4f31cb2108d99aefdd45%7C72f988bf86f141af91ab2d7cd011db47%7C0%7C0%7C637711177339778777%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=2BCGuB1g2yK%2BwFjvlbRmHrZK4VjbnpOn6wlAOWsbFW8%3D&reserved=0>

Re: [blink-dev] Intent to Ship: forced-color-adjust: preserve-parent-color

2021-10-29 Thread Fernando Serboncini 
Could you clarify if this I2S:
1. just adds the 'preserve-parent-color' value to forced-color-adjust and
doesn't apply it anywhere by default.
2. adds it and changes the default value of forced-color-adjust to be
"preserve-parent-color" on all SVG elements
3. adds it and changes the default value of forced-color-adjust to be "
preserve-parent-color" only on the SVG root element (that then gets
propagated down)

it seems that #3 was decided
 on
the CSSWG
,
but your summary and crbug's CL looks more like #1, while your motivation
seems like #2.


On Thu, Oct 28, 2021 at 5:45 PM 'Sara Tang' via blink-dev <
blink-dev@chromium.org> wrote:

> Contact emails sart...@microsoft.com, alison.ma...@microsoft.com
>
> Explainer
> https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/Accessibility/PreserveParentColor/explainer.md
>
> Specification
> https://www.w3.org/TR/css-color-adjust-1/#forced-color-adjust-prop
>
> Summary
>
> Adds the ‘preserve-parent-color' value to the ‘forced-color-adjust' CSS
> property. When Forced Colors Mode is enabled, the ‘color’ property is
> inherited, and we’ve set ‘forced-color-adjust: preserve-parent-color', the
> ‘color’ property will compute to the used value of its parent. Otherwise,
> ‘forced-color-adjust: preserve-parent-color' value behaves the same as
> ‘forced-color-adjust: none’.
>
> Contact emails sart...@microsoft.com, alison.ma...@microsoft.com
>
> Explainer
> https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/Accessibility/PreserveParentColor/explainer.md
>
> Specification
> https://www.w3.org/TR/css-color-adjust-1/#forced-color-adjust-prop
>
> Summary
>
> Adds the ‘preserve-parent-color' value to the ‘forced-color-adjust' CSS
> property. When Forced Colors Mode is enabled, the ‘color’ property is
> inherited, and we’ve set ‘forced-color-adjust: preserve-parent-color', the
> ‘color’ property will compute to the used value of its parent. Otherwise,
> ‘forced-color-adjust: preserve-parent-color' value behaves the same as
> ‘forced-color-adjust: none’.
>
> *Motivation*
>
> * ‘forced-color-adjust' is a CSS property that allows developers to opt
> out of Forced Colors Mode.  Previously, there were
> two supported values: ‘auto’ and ‘none’, which can be used to
> control whether or not an element’s styles are adjusted by the UA in Forced
> Colors Mode. A third value, ‘preserve-parent-color', has recently been
> introduced in the spec, which provides similar behavior to ‘none’,
> except that it also allows an element to inherit its parent's
> used ‘color’ value. In other words, ‘preserve-parent-color' provides the
> ability for an element to inherit its parent’s Forced Colors Mode
> adjusted ‘color’ value.  The intention of ‘preserve-parent-color’ is to get
> a reasonable behavior for SVG icons that utilize ‘currentColor’ when
> styling ‘fill’ and ‘stroke’ in Forced Colors Mode,
> as described in [css-color-adjust-1] Spec currently breaks use of
> currentColor for SVG icons in WHCM · Issue #6310 · w3c/csswg-drafts ·
> GitHub .  The use of
> ‘currentColor’ when styling an SVG icon is a common pattern used by authors
> to ensure an accessible experience in Forced Colors Mode. For example, in
> this sample logo,  an author
> would expect the logo to automatically adjust to use the ‘CanvasText’
> system color for ‘fill’ and ‘stroke’ in Forced Colors Mode, as a result of
> setting each to ‘currentColor’.   This behavior,
> however, became broken when we moved from forcing colors at computed value
> time to used value time: [css-color-adjust-1] Is forced color computed or
> used value? · Issue #4915 · w3c/csswg-drafts · GitHub
> . Instead of inheriting
> ‘CanvasText’, as before, the above sample logo
>  would inherit the computed
> ‘color’ value of its parent, resulting in a logo that is no longer readable
> in Forced Colors Mode.  The new ‘preserve-parent-color' value was added to
> address this common SVG use case. By changing the default value of
> ‘forced-color-adjust’ for SVGs from ‘none’ to ‘preserve-parent-color', SVG
> icons that make use of ‘currentColor’ will now inherit the used ‘color’
> value of its parent, as expected.  It is important to note that this may
> break SVGs that expect the opposite inheritance behavior for the ‘color’
> property. However, the behavior of `preserve-parent-color` handles the most
> common SVG use cases, and the behavior better matches legacy
> implementations of High Contrast Mode. *
>
> Blink component Blink>CSS
> 
>
> Search tags css , forced
>

[blink-dev] Intent to Ship: forced-color-adjust: preserve-parent-color

2021-10-28 Thread 'Sara Tang' via blink-dev 
Contact emails
sart...@microsoft.com, 
alison.ma...@microsoft.com

Explainer
https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/Accessibility/PreserveParentColor/explainer.md

Specification
https://www.w3.org/TR/css-color-adjust-1/#forced-color-adjust-prop

Summary

Adds the ‘preserve-parent-color' value to the ‘forced-color-adjust' CSS 
property. When Forced Colors Mode is enabled, the ‘color’ property is 
inherited, and we’ve set ‘forced-color-adjust: preserve-parent-color', the 
‘color’ property will compute to the used value of its parent. Otherwise, 
‘forced-color-adjust: preserve-parent-color' value behaves the same as 
‘forced-color-adjust: none’.

Contact emails
sart...@microsoft.com, 
alison.ma...@microsoft.com

Explainer
https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/Accessibility/PreserveParentColor/explainer.md

Specification
https://www.w3.org/TR/css-color-adjust-1/#forced-color-adjust-prop

Summary

Adds the ‘preserve-parent-color' value to the ‘forced-color-adjust' CSS 
property. When Forced Colors Mode is enabled, the ‘color’ property is 
inherited, and we’ve set ‘forced-color-adjust: preserve-parent-color', the 
‘color’ property will compute to the used value of its parent. Otherwise, 
‘forced-color-adjust: preserve-parent-color' value behaves the same as 
‘forced-color-adjust: none’.

Motivation

‘forced-color-adjust' is a CSS property that allows developers to opt out of 
Forced Colors Mode.

Previously, there were two supported values: ‘auto’ and ‘none’, which can be 
used to control whether or not an element’s styles are adjusted by the UA in 
Forced Colors Mode. A third value, ‘preserve-parent-color', has recently been 
introduced in the spec, which provides similar behavior to ‘none’, except that 
it also allows an element to inherit its parent's used ‘color’ value. In other 
words, ‘preserve-parent-color' provides the ability for an element to inherit 
its parent’s Forced Colors Mode adjusted ‘color’ value.

The intention of ‘preserve-parent-color’ is to get a reasonable behavior for 
SVG icons that utilize ‘currentColor’ when styling ‘fill’ and ‘stroke’ in 
Forced Colors Mode, as described in [css-color-adjust-1] Spec currently breaks 
use of currentColor for SVG icons in WHCM · Issue #6310 · w3c/csswg-drafts · 
GitHub.

The use of ‘currentColor’ when styling an SVG icon is a common pattern used by 
authors to ensure an accessible experience in Forced Colors Mode. For example, 
in this sample logo, an author 
would expect the logo to automatically adjust to use the ‘CanvasText’ system 
color for ‘fill’ and ‘stroke’ in Forced Colors Mode, as a result of setting 
each to ‘currentColor’.

This behavior, however, became broken when we moved from forcing colors at 
computed value time to used value time: [css-color-adjust-1] Is forced color 
computed or used value? · Issue #4915 · w3c/csswg-drafts · 
GitHub. Instead of inheriting 
‘CanvasText’, as before, the above sample 
logo would inherit the computed 
‘color’ value of its parent, resulting in a logo that is no longer readable in 
Forced Colors Mode.

The new ‘preserve-parent-color' value was added to address this common SVG use 
case. By changing the default value of ‘forced-color-adjust’ for SVGs from 
‘none’ to ‘preserve-parent-color', SVG icons that make use of ‘currentColor’ 
will now inherit the used ‘color’ value of its parent, as expected.

It is important to note that this may break SVGs that expect the opposite 
inheritance behavior for the ‘color’ property. However, the behavior of 
`preserve-parent-color` handles the most common SVG use cases, and the behavior 
better matches legacy implementations of High Contrast Mode.

Blink component
Blink>CSS

Search tags
css, 
forced, 
colors, 
forced-colors, 
forced-color-adjust,
 
preserve-parent-color

TAG review
https://github.com/w3ctag/design-reviews/issues/681

TAG review status
Issues open

Risks


Interoperability and Compatibility

Interoperability Risks Gecko has shipped a version of Forced Colors Mode 
without support for ‘forced-color-adjust’. Although there is an open bug for 
adding support (https://bugzilla.mozilla.org/show_bug.cgi?id=1591210), 
development has not been started yet. Compatibility Risks We are updating the 
defa