Spyrogimp 'Live' Preview Pollutes Undo History
Right now, when spyrogimp updates the on-canvas preview, everything is written to the undo tree which can quickly add up to 100+ items and multiple gigabytes if the 'flower' is dense or the image is large. I propose two changes:
-
Undo group the whole 'flower' instead of each revolution/stroke/however it's done now. I can't imagine any reason why someone would absolutely need a plugin's function split into sometimes over 20 separate undo items.
-
In absence of #999, freeze the history while the UI is open to prevent gradual memory/undo item accumulation. Then you just connect "OK" to remove layer - > thaw - > insert layer. This would create an undo group of just adding a fully-completed spirograph layer.
I can probably patch this in myself but before touching anything I wanted to see if my philosophy was on-track. Namely, is there a reason the history isn't grouped already, and is it better to just wait for someone smarter than me to implement #999 instead of the freezing layer insertion buggery?
Sys info
- Arch Linux
- Gimp master latest commit as of writing (82e945d9)
- Built/installed using pkgbuild gimp-develop-git