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

Issue 780747 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Closed: Oct 10
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Tab-order is wrong on the notifier settings panel

Project Member Reported by yoshiki@chromium.org, Nov 2 2017

Issue description

Google Chrome	64.0.3255.0 (Official Build) canary (64-bit)
Revision	0
Platform	10088.0.0 (Official Build) canary-channel eve

What steps will reproduce the problem?
(1) Open the message center
(2) Open the settings panel in the message center
(3) Press the tab-key

What is the expected result?
The first element gets the focus and the blue focus ring

What happens instead?
The focus ring doesn't appear on any element

Please use labels and text to provide additional information.
It looks hidden elements get the focus at first. After pressing the tab-key 8 times, the do-not-disturb button gets the focus.

 
It seems there is a hidden element between the close button and the do-not-disturb toggle.

> After pressing the tab-key 8 times

But I couldn't reproduce this part. I would investigate further.
Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Nov 6 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/13aed090ed75e1ad8a2698d09319c01cced58a84

commit 13aed090ed75e1ad8a2698d09319c01cced58a84
Author: Tetsui Ohkubo <tetsui@chromium.org>
Date: Mon Nov 06 18:22:31 2017

Adjust notifier settings a11y texts and focus behavior.

This CL adds a11y attributes to some elements in notifier settings view,
and also changes focus behavior of notifier settings to ACCESSIBLE_ONLY.

By these changes,
- ChromeVox now properly reads the a11y text of do-not-disturb toggle.
- Tab focus goes directly from the settings close button to
  the do-not-disturb toggle button.
- ChromeVox reads the contents of the notifier source list.

TEST=manual
BUG= 780747 

Change-Id: If2c97d03b160e981e6e6d68037fa4eade08fc5d6
Reviewed-on: https://chromium-review.googlesource.com/754241
Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org>
Cr-Commit-Position: refs/heads/master@{#514184}
[modify] https://crrev.com/13aed090ed75e1ad8a2698d09319c01cced58a84/ash/message_center/notifier_settings_view.cc
[modify] https://crrev.com/13aed090ed75e1ad8a2698d09319c01cced58a84/ash/message_center/notifier_settings_view.h

Status: Fixed (was: Started)

Comment 5 by tetsui@chromium.org, Nov 20 2017

Status: Assigned (was: Fixed)
Looks happening again.
Cc: yawano@chromium.org
Labels: -Pri-1 -M-64 M-67 Pri-2
It may be fixed already. Tetsui will check.
Status: WontFix (was: Assigned)
N/A for UnifiedSystemTray.

Sign in to add a comment