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

Extra quotes in settings json editor of non-current profile #156290

Closed
alexr00 opened this issue Jul 26, 2022 · 11 comments
Closed

Extra quotes in settings json editor of non-current profile #156290

alexr00 opened this issue Jul 26, 2022 · 11 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug config VS Code configuration, set up issues settings-editor VS Code settings editor issues verified Verification succeeded
Milestone

Comments

@alexr00
Copy link
Member

alexr00 commented Jul 26, 2022

Testing #155899

  1. Create 2 profiles, A and B.
  2. Switch to A. Open the settings.json file for the profile. Keep this file open.
  3. Switch to B.
  4. Use suggest to modify the value of a setting in A's json file.
  5. An extra quote will appear.

Recording 2022-07-26 at 14 23 19

@sandy081
Copy link
Member

I cannot reproduce it. Anyway this comes from JSON language server.

CC @aeschli

@aeschli
Copy link
Contributor

aeschli commented Jul 27, 2022

I'm also not able to reproduce
Did you press shift when selecting the completion (insert mode)?

@alexr00
Copy link
Member Author

alexr00 commented Jul 27, 2022

No, I just hit enter.

@aeschli
Copy link
Contributor

aeschli commented Jul 27, 2022

Can you open the settings and see what editor.suggest.insertMode is configured?
It's default setting is insert but json and jsonc overwrite the default to replace. I wonder if there's some problem with the configuration store.

@alexr00
Copy link
Member Author

alexr00 commented Jul 28, 2022

It started as an empty profile, so theme is the only thing I have set. Value of editor.suggest.insertMode:
image

@aeschli
Copy link
Contributor

aeschli commented Jul 28, 2022

JSON and JSONC should also be in that list.
I just tested again, switching to a Test profile and I even get even fewer entries:
image

Looks like there's something off.

@aeschli aeschli removed their assignment Jul 28, 2022
@sandy081
Copy link
Member

@aeschli Do you think it is related to profiles? If so can you please share any insights on what would have caused it?

@aeschli
Copy link
Contributor

aeschli commented Jul 28, 2022

Not clear if it's the settings UI that shows incomplete information, but after switching a profile, not all default language overwrites are shown. If the JSONC language overwrite for editor.suggest.insertMode is not loaded, that would explain what Alex is seeing.

@rzhao271
Copy link
Contributor

I am able to repro the issue and my settings look like #156290 (comment)

@rzhao271 rzhao271 added bug Issue identified by VS Code Team member as probable bug config VS Code configuration, set up issues settings-editor VS Code settings editor issues labels Jul 29, 2022
@rzhao271 rzhao271 added this to the August 2022 milestone Jul 29, 2022
@rzhao271
Copy link
Contributor

rzhao271 commented Aug 8, 2022

I'm unable to repro the issue now. I notice when I switch profiles and hover again over the label that it changes the list of languages depending on the extensions installed, but I always manage to have JSON and JSONC in the list.

@alexr00 are you able to repro the issue in the latest Insiders?

@rzhao271 rzhao271 added the info-needed Issue requires more information from poster label Aug 8, 2022
@alexr00
Copy link
Member Author

alexr00 commented Aug 9, 2022

I cannot repro with the latest insiders, though I could still repro with an insiders from some time last week. Looks like it's fixed.

@alexr00 alexr00 closed this as completed Aug 9, 2022
@rzhao271 rzhao271 removed the info-needed Issue requires more information from poster label Aug 11, 2022
@alexr00 alexr00 added the verified Verification succeeded label Aug 24, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Sep 24, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug config VS Code configuration, set up issues settings-editor VS Code settings editor issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

4 participants