OpenSmock / Pyramid

Pyramid is a Graphical User-Interface (GUI) builder and editor for Bloc and Toplo.
MIT License
33 stars 6 forks source link

When grouping a selection of BlElement, the order is not the same #103

Closed labordep closed 11 months ago

labordep commented 1 year ago

Create two elements and group them. The group contents have child but not in the same order that the selection.

Nyan11 commented 11 months ago

Probably fix when i updated the Tree element in #110. Need testing

Nyan11 commented 11 months ago

No it does not work

Nyan11 commented 11 months ago

Group command memento should not create new BlElement but instead recycle previously destroyed ones.

Example: Create a group, change order of this group (on background). Use undo button until the group is remove, then use redo button until the group order is change.

An error occurs, because group Element is a new reference and it should not be.

Nyan11 commented 11 months ago

New questions, how to test that no new reference are created by undo / redo memento ?

Add new test on test trait:

Nyan11 commented 11 months ago

131 is merged