[debbugs-tracker] bug#3408: closed (customize-face not working: seems to apply to frame-face)

[GNU bug Tracking System]

Your message dated Sun, 25 Nov 2012 13:04:32 +0800
with message-id <address@hidden>
and subject line Re: bug#3408: bug#3210: face customization fails after 
set-face-attribute
has caused the debbugs.gnu.org bug report #3408,
regarding customize-face not working: seems to apply to frame-face
to be marked as done.

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


-- 
3408: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=3408
GNU Bug Tracking System
Contact address@hidden with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> customize-face not working: seems to apply to frame-face
>
>
>
>
> Date: 
>
> Wed, 27 May 2009 20:40:37 -0400
>
>
>
>
> Emacs -Q
>
> (set-face-attribute 'default nil
>                     :height 130
>                     :family "Lucida Grande")
> ;; switches default face
>
> (customize-face 'default)
> ;; switch back manually to Monaco
>
> (make-frame-command)  ;; C-x 5 2
> ;; the new frame is shown in the Lucida face.  Why?
>
> The same thing works as I would expect in Emacs 22.  I would generally  
> assume that `customize-face' applies to faces in future frames as well.
>
> If this is no bug, I think an entry in NEWS under "Faces" would be  
> appropriate....
>
> Cc'ing Bob Halley who isolated this (but doesn't think it's a bug).
>
>
> PS.: text-scale-mode is mentioned twice in NEWS.
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#3408: bug#3210: face customization fails after	set-face-attribute
>
>
>
>
> Date: 
>
> Sun, 25 Nov 2012 13:04:32 +0800
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13) Emacs/24.2.90 (gnu/linux)
>
>
>
>
> "Drew Adams" <address@hidden> writes:
>
> > Bug #3408 is a _regression_, and it has not been fixed.
> > A regression is certainly NOT `notabug'.
>
> I disagree that this is a bug.  I have now taken another look at the
> code, and my earlier conclusion stands.  Modifications to internal Lisp
> faces (via set-face-attribute) take precedence over the effect of face
> specs, so the behavior is working as intended and as documented
> (including in the docstring of set-face-attribute).
>
> The trunk now contains a change to face-spec-set which allows it to be
> more easily called from Lisp code to override defface specs.  If you
> want to modify faces from Lisp, use that rather than set-face-attribute.
>
> Don't re-open this bug based on uninformed opinion.  Thanks.
>
>
> --- End Message ---