Opened 15 years ago

Closed 15 years ago

Last modified 15 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 15 years ago by ilia

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

comment:2 Changed 15 years ago by ilia

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

comment:3 Changed 15 years ago by ilia

Resolution: fixed
Status: newclosed

comment:4 Changed 15 years ago by (none)

Milestone: 0.4

Milestone 0.4 deleted

Note: See TracTickets for help on using tickets.