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

Selected text is very low contrast in light theme #21

Closed
jojanzing opened this issue Feb 7, 2024 · 4 comments
Closed

Selected text is very low contrast in light theme #21

jojanzing opened this issue Feb 7, 2024 · 4 comments
Assignees

Comments

@jojanzing
Copy link

Great implementation of the solarized theme, thank you!

Only one issue: selected text highlight is almost invisible while in the light theme, as the highlight colour barely differs from the background colour. In dark theme, selected text highlights as expected.

@edheltzel
Copy link
Owner

@jojanzing, could you send me a screenshot of the issue?

@edheltzel edheltzel self-assigned this Feb 8, 2024
@pintergreg
Copy link

pintergreg commented Apr 15, 2024

Képernyőkép_20240415_150731

The pivot_table part is selected, which is indicated only by the visible white spaces. It is practically unusable for me in this way.

edit: I forgot to tell, that this happens only in a Jupyter notebook environment.

Version: 1.88.0
Commit: 5c3e652f63e798a5ac2f31ffd0d863669328dc4c
Date: 2024-04-05T09:25:05.142Z
Electron: 28.3.0
ElectronBuildId: undefined
Chromium: 120.0.6099.291
Node.js: 18.18.2
V8: 12.0.267.19-electron.0
OS: Linux x64 6.8.5-arch1-1

@pintergreg
Copy link

Could be fixed using customization:

"workbench.colorCustomizations":{
  "notebook.editorBackground": "#eae0c0",
  "notebook.cellEditorBackground": "#fdf6e3"
},

Képernyőkép_20240416_115306

@edheltzel
Copy link
Owner

Awesome, I can push this through a little later today or you're welcome to create a PR

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

No branches or pull requests

3 participants