Opened 14 years ago

Closed 14 years ago

Last modified 13 years ago

#1221 closed defect (fixed)

TreeWithNodes.setChildren fires afterAddChild

Reported by: ornus Owned by: ilia
Priority: high Milestone:
Component: Widgets Version: 0.3
Keywords: tree Cc:
Blocked By: Blocking:


When nodes are created lazily afterAddChild shouldn't fire. This breaks data model, since node created lazily exists in the tree data source. There's no way to distinguish new nodes added to the tree from nodes create lazily.

It's more appropriate to use a separate event afterLazyChildCreate or something similar to that.

Change History (4)

comment:1 Changed 14 years ago by ilia

afterAddChild represents a new widget being added. setChildren creates (non-recursively) widgets from given array.

comment:2 Changed 14 years ago by ilia

Added special flag to indicate that a node was created lazily, not really added.

comment:3 Changed 14 years ago by ilia

Resolution: fixed
Status: newclosed

comment:4 Changed 13 years ago by (none)

Milestone: 0.4

Milestone 0.4 deleted

Note: See TracTickets for help on using tickets.