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

Not all changes to workspace preference take effect #5446

Closed
elaihau opened this issue Jun 13, 2019 · 2 comments · Fixed by #5498
Closed

Not all changes to workspace preference take effect #5446

elaihau opened this issue Jun 13, 2019 · 2 comments · Fixed by #5498
Labels
bug bugs found in the application preferences issues related to preferences

Comments

@elaihau
Copy link
Contributor

elaihau commented Jun 13, 2019

Description

Changes to some preferences don't take effect.

Reproduction Steps

  1. open preference editor
  2. update git.decorations.colors in the "User" tab of pref editor. At this point, changes take effect. Everything looks normal.
  3. switch to the "Workspace" tab of pref editor. Update git.decorations.colors.

what happens: Changes in the "Workspace" tab don't take effect
expected: changes in the "Workspace" tab should take effect, as Workspace preferences should override those in User preference.

Peek 2019-06-13 08-26

OS and Theia version:

Gitpod. Chrome browser. This problem can also be reproduced in the example browser running in Chrome, Ubuntu 16.04.

Theia version: master branch as of June 13 2019 https://github.com/theia-ide/theia/tree/741876b3ecba8a968ecf1200eb231639eb3b4074

Diagnostics:

  1. Changes made to editor.font in "Workspace" tab worked properly.

  2. Changes made to git.decorations.colors in the "Workspace" tab fires resource.onDidChangeContents, and there are no logical issues in the preference change handler either. Error occurred in this line, where this.getScope() returns 3 on the change made to the "Workspace" tab and schemaValidator considered it invalid.

@elaihau elaihau added bug bugs found in the application preferences issues related to preferences labels Jun 13, 2019
@elaihau
Copy link
Contributor Author

elaihau commented Jun 13, 2019

Could it be related to the refactoring made to preferences? @akosyakov

@akosyakov
Copy link
Member

@elaihau yes, it can be, one has to debug though to see the root cause.

elaihau pushed a commit to elaihau/theia that referenced this issue Jun 17, 2019
- FolderPreferenceProvider should be associated with PreferenceScope.Workspace when it is used as a delegate of WorkspacePreferenceProvider in a one-folder workspace, otherwise changes made to the workspace preferences would not be applied to the IDE.
- fixed eclipse-theia#5446

Signed-off-by: elaihau <[email protected]>
elaihau pushed a commit to elaihau/theia that referenced this issue Jun 25, 2019
- FolderPreferenceProvider should be associated with PreferenceScope.Workspace when it is used as a delegate of WorkspacePreferenceProvider in a one-folder workspace, otherwise changes made to the workspace preferences would not be applied to the IDE.
- fixed eclipse-theia#5446

Signed-off-by: elaihau <[email protected]>
elaihau pushed a commit to elaihau/theia that referenced this issue Jun 26, 2019
- FolderPreferenceProvider should be associated with PreferenceScope.Workspace when it is used as a delegate of WorkspacePreferenceProvider in a one-folder workspace, otherwise changes made to the workspace preferences would not be applied to the IDE.
- fixed eclipse-theia#5446

Signed-off-by: elaihau <[email protected]>
elaihau pushed a commit that referenced this issue Jun 26, 2019
- FolderPreferenceProvider should be associated with PreferenceScope.Workspace when it is used as a delegate of WorkspacePreferenceProvider in a one-folder workspace, otherwise changes made to the workspace preferences would not be applied to the IDE.
- fixed #5446

Signed-off-by: elaihau <[email protected]>
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.

2 participants