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

[themes] Foregound Color on Modals and Popups not respecting newly selected theme #58216

Closed
imoby opened this issue Sep 7, 2018 · 6 comments
Closed
Assignees
Labels
info-needed Issue requires more information from poster themes Color theme issues

Comments

@imoby
Copy link

imoby commented Sep 7, 2018

  • VSCode Version: 1.27.1
  • OS Version: macOS High Sierra

Steps to Reproduce:

  1. Select a different theme other than dark+ that has themed the foreground different than the default color (example is the shades of purple theme)
  2. Set that as the theme and reload vscode
  3. Open command palette and attempt to change to new theme
  4. Observe that even after selecting a new theme or navigating the themes list, the foreground of the current theme color is being selected instead of the new one or default one

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

Images:
first
issue

@vscodebot vscodebot bot added the new release label Sep 7, 2018
@vscodebot
Copy link

vscodebot bot commented Sep 7, 2018

@aeschli
Copy link
Contributor

aeschli commented Sep 12, 2018

@imoby Thanks a lot for reporting, I've seen something similar, but I'm not able to reproduce with your steps. What theme is that?
Did you by chance define any color customizations in your settings, like:

    "workbench.colorCustomizations": {
		"foreground": "#ff00ff"
	}

@aeschli aeschli added info-needed Issue requires more information from poster themes Color theme issues labels Sep 12, 2018
@aeschli aeschli changed the title Foregound Color on Modals and Popups not respecting newly selected theme [themes] Foregound Color on Modals and Popups not respecting newly selected theme Sep 12, 2018
@aeschli
Copy link
Contributor

aeschli commented Sep 12, 2018

I pushed a fix for what I believe is the problem. Maybe you can try in an insider build if this fixed it for you

@aeschli aeschli added this to the September 2018 milestone Sep 12, 2018
@imoby
Copy link
Author

imoby commented Sep 13, 2018

@aeschli the issue is still existing even in the latest insiders version. Notice the foreground color even though the Dark+ theme is selected

issue still

@imoby
Copy link
Author

imoby commented Sep 13, 2018

@aeschli heres a link to a gif on how to reproduce it https://d.pr/i/PK3oZb

@aeschli
Copy link
Contributor

aeschli commented Sep 13, 2018

@imoby Did you try with 1.28.0-insider
Commit: 42ffc0e
Date: 2018-09-13T05:22:02.945Z

?

@vscodebot vscodebot bot locked and limited conversation to collaborators Oct 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
info-needed Issue requires more information from poster themes Color theme issues
Projects
None yet
Development

No branches or pull requests

2 participants