Subject: Re: bug#51821: 29.0.50; Suggest add variable or frame parameter: line-height

From: Eli Zaretskii <[email-masked]>
Date: Sat, 20 Nov 2021 12:15:55 +0200

> From: Lars Ingebrigtsen <[email-masked]>
> Cc: [email-masked],  [email-masked],  [email-masked]
> Date: Sat, 20 Nov 2021 10:58:14 +0100
> 
> Eli Zaretskii <[email-masked]> writes:
> 
> >> Allow saying:
> >> 
> >> (defface separator-line
> >>   '((((type graphic) (background dark))
> >>      :pixel-height 1 :background "#505050")
> >
> > How is that different from :height?  It uses different units, but
> > that's not a problem for this use case, is it?
> 
> It's a way to say "don't scale this, ever", which was a problem here,
> wasn't it?  (And :height 0.1 doesn't give me a single pixel on this
> display -- we have no way to express that currently.)

No, float values are relative to the default face, which is not what
you want.  I meant ":height 1" or somesuch.  If that is not good
enough, the value of :height can be a function.