Skip to content

fix Solarized Light theme highlighting #24040#24292

Merged
aeschli merged 2 commits intomicrosoft:masterfrom
Ikuyadeu:solarizedLight
Apr 12, 2017
Merged

fix Solarized Light theme highlighting #24040#24292
aeschli merged 2 commits intomicrosoft:masterfrom
Ikuyadeu:solarizedLight

Conversation

@Ikuyadeu
Copy link
Contributor

@Ikuyadeu Ikuyadeu commented Apr 8, 2017

Fix solarized Light Themes Highlighting solve #24040 .
solarizedlight

Hex is
"editorLineHighlight": "#FDF6C0",
"editorSelection": "#FDF6AA"

Should I make config others?(ex: sideBarBackground)

@mention-bot
Copy link

@Ikuyadeu, thanks for your PR! By analyzing the history of the files in this pull request, we identified @egamma to be a potential reviewer.

@aeschli
Copy link
Contributor

aeschli commented Apr 10, 2017

@Ikuyadeu Sure, you can also fill in workbench colors such as 'sideBarBackground'. That's on our todo list..

@Ikuyadeu
Copy link
Contributor Author

Ikuyadeu commented Apr 11, 2017

@aeschli Thank you for review.
I updated color for sidebar etc...
But if you want to only fix #24040, please only use first commit or close.
newsolarized

@aeschli aeschli added the themes Color theme issues label Apr 12, 2017
@aeschli aeschli added this to the April 2017 milestone Apr 12, 2017
@aeschli aeschli merged commit 22946e2 into microsoft:master Apr 12, 2017
@aeschli
Copy link
Contributor

aeschli commented Apr 12, 2017

Thanks @Ikuyadeu. Looks good!

@Ikuyadeu Ikuyadeu deleted the solarizedLight branch April 12, 2017 08:35
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

themes Color theme issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants