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

The borders of the modeline are gone #3456

Closed
ernestas opened this issue Oct 18, 2015 · 9 comments
Closed

The borders of the modeline are gone #3456

ernestas opened this issue Oct 18, 2015 · 9 comments

Comments

@ernestas
Copy link
Contributor

The borders are gone in the develop branch. If this is a feature, please close this issue.

spacemacs-no-borders

System Info

  • OS: gnu/linux
  • Emacs: 24.5.1
  • Spacemacs: 0.105.0
  • Spacemacs branch: develop (rev. bd6716b)
  • Distribution: spacemacs
  • Layers:
(auto-completion better-defaults emacs-lisp git github restclient org markdown
                 (shell :variables shell-default-height 30 shell-default-position 'bottom)
                 (clojure :variables clojure-enable-fancify-symbols t)
                 javascript)
@TheBB
Copy link
Collaborator

TheBB commented Oct 18, 2015

We moved away from using our solarized fork and went with the solarized that's on MELPA. What you're probably seeing is the lack of those local modifications we had made. You can modify the modeline faces yourself, of course:

  • mode-line
  • mode-line-inactive
  • powerline-active1 and 2
  • powerline-inactive1 and 2

@ernestas
Copy link
Contributor Author

I have added this:

  (set-face-attribute 'mode-line nil :box '(:color "#839496" :line-width 1))
  (set-face-attribute 'mode-line-inactive nil :box '(:color "#586e75" :line-width 1))
  (set-face-attribute 'powerline-active1 nil :box '(:color "#839496" :line-width 1))
  (set-face-attribute 'powerline-inactive1 nil :box '(:color "#586e75" :line-width 1))
  (set-face-attribute 'powerline-active2 nil :box '(:color "#839496" :line-width 1))
  (set-face-attribute 'powerline-inactive2 nil :box '(:color "#586e75" :line-width 1))

but still not getting the full borders

spacemacs-no-borders-2

Will it be possible to pass in the :line-width value with the new theming layer #3486 ?

@TheBB
Copy link
Collaborator

TheBB commented Oct 20, 2015

That's strange. I don't know why that's not working for you. As for your question, :line-width works fine for me with #3486 and the Monokai theme.

@TheBB
Copy link
Collaborator

TheBB commented Oct 20, 2015

For the record, that layer is implemented using custom-set-faces and not set-face-attribute. I don't know if it matters. The docstring of the latter seems to imply it's primarily for internal use so I didn't want to touch it.

@TheBB
Copy link
Collaborator

TheBB commented Oct 21, 2015

I just tried with solarized light and I can reproduce your issue. Must be something weird about that theme. I get the same with solarized dark.

@razum2um
Copy link
Contributor

same with solarized (both themes)
it's definitely something wrong with them as sanityinc-solarized-dark works (both)

emacs macpro local 2015-10-22 06-58-58

@ernestas
Copy link
Contributor Author

bbatsov/solarized-emacs#174 looks related.

@robbyoconnor
Copy link
Contributor

I have since moved away from solarized -- I didn't like how it handled magit...so I now use monokai...It's not my favorite but it doesn't make my eyes bleed either...

@TheBB
Copy link
Collaborator

TheBB commented Jan 21, 2016

I think this can be closed now.

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

No branches or pull requests

5 participants