all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* 23.0.60; Different heights for customize faces
@ 2008-05-08  8:16 Lennart Borgman (gmail)
  2008-05-09 23:59 ` Juri Linkov
  0 siblings, 1 reply; 11+ messages in thread
From: Lennart Borgman (gmail) @ 2008-05-08  8:16 UTC (permalink / raw)
  To: emacs-pretest-bug

The heights for custom-variable-tag-face and custom-face-tag differs.


In GNU Emacs 23.0.60.1 (i386-mingw-nt5.1.2600)
  of 2008-05-07




^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: 23.0.60; Different heights for customize faces
  2008-05-08  8:16 23.0.60; Different heights for customize faces Lennart Borgman (gmail)
@ 2008-05-09 23:59 ` Juri Linkov
  2008-05-10 22:17   ` Lennart Borgman (gmail)
  0 siblings, 1 reply; 11+ messages in thread
From: Juri Linkov @ 2008-05-09 23:59 UTC (permalink / raw)
  To: Lennart Borgman (gmail); +Cc: emacs-pretest-bug

> The heights for custom-variable-tag-face and custom-face-tag differs.

Sorry, I see no bug.

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




^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: 23.0.60; Different heights for customize faces
  2008-05-09 23:59 ` Juri Linkov
@ 2008-05-10 22:17   ` Lennart Borgman (gmail)
  2008-05-11  0:06     ` Juri Linkov
  0 siblings, 1 reply; 11+ messages in thread
From: Lennart Borgman (gmail) @ 2008-05-10 22:17 UTC (permalink / raw)
  To: Juri Linkov; +Cc: emacs-pretest-bug

Juri Linkov wrote:
>> 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?




^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: 23.0.60; Different heights for customize faces
  2008-05-10 22:17   ` Lennart Borgman (gmail)
@ 2008-05-11  0:06     ` Juri Linkov
  2008-05-11  0:53       ` Lennart Borgman (gmail)
  0 siblings, 1 reply; 11+ messages in thread
From: Juri Linkov @ 2008-05-11  0:06 UTC (permalink / raw)
  To: Lennart Borgman (gmail); +Cc: emacs-pretest-bug

> Juri Linkov wrote:
>>> 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.
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?

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




^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: 23.0.60; Different heights for customize faces
  2008-05-11  0:06     ` Juri Linkov
@ 2008-05-11  0:53       ` Lennart Borgman (gmail)
  2008-05-11 20:10         ` Juri Linkov
  0 siblings, 1 reply; 11+ messages in thread
From: Lennart Borgman (gmail) @ 2008-05-11  0:53 UTC (permalink / raw)
  To: Juri Linkov; +Cc: emacs-pretest-bug

Juri Linkov wrote:
>> Juri Linkov wrote:
>>>> 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.

> 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.




^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: 23.0.60; Different heights for customize faces
  2008-05-11  0:53       ` Lennart Borgman (gmail)
@ 2008-05-11 20:10         ` Juri Linkov
  2008-05-11 22:35           ` Lennart Borgman (gmail)
  2008-07-29 17:14           ` bug#626: " Juri Linkov
  0 siblings, 2 replies; 11+ messages in thread
From: Juri Linkov @ 2008-05-11 20:10 UTC (permalink / raw)
  To: Lennart Borgman (gmail); +Cc: emacs-pretest-bug

>>>>> 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/




^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: 23.0.60; Different heights for customize faces
  2008-05-11 20:10         ` Juri Linkov
@ 2008-05-11 22:35           ` Lennart Borgman (gmail)
  2008-05-13  0:07             ` Juri Linkov
  2008-07-29 17:14           ` bug#626: " Juri Linkov
  1 sibling, 1 reply; 11+ messages in thread
From: Lennart Borgman (gmail) @ 2008-05-11 22:35 UTC (permalink / raw)
  To: Juri Linkov; +Cc: emacs-pretest-bug

Juri Linkov wrote:
>>> 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?

I think your proposal to reuse the font-lock colors is good. 
`font-lock-variable-name' seems appropriate to inherit in 
`custom-variable-tag-face' - at least as a mnemonic (and that is 
probably good for a quick overview).

Not sure about what `custom-face-tag' could inherit from, but 
font-lock-constant-face is not bad IMO.

> Meanwhile, I like to fix the original problem you reported
> by the following patch.  It changes the appearance of

Thanks.




^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: 23.0.60; Different heights for customize faces
  2008-05-11 22:35           ` Lennart Borgman (gmail)
@ 2008-05-13  0:07             ` Juri Linkov
  2008-05-13  0:39               ` Lennart Borgman (gmail)
  0 siblings, 1 reply; 11+ messages in thread
From: Juri Linkov @ 2008-05-13  0:07 UTC (permalink / raw)
  To: Lennart Borgman (gmail); +Cc: emacs-pretest-bug

>>>> 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?
>
> I think your proposal to reuse the font-lock colors is
> good. `font-lock-variable-name' seems appropriate to inherit in
> custom-variable-tag-face' - at least as a mnemonic (and that is probably
> good for a quick overview).
>
> Not sure about what `custom-face-tag' could inherit from, but
> font-lock-constant-face is not bad IMO.

One thing that it would be more logical, but another thing
how it would really look in Customize.  Does this look good?

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




^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: 23.0.60; Different heights for customize faces
  2008-05-13  0:07             ` Juri Linkov
@ 2008-05-13  0:39               ` Lennart Borgman (gmail)
  0 siblings, 0 replies; 11+ messages in thread
From: Lennart Borgman (gmail) @ 2008-05-13  0:39 UTC (permalink / raw)
  To: Juri Linkov; +Cc: emacs-pretest-bug

Juri Linkov wrote:
>>>>> 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?
>> I think your proposal to reuse the font-lock colors is
>> good. `font-lock-variable-name' seems appropriate to inherit in
>> custom-variable-tag-face' - at least as a mnemonic (and that is probably
>> good for a quick overview).
>>
>> Not sure about what `custom-face-tag' could inherit from, but
>> font-lock-constant-face is not bad IMO.
> 
> One thing that it would be more logical, but another thing
> how it would really look in Customize.  Does this look good?

I avoided that problem ...

If the colors is a problem from that point of view then maybe there 
could be a marker before the text with a color reflection the option 
type. The name of the object could be just black then.




^ permalink raw reply	[flat|nested] 11+ messages in thread

* bug#626: 23.0.60; Different heights for customize faces
  2008-05-11 20:10         ` Juri Linkov
  2008-05-11 22:35           ` Lennart Borgman (gmail)
@ 2008-07-29 17:14           ` Juri Linkov
  2008-07-29 17:26             ` bug#627: " Lennart Borgman (gmail)
  1 sibling, 1 reply; 11+ messages in thread
From: Juri Linkov @ 2008-07-29 17:14 UTC (permalink / raw)
  To: Lennart Borgman (gmail); +Cc: emacs-pretest-bug

> 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.

Fixed.

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






^ permalink raw reply	[flat|nested] 11+ messages in thread

* bug#627: 23.0.60; Different heights for customize faces
  2008-07-29 17:14           ` bug#626: " Juri Linkov
@ 2008-07-29 17:26             ` Lennart Borgman (gmail)
  0 siblings, 0 replies; 11+ messages in thread
From: Lennart Borgman (gmail) @ 2008-07-29 17:26 UTC (permalink / raw)
  To: Juri Linkov; +Cc: emacs-pretest-bug

Thanks.

Juri Linkov wrote:
>> 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.
> 
> Fixed.
> 






^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2008-07-29 17:26 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-05-08  8:16 23.0.60; Different heights for customize faces Lennart Borgman (gmail)
2008-05-09 23:59 ` Juri Linkov
2008-05-10 22:17   ` Lennart Borgman (gmail)
2008-05-11  0:06     ` Juri Linkov
2008-05-11  0:53       ` Lennart Borgman (gmail)
2008-05-11 20:10         ` Juri Linkov
2008-05-11 22:35           ` Lennart Borgman (gmail)
2008-05-13  0:07             ` Juri Linkov
2008-05-13  0:39               ` Lennart Borgman (gmail)
2008-07-29 17:14           ` bug#626: " Juri Linkov
2008-07-29 17:26             ` bug#627: " Lennart Borgman (gmail)

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.