unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: "Juanma Barranquero" <lekktu@gmail.com>
To: "Miles Bader" <miles@gnu.org>
Cc: Drew Adams <drew.adams@oracle.com>, emacs-devel@gnu.org
Subject: Re: face-remapping-alist client code
Date: Mon, 2 Jun 2008 17:09:36 +0200	[thread overview]
Message-ID: <f7ccd24b0806020809l2d99004dwa340a5395df57bba@mail.gmail.com> (raw)
In-Reply-To: <87tzgbzyjq.fsf@catnip.gol.com>

On Mon, Jun 2, 2008 at 4:44 PM, Miles Bader <miles@gnu.org> wrote:

> The size calculations I'm using are the same as before -- a simple
> constant factor height scaling for each step -- and probably will
> occasionally miss a step on platforms with only non-scalable fonts and
> poor font size coverage.  With scalable fonts, however, this simple
> calculation works very well.

With Vera Sans Mono, and using a smaller scale step (1.07) I get all
point sizes from 13 to 24 except for 19 and 23. Goint down it hits
every size, of course, though some of them appear twice because in
some steps the scaling is not big enough to reduce the font. All in
all, is quite useful and simple. Thanks.

A couple of small nitpicks:

 - the variables `text-scale-mode-lighter',
`text-scale-mode-remapping', `text-scale-mode-amount' and
`variable-pitch-mode-remapping' would  make more sense as
automatically buffer local, as they are always buffer-local'ized when
used.

 - the lighter for variable-pitch-mode is a bit of redundant, isn't
it? Either the user has a variable-pitch face different from the
default one (and so he doesn't need the reminder in the modeline), or
the mode is not going to be useful anyway...

 Juanma




  parent reply	other threads:[~2008-06-02 15:09 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-06-02 10:27 face-remapping-alist client code Miles Bader
2008-06-02 10:42 ` David Reitter
2008-06-02 10:45   ` Miles Bader
2008-06-02 13:13 ` Miles Bader
2008-06-02 14:19 ` Drew Adams
2008-06-02 14:44   ` Miles Bader
2008-06-02 14:50     ` Miles Bader
2008-06-02 15:09     ` Juanma Barranquero [this message]
2008-06-02 15:22       ` Miles Bader
2008-06-02 15:28         ` David Kastrup
2008-06-02 18:55           ` Stefan Monnier
2008-06-02 20:23             ` Drew Adams
2008-06-02 15:39         ` Juanma Barranquero
2008-06-02 18:54         ` Stefan Monnier
2008-06-02 15:56     ` Drew Adams
2008-06-02 15:09 ` Chong Yidong

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=f7ccd24b0806020809l2d99004dwa340a5395df57bba@mail.gmail.com \
    --to=lekktu@gmail.com \
    --cc=drew.adams@oracle.com \
    --cc=emacs-devel@gnu.org \
    --cc=miles@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).