New issue
Advanced search Search tips

Issue 793094 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 1
Type: Bug

Blocking:
issue 769402



Sign in to add a comment

MD Extensions: A11y: Origin Chip not announced by VoiceOver on Mac

Project Member Reported by hcarmona@chromium.org, Dec 7 2017

Issue description

1. Navigate to chrome://extensions
2. Make sure you have an extension installed from a non-webstore location. (Load unpacked, or by enterprise policy)
3. Use VocieOver to navigate to the chip (see screenshot)

It's expected that voice over will read the label, but voice over just says "Image"
 
unlabelled.png
26.3 KB View Download
Components: -Tests>Missing
Project Member

Comment 2 by bugdroid1@chromium.org, Dec 9 2017

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

commit 14097cda832fbb7a3cf87a4f05cb7d684b49a5dc
Author: Hector Carmona <hcarmona@chromium.org>
Date: Sat Dec 09 00:00:28 2017

MD Extension: A11y - Change aria-labelledby to aria-label for VoiceOver.

The element that was used for the aria-labelledby portion was hidden,
this caused VoiceOver to not read the text.

Bug:  793094 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I4905d677ad5ff7ae42f17890338e31fd8b0b6b14
Reviewed-on: https://chromium-review.googlesource.com/815680
Reviewed-by: Dave Schuyler <dschuyler@chromium.org>
Commit-Queue: Hector Carmona <hcarmona@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522927}
[modify] https://crrev.com/14097cda832fbb7a3cf87a4f05cb7d684b49a5dc/chrome/browser/resources/md_extensions/item.html

Labels: Merge-Request-64
Requesting merge to M64. This is not a complex fix and will address an a11y issue on MacOS. Issue is on canary, no regressions have been found.
Blocking: 769402
Adding blocking tag since this is important
Labels: -Merge-Request-64 Merge-Approved-65
Approving merge to M64. Branch:3282
Labels: -Merge-Approved-65 Merge-Approved-64
Project Member

Comment 7 by bugdroid1@chromium.org, Dec 11 2017

Labels: -merge-approved-64 merge-merged-3282
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/30ac6fa4930b51c6bfa0c15b0a3e82a1f34918ac

commit 30ac6fa4930b51c6bfa0c15b0a3e82a1f34918ac
Author: Hector Carmona <hcarmona@chromium.org>
Date: Mon Dec 11 22:44:29 2017

MD Extension: A11y - Change aria-labelledby to aria-label for VoiceOver.

The element that was used for the aria-labelledby portion was hidden,
this caused VoiceOver to not read the text.

Bug:  793094 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I4905d677ad5ff7ae42f17890338e31fd8b0b6b14
Reviewed-on: https://chromium-review.googlesource.com/815680
Reviewed-by: Dave Schuyler <dschuyler@chromium.org>
Commit-Queue: Hector Carmona <hcarmona@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#522927}(cherry picked from commit 14097cda832fbb7a3cf87a4f05cb7d684b49a5dc)
Reviewed-on: https://chromium-review.googlesource.com/820931
Reviewed-by: Hector Carmona <hcarmona@chromium.org>
Cr-Commit-Position: refs/branch-heads/3282@{#153}
Cr-Branched-From: 5fdc0fab22ce7efd32532ee989b223fa12f8171e-refs/heads/master@{#520840}
[modify] https://crrev.com/30ac6fa4930b51c6bfa0c15b0a3e82a1f34918ac/chrome/browser/resources/md_extensions/item.html

Status: Fixed (was: Started)
Cc: kkaluri@chromium.org
Labels: Needs-Feedback
Tested this issue on Mac 10.12.6 with chrome #64.0.3282.24

Steps followed:
1. Installed unpacked extension
2. Enabled voice over
3. Navigated to chrome://extensions


Observations:
1. On selecting extension icon, voiceover read as "Image"
2. Unable to read the extension Source indicator
3. On inspecting the Extension source indicator, observed the element used as "aria-label"

Attaching the screencast for reference. 

hcarmona@ Could you confirm that this is the expected behaviour of this issue.

Thank You...
793094.mp4
4.1 MB View Download
Icon is still an image, so "image" will also be read, but the bug fix is that it also reads the label on the orange icon. See screencast
Trying upload again
Unpacked Extension Image.mov
3.2 MB Download
Labels: -Needs-Feedback TE-Verified-64.0.3282.24
hcarmona@, thank you for the clarification. It is working as intended on Latest M64#64.0.3202.24 for OS X 10.13.3 Beta.

Sign in to add a comment