'Radio button' doesn't get selected in chrome://md-settings page.
Reported by
mni...@etouch.net,
Aug 3 2016
|
|||||
Issue descriptionVersion: 53.0.2785.45 70a7d200b65a4bc8fa957bfa8a8ad8bc8f429bdb-refs/branch-heads/2785@{#477} (32/64-bit) OS: Windows (7,8,8.1,10),Linux (14.04 LTS),Mac OS X(10.10.5, 10.11.4) What steps will reproduce the problem? 1) Launch chrome and navigate to chrome://md-settings/ 2) Now scroll down the page and check 'Show home button' under 'Appearance' 3) Now press 'Tab' key from keyboard so as focus stays on 'Other' text box,observe the 'Radio button' Actual : On pressing 'Tab' key from keyboard focus moves to 'Text box' but 'Radio button' doesn't get selected Expected : On pressing 'Tab' key from keyboard focus moves to 'Text box' at the same time 'Radio button' should get selected. This is a non-regression issue, seen from M-47 series as 'Other Text box' under 'Show home button' appears from build # 47.0.2518.0 Kindly review the attached video and screenshot for reference.
,
Aug 10 2016
,
Oct 7 2016
,
Oct 8 2016
Correct behavior is for the text box to not be in the tab order unless "other" has been selected. Old settings would disable the text box, but that looks a bit odd in MD. bettes@ please comment if you would prefer different behavior here.
,
Oct 12 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/046c41c1e58ebe49924a2628dc2cb0a3eee9fbd0 commit 046c41c1e58ebe49924a2628dc2cb0a3eee9fbd0 Author: hcarmona <hcarmona@chromium.org> Date: Wed Oct 12 15:39:03 2016 Skip over the URL input when the NTP is used for the Home Button. BUG= 633915 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2403623002 Cr-Commit-Position: refs/heads/master@{#424745} [modify] https://crrev.com/046c41c1e58ebe49924a2628dc2cb0a3eee9fbd0/chrome/browser/resources/settings/appearance_page/appearance_page.html [modify] https://crrev.com/046c41c1e58ebe49924a2628dc2cb0a3eee9fbd0/chrome/browser/resources/settings/controls/settings_input.html [modify] https://crrev.com/046c41c1e58ebe49924a2628dc2cb0a3eee9fbd0/chrome/browser/resources/settings/controls/settings_input.js
,
Oct 12 2016
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by ellyjo...@chromium.org
, Aug 4 2016Status: Untriaged (was: Unconfirmed)