532024 - update accessible tree when accessible reorder event is occurred

Status: RESOLVED FIXED

(Other Applications :: DOM Inspector, defect)

Description

[alexander :surkov (:asurkov)]

Attachment: patch

When accessible tree is mutation then reorder event is fired on the container of changed accessibles. It's worth to use it to update accessible tree view.

Comment 1

[alexander :surkov (:asurkov)]

(In reply to comment #0)
> Created an attachment (id=415331) [details]
> patch

the patch is a bit hacky but it sounds it's a simplest way.

Comment 2

[[e-mail]]

Comment on attachment 415331 [details] [diff] [review]
patch

>+    if (node.accessible == accessible) {
>+      // Toggle open state twice to update the children.
>+      this.toggleOpenState(idx);
>+      this.toggleOpenState(idx);
>+      break;
>+    }
Nit: only need to toggle twice if the index was already open.
(Still want to break of course.)

Comment 3

[Shawn Wilsher :sdwilsh]

Comment on attachment 415331 [details] [diff] [review]
patch

nit: s/var/let/ please

r=sdwilsh with that change.

Comment 4

[alexander :surkov (:asurkov)]

landed with addressed Neil's and Shawn's comments - http://hg.mozilla.org/dom-inspector/rev/2f604b4db3d0