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

Issue 786213 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Launcher: Results view is misaligned with search bar and overlaps with shelf

Project Member Reported by sdantul...@chromium.org, Nov 17 2017

Issue description

Google Chrome	64.0.3270.0 (Official Build) dev (64-bit)
Revision	0
Platform	10134.0.0 (Official Build) dev-channel eve

What steps will reproduce the problem?
(1) Open launcher
(2) Enter some text in search bar

What happens ?
Results view is misaligned with search bar and overlaps with shelf

Attached screenshot.
 
Screenshot 2017-11-16 at 4.31.06 PM.png
1.5 MB View Download
Labels: M-64

Comment 2 by vadimt@chromium.org, Nov 17 2017

Labels: Touch-Friendly-Launcher Touch-Friendly-Launcher-Triaged
Owner: newcomer@chromium.org
Please add affected OSs.
Labels: OS-Chrome

Comment 5 by vadimt@chromium.org, Nov 17 2017

Cc: krajshree@chromium.org brajkumar@chromium.org ajha@chromium.org
 Issue 786279  has been merged into this issue.
Project Member

Comment 6 by bugdroid1@chromium.org, Nov 20 2017

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

commit 89c4d0bd82a12f4f0f99a0e3808451e45a384fc6
Author: Alex Newcomer <newcomer@chromium.org>
Date: Mon Nov 20 23:02:23 2017

cros: fix SearchResultPageView's page bounds

During refactoring works to remove the fullscreen launcher flags,
the wrong block was removed in SearchResultPageView::GetPageBoundsForState

This CL restores the correct block of code, and fixes the page bounds
bug.

Bug:  786213 
Change-Id: Ib9c81ba41fbc1280fa4135ba3ddc5f705170bb7a
Reviewed-on: https://chromium-review.googlesource.com/780042
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Alex Newcomer <newcomer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517973}
[modify] https://crrev.com/89c4d0bd82a12f4f0f99a0e3808451e45a384fc6/ui/app_list/views/search_result_page_view.cc

Status: Fixed (was: Untriaged)
Status: Verified (was: Fixed)
10165.0.0, 64.0.3274.0

Sign in to add a comment