srcery-colors / srcery-emacs

The dark color scheme for Emacs/Spacemacs. GUI and terminal support
https://srcery.sh
131 stars 6 forks source link

Fringe needs to define a background ? #2

Closed gagbo closed 5 years ago

gagbo commented 5 years ago

I'm still not sure how I'm supposed to develop themes when I pull them from melpa using use-package, but if/when I find out I can take care of this.

The fringe face does not define the :background, so the background of the fringe just stays as it was defined. In this screenshot we can see a deep blue/gray column between cursor and line numbers because that's the background I defined in my .Xresources; but when changing theme from light to dark for example, the background of the fringe will stay white, so white strips are pillarboxing my buffers when I do the switch

fringe_srcery

To reproduce

Now you should have white columns pillarboxing the buffer

roosta commented 5 years ago

Added a fix. See commit above. As for developing a theme, what I usually do is just edit whatever I need in the theme then m-x load-file and the changes are active. There is an option in use-package called :load-path that could possibly be used to point it to a local location.

gagbo commented 5 years ago

Seems good to me !