emacs-devel
[Top][All Lists]
Advanced

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

Re: 23.0.60; Different heights for customize faces


From: Juri Linkov
Subject: Re: 23.0.60; Different heights for customize faces
Date: Sun, 11 May 2008 23:10:50 +0300
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/23.0.60 (x86_64-pc-linux-gnu)

>>>>> The heights for custom-variable-tag-face and custom-face-tag differs.
>>>> Sorry, I see no bug.
>>> Are those too faces the same height? Any one else seeing different heights?
>>
>> If you what to say that different heights look ugly then I argee.
>
> Eh, yes. I could perhaps have been clear, but I just could not imagine
> that the different heights were by intent.

It is currently defined as 1.2 height:

(defface custom-face-tag
  `((t (:weight bold :height 1.2 :inherit variable-pitch)))
  "Face used for face tags."
  :group 'custom-faces)

>> Also using the blue color for custom-variable-tag-face makes option names
>> similar to links.  Maybe we should use the same colors that font-lock
>> defines for font-lock-constant-face and font-lock-variable-name and just
>> make them bold?
>
> I think that would be worth to try.

Could you propose a better color?

Meanwhile, I like to fix the original problem you reported
by the following patch.  It changes the appearance of
custom-face-tag to be the same as custom-variable-tag.
These option types can be still distinguished in the
Customization buffer because faces have the explicit
text "face:" appended after the face name.

Index: lisp/cus-edit.el
===================================================================
RCS file: /sources/emacs/emacs/lisp/cus-edit.el,v
retrieving revision 1.346
diff -u -r1.346 cus-edit.el
--- lisp/cus-edit.el    6 May 2008 07:57:28 -0000       1.346
+++ lisp/cus-edit.el    11 May 2008 20:10:11 -0000
@@ -3218,7 +3218,7 @@
 ;;; The `custom-face' Widget.
 
 (defface custom-face-tag
-  `((t (:weight bold :height 1.2 :inherit variable-pitch)))
+  `((t :inherit custom-variable-tag))
   "Face used for face tags."
   :group 'custom-faces)
 ;; backward-compatibility alias

-- 
Juri Linkov
http://www.jurta.org/emacs/




reply via email to

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