Solarized Light scheme is not correct. #17802

Closed
opened 2026-01-31 05:53:51 +00:00 by claunia · 1 comment
Owner

Originally created by @kgfly on GitHub (Jun 26, 2022).

Windows Terminal version

1.13.11431.0

Windows build number

10.0.19044.1766

Other Software

No response

Steps to reproduce

In VS22 dev profile, add

            "colorScheme": "Solarized Light",

Expected Behavior

For warning string, the font color does not have any special color.

image

Actual Behavior

waring string should have a different color than normal font color. etc, in above, blue circle should different color than red circle

Originally created by @kgfly on GitHub (Jun 26, 2022). ### Windows Terminal version 1.13.11431.0 ### Windows build number 10.0.19044.1766 ### Other Software _No response_ ### Steps to reproduce In VS22 dev profile, add "colorScheme": "Solarized Light", ### Expected Behavior For warning string, the font color does not have any special color. ![image](https://user-images.githubusercontent.com/12282281/175831412-f498253f-43a8-41ac-a4c6-f08eeeabcfed.png) ### Actual Behavior waring string should have a different color than normal font color. etc, in above, blue circle should different color than red circle
Author
Owner

@zadjii-msft commented on GitHub (Jul 5, 2022):

DAYS SINCE I'VE REGRETTED SHIPPING SOLARIZED: 301 0

I'm suspecting that this tool is outputting text in "bright yellow", which in the default Solarized scheme is basically just gray.

image

Alas, by design, solarized doesn't have both a bright and dark yellow - it's only got one yellow. We're following the Solarized spec here, but feel free to modify it by hand to have both yellow and brightYellow be the same value.

@zadjii-msft commented on GitHub (Jul 5, 2022): ### [DAYS SINCE I'VE REGRETTED SHIPPING SOLARIZED: ~~301~~ 0](https://github.com/microsoft/terminal/issues/11171#issuecomment-914624178) I'm suspecting that this tool is outputting text in "bright yellow", which in the default Solarized scheme is basically just gray. ![image](https://user-images.githubusercontent.com/18356694/177303775-fb8f5beb-0746-4e94-b13f-632dbb483816.png) Alas, by design, solarized doesn't have both a bright and dark yellow - [it's only got one yellow](https://ethanschoonover.com/solarized/). We're following the Solarized spec here, but feel free to modify it by hand to have both `yellow` and `brightYellow` be the same value.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#17802