[L2Ork-dev] Undo support

[Ivica Bukvic]

Thank you, Guillem. This is fantastic news regarding your progress. As to
your question, I'm of the opinion that it would be more valuable to extend
the current undo to also include checkpoints (or whatever we wish to call
them), so that multi-step operations can be undone in a single step. This
is something that the existing implementation of infinite undo was in need
of even before your project. For instance, intelligent patching can
sometimes generate dozens of connect operations and the only way one can
undo them now is by pressing undo many times, one for each connection. So,
your implementation is not the only thing that needs this kind of feature
and therefore it would benefit more than just your project. All of that
said, I'm happy to support alternative approaches, especially ones that a
majority agrees to.

Best,

Ico

-- 
Ivica Ico Bukvic, D.M.A.
Director, Creativity + Innovation
Institute for Creativity, Arts, and Technology

Virginia Tech
Creative Technologies in Music
School of Performing Arts – 0141
Blacksburg, VA 24061
(540) 231-6139
ico at vt.edu

www.icat.vt.edu
www.performingarts.vt.edu
l2ork.icat.vt.edu
ico.bukvic.net

On Mon, Jun 15, 2020, 08:32 Guillem Bartrina Moreno <
guillembartrina at gmail.com> wrote:

> Hi all,
> I almost finished the new autosubpatching feature, only leaving undo/redo
> support.
> I have several questions regarding this topic.
> All the actions performed by the feature can be partitioned into 2+ basic
> actions that already have undo support: 1 cut + 1 paste + N connects (or 1
> cut + 1 paste + 1 special paste, if we slightly change the "dopaste"
> function, probably not the best option).
> The question is if it's worth it to implement a new undo type for the
> feature or instead would be better implement a new special undo type that
> implements a multi-step undo by having a private queue of basic undo steps
> and executing all of them in a row. This special undo type could be used
> not only in this feature but somewhere else in the future.
>
> What do you think?
> Especially to Ico, who implemented the infinite undo.
>
> --
> Guillem Bartrina Moreno
> _______________________________________________
> L2Ork-dev mailing list
> L2Ork-dev at disis.music.vt.edu
> https://disis.music.vt.edu/listinfo/l2ork-dev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://disis.music.vt.edu/pipermail/l2ork-dev/attachments/20200615/8d3f1865/attachment.html>