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

Rendered whitespace in selection hard to read #134

Closed
beeb opened this issue Mar 10, 2020 · 7 comments
Closed

Rendered whitespace in selection hard to read #134

beeb opened this issue Mar 10, 2020 · 7 comments

Comments

@beeb
Copy link

beeb commented Mar 10, 2020

When displaying whitespace characters in the selected code via editor.renderWhitespace: selection (which is the new default, by the way), the characters are hard to see. Pictured here is Python source code.

Dracula:
image

Dracula Soft:
image

Not sure how feasible it would be to update editorWhitespace.foreground to add some contrast without it looking ugly when unselected.

@dsifford
Copy link
Member

Open to suggestions you might have, but this is sort of an unfortunate result of how vscode layers their highlights.

See #61 (comment)

(theres a few more issues where there's discussion about highlight over comments with a similar sentiment as well)

@beeb
Copy link
Author

beeb commented Mar 10, 2020

Is there a way to easily try different colors on my end? Or do I need to clone the repo and install a local version or something? I'm not too familiar with VSCode extensions

@dsifford
Copy link
Member

@beeb
Copy link
Author

beeb commented Mar 11, 2020

Thanks! I found that #3b3b3b is just about where the symbols are visible both when unselected and selected. There are possible variations in saturation that work as well, like #3a3b3d maybe.

You might consider changing the indent guides color to the same shade too (didn't do it in the screenshot below)

Screenshot 2020-03-11 at 07 32 38

@beeb
Copy link
Author

beeb commented Mar 11, 2020

Additional option: use transparency, for instance #ffffff1f which gives this:

Screenshot 2020-03-11 at 07 43 21

To be honest this gives very good readability both in selected and unselected state.

@leandrocunha
Copy link
Contributor

I opened a PR for this improvement, really useful suggestion

@beeb
Copy link
Author

beeb commented Mar 19, 2020

Fixed by #135

@beeb beeb closed this as completed Mar 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants