Re: [blink-dev] Intent to Ship: CSS env variable for OS-level font scale

2025-06-17 Thread 'David Grogan' via blink-dev
Update: after further discussion with Torne, we've decided on a WebView plan, namely that for the time being: - env(preferred-text-scale) will be set only when the layout_mode is set to TEXT_AUTOSIZING (which is similar to how Chrome on Android works) - setTextZoom

Re: [blink-dev] Intent to Ship: CSS env variable for OS-level font scale

2025-05-30 Thread Vladimir Levin
LGTM3 On Fri, May 30, 2025 at 1:11 PM Philip Jägenstedt wrote: > LGTM2 > > Thank Philip and David for elaborating on the differences between mobile > and desktop platform that have led to this approach, and Domenic for your > summary of the situation. > > Thinking about the longer term, I think

Re: [blink-dev] Intent to Ship: CSS env variable for OS-level font scale

2025-05-30 Thread Philip Jägenstedt
LGTM2 Thank Philip and David for elaborating on the differences between mobile and desktop platform that have led to this approach, and Domenic for your summary of the situation. Thinking about the longer term, I think it's important that the addition of env(preferred-text-scale) and yet other me

Re: [blink-dev] Intent to Ship: CSS env variable for OS-level font scale

2025-05-29 Thread Domenic Denicola
Thanks for the patient explanation David. I'm convinced that this is one of those cases where the web has enough unfortunate legacy that we need to add extra complexity. LGTM1. To summarize back the main points of concern I had, and how they're actually OK: - I was concerned about forward-comp

Re: [blink-dev] Intent to Ship: CSS env variable for OS-level font scale

2025-05-29 Thread David Grogan
On Tue, May 27, 2025 at 6:37 PM Domenic Denicola wrote: > Thanks for the detailed response. However, I might not be understanding, > so let me try asking the following question. > > It sounds like the proposal is to ship two opt-ins for adapting to > OS-level font-scaling, one mobile-specific and

Re: [blink-dev] Intent to Ship: CSS env variable for OS-level font scale

2025-05-27 Thread Domenic Denicola
Thanks for the detailed response. However, I might not be understanding, so let me try asking the following question. It sounds like the proposal is to ship two opt-ins for adapting to OS-level font-scaling, one mobile-specific and one desktop-specific: - On mobile, authors would opt-in by add

Re: [blink-dev] Intent to Ship: CSS env variable for OS-level font scale

2025-05-27 Thread David Grogan
On Tue, May 27, 2025 at 7:50 AM Mike Taylor wrote: > On 5/27/25 10:46 AM, Torne (Richard Coles) wrote: > > On Tue, 27 May 2025 at 05:48, Peter Birk Pakkenberg > wrote: > >> Hi, >> >> It is not entirely clear if this will also be enabled for WebView as part >> of the Android enablement. >> WebVie

Re: [blink-dev] Intent to Ship: CSS env variable for OS-level font scale

2025-05-27 Thread David Grogan
Hi Peter, Yeah, we call out that WebView situation in the explainer: https://github.com/w3c/csswg-drafts/blob/main/css-env-1/explainers/env-preferred-text-scale.md#note-android-webview We recommend WebView authors whose pages use env(preferred-text-scale) to call setTextZoom(100)

Re: [blink-dev] Intent to Ship: CSS env variable for OS-level font scale

2025-05-27 Thread David Grogan
On Thu, May 22, 2025 at 7:07 PM Domenic Denicola wrote: > > > On Fri, May 23, 2025 at 9:00 AM Chromestatus < > ad...@cr-status.appspotmail.com> wrote: > >> Contact emails dgro...@chromium.org, p...@chromium.org >> >> Explainer https://davidsgrogan.github.io/env-explainer.html >> >> Specification

Re: [blink-dev] Intent to Ship: CSS env variable for OS-level font scale

2025-05-27 Thread Mike Taylor
On 5/27/25 10:46 AM, Torne (Richard Coles) wrote: On Tue, 27 May 2025 at 05:48, Peter Birk Pakkenberg wrote: Hi, It is not entirely clear if this will also be enabled for WebView as part of the Android enablement. WebView has an API to explicitly set the text zoom

Re: [blink-dev] Intent to Ship: CSS env variable for OS-level font scale

2025-05-27 Thread Torne (Richard Coles)
On Tue, 27 May 2025 at 05:48, Peter Birk Pakkenberg wrote: > Hi, > > It is not entirely clear if this will also be enabled for WebView as part > of the Android enablement. > WebView has an API to explicitly set the text zoom >

Re: [blink-dev] Intent to Ship: CSS env variable for OS-level font scale

2025-05-27 Thread Peter Birk Pakkenberg
Hi, It is not entirely clear if this will also be enabled for WebView as part of the Android enablement. WebView has an API to explicitly set the text zoom , which is applied to the FrameWidget here

Re: [blink-dev] Intent to Ship: CSS env variable for OS-level font scale

2025-05-23 Thread 'Philip Rogers' via blink-dev
Thanks Daniel! Responses inline. On Thursday, May 22, 2025 at 9:49:57 PM UTC-7 Daniel Herr wrote: I've skimmed through these docs, and personally it seems like an unnecessary feature. The explainer says a considered alternative is "font-size: medium would always be affected by the OS-level font

Re: [blink-dev] Intent to Ship: CSS env variable for OS-level font scale

2025-05-22 Thread Daniel Herr
I've skimmed through these docs, and personally it seems like an unnecessary feature. The explainer says a considered alternative is "font-size: medium would always be affected by the OS-level font scale." with cons "Sites are NOT built correctly and things that used to just look small would now b

Re: [blink-dev] Intent to Ship: CSS env variable for OS-level font scale

2025-05-22 Thread Domenic Denicola
On Fri, May 23, 2025 at 9:00 AM Chromestatus < ad...@cr-status.appspotmail.com> wrote: > Contact emails dgro...@chromium.org, p...@chromium.org > > Explainer https://davidsgrogan.github.io/env-explainer.html > > Specification https://drafts.csswg.org/css-env-1/#text-zoom > > Summary > > Exposes a

[blink-dev] Intent to Ship: CSS env variable for OS-level font scale

2025-05-22 Thread Chromestatus
Contact emails dgro...@chromium.org, p...@chromium.org Explainer https://davidsgrogan.github.io/env-explainer.html Specification https://drafts.csswg.org/css-env-1/#text-zoom Summary Exposes a user's preferred font scale to CSS. Currently, it is not practical for a page to detect if the use