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

Low contrast for ANSI colors #72

Closed
dominicegginton opened this issue Apr 16, 2021 · 1 comment · Fixed by #73
Closed

Low contrast for ANSI colors #72

dominicegginton opened this issue Apr 16, 2021 · 1 comment · Fixed by #73

Comments

@dominicegginton
Copy link

Not sure if this is a primitives issue but am opening just in case

GItHub's VSCode theme uses primers primitives ANSI color values for the integrated terminal color theme. There are several open issues in regards to low contrast (mostly pointing to the GitHub Light Default theme) rendering some text unreadable. Is this something the primer team can look into?

Please see primer/github-vscode-theme#157 & primer/github-vscode-theme#155

Screenshots

GitHub Light Default GitHub Dark Default GitHub Dark Dimmed
screenshot-Sat-17Apr21_00 17 screenshot-Sat-17Apr21_00 18 screenshot-Sat-17Apr21_00 18_000
@simurai
Copy link
Contributor

simurai commented Apr 19, 2021

Was wondering about the low contrast in VS Code's terminal too and I think the problem is that on github.com the Checks (that use the ANSI colors) have a "custom" dark background in light mode:

Screen Shot 2021-04-19 at 10 29 15

So the ANSI colors in light mode are kinda meant for a dark mode which doesn't really work outside of github.com. 🤔

Hmmm.. maybe we can look into a way where we can have a "permanent theme" (dark checks in light mode)? I guess as a short term fix we could add custom checks-ansi colors that work for dotcom and change the existing ansi colors to work on a light background.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants