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

Option to disable autosave. #482

Open
mbledkowski opened this issue May 5, 2022 · 11 comments
Open

Option to disable autosave. #482

mbledkowski opened this issue May 5, 2022 · 11 comments

Comments

@mbledkowski
Copy link

Hi, I would like to ask you to add an option to disable autosave. It would be useful for the people that keep their minder files in git repositories.
Best regards,
Maciej Błędkowski @mbledkowski

@phase1geo
Copy link
Owner

I'm not sure I understand the issue with auto-saving to a file that is managed by GitHub. How does this cause issues?

@mbledkowski
Copy link
Author

mbledkowski commented May 6, 2022

The issue in my case is that I keep minder mindmap with other code in one repository. When I want to check something in the mindmap, minder automatically makes changes to the file and saves them (moving around for some reason changes the data in the file).

@phase1geo
Copy link
Owner

The mindmap file is changed because Minder keeps track of the scroll position so that if the app is closed and reopened, the app looks the same as it did before the close which is a usability feature of elementary OS apps.

@phase1geo
Copy link
Owner

The elementary OS HIG is also the reason why the app auto saves. I don't wish to alter this feature of the app.

@mbledkowski
Copy link
Author

I am asking just for an option in app preferences, I am not saying that it should be disabled by default or anything like that.

@phase1geo
Copy link
Owner

FYI, I'm in the process of implementing a change for #503 that should at least fix the issue of Minder saving scroll and zoom changes to the Minder file. I'm not indicating that I won't consider adding a manual save feature to preferences, but at least that change should help to alleviate the bulk of your issues.

@daveiro
Copy link

daveiro commented Nov 9, 2022

Hi, just to add my support to have such option in the preferences; I've lost work a few times after a very difficult to reproduce bug that rarely happens makes many nodes disappear (involving cutting and pasting); I have to go back to some backup while if the autosave was disabled I wouldn't lose my work...

@titorgla
Copy link

titorgla commented Dec 6, 2023

2023, and this option has to exist, on multiple occasions I have lost hours of work when minder would suddenly crash, which would happen repeatedly when:

  • I undo too many times
  • I cut, copy or paste more than 50 nodes
  • some kind of error related to assertion failed

minder crashing in itself would not be that much of a problem if it weren't for the auto-save 'saving' my work before the crash taking the nodes or other info stored in its clipboard down with it

@mikdmst1
Copy link

mikdmst1 commented Mar 5, 2024

yea, please disable autosave, it crashed my work!

@phase1geo
Copy link
Owner

Do you have a stack trace of the crash?

@phase1geo
Copy link
Owner

I guess what I'm getting at here is to know if the problem was actually in the save code or not. If the code to save a mindmap has a bug which leads to an application crashing, that same could could cause the application to crash when manually saving. Understanding and fixing the source of the crash is more important than disabling auto-save.

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

5 participants