Bug 179780 – AX: VoiceOver in Safari does not read table header for first cell in first body row

• Status: RESOLVED
• Resolution: FIXED
• Component: Accessibility

[chris fleizach]

Summary: When using VoiceOver in Safari to read a HTML table, the column header (th) is not read for the first cell in the first row


Steps to Reproduce: Given this HTML markup: 
<table>
    <thead>
      <tr>
        <th>Morning</th>
        <th>Afternoon</th>
      </tr>
    </thead>
    <tbody>
      <tr>
        <td>Free</td>
        <td>Busy</td>
      </tr>
    </tbody>
  </table>

* Turn VoiceOver on
* Swipe to each item in the table


Expected Results:
VoiceOver should read:
* "Morning. Row One. Column 1. Column Start. Two Columns." 
* "Afternoon. Column 2."
* "Morning, Free. Row Two. Column 1."
* "Afternoon, Busy. Table End."

Actual Results:
VoiceOver reads:
* "Morning. Row One. Column 1. Column Start. Two Columns." 
* "Afternoon. Column 2."
* "Free. Row Two. Column 1."
* "Afternoon, Busy. Table End."

Version/Build:
iOS 11.0

<rdar://problem/34711324>

[Radar WebKit Bug Importer]

<rdar://problem/35590551>

[chris fleizach]

Created attachment 327071 [details]
patch

[Build Bot]

Comment on attachment 327071 [details]
patch

Attachment 327071 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/5260741

New failing tests:
accessibility/table-attributes.html

[Build Bot]

Created attachment 327082 [details]
Archive of layout-test-results from ews101 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews101  Port: mac-elcapitan  Platform: Mac OS X 10.11.6

[Build Bot]

Comment on attachment 327071 [details]
patch

Attachment 327071 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/5260819

New failing tests:
accessibility/table-attributes.html

[Build Bot]

Created attachment 327084 [details]
Archive of layout-test-results from ews104 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews104  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6

[Build Bot]

Comment on attachment 327071 [details]
patch

Attachment 327071 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/5260828

New failing tests:
accessibility/table-attributes.html

[Build Bot]

Created attachment 327086 [details]
Archive of layout-test-results from ews116 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews116  Port: mac-elcapitan  Platform: Mac OS X 10.11.6

[chris fleizach]

Created attachment 327103 [details]
patch

[Joanmarie Diggs]

Created attachment 327243 [details]
Rebaseline table-attributes.html results for WebKitGtk+

Chris: The attached patch contains the updated text expectations for table-attributes.html for WebKitGtk. If it's easy enough to land your patch with these changes included, that would be appreciated. If not, we'll do it as part of gardening.

[chris fleizach]

(In reply to Joanmarie Diggs (irc: joanie) from comment #10)
> Created attachment 327243 [details]
> Rebaseline table-attributes.html results for WebKitGtk+
> 
> Chris: The attached patch contains the updated text expectations for
> table-attributes.html for WebKitGtk. If it's easy enough to land your patch
> with these changes included, that would be appreciated. If not, we'll do it
> as part of gardening.

awesome! incorporating now

[chris fleizach]

Created attachment 327245 [details]
patch for landing

[chris fleizach]

Created attachment 327246 [details]
patch for landing

[WebKit Commit Bot]

Comment on attachment 327246 [details]
patch for landing

Clearing flags on attachment: 327246

Committed r224997: <https://trac.webkit.org/changeset/224997>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.