[NTP] Unnecessary 'Remove' button appears selected even when focus is shifted.
Reported by
rp...@etouch.net,
Aug 16
|
|||||||||||
Issue descriptionChrome Version: 70.0.3524.0 (Official Build)Revision af89913d666a6f95572c04f15ab3a0752343ebf3-refs/branch-heads/3524@{#1}(32/64-bit) OS: Windows(7,8,8.1,10), Mac(10.12.6, 10.13.1, 10.14, 10.13.6) and Linux(14.04 LTs). Pre-condition: Enable "Enable using the Google local NTP" and "New Tab Page Custom Links" flags from chrome://flags. What steps will reproduce the problem? 1. Launch Chrome and navigate to NTP. 2. Click on 'Add shortcut' iron icon on chrome webstore thumbnail to open 'Add shortcut' overlay. 3. Now press 'Tab' key to bring focus on 'Remove' button and press and hold 'Space bar' and then press 'Tab' key to move focus,observe 'Remove' button Actual: Unnecessary 'Remove' button appears selected even when focus is shifted. Expected: 'Remove' button shouldn't appear selected when focus is shifted. This is a non-regression issue, seen from M-70 series as 'Add shortcut' iron icon is introduced from build #70.0.3508.0 NOTE: This issue is also reproducible on Dev #70.0.3521.2 Kindly review the attached screen-cast for reference. Thank you.
,
Aug 16
This is super low priority but Weilun, would you mind taking a look. Basically if you hold space the CSS gets applied and never gets removed.
,
Aug 17
Could you verify the issue still exists? I can not reproduce the same problem in my Linux machine as well as MacOS Screencast: https://screencast.googleplex.com/cast/NjY1NTM2ODAwMDE3NjEyOHw4NWJlM2NjZC04Zg
,
Aug 21
,
Aug 28
,
Aug 29
With response to comment #3 and #5 : Rechecked this issue on Windows(7,8,8.1,10), Mac(10.12.6, 10.13.1, 10.14, 10.13.6) and Linux(14.04 LTs) using latest Canary chrome version : 70.0.3535.0 the issue is still reproducible.Kindly refer the attached screen cast for your reference. Thank you..!!
,
Aug 29
,
Sep 6
This is the default behavior of the space bar on a button. This is not only for the Remove button but all the other buttons. One way to fix this is to make all the button to listen on the keydown event. But I don't think it's worth to do that for such obscure case. I would like to leave it as it be. wdyt?
,
Sep 10
,
Nov 14
,
Dec 10
,
Jan 8
The NextAction date has arrived: 2019-01-08
,
Jan 8
|
|||||||||||
►
Sign in to add a comment |
|||||||||||
Comment 1 Deleted