ComfyUI_frontend icon indicating copy to clipboard operation
ComfyUI_frontend copied to clipboard

Improve Undo/Redo - Grouped Changes Mishandled

Open bryanray opened this issue 1 year ago • 3 comments

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

a

Additional information

No response

bryanray avatar Sep 06 '24 13:09 bryanray

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.

planb788 avatar Sep 07 '24 10:09 planb788

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

mcmonkey4eva avatar Sep 26 '24 08:09 mcmonkey4eva

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

bryanray avatar Sep 29 '24 08:09 bryanray

Resolved by #1022

webfiltered avatar Nov 23 '24 02:11 webfiltered