Closed Bug 614552 Opened 14 years ago Closed 14 years ago

fire reorder event on XUL tree when treeview is changed

Categories

(Core :: Disability Access APIs, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla2.0b8

People

(Reporter: surkov, Assigned: surkov)

References

Details

(Keywords: access)

Attachments

(1 file)

Attached patch patchSplinter Review
Now we fire platform show/hide events on XUL tree when tree view is changed. The reality is show/hide events aren't used by AT in general. But they listen for reorder events. It's more expected to use it instead of platform show/hide events. Ideally we should fire hide events for all old tree items and new for new tree items but I afraid of perf issue. Therefore I suggest to fire reorder event only and leave show/hide events issue for future.
Attachment #492988 - Flags: review?(marco.zehe)
Attachment #492988 - Flags: review?(bolterbugz)
Attachment #492988 - Flags: approval2.0?
I need this for bug 498015 where I'm going to process DOM events after layout. So that mochitest can't rely on TreeViewChanged DOM event anymore.
I think we need a try build here.
Comment on attachment 492988 [details] [diff] [review]
patch

Removing patch approval request as this blocks a blocker now.
Attachment #492988 - Flags: approval2.0?
Comment on attachment 492988 [details] [diff] [review]
patch

The patch looks straight-forward, but I'd like to see a try build before I can r+ this one.
Comment on attachment 492988 [details] [diff] [review]
patch

>@@ -591,36 +591,28 @@ nsXULTreeAccessible::TreeViewInvalidated

> void
> nsXULTreeAccessible::TreeViewChanged()

>-
>   mTree->GetView(getter_AddRefs(mTreeView));

What is this for?
(In reply to comment #6)

> >-
> >   mTree->GetView(getter_AddRefs(mTreeView));
> 
> What is this for?

line removal or mTreeView intializing when tree view was changed?
The latter. This call initializes the tree view?
Comment on attachment 492988 [details] [diff] [review]
patch

r=me, ignore my last comment/question (I totally missed the mTreeView somehow). Not this r+ is conditional on not breaking screen readers ;)
Attachment #492988 - Flags: review?(bolterbugz) → review+
Comment on attachment 492988 [details] [diff] [review]
patch

Yep this works, thanks! r=me.
Attachment #492988 - Flags: review?(marco.zehe) → review+
landed on 2.0 - http://hg.mozilla.org/mozilla-central/rev/6038dc8945d8
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: