New issue
Advanced search Search tips

Issue 787664 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Jan 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 1
Type: Bug
Team-Accessibility



Sign in to add a comment

A11Y: NVDA ignores new rows in layout tables marked with role="presentation"

Project Member Reported by dsexton@chromium.org, Nov 22 2017

Issue description

Windows: 10
Chrome: 64.0.3273.3 (Official Build) canary (64-bit) (cohort: Clang-64)
NVDA: 2017.3

# With NVDA running, open the attached file in Chrome
# Use arrow keys to review each section of the form

Form 1 presents as a table and table navigation commands work correctly.
Form 2 correctly does not present as a table, but line breaks at each <td> instead of <tr>
Form 3 and 4 present as a single line of <tr>'s and <td>'s

Expected: Tables regardless of role="presentation" should have lines broken at </tr>

Actual: Variable results from breaking at <td> to not breaking at all.
 
Presentation-table-form.html
1.5 KB View Download
Labels: win-a11y
Labels: tables
Labels: -Pri-2 Pri-1
Owner: aleventhal@chromium.org
Status: Started (was: Available)
Hi David, I have a CL to make it expose the content as Firefox does. However, down arrow navigates to the next cell in the table instead of by row. It is the same as what happens in Firefox wit NVDA and Firefox. Can you provide exact keys and expected output? Also, can you verify that it works as you expect in Firefox?
Project Member

Comment 6 by bugdroid1@chromium.org, Jan 31 2018

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

commit 112c7d585aab4bffd99773897731d3df996cab4e
Author: Aaron Leventhal <aleventhal@chromium.org>
Date: Wed Jan 31 04:40:23 2018

Table with role presentation should expose cells as generic containers

When screen readers traverse a <table role="presentation">, they need
generic container objects in place of the cells, to help them understand
and present formatted content. Specifically, this helps present the text
without jamming it together, as well as providing line breaks.

Bug:  787664 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I53b691f0ccae3f4090ca0f8ecfcbeb622bd3dfeb
Reviewed-on: https://chromium-review.googlesource.com/887599
Commit-Queue: Aaron Leventhal <aleventhal@chromium.org>
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#533202}
[modify] https://crrev.com/112c7d585aab4bffd99773897731d3df996cab4e/content/browser/accessibility/dump_accessibility_tree_browsertest.cc
[modify] https://crrev.com/112c7d585aab4bffd99773897731d3df996cab4e/content/test/data/accessibility/aria/aria-sort-html-table-expected-blink.txt
[modify] https://crrev.com/112c7d585aab4bffd99773897731d3df996cab4e/content/test/data/accessibility/aria/aria-sort-html-table-expected-mac.txt
[modify] https://crrev.com/112c7d585aab4bffd99773897731d3df996cab4e/content/test/data/accessibility/aria/aria-sort-html-table-expected-win.txt
[add] https://crrev.com/112c7d585aab4bffd99773897731d3df996cab4e/content/test/data/accessibility/html/table-presentation-expected-blink.txt
[add] https://crrev.com/112c7d585aab4bffd99773897731d3df996cab4e/content/test/data/accessibility/html/table-presentation-expected-mac.txt
[add] https://crrev.com/112c7d585aab4bffd99773897731d3df996cab4e/content/test/data/accessibility/html/table-presentation-expected-win.txt
[add] https://crrev.com/112c7d585aab4bffd99773897731d3df996cab4e/content/test/data/accessibility/html/table-presentation.html
[modify] https://crrev.com/112c7d585aab4bffd99773897731d3df996cab4e/third_party/WebKit/LayoutTests/accessibility/presentation-owned-elements-expected.txt
[modify] https://crrev.com/112c7d585aab4bffd99773897731d3df996cab4e/third_party/WebKit/LayoutTests/accessibility/role-change.html
[modify] https://crrev.com/112c7d585aab4bffd99773897731d3df996cab4e/third_party/WebKit/LayoutTests/accessibility/table-with-presentation-role.html
[modify] https://crrev.com/112c7d585aab4bffd99773897731d3df996cab4e/third_party/WebKit/Source/modules/accessibility/AXNodeObject.cpp

Status: Fixed (was: Started)
Labels: a11y-testing
Status: Verified (was: Fixed)
Labels: -a11y-testing

Sign in to add a comment