Improve Undo/Redo - Grouped Changes Mishandled

Comfy-Org / ComfyUI_frontend

Official front-end implementation of ComfyUI

https://www.comfy.org/

GNU General Public License v3.0

602 stars 108 forks source link

Improve Undo/Redo - Grouped Changes Mishandled #750

Open bryanray opened 2 months ago

bryanray commented 2 months ago

Is there an existing issue for this?

[X] I have searched the existing issues and checked the recent builds/commits

What would your feature do ?

Feature Idea UNdo + REdo = FUNction!!! Please!!!!

right now the undo redo function are little too much shabby.

Proposed workflow

Additional information

No response

planb788 commented 2 months ago

I believe the current undo function is not well implemented. When we batch copy a bunch of nodes to a certain location, what needs to be undone is the batch copy action, not individually undoing each node that was just copied.

mcmonkey4eva commented 1 month ago

Going to ignore original post content as Undo/Redo is in fact a working feature, but the comment from planb is valid - grouped changes, notably copying a group of nodes all at once, is mishandled as a series of individual changes instead of correctly tracked as one bulk action

bryanray commented 1 month ago

it is not working as it should, like the undo redo from photoshop, it has a list of history and everything is recorded.