Sebastian Klein <basti...@googlemail.com> writes:

> Upliner wrote:
>> By the way, what do you think about making merge layers operation
>> undonable? I think I can implement it, there is some groundwork for
>> in reverter and fuzzer(ext_tools) plugins.
>
> Cool. I see 2 paths here: First would be to have an undo stack for
> each data layer. Normally there is no reason you have to first undo
> all later operations on layer b before you can undo things on layer
> a. (Although I never had that problem.)
>
> Second, make higher level layer operations undoable. (Only thing that
> should be excluded is toggle of active layer and visibility.) Undo of
> merge layer would probably the most useful extension here.

A layer merge is essentially two operations: first merge the data into
the target layer and then delete the source layer.  The first part
would be covered by the target layer's undo stack.  The second could
be left to the user.  JOSM could simply not delete the source layer.

The problem with making such operations undoable is that this
potentially requires a lot of memory if there is a lot of data
involved. We might need to think about storing such things on disk.

Matthias

_______________________________________________
josm-dev mailing list
josm-dev@openstreetmap.org
http://lists.openstreetmap.org/listinfo/josm-dev

Reply via email to