New issue
Advanced search Search tips

Issue 886980 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 18
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug
Team-Accessibility

Blocked on:
issue 833348


Show other hotlists

Hotlists containing this issue:
Launcher-A11y


Sign in to add a comment

Launcher search doesn't respect docked magnifier, top is cut off

Project Member Reported by leberly@chromium.org, Sep 19

Issue description

Google Chrome	70.0.3538.16 (Official Build) dev (64-bit)
Firmware Version Google_Caroline.7820.384.0

Steps to repro:
# Launch docked magnifier either through Settings > Manage Accessibility Features or with search + ctrl + d.
# Open the Launcher to "peeking" view where you see the search field 
# Type in any search query, I used the word "testing"
Expected: able to see the top of the launcher as it opens more to show results
Actual: launcher opens "under" the docked magnifier, top is cut off

 
testing.png
380 KB View Download
Blockedon: 833348
Adding "blocked on" to the master bug (UI areas are too separate to dupe all into one bug)
Labels: M-72
Thanks! We will try to comply with this by M-72.
Labels: a11y-Launcher
Labels: -M-72 -m-72 M-73
Bulk moving <p-1's to the next milestone because we branched to M-73.
Owner: weidongg@chromium.org
Labels: -M-73 M-74
Project Member

Comment 7 by bugdroid1@chromium.org, Dec 18

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

commit 396a729a99562256bdffdad2f17da782e7ef6bcc
Author: weidongg@chromium.org <weidongg@chromium.org>
Date: Tue Dec 18 07:52:14 2018

Fix opened search box issue with docked magnifier

Opened search box from peeking state does not respect docked magnifer,
so change the height of the app list to make search box fit in work
area.

Bug:  886980 
Change-Id: I814bad77031767e29263a043083c1c76f6a20e8e
Reviewed-on: https://chromium-review.googlesource.com/c/1379595
Commit-Queue: Weidong Guo <weidongg@chromium.org>
Reviewed-by: Alex Newcomer <newcomer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#617406}
[modify] https://crrev.com/396a729a99562256bdffdad2f17da782e7ef6bcc/ash/app_list/app_list_presenter_delegate_unittest.cc
[modify] https://crrev.com/396a729a99562256bdffdad2f17da782e7ef6bcc/ash/app_list/views/app_list_view.cc

Status: Fixed (was: Available)

Sign in to add a comment