-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
preferences: toggling 'auto-save' should update user settings #9152
Comments
This issue is still open. |
Can I go a head to work on it |
@AbhiSahane0 unfortunately I do not believe it is a straighforward issue for beginners to pick up, we will require investigation, debugging and in the end discussions on how to fix the preferences-system (which is core to the framework) in order to achieve a proper solution rather than a patch fix. |
Okk , Sir |
I'll pick this up, since I'm pretty familiar with the workings of the preference system. |
I am not sure if this is a bug. I also prefer the VS Code UX, (save this config with a user scope) but you cannot just change the runtime behavior that downstreams have been using for years now. What about adjusting it for electron? Thoughts? |
Bug Description:
The framework incorrectly updates the
editor.autoSave
preference when toggled (through the menu or command), by updating the value in the workspacescope
rather thanuser
. We should align the behavior to use theuser
scope by default similarly to vscode.autoSave.mp4
Steps to Reproduce:
rm -rf ~/.theia
)..theia
should be present).autoSave
command..theia
folder is created in the workspace with the setting.The text was updated successfully, but these errors were encountered: