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

Background color incorrect in some terminals #16

Open
PythonNut opened this issue Jan 20, 2015 · 5 comments
Open

Background color incorrect in some terminals #16

PythonNut opened this issue Jan 20, 2015 · 5 comments

Comments

@PythonNut
Copy link

This theme seems to suffer from a similar issue to bbastov/solarized-emacs #18. In particular, the background face is a bright blue, when the theme is loaded in a terminal. For example, here is a screenshot of emacs -Q in a GNOME terminal.

color_theme_sanityinc_dolarized_dark

I can reproduce s similar result in xfce4-terminal. In contrast the canonical vim solarized theme colors the background a dark grey, although I don't necessarily agree with some of the other color choices.
selection_003

@purcell
Copy link
Owner

purcell commented Jan 20, 2015

This is probably a rounding issue: we only specify full sRGB colour values, but the terminal will quantise those colours into its palette of 16 or 256 colours. Or perhaps Emacs does that quantisation, and that's what causes the clumsy rounding. In any case, I see the same behaviour on my own machine, in iTerm on OS X.

The solution would probably be to extend the theme with 256-colour-palette definitions for all the colours, but that's a lot of work and I'm unlikely to do it in the near future.

@localredhead
Copy link

Hello - I think this can be mitigated by installing a solarized-dark theme for iTerm2 itself. This is the only way I got semi-accurate solarized-dark in the terminal.

I should note that the solarized-dark in this manner was still slightly too dark but it was manageable.

@cpaulik
Copy link

cpaulik commented Mar 18, 2016

What about introducing a setting that would make this compatible with a terminal configured to use the 16 solarized colors?

@purcell
Copy link
Owner

purcell commented Mar 18, 2016

@cpaulik How would that work?

@cpaulik
Copy link

cpaulik commented Mar 19, 2016

Take a look at https://github.com/sellout/emacs-color-theme-solarized/blob/master/solarized-definitions.el They use brightblack for base03 and so on if solarized-termcolors is set to 16 and emacs is started in a terminal. So in this theme you could introduce a variable like solarized-termcolors that then uses the 16 standard terminal color names that you already have in your list

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

No branches or pull requests

4 participants