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

Issue 730928 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 1
Type: Bug
Team-Accessibility



Sign in to add a comment

Chrome does not calculate coordinates for ARIA tables (role table/row/cell)

Project Member Reported by ja...@nvaccess.org, Jun 8 2017

Issue description

Chrome Version: 61.0.3122.0 (Official Build) canary(64-bit)
OS: Windows 10 Version 1703 (OS Build 16199.1000) 64-bit

What steps will reproduce the problem?
(1) Start Chrome and the NVDA screen reader.
(2) Open this URL: data:text/html,before<div role="table"><div role="row"><div role="cell">A</div><div role="cell">B</div></div></div>
(3) Press control+home, then down arrow to move to the table.
Expected: NVDA should say "table with 1 rows and 2 columns  row 1 column 1  A"
Actual: NVDA says "table with 0 rows and 0 columns  row 1 column 1  A"
(4) Press down arrow.
Expected: NVDA should say "column 2  B"
Actual: NVDA says just "B"

IAccessibleTable2::nRows and IAccessible2::nColumns both report 0 for this table. They should report 1 and 2 respectively. IAccessibleTableCell::rowNumber and columnNumber both report 0 for both cells. columnNumber should report 1 for the second cell.

I've used these new roles in the GitHub axSGrease script (https://github.com/nvaccess/axSGrease) to make otherwise inaccessible GitHub tables accessible. This works in Firefox, but not in Chrome (because of this bug).
 
Labels: triage-nektar
Labels: triage-aaron
Cc: nek...@chromium.org
Components: -UI>Accessibility UI>Accessibility>Compatibility
Labels: -Pri-3 -triage-aaron -triage-nektar Pri-2
Status: Available (was: Untriaged)
Owner: ----
Labels: win-a11y
Labels: tables
Labels: -Pri-2 Pri-1
Owner: dmazz...@chromium.org
Status: Started (was: Available)
Project Member

Comment 10 by bugdroid1@chromium.org, Feb 22 2018

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

commit 0c8b6d1ede042904a1ec42ccdc2b4d6b89d84931
Author: Dominic Mazzoni <dmazzoni@chromium.org>
Date: Thu Feb 22 20:50:30 2018

The ARIA Grid classes should handle ARIA Tables too

ARIA introduced role="table" in ARIA 1.1, but to the browser it behaves
the same as role="grid". We should be using AXARIAGrid and
AXARIAGridCell for role="table" and role="cell", respectively.

As a follow-up we could rename the classes, too - but first this is
the simplest change that just fixes the user-facing bug.

Bug:  730928 
Change-Id: I63bbe927b54732e2e4f5027c81933a7399deb631
Reviewed-on: https://chromium-review.googlesource.com/930130
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: Nektarios Paisios <nektar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#538554}
[modify] https://crrev.com/0c8b6d1ede042904a1ec42ccdc2b4d6b89d84931/content/test/data/accessibility/aria/aria-cell-expected-blink.txt
[modify] https://crrev.com/0c8b6d1ede042904a1ec42ccdc2b4d6b89d84931/content/test/data/accessibility/aria/aria-cell-expected-win.txt
[modify] https://crrev.com/0c8b6d1ede042904a1ec42ccdc2b4d6b89d84931/content/test/data/accessibility/aria/aria-table-expected-blink.txt
[modify] https://crrev.com/0c8b6d1ede042904a1ec42ccdc2b4d6b89d84931/content/test/data/accessibility/aria/aria-table-expected-mac.txt
[modify] https://crrev.com/0c8b6d1ede042904a1ec42ccdc2b4d6b89d84931/content/test/data/accessibility/aria/aria-table-expected-win.txt
[modify] https://crrev.com/0c8b6d1ede042904a1ec42ccdc2b4d6b89d84931/content/test/data/accessibility/aria/aria-table.html
[modify] https://crrev.com/0c8b6d1ede042904a1ec42ccdc2b4d6b89d84931/third_party/WebKit/Source/modules/accessibility/AXObjectCacheImpl.cpp

Status: Fixed (was: Started)

Sign in to add a comment