Re: [storm@cua.dk: Re: Patch to disable links line in *info* buffer]

[gerd.moellmann@t-online.de (Gerd Moellmann)]

Richard Stallman <rms@gnu.org> writes:

> Would you please help us with this?  You know this code more
> than anyone.
> 
> From: storm@cua.dk (Kim F. Storm)
> Subject: Re: Patch to disable links line in *info* buffer
> To: rms@gnu.org
> Cc: "Stefan Monnier" <monnier+gnu/emacs@rum.cs.yale.edu>, jasonr@gnu.org,
>    bob@rattlesnake.com, emacs-devel@gnu.org
> Date: 22 Jun 2002 01:56:31 +0200
> 
> no-spam@cua.dk (Kim F. Storm) writes:
> 
> > "Stefan Monnier" <monnier+gnu/emacs@rum.cs.yale.edu> writes:
> > 
> > > > The problem is that there is no named face which has this appearence,
> > > > so I cannot simply put a (face mode-line-bold) property on the string
> > > > returned by format-mode-line.
> > > 
> > > Any reason why you can't use (face (mode-line bold)) ?
> > 
> > Not anymore :-)
> 
> Ok, I have changed format-mode-line to return a propertized string
> (changes are not committed yet), but the "face merging" doesn't always
> give the same result as the actual mode line:
> 
> This combination works (*Help* is bold):
> 
> (insert '#("-1:%%  *Help*  ..."
>          7 13 (face (mode-line :weight bold))))
> 
> but this doesn't (*Help* isn't bold):
> 
> (insert '#("-1:%%  *Help*  ..."
>          7 13 (face (mode-line-inactive :weight bold))))

The above list form of face property value doesn't look right.  I
think it should be something like (mode-line (:weight bold)) instead.
Each element of the list should be either a named face or a list of
face attributes; see also the Lisp Reference.  The face property value
above looks like it will use the non-existent named face `:weight'.

Since faces from a list-valued face property are merged from left to
right (AFAIR), the faces in the list should also be reversed:
((:weight bold) mode-line).  Otherwise, if mode-line explicitly
specifies a weight, the (:weight bold) will have no effect.