[debbugs-tracker] bug#13875: closed (24.3.50; whitespace-mode should use 'prepend for whitespace-line face)

[GNU bug Tracking System]

Your message dated Tue, 19 Mar 2013 08:24:05 +0400
with message-id <address@hidden>
and subject line Re: bug#13875: 24.3.50; whitespace-mode should use 'prepend 
for whitespace-line face
has caused the debbugs.gnu.org bug report #13875,
regarding 24.3.50; whitespace-mode should use 'prepend for whitespace-line face
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
13875: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=13875
GNU Bug Tracking System
Contact address@hidden with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> 24.3.50; whitespace-mode should use 'prepend for whitespace-line face
>
>
>
>
> Date: 
>
> Tue, 05 Mar 2013 20:36:21 +0400
>
>
>
>
> Tags: patch
>
> Right now, there's no way to tell whitespace-mode to keep the foreground
> color of the underlying text when highlighting lines that are too long.
>
> Related SO question:
> http://stackoverflow.com/questions/14636786/how-to-unset-the-foreground-color-of-whitespace-mode-for-emacs
>
> With the patch at the bottom applied, the following code will make it
> keep the text's foreground and just change background:
>
>   (set-face-attribute 'whitespace-line nil :foreground nil
>                                            :background "gainsboro")
>
> (Changing :backround is not necessary, but gray20 looks bad with the
> default foreground color.)
>
> === modified file 'lisp/whitespace.el'
> --- lisp/whitespace.el  2013-01-11 23:08:55 +0000
> +++ lisp/whitespace.el  2013-03-05 16:25:51 +0000
> @@ -2291,7 +2291,7 @@
>          (if (memq 'lines whitespace-active-style)
>              0                          ; whole line
>            2)                           ; line tail
> -        whitespace-line t))
> +        whitespace-line 'prepend))
>         t))
>      (cond
>       ((memq 'space-before-tab whitespace-active-style)
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#13875: 24.3.50;	whitespace-mode should use 'prepend for whitespace-line face
>
>
>
>
> Date: 
>
> Tue, 19 Mar 2013 08:24:05 +0400
>
>
>
>
> User-agent: 
>
> Mozilla/5.0 (Windows NT 6.1; WOW64;	rv:17.0) Gecko/20130307 Thunderbird/17.0.4
>
>
>
>
> Version: 24.4
>
> On 18.03.2013 2:27, Stefan Monnier wrote:
> > > -        whitespace-line t))
> > > +        whitespace-line 'prepend))
> >
> > No objection on my part,
>
> Thanks, installed.
>
>
> --- End Message ---