Closed
Bug 370276
Opened 18 years ago
Closed 18 years ago
Expose buttons with popup as ROLE_SYSTEM_BUTTONMENU
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: aaronlev, Assigned: aaronlev)
References
(Blocks 1 open bug)
Details
(Keywords: access)
Attachments
(1 file)
Currently we expose all MSAA buttons as ROLE_SYSTEM_PUSHBUTTON.
We indicate the popup state with STATE_SYSTEM_HASPOPUP.
However we should also change the role in that case, to ROLE_SYSTEM_BUTTONMENU.
|
Assignee | |
Comment 1•18 years ago
|
||
Attachment #254946 -
Flags: review?(david.bolter)
|
||
Comment 2•18 years ago
|
||
Aaron, your changes can't fit to rolemap.h, right?
|
|
Assignee | |
Comment 3•18 years ago
|
||
No, and I don't see modifying the role map mechanism just for this one item.
|
||
Comment 4•18 years ago
|
||
Comment on attachment 254946 [details] [diff] [review]
A small hack, but better than alternatives. When role==button and state&haspopup, change role to ROLE_SYSTEM_BUTTONMENU
Looks good Aaron. This seems a reasonable place to put this fix. (I guess the new role map stuff would have to get complicated to handle cases like this?)
Attachment #254946 -
Flags: review?(david.bolter) → review+
|
|
Assignee | |
Comment 5•18 years ago
|
||
Changing how role map works is not worth the complexity for this one case. If we get more cases we can think of it.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•