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

VERIFIED FIXED

Status

()

VERIFIED FIXED
17 years ago
17 years ago

People

(Reporter: dsirnapalli, Assigned: yuanyi21)

Tracking

({testcase})

Trunk
x86
Windows 2000
testcase
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

17 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

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

Updated

17 years ago
Keywords: testcase
(Assignee)

Comment 2

17 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

17 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

17 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

17 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

17 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

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

Comment 8

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