ondeselect call: proposed code change

4 views

Skip to first unread message

Kostrowsky

unread,

Feb 9, 2010, 11:24:41 AM2/9/10

to jsTree

Tinkering with the jquery.tree.js code, I moved the callback at line
747

this.callback("ondeselect", [obj.get(0), _this]);

to line 760 (after the final else) so the callback would be invoked
AFTER any tree change happening in the deselect callback set as an
option.

I did that code change after finding that the selected_arr property
was not properly set in my own ondeselect callback after I
programmatically deselected some branches.

May this be a valid and my first extra small contribution to this very
nice library.

vakata

unread,

Feb 13, 2010, 9:17:23 AM2/13/10

to jsTree

Thank you for the suggestion, you will find that version 1.0 is much
changed, and events are handled differently now - wait a few days till
I release beta3 or use beta2 :)