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

Issue 881039 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Sep 7
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Fix autocomplete interaction with left/right keys.

Project Member Reported by oxyflush@chromium.org, Sep 5

Issue description

Autocomplete feature currently resets text after each arrow keypress, regardless of if there is a suggestion or not, causing unwanted interactions.

Fix: Handle arrow keys only if there is currently autocomplete text in AppList search box.
 
Labels: -M-70 M-71
Labels: M-70-Bug-Bash
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 7

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

commit 81ac5803ecec7d4b1181a37a1cb83f8c07109d3d
Author: Daniel Zhang <oxyflush@google.com>
Date: Fri Sep 07 18:05:55 2018

cros: Fix autocomplete interaction with arrow keys

Currently, AppList autocomplete feature handles arrow
key events regardless of whether or not there is
autocomplete text present, causing unwanted
interactions.

Change autocomplete so that it only handles arrow keys
when autocomplete text is present.

Bug:  881039 
Change-Id: I7041646e87e5cd4e95d6299017fa2259899f8994
Reviewed-on: https://chromium-review.googlesource.com/1208718
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Reviewed-by: Alex Newcomer <newcomer@chromium.org>
Commit-Queue: Daniel Zhang <oxyflush@google.com>
Cr-Commit-Position: refs/heads/master@{#589582}
[modify] https://crrev.com/81ac5803ecec7d4b1181a37a1cb83f8c07109d3d/ash/app_list/views/search_box_view.cc
[modify] https://crrev.com/81ac5803ecec7d4b1181a37a1cb83f8c07109d3d/ash/app_list/views/search_box_view.h

Status: Fixed (was: Started)

Sign in to add a comment