1407724 - "Strong" ARIA roles applied to HTML table elements result in descendants with unknown platform role

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect, P2)

Description

[Joanmarie Diggs]

Step to reproduce:

1. Load one of the following test cases in Firefox and use an accessibility inspector (e.g. Accerciser in Linux) to examine it:

<table role="dialog">
  <tr><td>foo</td><td>bar</td></tr>
  <tr><td>bar</td><td>baz</td></tr>
</table>

<ul role="dialog"><li>foo</li></ul>

Expected result: the descendants of the table and list would be exposed with a valid platform accessibility role.

Actual result: At least in the case of ATK, the descendants of the table and list are exposed with ROLE_UNKNOWN. And from the creation of a quick mochitest, it appears that they have internal accessibility role of roles:NOTHING. So I'm guessing it's broken across the board.

Related aside, the accessible with role dialog also claims to implement the accessible table interface.

Comment 1

[David Bolter [:davidb] (NeedInfo me for attention)]

This sounds bad. What version of Firefox did this start in?

Comment 2

[Joanmarie Diggs]

To be honest, I don't know. I was hoping to just provide a patch for bug 1407287. :) Then I came across this. Since I'm trying to implement or fix some of the remaining ARIA 1.1 and Core AAM 1.1 issues in time for an upcoming W3C deadline, I don't have time to figure out when it was broken. Sorry.

Comment 3

[alexander :surkov (:asurkov)]

(In reply to Joanmarie Diggs from comment #0)

> <table role="dialog">
>   <tr><td>foo</td><td>bar</td></tr>
>   <tr><td>bar</td><td>baz</td></tr>
> </table>
> 
> <ul role="dialog"><li>foo</li></ul>
> 
> Expected result: the descendants of the table and list would be exposed with
> a valid platform accessibility role.

Joanie, just to clarify, what is valid platform accessibility role?

dialog
  table_row
    table_cell

or

dialog
  section
    section

just a generic one, correct?

Comment 4

[James Teh [:Jamie]]

This one seems to be fixed:
<ul role="dialog"><li>foo</li></ul>
Table still broken though.

Comment 5

[Eitan Isaacson [:eeejay]]

Attachment: Bug 1407724 - Provide role of TEXT_CONTAINER to descendants of "strong" aria roles. r?Jamie!

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/742cde14ec80
Provide role of TEXT_CONTAINER to descendants of "strong" aria roles. r=Jamie

Comment 8

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/742cde14ec80