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

Issue 739413 link

Starred by 4 users
Status: Archived
Owner:
newcomer@chromium.org
Closed: Jul 2017
Cc:
█ maajid@chromium.org
█ vadimt@chromium.org
weidongg@chromium.org
█ warx@chromium.org
Components:
EstimatedDays: 1
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Adjust AppListView heights for peeking and half states.

Project Member Reported by newcomer@chromium.org, Jul 5 2017 
Peeking should be 320px, half should be 561 px.

Comment 1 by newcomer@chromium.org, Jul 5 2017

Status: Started (was: Available)
Project Member

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

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

commit 3fb847da9e4ea7d740f050b500c8bee941520ffa
Author: newcomer <newcomer@chromium.org>
Date: Tue Jul 11 22:30:38 2017

Fixed bug where the display height was being computed incorrectly, leading to the AppListView being displayed 48 px higher than desired.

BUG= 739413 

Review-Url: https://codereview.chromium.org/2969303002
Cr-Commit-Position: refs/heads/master@{#485698}

[modify] https://crrev.com/3fb847da9e4ea7d740f050b500c8bee941520ffa/ui/app_list/views/app_list_view.cc

Comment 3 by newcomer@chromium.org, Jul 21 2017

Status: Fixed (was: Started)

Comment 4 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)
►

Sign in to add a comment