Issue 26: Fix undo and redo of pivot and set root commands
Status:  WontFix
Owner:
Closed:  Aug 23
Project Member Reported by ejrh00@gmail.com, Nov 20, 2011
The problem here is that set root is implemented by a sequence of pivots.  Pivots have correct undo and redo, but the sequence as a whole is put on the undo stack as one big command.  It is not correct because it doesn't record the position of each item in a node's list of children.  This information gets messed up when the whole sequence is replayed by the inverse set root command.

The stack records the state as before the set root command, and tries to set root back to that state.

One way to fix this might be to let groups of subcommands be put on the stack; each subcommand is undone is order, not as a whole.

But it's also possible the idea of having a list of children is misguided.  Perhaps a set of children with optional hints on how to order then would work better.

I'm not sure I've explained it very well, but we'll see...

Aug 23, 2015
Project Member #1 ejrh00@gmail.com
(No comment was entered for this change.)
Status: WontFix