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

Back to classic Solarized powerline, except better #1018

Closed
wants to merge 1 commit into from

Conversation

trishume
Copy link
Contributor

@trishume trishume commented Apr 1, 2015

Use a more classic Spacemacs Powerline colours for Solarized Light. Unlike the old powerline this one does have better text contrast and no weird light grey empty bit. This change does not affect the new high-contrast powerline and only very subtly tweaks the look of the Solarized Dark powerline.

It's my personal belief that this does not need to be config variable gated but I'm open to it.
I personally can't see why anyone would want the super light version without contrasting sections, but it is highly subjective.

However I think @ryanartecona's high contrast light powerline looks pretty awesome so I left that in (see bbatsov/solarized-emacs#169). The dark powerline also looked fine as it is.

New but also old (this PR)

screen shot 2015-03-31 at 5 20 44 pm

Old but also new (on develop as of 9885406)

old but new powerline

@trishume
Copy link
Contributor Author

trishume commented Apr 2, 2015

As of bbatsov/solarized-emacs#174 @ryanartecona's version looks much better so I retract my decision and think this should probably be feature gated. The question is if it should be the default or alternate.

@syl20bnr
Copy link
Owner

I tried it and now I find the original spacemacs colors worst than the new ones -__________-
(especially because of the aliasing).

@trishume
Copy link
Contributor Author

Yah I am used to the new ones as well now :|
This was just old-timer aversion to change I guess, just needed some time.

Also bbatsov/solarized-emacs#174 should make it even better.

@trishume trishume closed this Apr 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants