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

Unchecking "Apply Setting to all Profiles" reverts setting in non-default profile #188794

Closed
chrmarti opened this issue Jul 25, 2023 · 6 comments · Fixed by #188908 or #189130
Closed

Unchecking "Apply Setting to all Profiles" reverts setting in non-default profile #188794

chrmarti opened this issue Jul 25, 2023 · 6 comments · Fixed by #188908 or #189130
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug candidate Issue identified as probable candidate for fixing in the next release insiders-released Patch has been released in VS Code Insiders user-profiles User profile management verified Verification succeeded
Milestone

Comments

@chrmarti
Copy link
Contributor

Testing #188614

  • Default and testing profiles.
  • In default profile change color theme and enable "Apply Setting to all Profiles".
  • Switch to testing profile and uncheck "Apply Setting to all Profiles". The color theme reverts to the previous one. Maybe this is intentional, but I expect it to stay the same.
@sandy081 sandy081 added bug Issue identified by VS Code Team member as probable bug user-profiles User profile management labels Jul 26, 2023
@sandy081 sandy081 added this to the July 2023 milestone Jul 26, 2023
sandy081 added a commit that referenced this issue Jul 26, 2023
sandy081 added a commit that referenced this issue Jul 26, 2023
@vscodenpa vscodenpa added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Jul 26, 2023
@amunger amunger added the verified Verification succeeded label Jul 27, 2023
@amunger
Copy link
Contributor

amunger commented Jul 27, 2023

  1. selected a new theme
  2. selected "Apply settings to all profiles"
  3. switched profiles
    🐛 new theme setting did not apply to the profile I switched to

🐛 Now I can't uncheck "Apply settings to all profiles"

Recording 2023-07-27 at 14 09 46

@amunger amunger reopened this Jul 27, 2023
@vscodenpa vscodenpa removed the insiders-released Patch has been released in VS Code Insiders label Jul 27, 2023
@amunger amunger added verification-found Issue verification failed and removed verified Verification succeeded labels Jul 27, 2023
@amunger
Copy link
Contributor

amunger commented Jul 27, 2023

I noticed after posting the gif that the toggle did turn off at some point, but that wasn't happening very often. Reloading seems to have fixed it though

@amunger
Copy link
Contributor

amunger commented Jul 27, 2023

Here was the first bug, the second one in the gif above isn't happening anymore
Recording 2023-07-27 at 14 18 48

@amunger
Copy link
Contributor

amunger commented Jul 27, 2023

I also tried with only built-in themes - once I had the OP behavior, and on the second try I was unable to uncheck the "Apply settings to all profiles" again

@sandy081
Copy link
Member

sandy081 commented Jul 28, 2023

I noticed after posting the gif that the toggle did turn off at some point, but that wasn't happening very often. Reloading seems to have fixed it though

I could not reproduce this. Can you please check if the setting you are toggling is getting updated in the setting workbench.settings.applyToAllProfiles correctly in the application settings file? To open this file you can use following commands

  • If you are in default profile, "Open User Settings (JSON)"
  • If you are in non default profile, "Open Application Settings (JSON)"

If the setting is getting updated correctly then it could be a UI bug. Please check and if you can repro, file a separate issue with steps. Thanks.

Regarding first issue

selected a new theme
selected "Apply settings to all profiles"
switched profiles
🐛 new theme setting did not apply to the profile I switched to

I can repro and it seems to be a separate issue and not related to this. Hence filed a new issue for it - #189129. Lets track this there.

The fix is not working correctly for the reported issue. Hence reopening it.

@sandy081
Copy link
Member

To verify:

  • Have Default and Testing Profile
  • In Default profile change files.autoSave to afterDelay and enable "Apply Setting to all Profiles".
  • Switch to Testing profile and uncheck "Apply Setting to all Profiles".
  • files.autoSave value should remain to afterDelay

@sandy081 sandy081 added verification-found Issue verification failed and removed verification-found Issue verification failed labels Jul 28, 2023
@sandy081 sandy081 reopened this Jul 28, 2023
@sandy081 sandy081 added the candidate Issue identified as probable candidate for fixing in the next release label Jul 28, 2023
sandy081 added a commit that referenced this issue Jul 28, 2023
sandy081 added a commit that referenced this issue Jul 28, 2023
sandy081 added a commit that referenced this issue Jul 28, 2023
@vscodenpa vscodenpa added the unreleased Patch has not yet been released in VS Code Insiders label Jul 28, 2023
@sandy081 sandy081 removed the verification-found Issue verification failed label Jul 28, 2023
@chrmarti chrmarti added the verified Verification succeeded label Jul 28, 2023
@vscodenpa vscodenpa added insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Aug 3, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Sep 11, 2023
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 candidate Issue identified as probable candidate for fixing in the next release insiders-released Patch has been released in VS Code Insiders user-profiles User profile management verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants