After changing the display settings, launcher all apps view doesn't cover the entire external display |
||||||||||||||
Issue descriptionChrome OS Version: 10452.30.0, 66.0.3359.67 beta channel kevin, elm What steps will reproduce the problem? (1) Connect external monitor to the device (say HP LP2465) (2) On the external display, open launcher and view all apps screen (3) Goto chrome://settings/display page and change the display arrangement. My arrangement was as shown in attached screenshot. (4) Again on the external display, open launcher and view all apps screen What happens instead? Launcher all apps screen is displaced. Certain regions of external display are not covered by the launcher view. This happens post changing the display settings to any arrangement. Expected behavior? Launcher all apps view must cover the entire external display region. Please refer the attached issue video.
,
Apr 2 2018
,
Apr 4 2018
warx@ please triage
,
Apr 4 2018
weidongg@, please take a look. If it's not a complex fix we should consider merging it. Stable cut is the 12th!
,
Apr 5 2018
,
Apr 5 2018
Sure, the blocking bug 829190 is fixed in ToT, I will work on it.
,
Apr 5 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6e843f40a378754230e7e2dffae361575c81e84b commit 6e843f40a378754230e7e2dffae361575c81e84b Author: Weidong Guo <weidongg@chromium.org> Date: Thu Apr 05 21:50:02 2018 Fix fullscreen app list bounds issue Changes: Use the relative y of work area to display bounds to set fullscreen app list bounds. This fixes the app list bounds in non-primary displays. BUG= 828207 Change-Id: I2bf2ebc49c617240ef9df869020c95996ef18e3f Reviewed-on: https://chromium-review.googlesource.com/998517 Reviewed-by: Yury Khmel <khmel@chromium.org> Commit-Queue: Weidong Guo <weidongg@chromium.org> Cr-Commit-Position: refs/heads/master@{#548568} [modify] https://crrev.com/6e843f40a378754230e7e2dffae361575c81e84b/ui/app_list/views/app_list_view.cc
,
Apr 5 2018
,
Apr 26 2018
Issue 835700 has been merged into this issue.
,
Apr 26 2018
,
May 1 2018
Changing status so hopefully this gets seen.
,
May 1 2018
Have you confirmed changes in ToT/M67?
,
May 1 2018
Yes, I confirm the change is in M67. (67.0.3390.0)
,
May 2 2018
,
May 2 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d732b8ef3697af2d08d8bcecb8f6b67826030c18 commit d732b8ef3697af2d08d8bcecb8f6b67826030c18 Author: Weidong Guo <weidongg@chromium.org> Date: Wed May 02 17:14:43 2018 Fix fullscreen app list bounds issue Changes: Use the relative y of work area to display bounds to set fullscreen app list bounds. This fixes the app list bounds in non-primary displays. BUG= 828207 TBR=khmel@chromium.org Change-Id: I2bf2ebc49c617240ef9df869020c95996ef18e3f Reviewed-on: https://chromium-review.googlesource.com/998517 Reviewed-by: Yury Khmel <khmel@chromium.org> Commit-Queue: Weidong Guo <weidongg@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#548568}(cherry picked from commit 6e843f40a378754230e7e2dffae361575c81e84b) Reviewed-on: https://chromium-review.googlesource.com/1040165 Reviewed-by: Weidong Guo <weidongg@chromium.org> Cr-Commit-Position: refs/branch-heads/3359@{#786} Cr-Branched-From: 66afc5e5d10127546cc4b98b9117aff588b5e66b-refs/heads/master@{#540276} [modify] https://crrev.com/d732b8ef3697af2d08d8bcecb8f6b67826030c18/ui/app_list/views/app_list_view.cc
,
May 2 2018
,
Jul 3
Verified on M67 (10575.58.0, 67.0.3396.99) elm. |
||||||||||||||
►
Sign in to add a comment |
||||||||||||||
Comment 1 by mkarkada@chromium.org
, Apr 2 2018