W dniu 27 lipca 2010 10:57 użytkownik ~suv <suv-sf@...58...> napisał:
- Undo does not work correctly: when modifying a parameter using a
slider, undo does not restore the slider to its value at the start of the drag, but to intermediate positions during the drag. (Fixing this likely requires a substantial rewrite.)
579932 Too much states saved for undo/redo when using sliders https://bugs.launchpad.net/inkscape/+bug/579932
It turns out that the author of the dialog was thinking correctly but did not know that some Inkscape APIs were braindead. The undo spam was triggered because the undo key passed to sp_document_maybe_done was dynamically created in a Glib::ustring, and that function expects a constant string. I fixed it to store a duplicate of the last undo action key in SPDocument (via g_strdup), rather than simply assigning a char pointer. The destructor was adjusted as well.
The patch can be applied to 0.48 branch as well.
Regards, Krzysztof