Closed
Bug 109221
Opened 23 years ago
Closed 23 years ago
Active Accessibility: support <colorpicker>
Categories
(SeaMonkey :: General, defect, P3)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla0.9.9
People
(Reporter: aaronlev, Assigned: aaronlev)
References
Details
(Keywords: access, Whiteboard: need r=, sr=)
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
mozilla
:
review+
|
Details | Diff | Splinter Review |
I'm not sure what supporting <colorpicker> via MSAA will look like, but it
should be something like our <grid> support, using:
ROLE_TABLE
ROW_COLUMN
ROLE_ROW
ROLE_CELL
Each color needs a name, so we might need to put tooltiptext on the colors in
colorpicker.
Assignee | ||
Updated•23 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P3
Target Milestone: --- → mozilla0.9.9
Assignee | ||
Comment 1•23 years ago
|
||
Assignee | ||
Comment 2•23 years ago
|
||
Oh yeah, the patch doesn't support rows and columns, just a list of tile
children, but that's fine.
Looking for reviews.
Assignee | ||
Comment 3•23 years ago
|
||
Attachment #57515 -
Attachment is obsolete: true
Comment 4•23 years ago
|
||
Comment on attachment 57569 [details] [diff] [review]
Better patch -- tested and works well
don't forget we need mcp file changes for this -- new file.
remove the comments about reusing the HTML widget in nsAccessibilityService
Doesn't the nsFromControlAccessible::GetAccState tell us if we are focused? Why
the extra method in the XULColorPicker impl? Just for hover?
extra comment in XULColorPickerAcc about being a pushbutton
fix these and
r=jgaunt
Attachment #57569 -
Flags: review+
Assignee | ||
Comment 5•23 years ago
|
||
John, we needed to implement something special for focus because the colorpicker
tiles never actually receive NS or DOM focus, similar to menuitems. However, we
still need to generate MSAA Focus events.
When they're hovered over or arrowed to, it only changes attributes. Therefore,
we now generate "DOMMenuActive" events on the items so we can catch where the
user's keyboard state of regard is.
Comment 6•23 years ago
|
||
sr=hyatt
Assignee | ||
Comment 7•23 years ago
|
||
-> checked in
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•