Skip to content
This repository has been archived by the owner on May 7, 2021. It is now read-only.

Guru highlighting colour makes identifier text hard to read #621

Closed
mkraft opened this issue Apr 13, 2017 · 3 comments
Closed

Guru highlighting colour makes identifier text hard to read #621

mkraft opened this issue Apr 13, 2017 · 3 comments

Comments

@mkraft
Copy link

mkraft commented Apr 13, 2017

Guru highlighting makes the text of the highlighted identifier difficult to read with the following popular theme combination:

UI Theme: Atom Dark
Syntax Theme: Solarized Dark

screen shot 2017-04-13 at 8 01 01 am

It's a great feature that I want to keep enabled, but I also don't want to change my theme combination. Could we please make it configurable or change the colour for everyone?

@joefitzgerald
Copy link
Owner

This is actually a theme issue; we're just using the selected background color (

background-color: @background-color-selected;
). atom/solarized-dark-syntax#85 seems to describe how it should look with those combinations. The question is: why does yours look different?

@zmb3
Copy link
Collaborator

zmb3 commented Apr 17, 2017

I'm noticing the same behavior with the solarized light syntax theme.

@zmb3
Copy link
Collaborator

zmb3 commented Apr 17, 2017

It turns out @background-color-selected is for selections in the UI (tree-view, for example). It is not the background color for selected text. For that, we should be using @syntax-selection-color from syntax-variables.less.

Current behavior (@background-color-selected):

screen shot 2017-04-17 at 10 11 24 am

With fix (@syntax-selection-color):

screen shot 2017-04-17 at 10 12 22 am

Should be a super quick fix. I'll get to it tonight if someone doesn't beat me to it.

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

No branches or pull requests

3 participants