Issue metadata
Sign in to add a comment
|
Regression: Tab focus is not seen on options under 'View site info' overlay.
Reported by
rk...@etouch.net,
May 27 2016
|
||||||||||||||||||||||
Issue descriptionChrome Version: 53.0.2750.0 Revision c84885e83ecac137b93ed2b6c8b7323d8e91ffe1-refs/heads/master@{#396337} OS: Windows(7,8,10) What steps will reproduce the problem? (1) Launch chrome, navigate to google.com (2) Click on 'View site info' icon and press Tab key, observe focus traverse. Tab focus is not seen on option under 'Permissions' Tab focus should be seen on option under 'Permissions' This is a regression issue, broken in 'M-53', will soon update the other info: Good Build: 53.0.2749.0 Bad Build: 53.0.2750.0
,
May 27 2016
This UI is not a part of Blink. Forwarding to permissions UI.
,
May 27 2016
Above issue is also seen Linux OS.
,
May 27 2016
Marking the above issue as RB-Stable as this is a recent regression. Thank you!
,
May 27 2016
will be a lot easier to fix when https://codereview.chromium.org/2011963002/ lands --- we can just switch over to using comboboxes for the MD version.
,
Jun 15 2016
Issue 618182 has been merged into this issue.
,
Jun 15 2016
,
Jun 20 2016
Just to update, Still able to reproduce the issue on windows 7 using chrome version 53.0.2773.0. estade@ Gentle Ping !
,
Jun 20 2016
I'm working on it, see above link
,
Jun 22 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/399800ecfbe6b923f0a2c64bd83d21d50a97d399 commit 399800ecfbe6b923f0a2c64bd83d21d50a97d399 Author: estade <estade@chromium.org> Date: Wed Jun 22 16:29:22 2016 MD - Use real comboboxes in website settings popup. In MD, 1. Hide borders (but leave focus border intact). 2. Move arrow closer to text instead of sizing based on widest option text. Also fix bug where button focus borders depended on the --top-chrome-md flag even if they were in secondary UI. Thus with top chrome md enabled and secondary chrome md disabled, we get the normal pre-MD focus rects. BUG= 615308 Review-Url: https://codereview.chromium.org/2069733002 Cr-Commit-Position: refs/heads/master@{#401301} [modify] https://crrev.com/399800ecfbe6b923f0a2c64bd83d21d50a97d399/chrome/browser/ui/views/bookmarks/bookmark_bar_view.cc [modify] https://crrev.com/399800ecfbe6b923f0a2c64bd83d21d50a97d399/chrome/browser/ui/views/toolbar/app_menu_button.cc [modify] https://crrev.com/399800ecfbe6b923f0a2c64bd83d21d50a97d399/chrome/browser/ui/views/toolbar/toolbar_action_view.cc [modify] https://crrev.com/399800ecfbe6b923f0a2c64bd83d21d50a97d399/chrome/browser/ui/views/website_settings/permission_selector_view.cc [modify] https://crrev.com/399800ecfbe6b923f0a2c64bd83d21d50a97d399/chrome/browser/ui/views/website_settings/permissions_bubble_view.cc [modify] https://crrev.com/399800ecfbe6b923f0a2c64bd83d21d50a97d399/ui/views/controls/button/blue_button_unittest.cc [modify] https://crrev.com/399800ecfbe6b923f0a2c64bd83d21d50a97d399/ui/views/controls/button/label_button.cc [modify] https://crrev.com/399800ecfbe6b923f0a2c64bd83d21d50a97d399/ui/views/controls/combobox/combobox.cc [modify] https://crrev.com/399800ecfbe6b923f0a2c64bd83d21d50a97d399/ui/views/controls/combobox/combobox.h
,
Jun 22 2016
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by rk...@etouch.net
, May 27 2016Status: Assigned (was: Unconfirmed)