-
Notifications
You must be signed in to change notification settings - Fork 3.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Tooling] Support undo/redo #14042
Comments
I have a generic script for "Commands" that you could wrap all the existing functionality in pretty easy. https://github.com/Pryme8/Doomish/blob/main/src/Editor/Commands/Command.ts Not sure if this would do it for you, but it has worked in every project that I have made that needs undo redo. You can see how its used by the editor here: and then a simple example command would be like this: This can also be tied into a notification system that stacks little popups to the user notifying them when they have ran/undone a command etc. If you are interested let me know, if not no worries! |
Hi, I'm curious if there are any updates on this feature? It would really improve the ux of the editor :) |
This is still on our agenda. It is simply not that trivial with our current architecture. If we want to deliver something, we want to deliver it the best we can. |
I understand, thanks for the reply! |
The link you provided cannot be opened, I would like to refer to it |
Since NME etc. are mainly based on JSON it might be very simple to use immer and Json patches? Immer however produces immutable data structures. And every change on the json would need to be wrapped by immer then. But it is pretty convenient. |
This issue has been automatically staled because it has been inactive for more than 14 days. Please update to "unstale". |
Are you making progress |
This would improve user experience on NME, GUI Editor, etc... And a generic solution can be used in future tools.
The text was updated successfully, but these errors were encountered: