Issue metadata
Sign in to add a comment
|
Too many accessible nodes marked as clickable on Android |
||||||||||||||||||||||||
Issue descriptionPrevious versions of TalkBack explored the accessibility tree and skipped over some important nodes, so we marked anything we thought was interesting as clickable. Now TalkBack calls ACTION_NEXT_HTML_ELEMENT to linearly navigate the tree, so we can do things: 1. Relax "clickable" and only expose that on nodes that really are clickable 2. Update ACTION_NEXT_HTML_ELEMENT so that it doesn't miss any interesting nodes
,
Oct 21 2016
,
Dec 1 2017
This issue has reappeared as elements with action verb "click-ancestor" are also considered as clickable in android. This is incorrect as "click-ancestor" only means that element has elements with click listener in its ancestor chain and is not self clickable.
,
Dec 11 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b423714978378371052144e1548480072e4cfea4 commit b423714978378371052144e1548480072e4cfea4 Author: Isha Bobra <ibobra@google.com> Date: Mon Dec 11 21:23:07 2017 Android Accessibility: Avoiding incorrect marking of elements as clickable For the elements where a click listener is present in its ancestry chain, a default action verb "click-ancestor" is used. So for android this CL makes sure that such elements are not marked as clickable. Bug: 645167 Change-Id: I0ce047e8bf619bb6eaa3c77bc4bec4d2d999e203 Reviewed-on: https://chromium-review.googlesource.com/804202 Commit-Queue: Isha Bobra <ibobra@google.com> Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by: Nektarios Paisios <nektar@chromium.org> Cr-Commit-Position: refs/heads/master@{#523210} [modify] https://crrev.com/b423714978378371052144e1548480072e4cfea4/content/browser/accessibility/browser_accessibility_android.cc [modify] https://crrev.com/b423714978378371052144e1548480072e4cfea4/content/browser/accessibility/dump_accessibility_tree_browsertest.cc [add] https://crrev.com/b423714978378371052144e1548480072e4cfea4/content/test/data/accessibility/html/clickable-ancestor-expected-android.txt [add] https://crrev.com/b423714978378371052144e1548480072e4cfea4/content/test/data/accessibility/html/clickable-ancestor.html
,
Dec 18 2017
Issue 795732 has been merged into this issue.
,
Dec 20 2017
Issue 771423 has been merged into this issue.
,
Feb 26 2018
|
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by bugdroid1@chromium.org
, Oct 17 2016