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

Fix search marker translucency (currently ignoring scheme) #1490

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Galland
Copy link

@Galland Galland commented May 3, 2017

On filetypes common it says: "# translucency for the line marker(first argument) and the search marker (second argument)
marker_translucency=255;255"
However changing the 2nd argument does NOT affect search marker translucency.
The proposed change addresses this.

On the downside: current hard-coded value of 60 (very transparent) would become filetypes.common's default of 255 (no transparency).

On the upside:

  1. we would be consistent with what is indicated in filetypes.common about translucency for the search marker
  2. this change may help all DARK themes, since marked words are barely visible no matter the GCS_MARKER_SEARCH background color used. By changing the translucency we can make them standout, just like in white themes.

On filetypes common it says: "# translucency for the line marker(first argument) and the search marker (second argument)
marker_translucency=255;255"
However changing the 2nd argument does NOT affect search marker translucency.
The proposed change addresses this.

On the downside: current hard-coded value of 60 (very transparent) would become filetypes.common's default of 255 (no transparency).

On the upside:
1) we would be consistent with what is indicated in filetypes.common about translucency for the search marker
2) this change may help all DARK themes, since marked words are barely visible no matter the GCS_MARKER_SEARCH background color used. By changing the translucency we can make them standout, just like in white themes.
@elextr
Copy link
Member

elextr commented May 4, 2017

Agree its a problem worth solving,

The MARKER_TRANSLUCENCY background value has already been used for the translucency of a margin marker. It is probably better not to use it for two unrelated things, and not to change an existing usage.

I don't know where the comment in filetypes.common came from, the translucency has been fixed as far back as I looked (2010). The setting being used for a margin marker may be a confusion between "mark", "marker" and "indicator" that is used as a marker.

The thing to do is to make a new setting adding it to filetypes.common as a named style with the default 60 and then using the named style in the styling section. Then colour schemes can change it to suit themselves.

Would probably be good to make translucency and marker_translucency named styles as well "while you are there" ;-).

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

Successfully merging this pull request may close these issues.

None yet

2 participants