-
Notifications
You must be signed in to change notification settings - Fork 29k
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
Unable to write to dirty settings file #19161
Comments
Related to #19625 |
We can write into the opened buffer but saving the file automatically might be problematic. Because user might have changes which he does not want to save. One approach is to inform user about saving the Settings for the action to take affect |
Enable auto save for settings, my suggestion :) |
@sandy081 I get a nice error dialog to open settings, however I do not get the option to |
@isidorn Please try when the file is dirty without any errors. If there are errors, then I show a message to open and fix errors. |
Hei, similary, would not allow to rewrite because of diry file { please help |
Now we have more and more menu actions that write immediatly to settings, for instance toggle word wrap.
If you have a dirty settings.json all these actions are broken as I always get the erros that I am unable to write to a dirty settings file
I believe this can hurt a lot of our users as they usualy do not have auto save on, and showing an error message in this case looks bad imho.
The text was updated successfully, but these errors were encountered: