400539 - onClick, role or tabindex in table row alters table hierarchy

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Joanmarie Diggs]

Attachment: test case

Steps to reproduce:

1. Launch Accerciser and use it to examine the table in the attached test case.

Expected results:  The parent of the table cell would be the table.
Actual results: The parent of the table cell is an object of ROLE_TEXT

2. Remove the onclick from the table row, reload the page, and re-examine the hierarchy.

Expected and Actual results: The parent of the table cell is the table.

Comment 1

[alexander :surkov (:asurkov)]

1) table@onclick
table (html:table)
  text container (html:tbody)
    text container (html:tr)
      cell (html:td)
        text leaf (text node)


2) table
table (html:table)
  text container (html:tbody)
    cell (html:td)
      text leaf (text node)

So I guess the problem we create accessible for html:tr whereas we shouldn't?

Comment 2

[Aaron Leventhal]

We can't guarantee that the table rows won't have accessibles,  because they need an accessible object if they:
- are focusable (need to be able to fire events)
- have an action (like onclick)
- have a relationship to another accessible
etc.

Does the table interface still work?

Comment 3

[alexander :surkov (:asurkov)]

It looks this leads to problem described in http://groups.google.com/group/mozilla.dev.accessibility/browse_frm/thread/4f54984000b93b98 (windows eyes doesn't announce table headers)

Comment 4

[Aaron Leventhal]

I think this needs to be solved on the AT end.

Comment 5

[alexander :surkov (:asurkov)]

Would be nice to get some tests on question from comment #2. I think if table interface still works then the bug is invalid.

Comment 6

[Marco Zehe (:MarcoZ)]

The fact that accessibles on table rows currently break part of our table interface is documented in bug 410052, and a result of that is in bug 409439. So once bug 410052 is fixed, we should revisit this one and see if there's still a problem.

Comment 7

[Joanmarie Diggs]

I've worked around this issue.  Close it as INVALID if you'd like.

Comment 8

[Aaron Leventhal]

I want to keep it open and look at it after Firefox 3 ships.

Comment 9

[alexander :surkov (:asurkov)]

bug 410052 has been landed on trunk.

Comment 10

[alexander :surkov (:asurkov)]

When we land bug 493695 then this bug becomes worksforme because onclick and etc on row won't change table hierarchy.

Comment 11

[alexander :surkov (:asurkov)]

fixed in bug 493695 because onclick and etc doesn't change hierarchy any more, row accessibles are exposed always.