GNU bug report logs - #67404
30.0.50; Issue with :box face attribute

Previous Next

Full log

Message #8 received at 67404 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: David Ponce <da_vid <at> orange.fr>, Mauro Aranda <maurooaranda <at> gmail.com>
Cc: 67404 <at> debbugs.gnu.org
Subject: Re: bug#67404: 30.0.50; Issue with :box face attribute
Date: Sun, 26 Nov 2023 12:27:23 +0200

> Date: Thu, 23 Nov 2023 16:23:15 +0100
> From: David Ponce <da_vid <at> orange.fr>
> 
> Hello,
> 
> The below face with specified :box attribute can be defined and further
> used without problem:
> 
> (defface test-face
>    '((t
>       (:box (:style released-button :line-width (3 . 0)))
>       ))
>    "Test face.")
> 
> (insert (propertize "XXXXX" 'face 'test-face))
> 
> However, it is not possible to apply any change to this face with
> `customize-face', for example to change :line-width (3 . 0) to
> :line-width (2 . 0), or to add a :foreground color, etc.  This fails
> with the message "set-face-attribute: Invalid face box: :line-width,
> <value>, :style, <value>", and the :box attribute is discarded.
> 
> AFAICS, :box attribute with any zero :line-width value works as
> expected: the box line with zero width is not displayed.  But zero is
> not considered as a valid value by `set-face-attribute' (in fact, the
> internal function `internal-set-lisp-face-attribute').
> 
> For consistency, a zero :line-width value either should be accepted by
> `internal-set-lisp-face-attribute', or never accepted at all.

Mauro, any comments about these issues?

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.