New issue
Advanced search Search tips

Issue 911726 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 13
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug
Team-Accessibility



Sign in to add a comment

ChromeVox: tapping on an ARIA button highlights text, not the button

Project Member Reported by dmazz...@chromium.org, Dec 4

Issue description

Internal Google bug affecting CfM / Meet: b/118257269

If you have an ARIA button like this:

  <div tabIndex="0" role="button">
    <p>
      Button Text
    </p>
  </div>

Tapping on the button text selects the text, but it should select the button.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Dec 6

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

commit 672f22463f1d416359f1db16e11ab6fe23accc88
Author: Dominic Mazzoni <dmazzoni@chromium.org>
Date: Thu Dec 06 01:15:57 2018

When ChromeVox gets a hover event, it should target a leaf node.

In particular, the hover might be fired on a text node inside something
like a button. The node that should be focused by ChromeVox is the
button, not the text inside.

Bug:  911726 
Change-Id: Ibdbdbe5991e693bd8bbc575ab3e51e7a0fed3e07
Reviewed-on: https://chromium-review.googlesource.com/c/1361622
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: David Tseng <dtseng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#614215}
[modify] https://crrev.com/672f22463f1d416359f1db16e11ab6fe23accc88/chrome/browser/resources/chromeos/chromevox/cvox2/background/automation_predicate.js
[modify] https://crrev.com/672f22463f1d416359f1db16e11ab6fe23accc88/chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs
[modify] https://crrev.com/672f22463f1d416359f1db16e11ab6fe23accc88/chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.js

Labels: -Pri-3 ReleaseBlock-Stable Merge-Request-72 OS-Chrome Pri-1
b/118257269 seems to be fixed. The CfM team would like this in 72 if possible.


Components: UI>Accessibility>ChromeVox
Project Member

Comment 4 by sheriffbot@chromium.org, Dec 8

Labels: -Merge-Request-72 Hotlist-Merge-Approved Merge-Approved-72
Your change meets the bar and is auto-approved for M72. Please go ahead and merge the CL to branch 3626 manually. Please contact milestone owner if you have questions.
Owners: govind@(Android), kariahda@(iOS), djmm@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 5 by sheriffbot@chromium.org, Dec 12

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Fixed (was: Started)
Project Member

Comment 7 by bugdroid1@chromium.org, Dec 13

Labels: -merge-approved-72 merge-merged-3626
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/745deb1e5599ab79a19e89776fa495ae9ee74c3a

commit 745deb1e5599ab79a19e89776fa495ae9ee74c3a
Author: Dominic Mazzoni <dmazzoni@chromium.org>
Date: Thu Dec 13 17:56:30 2018

When ChromeVox gets a hover event, it should target a leaf node.

In particular, the hover might be fired on a text node inside something
like a button. The node that should be focused by ChromeVox is the
button, not the text inside.

Bug:  911726 
Change-Id: Ibdbdbe5991e693bd8bbc575ab3e51e7a0fed3e07
Reviewed-on: https://chromium-review.googlesource.com/c/1361622
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: David Tseng <dtseng@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#614215}(cherry picked from commit 672f22463f1d416359f1db16e11ab6fe23accc88)
Reviewed-on: https://chromium-review.googlesource.com/c/1376309
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/branch-heads/3626@{#328}
Cr-Branched-From: d897fb137fbaaa9355c0c93124cc048824eb1e65-refs/heads/master@{#612437}
[modify] https://crrev.com/745deb1e5599ab79a19e89776fa495ae9ee74c3a/chrome/browser/resources/chromeos/chromevox/cvox2/background/automation_predicate.js
[modify] https://crrev.com/745deb1e5599ab79a19e89776fa495ae9ee74c3a/chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs
[modify] https://crrev.com/745deb1e5599ab79a19e89776fa495ae9ee74c3a/chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.js

Labels: Merge-Merged-72-3626
The following revision refers to this bug: 
https://chromium.googlesource.com/chromium/src.git/+/745deb1e5599ab79a19e89776fa495ae9ee74c3a

Commit: 745deb1e5599ab79a19e89776fa495ae9ee74c3a
Author: dmazzoni@chromium.org
Commiter: dmazzoni@chromium.org
Date: 2018-12-13 17:56:30 +0000 UTC

When ChromeVox gets a hover event, it should target a leaf node.

In particular, the hover might be fired on a text node inside something
like a button. The node that should be focused by ChromeVox is the
button, not the text inside.

Bug:  911726 
Change-Id: Ibdbdbe5991e693bd8bbc575ab3e51e7a0fed3e07
Reviewed-on: https://chromium-review.googlesource.com/c/1361622
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: David Tseng <dtseng@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#614215}(cherry picked from commit 672f22463f1d416359f1db16e11ab6fe23accc88)
Reviewed-on: https://chromium-review.googlesource.com/c/1376309
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/branch-heads/3626@{#328}
Cr-Branched-From: d897fb137fbaaa9355c0c93124cc048824eb1e65-refs/heads/master@{#612437}

Sign in to add a comment