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

string color hard to see color in dark theme #8158

Closed
blaylockbk opened this issue Apr 2, 2020 · 9 comments · Fixed by #8181
Closed

string color hard to see color in dark theme #8158

blaylockbk opened this issue Apr 2, 2020 · 9 comments · Fixed by #8181
Labels
status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion. tag:Design and UX
Milestone

Comments

@blaylockbk
Copy link

I love the dark theme, but the dark red string color is very difficult to see against the dark background (I have been accused of color blindness)

--jp-mirror-editor-string-color: #ba2121;

Can it be changed to have a higher contrast ratio against the black background

from #ba2121
image

to #ff7070
image


I changed this in my personal environment, but thought it might be helpful for other people using Jupyter's Dark Theme.

Before...
image

After...
image

@jasongrout
Copy link
Contributor

CC @tgeorgeux

@jasongrout
Copy link
Contributor

See also #7779, which advocates for much larger changes. I agree the red is hard to read - perhaps just this could be changed with much less controversy?

@jasongrout
Copy link
Contributor

I changed this in my personal environment,

Just curious, how did you change this?

@blaylockbk
Copy link
Author

I changed the line in the file

~/anaconda3/envs/best/share/jupyter/lab/themes/@jupyterlab/theme-dark-extension/index.css

@jasongrout
Copy link
Contributor

Thanks. I was wondering if you had used an "official" way to change the color (like making your own theme), or if you just hacked it in.

@tgeorgeux
Copy link
Contributor

That change looks great to me. I ran it through webAims contrast checker to be sure and it passes with flying colors!

Screen Shot 2020-04-02 at 11 51 17 AM

@jasongrout jasongrout added this to the 2.1 milestone Apr 2, 2020
@jasongrout
Copy link
Contributor

@tgeorgeux - how about you make the final call on if this is the shade of red we want, and merge it if it's good, or if you want to tweak it more.

@tgeorgeux
Copy link
Contributor

I think this color is better than what we're currently using, the changes in #7779 are larger and need more consideration, but for 2.1 this is a good change that we should implement. I can make a PR to try to get this in before Monday.

@blaylockbk
Copy link
Author

Thanks for making the change! I think a lot of people will appreciate it.

@lock lock bot added the status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion. label May 20, 2020
@lock lock bot locked as resolved and limited conversation to collaborators May 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion. tag:Design and UX
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants