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

User configuration is buggy #6098

Closed
a1994846931931 opened this issue Sep 4, 2019 · 5 comments
Closed

User configuration is buggy #6098

a1994846931931 opened this issue Sep 4, 2019 · 5 comments
Labels
bug bugs found in the application help wanted issues meant to be picked up, require help preferences issues related to preferences

Comments

@a1994846931931
Copy link
Contributor

a1994846931931 commented Sep 4, 2019

Description

Summary:

  1. Change of user configuration doesn't take effect
  2. Editor of the user configuration is buggy

1. Change of user configuration doesn't take effect

Change Workspace Configuration (good - takes a while to take effect though):
workspace-configuration-valid

Change User Configuration (buggy):
user-configuration-invalid

2. Editor of the user configuration is buggy

"Add Value" Doesn't Refresh the Editor
user-configuration-editor-2

Editor's Weird Behaviour
user-configuration-editor
Notice how the user configuration editor becomes dirty every time I click on it. It only happens with "editor.formatOnSave" on. In addition, the font size configuration doesn't affect user configuration editor.

Reproduction Steps

See gifs.

OS and Theia version:
OS: Ubuntu 18.04
Theia: 4852766

Diagnostics:

@akosyakov
Copy link
Member

@a1994846931931 workspace settings are overriding user settings, are you sure that workspace settings don't contain editor.formatOnSave when you edit user settings?

@akosyakov akosyakov added bug bugs found in the application help wanted issues meant to be picked up, require help preferences issues related to preferences labels Sep 4, 2019
@a1994846931931
Copy link
Contributor Author

@a1994846931931 workspace settings are overriding user settings, are you sure that workspace settings don't contain editor.formatOnSave when you edit user settings?

@akosyakov Do you mean in "Editor's Weird Behaviour"? Yes, it does. The workspace settings contain editor.formatOnSave, and it's set to true.

@akosyakov
Copy link
Member

@a1994846931931 oh, sorry, i meant editor.fontSize and Change User Configuration (buggy):. It looks like an event is missing, we have a good coverage of tests for resolving preferences but not emitting change events.

@a1994846931931
Copy link
Contributor Author

a1994846931931 commented Sep 4, 2019

@a1994846931931 oh, sorry, i meant editor.fontSize and Change User Configuration (buggy):. It looks like an event is missing, we have a good coverage of tests for resolving preferences but not emitting change events.

@akosyakov No. As you can see in the gif, I've deleted settings.json ( thus no Workspace is shown in Preferences widget ) before testing on user configuration.

@colin-grant-work
Copy link
Contributor

I'm unable to reproduce this behavior after #7105.

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 help wanted issues meant to be picked up, require help preferences issues related to preferences
Projects
None yet
Development

No branches or pull requests

3 participants