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

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

Open
rayx opened this issue Jul 16, 2019 · 2 comments
Open

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

rayx opened this issue Jul 16, 2019 · 2 comments
Labels
bug Something isn't working

Comments

@rayx
Copy link

rayx commented Jul 16, 2019

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:

  • Ranger solarized theme uses 256 colors

  • It also uses default fg and bg color of the terminal. This is where the terminal's theme comes into play.

  • In ranger's solarized theme, the fb color of copied item is 234 (see code below). Googling it on the net shows that it's #1c1c1c.

      if not context.selected and (context.cut or context.copied):
          fg = 234
          attr |= bold
    
  • In gnome-terminal's solarized-dark theme, the default bg coloar is #002B36

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?

@rayx rayx changed the title fg/bg issue with copied item when running ranger (with solarized theme) in gnome-terminal (with solarized-dark theme) solarized colortheme: issue with copied item's fg/bg color Jul 16, 2019
@rayx rayx changed the title solarized colortheme: issue with copied item's fg/bg color solarized colortheme issue: fg/bg color of copied item Jul 16, 2019
@toonn
Copy link
Member

toonn commented Jul 16, 2019

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

@toonn toonn transferred this issue from ranger/ranger Jul 16, 2019
@rayx
Copy link
Author

rayx commented Jul 17, 2019

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.

@toonn toonn added the bug Something isn't working label Jul 23, 2019
nhap69 added a commit to nhap69/colorschemes that referenced this issue Jul 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants