298967 - Incorporate latest feedback into role and states table

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Aaron Leventhal]

The attributes selected and checked should act like expanded, in that setting
them to "false" is different from not having them.

checked="false" means checkable, no checked attribute means not checkable
selected="false" means selectable, no selected attribute means not selectable

Putting menuitemcheckable back in for 2 reasons:
1. to be symmetrical menuitemradio which we need
2. because both it and menuitemradio can't have a submenu (no haspopup support)

Comment 1

[Aaron Leventhal]

Attachment: Simple table change to implement tweaks

Comment 2

[Aaron Leventhal]

Comment on attachment 187473 [details] [diff] [review]
Simple table change to implement tweaks

Another small patch.

Bz is gone for a few more days.

Comment 3

[parente]

Comment on attachment 187473 [details] [diff] [review]
Simple table change to implement tweaks

Do ATs expect grid cells and headers to be selectable but not list items? Seems
like a list is a degenerate (1D) table and should behave as such.

Comment 4

[Aaron Leventhal]

Good catch. List item should have used the same rules as grid in terms of
selected/selectable.

The AT's should only expect that something is selectable if it says it is, not
based on the role.

Comment 5

[Mike Kaply [:mkaply]]

Comment on attachment 187473 [details] [diff] [review]
Simple table change to implement tweaks

a=mkaply

Comment 6

[Aaron Leventhal]

Checking in src/base/nsAccessible.cpp;
/cvsroot/mozilla/accessible/src/base/nsAccessible.cpp,v  <--  nsAccessible.cpp
new revision: 1.155; previous revision: 1.154
done
Checking in src/base/nsAccessible.h;
/cvsroot/mozilla/accessible/src/base/nsAccessible.h,v  <--  nsAccessible.h
new revision: 1.66; previous revision: 1.65
done