NodeGraphProcessor icon indicating copy to clipboard operation
NodeGraphProcessor copied to clipboard

Undo will lose all group view !

Open bbsuuo opened this issue 3 years ago • 3 comments

when undo , it's will reload graph view, and it's will remove all elements

remove element will call GraphViewChangedCallback function

and when remove element, group view will check this node on group , and remove from base graph data

it's will lose all group view ,this kind of incorrect call can lead to many potential problems later

bbsuuo avatar Feb 25 '22 06:02 bbsuuo

I just changed the order of the clear group view ()

the order of calls should clear the group first

and i remove graphViewChanged before reload and add later

some callback chain messed up , i try to code refactoring , and the code and ideas are valuable

bbsuuo avatar Feb 25 '22 06:02 bbsuuo

I also want to know how to solve it in the end?

DJ-Huang avatar May 25 '23 13:05 DJ-Huang