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

Issue 757592 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug
Team-Accessibility



Sign in to add a comment

Not all objects that are clickable are marked as such in the a11y tree

Project Member Reported by nek...@chromium.org, Aug 21 2017

Issue description

Steps to reproduce:
(This is one example but there are more.)
data:text/html,<details><summary>summary</summary>details</details>

Cause:
The following roles should be marked as clickable and should return themselves as the |ActionElement| in AXNodeObject:
searchbox, switch, combobox, menu list option, disclosure triangle.

Actual:
Calling IAccessible::DoDefaultAction on any of the above elements doesn't work.

Expected:
Calling IAccessible::DoDefaultAction should click the element.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 23 2017

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

commit 3c944b9a8936705ca717477395531942ca02f641
Author: Nektarios Paisios <nektar@chromium.org>
Date: Wed Aug 23 23:03:59 2017

Simplified logic in Blink that detects clickable elements and added more roles that should be marked as clickable.

Bug:  757592 
R=dmazzoni@chromium.org
TESTED=Manually with some roles, Layout test

Change-Id: Iae296aebc5615d3fb798b282b44acf16f01c12ae
Reviewed-on: https://chromium-review.googlesource.com/624835
Commit-Queue: Nektarios Paisios <nektar@chromium.org>
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#496848}
[modify] https://crrev.com/3c944b9a8936705ca717477395531942ca02f641/content/test/data/accessibility/aria/aria-autocomplete-expected-android.txt
[modify] https://crrev.com/3c944b9a8936705ca717477395531942ca02f641/content/test/data/accessibility/aria/aria-listbox-expected-android.txt
[modify] https://crrev.com/3c944b9a8936705ca717477395531942ca02f641/content/test/data/accessibility/aria/aria-multiselectable-expected-android.txt
[modify] https://crrev.com/3c944b9a8936705ca717477395531942ca02f641/content/test/data/accessibility/aria/aria-option-expected-android.txt
[modify] https://crrev.com/3c944b9a8936705ca717477395531942ca02f641/content/test/data/accessibility/aria/aria-orientation-expected-android.txt
[modify] https://crrev.com/3c944b9a8936705ca717477395531942ca02f641/content/test/data/accessibility/aria/aria-posinset-expected-android.txt
[modify] https://crrev.com/3c944b9a8936705ca717477395531942ca02f641/content/test/data/accessibility/aria/aria-readonly-expected-android.txt
[modify] https://crrev.com/3c944b9a8936705ca717477395531942ca02f641/content/test/data/accessibility/aria/aria-selected-expected-android.txt
[modify] https://crrev.com/3c944b9a8936705ca717477395531942ca02f641/content/test/data/accessibility/aria/aria-setsize-expected-android.txt
[modify] https://crrev.com/3c944b9a8936705ca717477395531942ca02f641/content/test/data/accessibility/html/action-verbs-expected-blink.txt
[modify] https://crrev.com/3c944b9a8936705ca717477395531942ca02f641/content/test/data/accessibility/html/action-verbs.html
[modify] https://crrev.com/3c944b9a8936705ca717477395531942ca02f641/third_party/WebKit/LayoutTests/accessibility/clickable-expected.txt
[modify] https://crrev.com/3c944b9a8936705ca717477395531942ca02f641/third_party/WebKit/LayoutTests/accessibility/clickable.html
[modify] https://crrev.com/3c944b9a8936705ca717477395531942ca02f641/third_party/WebKit/LayoutTests/accessibility/press-works-on-text-fields.html
[modify] https://crrev.com/3c944b9a8936705ca717477395531942ca02f641/third_party/WebKit/Source/modules/accessibility/AXNodeObject.cpp
[modify] https://crrev.com/3c944b9a8936705ca717477395531942ca02f641/third_party/WebKit/Source/modules/accessibility/AXObject.cpp

Comment 2 by nek...@chromium.org, Dec 10 2017

Status: Fixed (was: Started)

Sign in to add a comment