emacs-pretest-bug
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: problems with face-spec-reset-face and set-face-attribute


From: Eli Zaretskii
Subject: Re: problems with face-spec-reset-face and set-face-attribute
Date: Wed, 20 Jul 2005 20:45:35 +0300

> From: "Drew Adams" <address@hidden>
> Date: Sun, 17 Jul 2005 21:09:06 -0700
> 
> emacs -q
> (setq pop-up-frames t)
> M-x list-faces-display
> Click mouse-2 on completions-first-difference. It shows Inherit: bold.
> M-: (set-face-attribute 'completions-first-difference nil :inherit
> 'unspecified)
> C-x 5 f whatever (create a new frame)
> M-x list-faces-display (from the new frame)
> Click mouse-2 on completions-first-difference. It shows "Inherit: bold" (the
> original value).
> 
> However, doing list-faces-display from the same buffer where you did
> `set-face-attribute', and clicking completions-first-difference, shows
> "Inherit: unspecified".

Thanks for the recipe.

This seems to be a bug in internal-set-lisp-face-attribute that is
specific to the :inherit attribute.  When I try your recipe with other
attributes, like :foreground or :slant, everything works as expected.
(A good face to try this is `fringe': when it changes, you see the
effect immediately.)

>     If FRAME is t, then set-face-attribute _does_ set the attribute for
>     new frames.  If it doesn't for you, please post a complete test case
>     to reproduce that bug.
> 
> Same test as above, with t instead of nil. The only difference is that the
> value is not changed for any existing frames (which is normal). In both
> cases, new frames do not have the `unspecified' value for the face's
> :inherit property - they still have value `bold'.

Likewise: try with something other than :inherit, it works as
advertised.

I'll look into the problem with :inherit when I have time, unless
someone beats me to it.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]