Virtual Keyboard doesn't always show up when tapping the searchbox |
|||||||||
Issue descriptionThe search box view has two parts, the textfield and the white background of the searchbox. When a tap event is received on the textfield the virtual keyboard summons as normal. When a tap event is received on the white background (SearchBoxView::OnGestureEvent()) the virtual keyboard does not show up. Steps to Repro: Tap on the edges of the search box in maximize mode, the keyboard should show up but it does not.
,
Jul 11 2017
Current behavior: v keyboard shows up until the AppListView is dismissed regardless of searchbox focus.
,
Jul 13 2017
Clearing the searchbox should not dismiss the keyboard
,
Jul 26 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ee1e914346a3db5b7e0c59ac5ca31813a4631e84 commit ee1e914346a3db5b7e0c59ac5ca31813a4631e84 Author: Alex Newcomer <newcomer@chromium.org> Date: Wed Jul 26 19:35:43 2017 cros: Virtual Keyboard behavior in the new launcher. When the searchbox is active and the device is in tablet mode the virtual keyboard will be shown. Bug: 740669 Change-Id: I25523b905ced2b5267f08bae56622aac6f8abc18 Reviewed-on: https://chromium-review.googlesource.com/576609 Commit-Queue: Alex Newcomer <newcomer@chromium.org> Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by: Yury Khmel <khmel@chromium.org> Cr-Commit-Position: refs/heads/master@{#489724} [modify] https://crrev.com/ee1e914346a3db5b7e0c59ac5ca31813a4631e84/ui/app_list/BUILD.gn [modify] https://crrev.com/ee1e914346a3db5b7e0c59ac5ca31813a4631e84/ui/app_list/DEPS [modify] https://crrev.com/ee1e914346a3db5b7e0c59ac5ca31813a4631e84/ui/app_list/views/app_list_view.cc [modify] https://crrev.com/ee1e914346a3db5b7e0c59ac5ca31813a4631e84/ui/app_list/views/app_list_view.h [modify] https://crrev.com/ee1e914346a3db5b7e0c59ac5ca31813a4631e84/ui/app_list/views/search_box_view.cc [modify] https://crrev.com/ee1e914346a3db5b7e0c59ac5ca31813a4631e84/ui/app_list/views/search_box_view.h
,
Jul 26 2017
,
Jul 27 2017
,
Jul 27 2017
This bug requires manual review: DEPS changes referenced in bugdroid comments. Please contact the milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), ketakid @(ChromeOS), govind@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 31 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b24785517b115262f7113f09598fbcfef252deaf commit b24785517b115262f7113f09598fbcfef252deaf Author: Maajid <maajid@chromium.org> Date: Mon Jul 31 21:07:13 2017 cros: Virtual Keyboard behavior in the new launcher. When the searchbox is active and the device is in tablet mode the virtual keyboard will be shown. TBR=newcomer@chromium.org (cherry picked from commit ee1e914346a3db5b7e0c59ac5ca31813a4631e84) Bug: 740669 Change-Id: I25523b905ced2b5267f08bae56622aac6f8abc18 Reviewed-on: https://chromium-review.googlesource.com/576609 Commit-Queue: Alex Newcomer <newcomer@chromium.org> Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by: Yury Khmel <khmel@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#489724} Reviewed-on: https://chromium-review.googlesource.com/595031 Reviewed-by: Vadim Tryshev <vadimt@chromium.org> Cr-Commit-Position: refs/branch-heads/3163@{#181} Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528} [modify] https://crrev.com/b24785517b115262f7113f09598fbcfef252deaf/ui/app_list/BUILD.gn [modify] https://crrev.com/b24785517b115262f7113f09598fbcfef252deaf/ui/app_list/DEPS [modify] https://crrev.com/b24785517b115262f7113f09598fbcfef252deaf/ui/app_list/views/app_list_view.cc [modify] https://crrev.com/b24785517b115262f7113f09598fbcfef252deaf/ui/app_list/views/app_list_view.h [modify] https://crrev.com/b24785517b115262f7113f09598fbcfef252deaf/ui/app_list/views/search_box_view.cc [modify] https://crrev.com/b24785517b115262f7113f09598fbcfef252deaf/ui/app_list/views/search_box_view.h
,
Aug 9 2017
Verified on Chrome OS 9824.0.0, 62.0.3176.0 dev build.
,
Sep 5 2017
Approving merge to M61 and M62.
,
Sep 11 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Sep 15 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Dec 17
|
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by newcomer@chromium.org
, Jul 11 2017