Re: [blink-dev] Intent to remove: zoom CSS property

2023-07-21 Thread Chris Harrelson
Update: the CSSWG just resolved to specify zoom, so please consider this intent retracted. On Thu, Jun 22, 2023 at 8:10 AM Chris Harrelson wrote: > Hi Robin, > > This intent-to-ship is currently on hold while I investigate

Re: [blink-dev] Intent to remove: zoom CSS property

2023-06-22 Thread Chris Harrelson
Hi Robin, This intent-to-ship is currently on hold while I investigate further the use cases and compatibility risk. On Thu, Jun 22, 2023 at 8:09 AM Robin Haegeman wrote: > What's the latest on this topic? Is this still an ongoing discussion or is > it decided to become deprecated from Chrome 1

Re: [blink-dev] Intent to remove: zoom CSS property

2023-06-22 Thread Robin Haegeman
What's the latest on this topic? Is this still an ongoing discussion or is it decided to become deprecated from Chrome 117 onwards? Our application relies heavily on it so ideally we have at least a couple of months to adapt to this (and have some warnings in the console first a couple of relea

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-24 Thread Malte Nuhn
Not to our knowledge, but we’ll dig deep into this. However, I can confirm that simply using transform: scale is not one of them: I’ve just done this in our app, and immediately run into the same issues we’d seen the last time we tried: scroll performance is awful, there is scroll jumpiness (

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-24 Thread Yoav Weiss
Are there alternative ways to achieve the same effect that don't suffer from blurriness or other UX issues? On Mon, Apr 24, 2023 at 6:25 PM Malte Nuhn wrote: > Similarly, online web design and authoring tools (like Framer, or our OSS > project at Utopia) rely on the zoom property for working whe

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-24 Thread Malte Nuhn
Similarly, online web design and authoring tools (like Framer, or our OSS project at Utopia) rely on the zoom property for working when "zoomed in". In Firefox (w/ scale as fallback) the result is a degraded (eg blurry) experience - sometimes severely so, especially when shadows, serif fonts, a

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-24 Thread Christoph Nakazawa
In a previous response it was stated that the removal of this property leads to only a small amount of code being removed, which I assume also means that there is little impact on reducing complexity in the engine. Maybe I missed it but is there an in-depth explanation of the intention and impa

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-24 Thread Noam Helfman
> > Thanks for sharing Noam, that's good to know! So is Excel Online > unsupported or completely broken for Firefox users then? The feature is disabled for Firefox. Since it represents a very small fraction of our users it is less of a concern. On Mon, Apr 24, 2023 at 5:04 PM Rick Byers wrote:

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-24 Thread Rick Byers
On Mon, Apr 24, 2023 at 9:50 AM Noam Helfman wrote: > I would like to point out that Microsoft Excel Online utilizes zoom CSS > property heavily to perform the Excel grid zoom operations. > Removing it would completely break our zoom functionality in the product > and impact 100s of millions of u

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-24 Thread Noam Helfman
I would like to point out that Microsoft Excel Online utilizes zoom CSS property heavily to perform the Excel grid zoom operations. Removing it would completely break our zoom functionality in the product and impact 100s of millions of users. On Thursday, April 20, 2023 at 10:42:17 PM UTC+3 Ch

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-20 Thread Chris Harrelson
On Thu, Apr 20, 2023 at 12:01 PM Alex Russell wrote: > I agree that this is probably too risky right now. Are you willing to > modify the plan you posted to gate #4 on a UKM analysis and/or driving use > below a negotiated threshold, Chris? I can do the UKM analysis if that's needed. As for thr

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-20 Thread Chris Harrelson
On Thu, Apr 20, 2023 at 12:30 PM Stephen Chenney wrote: > Back to the testing question. There are hundreds of blink web tests that > set zoom via CSS. Most of them are not testing the zoom CSS feature itself > (though some are) so most of them would need to be updated to test browser > zoom situa

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-20 Thread Stephen Chenney
Back to the testing question. There are hundreds of blink web tests that set zoom via CSS. Most of them are not testing the zoom CSS feature itself (though some are) so most of them would need to be updated to test browser zoom situations. It's tractable, particularly if we modify the existing test

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-20 Thread Alex Russell
I agree that this is probably too risky right now. Are you willing to modify the plan you posted to gate #4 on a UKM analysis and/or driving use below a negotiated threshold, Chris? Thanks, Alex On Thursday, April 20, 2023 at 11:15:32 AM UTC-7 Chris Harrelson wrote: > Comments below, but here

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-20 Thread Chris Harrelson
Comments below, but here is a concrete shipping plan proposal: 1. Blog post describing what is happening, why, and how to fix your code. 2. Start a deprecation for 3 milestones (M114-116), with a devtools console warning. Notify enterprises and webview clients of the deprecation. 3. In parallel wi

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-20 Thread Chris Harrelson
On Thu, Apr 20, 2023 at 11:00 AM Jeremy Roman wrote: > How does this removal interact with the browser zoom feature? Today this > seems to affect the CSS zoom property: > > getComputedStyle(document.documentElement).zoom > Browser zoom is implemented using the same machinery as CSS zoom. Removin

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-20 Thread Jeremy Roman
How does this removal interact with the browser zoom feature? Today this seems to affect the CSS zoom property: getComputedStyle(document.documentElement).zoom How much of the code do you get to remove, vs how much is still needed to support browser zoom but will have reduced web exposure? On Th

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-20 Thread Rick Byers
On Wed, Apr 19, 2023 at 6:53 PM Chris Harrelson wrote: > Mike said: *"It would also be good to go through all duplicates and "See > Also" bugs linked at https://bugzilla.mozilla.org/show_bug.cgi?id=390936 > and see how we fare > with a build t

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-19 Thread Chris Harrelson
Mike said: *"It would also be good to go through all duplicates and "See Also" bugs linked at https://bugzilla.mozilla.org/show_bug.cgi?id=390936 and see how we fare with a build that has zoom disabled."* Good idea. I checked all 37 of the site

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-19 Thread Chris Harrelson
On Mon, Apr 17, 2023 at 1:55 PM Morten Stenshorne wrote: > Chris Harrelson writes: > > > On Sun, Apr 16, 2023 at 11:45 PM Morten Stenshorne < > msten...@chromium.org> wrote: > > > > Chris Harrelson writes: > > > > > On Fri, Apr 14, 2023 at 5:09 PM PhistucK wrote: > > > > > > Any alternati

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-19 Thread Chris Harrelson
Hi Christoph, Thanks for the feedback. I left a omment inline below. On Tue, Apr 18, 2023 at 8:19 AM Christoph Nakazawa < christoph.po...@gmail.com> wrote: > Hey everyone, > > I currently depend on the `zoom` CSS property since using `scale` in > conjunction with perspective transforms results i

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-18 Thread Christoph Nakazawa
Hey everyone, I currently depend on the `zoom` CSS property since using `scale` in conjunction with perspective transforms results in a significant color washout in Chrome. My primary goal is to achieve pixel-perfect precision, and I'd be more than willing to transition away from `zoom` if this

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-17 Thread Rick Byers
I'm supportive of aligning with the CSSWG and Firefox here. But this does seem at least a little risky to me, so I think we need to carefully consider the launch plan. First, you'll have a flag so we can kill-switch it if we see any non-trivial breakage in practice, right? WebView seems particular

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-17 Thread Morten Stenshorne
Chris Harrelson writes: > On Sun, Apr 16, 2023 at 11:45 PM Morten Stenshorne > wrote: > > Chris Harrelson writes: > > > On Fri, Apr 14, 2023 at 5:09 PM PhistucK wrote: > > > > Any alternatives? I thought there was a section in the intent templates > for that... > > > > One alternative

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-17 Thread Chris Harrelson
On Sun, Apr 16, 2023 at 11:45 PM Morten Stenshorne wrote: > Chris Harrelson writes: > > > On Fri, Apr 14, 2023 at 5:09 PM PhistucK wrote: > > > > Any alternatives? I thought there was a section in the intent templates > for that... > > > > One alternative for the use case mentioned in my earli

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-16 Thread Morten Stenshorne
Chris Harrelson writes: > On Fri, Apr 14, 2023 at 5:09 PM PhistucK wrote: > > Any alternatives? I thought there was a section in the intent templates for > that... > > One alternative for the use case mentioned in my earlier email is to > apply a CSS transform instead. This will magnify the su

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-16 Thread Yoav Weiss
On Sat, Apr 15, 2023 at 6:29 PM Mike Taylor wrote: > > On 4/14/23 8:03 PM, Chris Harrelson wrote: > > Contact emails chris...@chromium.org > > Specification https://developer.mozilla.org/en-US/docs/Web/CSS/zoom > > Summary > > Removes support for the non-standard "zoom" CSS property. This CSS > p

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-15 Thread Mike Taylor
On 4/14/23 8:03 PM, Chris Harrelson wrote: Contact emails chris...@chromium.org Specification https://developer.mozilla.org/en-US/docs/Web/CSS/zoom Summary Removes support for the non-standard "zoom" CSS property. This CSS property causes computed lengths for an

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-15 Thread PhistucK
Not sure which e-mail you are referring to. I tried transform: scale(100) and I was pleasantly surprised to see that it did not make anything pixelated (I did not try raster image formats, of course, that must be pixelated even with zoom). I was worried it was not a good replacement, but it is for

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-14 Thread Stephen Chenney
I seem to recall tests making use of CSS zoom. Is that not a concern? How do we write tests for issues that only manifest at particular browser zoom levels, because transform behaves differently than zoom (aka device scale factor)? Stephen. On Fri, Apr 14, 2023 at 8:14 PM Chris Harrelson wrote:

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-14 Thread Chris Harrelson
On Fri, Apr 14, 2023 at 5:09 PM PhistucK wrote: > Any alternatives? I thought there was a section in the intent templates > for that... > One alternative for the use case mentioned in my earlier email is to apply a CSS transform instead. This will magnify the subtree visually but not cause a zoo

Re: [blink-dev] Intent to remove: zoom CSS property

2023-04-14 Thread PhistucK
Any alternatives? I thought there was a section in the intent templates for that... ☆*PhistucK* On Sat, Apr 15, 2023 at 1:03 AM Chris Harrelson wrote: > Contact emailschris...@chromium.org > > Specificationhttps://developer.mozilla.org/en-US/docs/Web/CSS/zoom > > Summary > > Removes support fo

[blink-dev] Intent to remove: zoom CSS property

2023-04-14 Thread Chris Harrelson
Contact emailschris...@chromium.org Specificationhttps://developer.mozilla.org/en-US/docs/Web/CSS/zoom Summary Removes support for the non-standard "zoom" CSS property. This CSS property causes computed lengths for an element to be multiplied by the specified zoom factor. Blink componentBlink>