mitchcurtis / slate

Pixel Art Editor
GNU General Public License v3.0
1.05k stars 102 forks source link

See if it's possible to de-duplicate undo command code #172

Open mitchcurtis opened 2 years ago

mitchcurtis commented 2 years ago

For the commands that just take a new and old image (or set of images, in the case of layers), we should see if it's possible to reduce them down to more generic commands that just take an image (or set of images). To do this we have to check that the commands can still be accurately represented in a history panel (undo view). To do that we need a model that can represent undo actions:

https://bugreports.qt.io/browse/QTBUG-95640

As we use macros (for reasons I don't remember), it should be possible to use the text of each macro to identify the command, even after removing the duplicate classes.