[feature request] undo-redo

[PladsElsker]

I would like to be able to accidentally delete 40 nodes, and hit "ctrl-z" to bring them back with the correct states. An action manager would be quite nice, although I understand this feature can be annoying to implement in an already existing software.

[paulo-coronado]

Undo/Redo would be a nice addition to the UI! :)

[Natotela]

I keep banging my head everytime I do something and "OH right, there's no ctrl+z"

[bmad4ever]

https://gist.github.com/bmad4ever/9b35f6857ff9d1be3d79549c839356be

This is not an action manager, and is somewhat dirty and untested, but should help.

[bmad4ever]

Just finished implementing a significantly more robust solution as an extension:
https://github.com/bmad4ever/ComfyUI-Bmad-DirtyUndoRedo

[Lex-DRL]

Ummm... bump?

I consider such functionality essential. Even though, yeah, it's a real pain in the... (somwhere) to implement.

[comfyanonymous]

#2080

[PladsElsker]

Shouldn't we wait for the PR to be merged before closing this?

[Lex-DRL]

It's already merged, isn't it?
Works for me, with Bmad's custom node removed (which you should do).

However, same as Bmad's implementation, the one in the core resets all the previews on undo. While under the hood all the nodes stay marked as calculated. So each time you ctrl+Z, you also have to change something at the very start of your graph and re-calculate everything. It makes you fear of ctrl+z, especially when your image is generated at stages.

[PladsElsker]

Hmm not sure why I thought it was not merged, I probably got confused with another PR.

[PladsElsker]

Ah it's arnon-1's PR that's not merged. It implements ctrl+z as well.

[arnon-1]

NOOO, I got rejected! XD