Skip to content

Loading…

Trailing whitespace face fix #46

Closed
wants to merge 1 commit into from

2 participants

@sphynx

Hi, Bozhidar!

I have noticed that trailing whitespace visual highlighting is not working when using zenburn theme, so I fixed it. Zenburn used "red" color as foreground, but since whitespace is usually not visible, this foreground color change was not affecting anything.

P.S. Just in case, my whitespace style settings are:

(require 'whitespace)
(setq whitespace-style '(face trailing tabs))

@bbatsov
Owner

Unfortunately it's not as simple as that. Most people that use whitespace mode use it to visualize the spaces as dots and the foreground red color makes those dots stand out for trailing whitespace. Try adding spaces after tabs in your whitespace-style and you'll see what I mean.

@bbatsov bbatsov closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Showing with 1 addition and 1 deletion.
  1. +1 −1 zenburn-theme.el
View
2 zenburn-theme.el
@@ -463,7 +463,7 @@
`(whitespace-hspace ((,class (:background ,zenburn-bg :foreground ,zenburn-bg+1))))
`(whitespace-tab ((,class (:background ,zenburn-bg :foreground ,zenburn-red))))
`(whitespace-newline ((,class (:foreground ,zenburn-bg+1))))
- `(whitespace-trailing ((,class (:foreground ,zenburn-red :background ,zenburn-bg))))
+ `(whitespace-trailing ((,class (:background ,zenburn-red))))
`(whitespace-line ((,class (:background ,zenburn-bg-05 :foreground ,zenburn-magenta))))
`(whitespace-space-before-tab ((,class (:background ,zenburn-orange :foreground ,zenburn-orange))))
`(whitespace-indentation ((,class (:background ,zenburn-yellow :foreground ,zenburn-red))))
Something went wrong with that request. Please try again.