Closed
Bug 154196
Opened 22 years ago
Closed 22 years ago
nsIAccessible's accState returns wrong value for XUL Treeitem node.
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: dsirnapalli, Assigned: yuanyi21)
Details
(Keywords: testcase)
Attachments
(2 files, 1 obsolete file)
4.11 KB,
application/vnd.mozilla.xul+xml
|
Details | |
3.00 KB,
patch
|
aaronlev
:
review+
jag+mozilla
:
superreview+
|
Details | Diff | Splinter Review |
-- Test case attached. The alert shows "1048576" for state. that means its focusable. But i implemented the code in the test case so that the node gets the focus. The last alert shows "Focused" for NodeFocus. When i got this alert that means focus is happening. Therefore the alert for state should show "1048580" (STATE_FOCUSED + STATE_FOCUSABLE). The focus is happening but the state is not picking up the focus value.
Reporter | ||
Comment 1•22 years ago
|
||
Tested in windows, I found all windows widgets have neither STATE_FOCUSABLE nor STATE_FOCUSED state. Aaron, it's a windows' bug or our redundant?
Comment 3•22 years ago
|
||
Here is the rule we should follow: If you can tab to something, it should be marked STATE_FOCUSABLE. If it can be selected, it should be marked STATE_SELECTABLE. However, I don't think anything bad will happen if we report some things that are STATE_SELECTABLE also as STATE_FOCUSABLE.
now, treeitem has focusable and selectable state, but tree has neither. seeking r=
Attachment #89225 -
Attachment is obsolete: true
Comment 5•22 years ago
|
||
Comment on attachment 89239 [details] [diff] [review] better fix. override acctakefocus, acctakeselection, accremoveselection for treeitem r=aaronl
Attachment #89239 -
Flags: review+
Comment 6•22 years ago
|
||
Comment on attachment 89239 [details] [diff] [review] better fix. override acctakefocus, acctakeselection, accremoveselection for treeitem sr=jag
Attachment #89239 -
Flags: superreview+
checked in!
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 8•22 years ago
|
||
-- Verified in todays trunk build. Works fine. Marking the bug as verified.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•