523069 - ARIA treegrid should expose group attributes

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Comment 1

[alexander :surkov (:asurkov)]

The ARIA spec doesn't allow aria-setsize and aria-posinset to be used on the ARIA row. But it sounds like it's sort of mistake that should be fixed. I filed the question to the [e-mail] mailing list.

Comment 2

[alexander :surkov (:asurkov)]

Attachment: patch

It seems ARIA group agreed on this issue.

1) expose posinset and setsize on row and gridcell
2) expose level on row of the treegrid

Comment 3

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

Comment on attachment 418806 [details] [diff] [review]
patch

r=me thanks. Nit: do you really think we need a mochitest/attributes directory?

Comment 4

[alexander :surkov (:asurkov)]

(In reply to comment #3)
> (From update of attachment 418806 [details] [diff] [review])
> r=me thanks. Nit: do you really think we need a mochitest/attributes directory?

Yes. In general I'd like to group our tests by directories. In this case tests have shorter names and tree of tests is nicer, it's easier to find interesting tests.

Comment 5

[alexander :surkov (:asurkov)]

landed on 1.9.3 - http://hg.mozilla.org/mozilla-central/rev/40a3ba9e110c

Comment 6

[alexander :surkov (:asurkov)]

Comment on attachment 418806 [details] [diff] [review]
patch

It's major fix needed by screen readers for better support of accessible tables. It should be a part of accessible table work performed for 1.9.2 (Firefox 3.6).

Comment 7

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 418806 [details] [diff] [review]
patch

We should get this early in the next cycle, but it's baked long and comes with tests.

Comment 8

[Marco Zehe (:MarcoZ)]

Comment on attachment 418806 [details] [diff] [review]
patch

1.9.2.4 is done, bumping approval request to 1.9.2.5.

Comment 9

[LegNeato]

Comment on attachment 418806 [details] [diff] [review]
patch

We will not be taking this for 3.6.6. Moving approval request forward.

If you disagree, send me an email.