Thank you for LGTMs.

FYI, TAG review was positive 
https://github.com/w3ctag/design-reviews/issues/915#issuecomment-1828305940 
<https://github.com/w3ctag/design-reviews/issues/915>.
On Tuesday, November 28, 2023 at 7:53:33 AM UTC+1 rby...@chromium.org wrote:

> LGTM3
>
> On Sun, Nov 26, 2023 at 3:09 AM Mike Taylor <mike...@chromium.org> wrote:
>
>> LGTM2
>> On 11/23/23 8:39 AM, Yoav Weiss wrote:
>>
>> LGTM1 assuming there aren't any surprises on that front. Thanks!!
>>
>> On Thu, Nov 23, 2023 at 2:33 PM Dominik Röttsches <dr...@google.com> 
>> wrote:
>>
>>> Hi Yoav,
>>>
>>> On Wed, Nov 22, 2023 at 6:46 PM Yoav Weiss <yoav...@chromium.org> wrote:
>>>
>>>> https://github.com/w3c/csswg-drafts/issues/8922#issuecomment-1720930646 
>>>> indicates that some parts of this were resolved and other parts are still 
>>>> to be bikeshedded. 
>>>>
>>>> Can you expand on that? What's the issue around color space 
>>>> interpolation?
>>>>
>>>
>>> In my response further down in the thread I untangled that 
>>> <https://github.com/w3c/csswg-drafts/issues/8922#issuecomment-1792237695>. 
>>> There are two situations in which we need an explicit color space 
>>> representation:
>>> 1) In mixing two palettes, to describe it as a static state, when an 
>>> animation or transition has completed or when we're describing an interim 
>>> state of an animation or transition. For this, we need an explicit 
>>> interpolation method in the grammar of palette-mix() - to be consistent 
>>> with color-mix() and to prepare to capture a static interpolation value 
>>> when animations will have color space control, see below.
>>> The "remaining bike shedding to be done" that's captured in the notes 
>>> solely originated from a question from emilio@ on whether that color 
>>> interpolation tag is needed. Munira and I discussed that with him in the 
>>> directly follow break at resolution at TPAC - but that was offline and not 
>>> minuted. I am expecting emilio@ to comment on this - otherwise there was 
>>> nothing open. 
>>>
>>> 2) What needs to be discussed as a wider issue in CSS is controlling the 
>>> interpolation method during animation or transitions, but that's outside 
>>> the scope of the issue we resolved on. Interpolation color space control 
>>> during animations and transitions are discussed in #7741 
>>> <https://github.com/w3c/csswg-drafts/issues/7741> and #7063 
>>> <https://github.com/w3c/csswg-drafts/issues/7063>.
>>>
>>> I'll ping Emilio again to comment on the issue. Assuming that happens, I 
>>> really see no blockers at all, and Chris Lilley seems to agree with us on 
>>> that.
>>>
>>> Dominik
>>>  
>>>
>>>>
>>>> On Thursday, November 9, 2023 at 4:02:34 PM UTC+1 moo...@google.com 
>>>> wrote:
>>>>
>>>>> Submitted:
>>>>>
>>>>>    - TAG review: https://github.com/w3ctag/design-reviews/issues/915 
>>>>>    - Requested signals from the other vendors:
>>>>>    - Gecko: https://github.com/mozilla/standards-positions/issues/925  
>>>>>       - Webkit: 
>>>>>       https://github.com/WebKit/standards-positions/issues/276 
>>>>>    
>>>>> Thank you, Daniel!
>>>>>
>>>>> On Wed, Nov 8, 2023 at 4:52 PM Daniel Bratell <brat...@gmail.com> 
>>>>> wrote:
>>>>>
>>>>>> Hi, could you please request signals from the other vendors? 
>>>>>> https://bit.ly/blink-signals will tell you how they work.
>>>>>>
>>>>>> Also, you may want to file a TAG review, or give them a FYI or let us 
>>>>>> know why that is not necessary.
>>>>>>
>>>>>> /Daniel
>>>>>> On 2023-11-08 13:56, 'Munira Tursunova' via blink-dev wrote:
>>>>>>
>>>>>> Contact emails moo...@google.com, dr...@google.com
>>>>>>
>>>>>> Explainer 
>>>>>> https://docs.google.com/document/d/1XMTrKH003KBOes6hxzI-3E7LTwp5YwFC-rnzoFpFrfw/edit?usp=sharing
>>>>>>
>>>>>> Specification https://drafts.csswg.org/css-fonts-4/#font-palette-prop
>>>>>>
>>>>>> Summary 
>>>>>>
>>>>>> The CSS font-palette property allows selection of a specific palette 
>>>>>> used to render a font. The CSS Fonts 4 spec defines the animation 
>>>>>> behavior 
>>>>>> of this property as discrete, which is insufficient to achieve a smooth 
>>>>>> transition between two selected palettes. Instead, animating the 
>>>>>> font-palette property should happen by interpolating each of the colour 
>>>>>> record values from the defined palette, i.e. if the start or the end of 
>>>>>> the 
>>>>>> animation has a different colour value for some record in the palette, 
>>>>>> such 
>>>>>> colour value should be interpolated.
>>>>>>
>>>>>>
>>>>>> Blink component Blink>Fonts 
>>>>>> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3EFonts>
>>>>>>
>>>>>> Search tags font-palette 
>>>>>> <https://chromestatus.com/features#tags:font-palette>, animation 
>>>>>> <https://chromestatus.com/features#tags:animation>, transition 
>>>>>> <https://chromestatus.com/features#tags:transition>, 
>>>>>> font-palette-values 
>>>>>> <https://chromestatus.com/features#tags:font-palette-values>, color 
>>>>>> fonts <https://chromestatus.com/features#tags:color%20fonts>
>>>>>>
>>>>>> TAG review None
>>>>>>
>>>>>> TAG review status Not applicable
>>>>>>
>>>>>> Risks 
>>>>>>
>>>>>>
>>>>>> Interoperability and Compatibility 
>>>>>>
>>>>>> Low, new feature, was resolved by W3C working group 
>>>>>> https://github.com/w3c/csswg-drafts/issues/8922#issuecomment-1720930646, 
>>>>>> but not yet implemented in other browsers.
>>>>>>
>>>>>> *Gecko*: No signal Not implemented.
>>>>>>
>>>>>> *WebKit*: No signal Not implemented.
>>>>>>
>>>>>> *Web developers*: Positive (
>>>>>> https://css-tricks.com/colrv1-and-css-font-palette-web-typography/#:~:text=Another%20limitation%3A%20animations%20and%20transitions%20from%20one%20font%2Dpalette%20to%20another%20don%E2%80%99t%20interpolate%20%E2%80%94%20meaning%20you%20can%20switch%20instantly%20from%20one%20palette%20to%20another%2C%20but%20can%E2%80%99t%20gradually%20animate%20between%20them.%20My%20dream%20of%20a%20luridly%20animated%20emoji%20font%20is%20sadly%20unrealized)
>>>>>>  
>>>>>> Ollie Williams expressed his interest in the feature in his article on 
>>>>>> CSS 
>>>>>> Tricks. Scott Kellum (of typetura.com) has also been suggesting it 
>>>>>> as a useful feature for the web (origin: a Twitter thread and email 
>>>>>> conversation, Scott in the meantime deleted their Twitter account).
>>>>>>
>>>>>> *Other signals*:
>>>>>>
>>>>>> WebView application risks 
>>>>>>
>>>>>> Does this intent deprecate or change behavior of existing APIs, such 
>>>>>> that it has potentially high risk for Android WebView-based applications?
>>>>>>
>>>>>> None
>>>>>>
>>>>>>
>>>>>> Debuggability 
>>>>>>
>>>>>> Same as any other CSS property, font-palette property is inspectable 
>>>>>> in DevTools.
>>>>>>
>>>>>>
>>>>>> Will this feature be supported on all six Blink platforms (Windows, 
>>>>>> Mac, Linux, Chrome OS, Android, and Android WebView)? Yes
>>>>>>
>>>>>> Is this feature fully tested by web-platform-tests 
>>>>>> <https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md>
>>>>>> ? Yes 
>>>>>>
>>>>>> https://wpt.fyi/results/css/css-fonts/palette-mix-computed.html 
>>>>>> https://wpt.fyi/results/css/css-fonts/animations/font-palette-interpolation.html
>>>>>>
>>>>>>
>>>>>> Flag name on chrome://flags FontPaletteAnimation
>>>>>>
>>>>>> Finch feature name None
>>>>>>
>>>>>> Non-finch justification None
>>>>>>
>>>>>> Requires code in //chrome? False
>>>>>>
>>>>>> Tracking bug 
>>>>>> https://bugs.chromium.org/p/chromium/issues/detail?id=1400620
>>>>>>
>>>>>> Sample links 
>>>>>> https://drafts.csswg.org/css-fonts-4/images/nabla-animated.webp
>>>>>>
>>>>>> Estimated milestones 
>>>>>> Shipping on desktop 121 
>>>>>> DevTrial on desktop 119 
>>>>>> Shipping on Android 121 
>>>>>> DevTrial on Android 119 
>>>>>> Shipping on WebView 121 
>>>>>>
>>>>>> Anticipated spec changes 
>>>>>>
>>>>>> Open questions about a feature may be a source of future web compat 
>>>>>> or interop issues. Please list open issues (e.g. links to known github 
>>>>>> issues in the project for the feature specification) whose resolution 
>>>>>> may 
>>>>>> introduce web compat/interop risk (e.g., changing to naming or structure 
>>>>>> of 
>>>>>> the API in a non-backward-compatible way).
>>>>>> None
>>>>>>
>>>>>> Link to entry on the Chrome Platform Status 
>>>>>> https://chromestatus.com/feature/5177171439517696
>>>>>>
>>>>>> Links to previous Intent discussions Intent to prototype: 
>>>>>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAAO7W_DFvgY9yqz_Tr%2B2sHMwsydbWMQ66yZWwF7ZoxDZ2yE1QA%40mail.gmail.com
>>>>>>
>>>>>> This intent message was generated by Chrome Platform Status 
>>>>>> <https://chromestatus.com/>.
>>>>>> -- 
>>>>>> You received this message because you are subscribed to the Google 
>>>>>> Groups "blink-dev" group.
>>>>>> To unsubscribe from this group and stop receiving emails from it, 
>>>>>> send an email to blink-dev+...@chromium.org.
>>>>>> To view this discussion on the web visit 
>>>>>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAAO7W_Bao93XoyHRT6NKG6p4pP87pQ0bvTedXOiGc_2hX3E7Jw%40mail.gmail.com
>>>>>>  
>>>>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAAO7W_Bao93XoyHRT6NKG6p4pP87pQ0bvTedXOiGc_2hX3E7Jw%40mail.gmail.com?utm_medium=email&utm_source=footer>
>>>>>> .
>>>>>>
>>>>>> -- 
>>>> You received this message because you are subscribed to the Google 
>>>> Groups "blink-dev" group.
>>>> To unsubscribe from this group and stop receiving emails from it, send 
>>>> an email to blink-dev+...@chromium.org.
>>>> To view this discussion on the web visit 
>>>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/85a8ff9f-03e2-4434-bc29-f7d02130e8e3n%40chromium.org
>>>>  
>>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/85a8ff9f-03e2-4434-bc29-f7d02130e8e3n%40chromium.org?utm_medium=email&utm_source=footer>
>>>> .
>>>>
>>> -- 
>> You received this message because you are subscribed to the Google Groups 
>> "blink-dev" group.
>> To unsubscribe from this group and stop receiving emails from it, send an 
>> email to blink-dev+...@chromium.org.
>> To view this discussion on the web visit 
>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAL5BFfU%3DKmysPKGBfykEE7Kv1g%3D9tO8bEq87Tfaw9knb9-uAZg%40mail.gmail.com
>>  
>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAL5BFfU%3DKmysPKGBfykEE7Kv1g%3D9tO8bEq87Tfaw9knb9-uAZg%40mail.gmail.com?utm_medium=email&utm_source=footer>
>> .
>>
>> -- 
>> You received this message because you are subscribed to the Google Groups 
>> "blink-dev" group.
>> To unsubscribe from this group and stop receiving emails from it, send an 
>> email to blink-dev+...@chromium.org.
>>
> To view this discussion on the web visit 
>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/ce826ae3-ed8a-44c9-a62f-64d7ad1bb36e%40chromium.org
>>  
>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/ce826ae3-ed8a-44c9-a62f-64d7ad1bb36e%40chromium.org?utm_medium=email&utm_source=footer>
>> .
>>
>

-- 
You received this message because you are subscribed to the Google Groups 
"blink-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to blink-dev+unsubscr...@chromium.org.
To view this discussion on the web visit 
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/94a843fb-8e98-4b16-9300-e3258b7f86fen%40chromium.org.

Reply via email to