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

Workbench Appearance color theme keeps being overridden #105102

Closed
noelrappin opened this issue Aug 20, 2020 · 6 comments
Closed

Workbench Appearance color theme keeps being overridden #105102

noelrappin opened this issue Aug 20, 2020 · 6 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug editor-theming Issues related to the way themes are applied to editors insiders-released Patch has been released in VS Code Insiders verified Verification succeeded
Milestone

Comments

@noelrappin
Copy link

  • VSCode Version: 1.48.1
  • OS Version: Mac 10.15.6

Steps to Reproduce:

  1. In Settings, set a "Preferred Dark Color Theme", a "Preferred Light Color Theme"
  2. Set the "Color Theme" to the same as the "Preferred Light Color Theme"
  3. Have 'window.autoDetectColorScheme' enabled
  4. Restart VS Code or open any new window

The Workbench Color Theme will be reset to Default Light+ and that theme will be re-adapted as part of the restart process.

Does this issue occur when all extensions are disabled?: No

But I'm not sure which extension is causing this.

@vscodebot
Copy link

vscodebot bot commented Aug 20, 2020

@noelrappin
Copy link
Author

This might be related to #102795?

@IllusionMH
Copy link
Contributor

Looks more like duplicate of #91231

@noelrappin
Copy link
Author

Maybe, but it's not clear to me that the other issue is actually changing the theme on already open windows if I open a new window, which is the behavior that I'm seeing.

@aeschli aeschli modified the milestones: On Deck, August 2020 Aug 24, 2020
@aeschli
Copy link
Contributor

aeschli commented Sep 1, 2020

It's a bug. The preferred theme should only be applied when OS color scheme is detected.
That's either when the the scheme change event happens while VSCode is open, or when we detect a scheme change when the window is opened. For that we remember the last seen scheme.

Also, we should not modify the workspace theme, only the user theme.

@aeschli aeschli added bug Issue identified by VS Code Team member as probable bug editor-theming Issues related to the way themes are applied to editors labels Sep 1, 2020
@aeschli aeschli closed this as completed in 73ce0a4 Sep 1, 2020
@aeschli
Copy link
Contributor

aeschli commented Sep 3, 2020

To verify (on Windows or Mac):

  • Have 'window.autoDetectColorScheme' enabled and set a "Preferred Dark Color Theme", a "Preferred Light Color Theme"
  • switch the OS color scheme (for dark to light or vice versa). Verify that the theme in VSCode changes to one of the preferred themes
  • use the theme picker and change to any other theme
  • restart VSCode: Verify that no theme switch happens
  • close VSCode, switch the OS color scheme, start VS Code: verify that the theme in VSCode changes.

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 editor-theming Issues related to the way themes are applied to editors insiders-released Patch has been released in VS Code Insiders verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

5 participants
@noelrappin @IllusionMH @aeschli @miguelsolorio and others