Persistent Undo

Bug #169990 reported by Romgen
4
Affects Status Importance Assigned to Milestone
Inkscape
Won't Fix
Medium
Unassigned

Bug Description

Hello guys,

I just put an updated version of this patch:
http://people.csail.mit.edu/sarasu/history/

into the tracker.

Fix:
- Compiles with gcc 4.1.2 (had to add some brackets in "case" structures)

- I had to fix some functions to match new SVN behaviour

- Patch is against todays SVN 19 Okt 2007

TODO:

As already mentioned on the mailing by others, it seems that this patch
needs cleanup in various ways

1. remove all commented code
2. remove debug output

Greetings

Genscher / romgen

Revision history for this message
Romgen (romgen) wrote :
nightrow (jb-benoit)
Changed in inkscape:
importance: Undecided → Medium
status: New → Triaged
Revision history for this message
bbyak (buliabyak) wrote :

Unfortunately I don't think this patch will be accepted as is, because it does not solve the hardest problem: how to store deleted nodes in undo history so that they can be restored. Instead it just "simulates" deletion which is hardly acceptable.

jazzynico (jazzynico)
tags: added: undo
tags: added: patch-rejected-upstream
Revision history for this message
ScislaC (scislac) wrote :

Changing to Won't Fix as the patch was tightly integrated to the removed Whiteboard feature and did not ever address the node deletion tracking. If you feel this was closed in error, please feel free to reopen... with an updated patch not reliant on deprecated features. :)

Changed in inkscape:
status: Triaged → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.