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

Preference changes not registered when settings.json open in editor. #7721

Closed
colin-grant-work opened this issue May 1, 2020 · 0 comments · Fixed by #7775
Closed

Preference changes not registered when settings.json open in editor. #7721

colin-grant-work opened this issue May 1, 2020 · 0 comments · Fixed by #7775
Labels
bug bugs found in the application preferences issues related to preferences

Comments

@colin-grant-work
Copy link
Contributor

Bug Description:

This bug is a follow up to ongoing work on #7105. When a preference file is open in the editor, changes in the GUI are written to the file, but the file is not saved. In VSCode, if the file is not dirty when a preference change is made in the GUI, the file is automatically saved.

Steps to Reproduce:

  1. Using the code from Added Preferences UI widget #7105,
  2. Open the preferences GUI
  3. Open the preferences file corresponding to your current scope (e.g. by clicking the {} in the upper right.)
  4. Modify preferences in the GUI
  5. Observe that the preferences changes are not registered by the GUI (no blue bars appear), that the file is marked as dirty, and that the preference changes are written to it, but not saved.

Additional Information

The current behavior is governed by a call to applyBackgroundEdit here:

await this.workspace.applyBackgroundEdit(this.model, editOperations);

That method explicitly does not save the file if the editor is open.

  • Operating System: RHEL7
  • Theia Version: Master (1.1.0)
@vince-fugnitto vince-fugnitto added the preferences issues related to preferences label May 2, 2020
@akosyakov akosyakov added the bug bugs found in the application label May 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bugs found in the application preferences issues related to preferences
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants