New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 759494 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression : Focus doesn't traverse in chrome://settings/changePicture page on using arrow keys after discarding photo

Project Member Reported by mmanchala@chromium.org, Aug 28 2017

Issue description

Chrome Version: 62.0.3197.0/9885.0.0 dev-channel Reks,Candy and Peppy
OS: Chrome

What steps will reproduce the problem?
(1)Sign into User -> Go to chrome://settings/changePicture page
(2)Now click on 'Take photo' option -> Capture it -> Click on 'Discard photo'
(3)Now focus is on 'Google profile photo' option -> Use Up/Down/Right/left arrow buttons from keyboard and observe Focus doesn't traverse

Expected: Focus should traverse to other pictures on using arrow keys
Actual: Instead Focus doesn't traverse on using arrow keys

This is Regression Issue as same is working fine in old chrome://settings

@stevenjb : Please confirm the Issue
 
Actual_Focus.mp4
15.3 MB Download
Labels: -Type-Bug -Pri-3 ReleaseBlock-Stable M-62 Proj-MaterialDesign-WebUI OS-Chrome Pri-1 Type-Bug-Regression
Summary: Regression : Focus doesn't traverse in chrome://settings/changePicture page on using arrow keys after discarding photo (was: Regression : )
Owner: hcarmona@chromium.org
I believe this should be fixed with the recent a11y fixes?

Cc: steve...@chromium.org reve...@chromium.org
Labels: -Pri-1 Pri-2
Just noticed the "after discarding photo" part; lowering priority, but we should make sure that we do the right thing here.

Status: Fixed (was: Assigned)
Not able to reproduce after fixing a11y issue. Marking as fixed by for for Issue 753555
Labels: Merge-TBD
[Auto-generated comment by a script] We noticed that this issue is targeted for M-62; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-62 label, otherwise remove Merge-TBD label. Thanks.
Labels: -Merge-TBD
The CL references in the issue landed 2 days ago, no need to merge.

Sign in to add a comment