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

Issue 775167 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug



Sign in to add a comment

VoiceOver: table column headers not always read

Project Member Reported by aleventhal@chromium.org, Oct 16 2017

Issue description

Steps:
1. Load the attached file with VoiceOver
2. Use Ctrl+alt down/right to get to bottom right cell
3. Use Ctrl+alt+c to read the column header

Column header is not read, but it is read in Safari with the same example. 
One difference with Safari is that we expose every column header in the table for both cells, whereas Safari only exposes the matching column header.

We do not have the same issue with rowheaders, even though we expose all the rowheaders for each cell.

 
table-th-colheader.html
376 bytes View Download
Description: Show this description
Project Member

Comment 2 by bugdroid1@chromium.org, Oct 31 2017

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

commit e30fa13612ca786e25c0268270a5064943effbb8
Author: Aaron Leventhal <aleventhal@chromium.org>
Date: Tue Oct 31 01:51:58 2017

Table column headers not exposed appropriately in Mac accessibility

- On existing column headers, we should expose nothing for
AXColumnHeaderUIElements.
- For cells, we should expose only the column headers that are used for that cell,
not all of them as we currently do.
- The children of a column should be exposed (this is what actually fixes the VoiceOver issue in the bug).
- The children and visibleChildren should be the same.

Bug:  775167 
Change-Id: Ief4d1ee80cd759250041e78e95e643030c14b361
Reviewed-on: https://chromium-review.googlesource.com/721420
Commit-Queue: Aaron Leventhal <aleventhal@chromium.org>
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#512707}
[modify] https://crrev.com/e30fa13612ca786e25c0268270a5064943effbb8/content/browser/accessibility/browser_accessibility_cocoa.h
[modify] https://crrev.com/e30fa13612ca786e25c0268270a5064943effbb8/content/browser/accessibility/browser_accessibility_cocoa.mm
[modify] https://crrev.com/e30fa13612ca786e25c0268270a5064943effbb8/content/renderer/accessibility/blink_ax_tree_source.cc
[modify] https://crrev.com/e30fa13612ca786e25c0268270a5064943effbb8/content/test/data/accessibility/aria/aria-columnheader-expected-mac.txt
[modify] https://crrev.com/e30fa13612ca786e25c0268270a5064943effbb8/content/test/data/accessibility/aria/aria-rowheader-expected-mac.txt
[modify] https://crrev.com/e30fa13612ca786e25c0268270a5064943effbb8/content/test/data/accessibility/html/table-th-colheader-expected-mac.txt
[modify] https://crrev.com/e30fa13612ca786e25c0268270a5064943effbb8/content/test/data/accessibility/html/table-th-rowheader-expected-mac.txt

Status: Fixed (was: Assigned)

Sign in to add a comment