Autohide Shelf doesn't work for an app launched from overflow menu of Shelf |
||||||||
Issue descriptionChrome Version: 51.0.2704.103, 53.0.2773.0 Chrome OS Version: 8172.60.0, 8481.0.0 Chrome OS Platform: Pixel 2015 How to reproduce. Preparation 1. Sets Shelf position to the left or right (not necessary but useful to create overflow in the next step) Preparation 2. Pins a lot of apps to Shelf until overflow menu appears on Shelf. Step 1. Enable Autohide shelf. Step 2. Launch an app from overflow menu of Shelf (See 2nd screenshot). Expected behavior: After step 2, Shelf hides automatically and the app is maximized. Observed behavior: After step 2, The app is maximized but Shelf remains on the screen (See 3rd attached screenshot). This issue happens only from overflow. Autohide Shelf works fine for an app on Shelf itself.
,
Aug 8 2016
,
Aug 9 2016
Note that this issue happens for all type of apps, not specific to ARC apps.
,
Feb 21 2017
,
Feb 28 2017
,
Apr 6 2017
I can look at this one, Zach.
,
Apr 6 2017
,
Apr 7 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/4242daf2b69dfc2c76b92adeeeb5d7efc756d59f commit 4242daf2b69dfc2c76b92adeeeb5d7efc756d59f Author: sammiequon <sammiequon@chromium.org> Date: Fri Apr 07 15:42:47 2017 Shelf: Hide overflow bubble when app launched from overflow. When the shelf is in autohide, the shelf does not hide after launching from overflow shelf. This is because the overflow shelf remains visible after lauching an app, which this CL fixes. BUG= 625924 TEST=manual,ash_unittests --gtest_filter="ShelfViewTest.*" Review-Url: https://codereview.chromium.org/2807473003 Cr-Commit-Position: refs/heads/master@{#462869} [modify] https://crrev.com/4242daf2b69dfc2c76b92adeeeb5d7efc756d59f/ash/common/shelf/shelf_view.cc [modify] https://crrev.com/4242daf2b69dfc2c76b92adeeeb5d7efc756d59f/ash/shelf/shelf_view_unittest.cc
,
Apr 10 2017
,
Apr 24 2017
Chrome OS 9460.5.0/59.0.3071.15 |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by takaoka@chromium.org
, Jul 6 2016