Skip to content
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

Add auto-save #549

Open
AlexBxl opened this issue Dec 2, 2024 · 0 comments
Open

Add auto-save #549

AlexBxl opened this issue Dec 2, 2024 · 0 comments

Comments

@AlexBxl
Copy link
Contributor

AlexBxl commented Dec 2, 2024

Overview

I get why this is not done in other graph editors, with async execution of nodes etc you really don't want to auto save. But manual saving is not an expected behavior for web documents anymore, I lost a ton of work because I just closed/refreshed the browser.

Always save the state, if it needs some external execution, don't persist the execution!

When the user goes back to the document, prompt them if they want to load the current state or the saved state. If your graphs have no side effects just save it.

/rant I really hate this UI pattern for graph editors 😆

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant