1728676 - Table interfaces broken/not exposed when table roles used in SVG

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[James Teh [:Jamie] (away returning 3 Oct)]

Originally filed as NVDA issue: https://github.com/nvaccess/nvda/issues/12767

STR (with the NVDA screen reader):

1. Visit https://cdpn.io/aardrian/debug/xxqYyaa
2. Press 2 to navigate to the heading "3. SVG with Table Roles".
3. Press T to navigate to the table that immediately follows that heading.
4. Navigate into the table and then move among cells using Ctrl + Alt + arrow key.
   • Actual behavior: NVDA reports "Not in a table cell".
   • Expected behavior: I can navigate the SVG as any other well-formed HTML table.

I investigated this briefly. The IAccessibleTable and IAccessibleTable2 interfaces are exposed on the table, but calling nRows, nColumns, etc. returns CO_E_OBJNOTCONNECTED. On the cells, the IAccessibleTableCell interface isn't exposed at all.

I'm not sure whether this is Windows specific or not; I haven't dug into that.

Comment 1

[James Teh [:Jamie] (away returning 3 Oct)]

Bug 1832228 should make fixing this a lot easier. However, we'll need to call MaybeCreateSpecificARIAAccessible when handling the creation of SVG elements as well as HTML elements.

Actually, I wonder whether we can call MaybeCreateSpecificARIAAccessible much earlier in CreateAccessible and thus avoid duplicate calls.

Comment 2

[James Teh [:Jamie] (away returning 3 Oct)]

Attachment: Bug 1728676: Correctly expose ARIA table cells in SVG.

Previously, the code to create ARIAGridCellAccessible only ran for HTML elements.
This meant that ARIA cells weren't exposed correctly in SVG.
This code has now been moved outside of the HTML check so that it works for SVG as well.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/mozilla-central/rev/78204d8945ed
Correctly expose ARIA table cells in SVG. r=nlapre

Comment 4

[Sandor Molnar]

https://hg.mozilla.org/mozilla-central/rev/78204d8945ed

Comment 5

[Monica Chiorean]

I was able to reproduce the issue on Win10x64 using FF build 93.0a1(20210901214553) and the steps from description and got: NVDA reports "Not in a table cell".
But I see no difference between the old build and latest 116.0 (20230724170120) or Nightly 117.0a1(20230726154100). Is there something that I am missing? Thank you.

Comment 6

[James Teh [:Jamie] (away returning 3 Oct)]

Hi! I'm guessing the piece you're probably missing is in step 4:

4. Navigate into the table and then move among cells using Ctrl + Alt + arrow key.

When you press t to move to the table, you land on the caption, which is not a cell and thus won't allow table cell navigation. To "navigate into the table" and move to the first cell, you'll need to press down arrow after pressing t. At that point, you should be able to use control+alt+arrows to move around the cells.

Sorry for the lack of clarity.

Comment 7

[Monica Chiorean]

I was able to reproduce the issue on Win10x64 using FF build 93.0a1(20210901214553) using the steps from description and the steps from comment #6. Indeed there is a difference now using the steps from last comment. Thank you James for your help.
Verified as fixed on Win10x64 using FF builds 116.0 (20230724170120) and 117.0a1(20230727034425).