New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 773785 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

App folder icon does not disapper when going back to PEEKING mode.

Project Member Reported by weidongg@chromium.org, Oct 11 2017

Issue description

What steps will reproduce the problem?
(1) Open app list and go to FULLSCREEN ALL APPs state
(2) Open a folder
(3) Scroll to PEEKING state

What is the expected result?
App folder icon disappears
What happens instead?
App folder icon still shows.

 
Labels: M-63
Owner: newcomer@chromium.org
Taking it!
Project Member

Comment 2 by bugdroid1@chromium.org, Oct 11 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4dd2acbb826afea00dfcb339926ead7720d482ff

commit 4dd2acbb826afea00dfcb339926ead7720d482ff
Author: Alex Newcomer <newcomer@chromium.org>
Date: Wed Oct 11 22:02:31 2017

cros fixes ui artifacts after going to PEEKING

Entering folder view then going to peeking used to
go into an undesired state, this was fixed in 767644.
This fix revealed a bug in how the AppListView was
resetting the folder view.

This CL contains a fix and a test.

Bug:  773785 
Change-Id: I2b36f4a4020d984334d8ac6749d67577caede8b7
Reviewed-on: https://chromium-review.googlesource.com/714120
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Alex Newcomer <newcomer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#508138}
[modify] https://crrev.com/4dd2acbb826afea00dfcb339926ead7720d482ff/ui/app_list/views/app_list_view.cc
[modify] https://crrev.com/4dd2acbb826afea00dfcb339926ead7720d482ff/ui/app_list/views/app_list_view_unittest.cc

Comment 3 by vadimt@chromium.org, Oct 14 2017

Status: Fixed (was: Untriaged)

Sign in to add a comment