Issue metadata
Sign in to add a comment
|
3072 x 1728 Resolution lost on 4K Screens as 125 % Zoom is missing
Reported by
norb...@rittel.de,
Apr 27 2018
|
||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (X11; CrOS x86_64 10575.17.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/67.0.3396.19 Safari/537.36 Platform: 10575.17.0 (Official Build) dev-channel panther Steps to reproduce the problem: With 67.0.3396.19 the so far optional chrome://flags/#enable-display-zoom-setting has been enabled by default. This replaces the old display resolution settings with the new display zoom setting. With a 4K monitor attached this makes the user loose the important 125% zoom level. What is the expected behavior? With the old display settings UI you were able to set a 4K screen to be used with a target resolution of 3072 x 1728 while still sending the native 3840 x 2160 to the monitor. This translates to a zoom factor of 125 % (3840/3072 = 1,25) This factor should be available in the new UI, too. What went wrong? The new display zoom settings UI immediately jumps from 100% to 150% (previously 2560 x 1440) which is way too high for normal usage. While at 100% the screen elements tend to be much too small on a typical 4K display (28 inch in my case), a zoom level of 150% already wastes way too much screen estate. 125% used to be the sweet spot here over and should not be lost in this makeover. Did this work before? Yes 67.0.3396.17 Chrome version: 67.0.3396.19 Channel: dev OS Version: 10575.17.0 Flash Version: I can't test this here but I assume this now also happens with (built-in) HD screens (1920 x 1080) which you could drive at 1536 x 864 (same 125% zoom level). This also proved to be the sweet spot on 14 to 15 inch displays (same math as for 28 to 30 inch at 4K).
,
Apr 27 2018
,
Apr 27 2018
Just saw a public G+ post about this as well. How hard would it be to re-add the 125% step? (I get that figuring out what the steps should be is tough)
,
Apr 27 2018
Its only a couple of line change which is already under review: https://chromium-review.googlesource.com/c/chromium/src/+/1033587 New zoom levels for 4K are: 3840: {100, 125, 150, 175, 200, 225, 250, 275, 300}
,
Apr 27 2018
Awesome! Thanks.
,
May 9 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/488aefc58bccc019443fe17fe2de4b4816f711c5 commit 488aefc58bccc019443fe17fe2de4b4816f711c5 Author: Malay Keshav <malaykeshav@chromium.org> Date: Wed May 09 04:39:29 2018 Soften the display zoom jump amount for higher resolution displays The current logic to compute list of zoom levels for a display leads to very big jumps on 4k or 8k displays. This patch reduces the jump interval for higher resolution displays. This patch also ensures that the inverse of the internal device scale factor is present in the list of zoom levels. This ensures that the user can reach the native resolution on the internal display. This patch also changes the format in which we persistently store the zoom level from an int to double. This is necessary because the inverse dsf that we introduced can be an irrational number and we do not want to lose precision due to multiple store -> retrieve -> store operations. Bug: 837512 Change-Id: If6870153f203f218ad52eafcbf1175ed588e0783 Component: Display zoom, settings slider, display pref Reviewed-on: https://chromium-review.googlesource.com/1033587 Commit-Queue: Malay Keshav <malaykeshav@chromium.org> Reviewed-by: Ahmed Fakhry <afakhry@chromium.org> Cr-Commit-Position: refs/heads/master@{#557093} [modify] https://crrev.com/488aefc58bccc019443fe17fe2de4b4816f711c5/ash/display/cros_display_config.cc [modify] https://crrev.com/488aefc58bccc019443fe17fe2de4b4816f711c5/ash/display/display_manager_unittest.cc [modify] https://crrev.com/488aefc58bccc019443fe17fe2de4b4816f711c5/ash/display/display_prefs.cc [modify] https://crrev.com/488aefc58bccc019443fe17fe2de4b4816f711c5/ash/display/display_prefs_unittest.cc [modify] https://crrev.com/488aefc58bccc019443fe17fe2de4b4816f711c5/ui/display/manager/display_manager.cc [modify] https://crrev.com/488aefc58bccc019443fe17fe2de4b4816f711c5/ui/display/manager/display_util.cc [modify] https://crrev.com/488aefc58bccc019443fe17fe2de4b4816f711c5/ui/display/manager/display_util.h [modify] https://crrev.com/488aefc58bccc019443fe17fe2de4b4816f711c5/ui/display/manager/display_utils_unittest.cc
,
May 9 2018
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by tbuck...@chromium.org
, Apr 27 2018Components: -UI UI>Shell>MultipleMonitor
Owner: ovanieva@chromium.org
Status: Assigned (was: Unconfirmed)