ranger / colorschemes

Repository that collects ranger colorschemes implemented by the community
GNU General Public License v3.0
55 stars 20 forks source link

solarized colortheme issue: fg/bg color of copied item #6

Open rayx opened 5 years ago

rayx commented 5 years ago

When running ranger (with its solarized theme) in gnome-terminal (with solarized-dark theme), the fg and bg color of copied item are very close, which makes the item's text almost impossible to read. Below are steps to reproduce the issue:

1) Install solarized-dark theme for gnome-terminal. See https://github.com/aruhier/gnome-terminal-colors-solarized 2) Setup ranger to use solarized theme: set colorscheme solarized 3) Start ranger and press yy to yank one file to copy buffer, then move cursor down to next item so that the copied item is visible.

Then you should notice that the copied item's text is hard to recognize because its fg and bg colors are close.

The issue is gone if changing gnome-terminal's default fg and bg color to something other than "solaried dark". The procedure is: 1) Right click in gnome-terminal, choose Preference, a window should pop up. 2) Click Colors tab, and change Built-in schemes under Text and Background Color from Solarized dark to something like Solarized light. 3) Click Close button to close the window.

I observed the issue on Ubuntu 18.04.2 LTS and GNOME Terminal 3.28.2 using VTE 0.52.2, though I don't think the versions matter.

I think the following factors lead to the issue:

As you see, the two colors are very close, so using them as fg and bg colors makes the text hard to recognize.

Marked items don't have this issue because their fg and bg colors are swapped.

if context.selected:
    attr = reverse

It seems to me the only workaround is to modify the code to change fg or bg color, but it's hard to find values that work in all situations.

Any suggestions?

toonn commented 5 years ago

Sounds like a great fix to PR : ) (I'm transferring this issue to our colorschemes repo.)

rayx commented 5 years ago

Just FYI. I work around the issue by modifying fg value to 223. It's sort of a hack because it probably doesn't work with solarized-light theme.