If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

nsIAccessible's accState returns wrong value for XUL Treeitem node.

VERIFIED FIXED

Status

()

Core
Disability Access APIs
VERIFIED FIXED
16 years ago
15 years ago

People

(Reporter: dsirnapalli, Assigned: Kyle Yuan)

Tracking

({testcase})

Trunk
x86
Windows 2000
testcase
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

16 years ago
-- 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

16 years ago
Created attachment 89114 [details]
Test Case to reproduce the bug.

Updated

16 years ago
Keywords: testcase
(Assignee)

Comment 2

16 years ago
Created attachment 89225 [details] [diff] [review]
remove STATE_FOCUSABLE from tree and treeitem


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

16 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.
(Assignee)

Comment 4

16 years ago
Created attachment 89239 [details] [diff] [review]
better fix. override acctakefocus, acctakeselection, accremoveselection for treeitem


now, treeitem has focusable and selectable state, but tree has neither.

seeking r=
Attachment #89225 - Attachment is obsolete: true

Comment 5

16 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

15 years ago
Comment on attachment 89239 [details] [diff] [review]
better fix. override acctakefocus, acctakeselection, accremoveselection for treeitem

sr=jag
Attachment #89239 - Flags: superreview+
(Assignee)

Comment 7

15 years ago
checked in!
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
(Reporter)

Comment 8

15 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.