fniessen / emacs-leuven-theme

This Emacs theme reduces eye strain with a light, high-contrast color scheme, syntax highlighting, and support for multiple modes. Enhance your coding experience! #emacs #theme #coding #orgmode
GNU General Public License v3.0
696 stars 59 forks source link

sx.el colors missing and unreadable #23

Closed dakrone closed 8 years ago

dakrone commented 9 years ago

Colors for sx.el are missing, which causes the headline to show up like:

screenshot_2015-09-29_07-08-45

And makes it really difficult to read the white-on-yellow

fniessen commented 9 years ago

Would you like to come up with a proposition, preferably as a code block which I can directly inject into the theme file, or as a GitHub PR?

brabalan commented 8 years ago

I guess the same colors are used for elfeed: screen shot 2016-01-14 at 14 54 22

I have no idea how themes work, however. I had a quick look at the code, and it seems the first step is to find the name of the face that one wants to change. Is it possible through the interface (taking into account the fact that I cannot move the cursor to that point), or can it only be done by looking at the code?

brabalan commented 8 years ago

I’ve looked into this, and both modes use mode-line-buffer-id. Elfeed declares a face for it:

  (defface elfeed-search-filter-face
    '((t :inherit mode-line-buffer-id))
    "Face for showing the current Elfeed search filter."
    :group 'elfeed)

so it can be easily changed, but sx does not:

(defconst sx-inbox--header-line
  '("    "
    (:propertize "n p j k" face mode-line-buffer-id)
    ": Navigate"
    "    "

@dakrone I guess you should file an issue with sx.el asking them to create a face that inherits from mode-line-buffer-id so that the leuven theme could customize it.

fniessen commented 8 years ago

@brabalan Thanks for looking into it.

Your recommandation is clearly the right one! Best regards.

brabalan commented 8 years ago

@fniessen here is some code you could add to the theme for elfeed:

;; elfeed
`(elfeed-search-filter-face ((,class (:foreground "dark gray"))))

(edit: I tried a few variations and a little darker looks better.)

fniessen commented 8 years ago

Pushed.