Issue metadata
Sign in to add a comment
|
Regression: Auto-scrolling is seen missing in Keyboard/Network list of Uber tray on using tab key . |
||||||||||||||||||||||
Issue descriptionVersion: 57.0.2926.0/9014.0.0 dev-channel Paine, Falco, Gnawty OS:Chrome. Pre-condition:Select any uber tray option which has long list such that scrollbar will be seen. What steps will reproduce the problem? (1)Recover build via USB/Sign in to chrome>> Go to Uber tray and select keyboard/Network option (2)Now hit tab till you reach to the bottom option and Observe (Please refer video) Expected: As you reach to the bottom options keyboard/network list should be auto-scrolled accordingly. Actual: Instead auto-scrolling is seen missing. (Only focus is getting shifted but options are not seen until you manually scroll using touch-pad/by dragging the scrollbar.) This is regression issue as it is working fine in 54.0.2840.101/8743.85.0 stable channel butterfly Attaching screen-cast for reference.
,
Nov 23 2016
,
Nov 23 2016
,
Nov 29 2016
Not blink related Not convinced this should be a release blocker
,
Nov 29 2016
Agreed this is not a release blocker. I suspect this actually regressed in M-55 when the opt-in IME menu refactoring took place; on canary I do not see the same issue on other detailed views (the network detailed view, for instance). azurewei@, can you please take a look? Let's consider for a merge back into M-56 if the fix is low-risk.
,
Nov 30 2016
I wonder if it relates to the shadow at top of TrayDetailsView when scrolled. It seems not related to opt-in IME menu refactoring since network list also repo the bug. Let me debug and find out what goes wrong.
,
Dec 1 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/c5f7e4111cf2110cc972a4d7bcb8f90651dfcae6 commit c5f7e4111cf2110cc972a4d7bcb8f90651dfcae6 Author: azurewei <azurewei@chromium.org> Date: Thu Dec 01 00:34:36 2016 Scroll IME list to visible when using tab key. When switches the focused IME item with Tab key, scroll the IME detailed view to make the focused item visible. BUG= 668035 TEST=Verified on local build. Review-Url: https://codereview.chromium.org/2537893003 Cr-Commit-Position: refs/heads/master@{#435494} [modify] https://crrev.com/c5f7e4111cf2110cc972a4d7bcb8f90651dfcae6/ash/common/system/chromeos/ime_menu/ime_list_view.cc
,
Dec 1 2016
,
Dec 1 2016
,
Dec 1 2016
,
Dec 1 2016
Your change meets the bar and is auto-approved for M56 (branch: 2924)
,
Dec 2 2016
,
Dec 2 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e4756740d27df15f3035530810333aaca3d8e9db commit e4756740d27df15f3035530810333aaca3d8e9db Author: azurewei <azurewei@chromium.org> Date: Fri Dec 02 03:20:38 2016 Scroll IME list to visible when using tab key. When switches the focused IME item with Tab key, scroll the IME detailed view to make the focused item visible. TBR=tdanderson@chromium.org NOPRESUBMIT=true NOTRY=true BUG= 668035 TEST=Verified on local build. Review-Url: https://codereview.chromium.org/2537893003 Cr-Commit-Position: refs/heads/master@{#435494} (cherry picked from commit c5f7e4111cf2110cc972a4d7bcb8f90651dfcae6) Review-Url: https://codereview.chromium.org/2549673003 Cr-Commit-Position: refs/branch-heads/2924@{#282} Cr-Branched-From: 3a87aecc31cd1ffe751dd72c04e5a96a1fc8108a-refs/heads/master@{#433059} [modify] https://crrev.com/e4756740d27df15f3035530810333aaca3d8e9db/ash/common/system/chromeos/ime_menu/ime_list_view.cc
,
Dec 15 2016
Verified on 56.0.2924.28/9000.29.0 |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by jbanavatu@chromium.org
, Nov 23 2016