unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Stephen Gildea <stepheng+emacs@gildea.com>
Cc: emacs-devel@gnu.org
Subject: Re: anti-alias fix results in ugly scaled fonts
Date: Sat, 15 Oct 2022 10:32:30 +0300	[thread overview]
Message-ID: <83k0517d9t.fsf@gnu.org> (raw)
In-Reply-To: <517174.1665805249@pental> (message from Stephen Gildea on Fri, 14 Oct 2022 20:40:49 -0700)

> From: Stephen Gildea <stepheng+emacs@gildea.com>
> Date: Fri, 14 Oct 2022 20:40:49 -0700
> 
> At HEAD, I get an ugly scaled font for my sans-serif text.
> I noticed it in MH-E, and you can reproduce it as follows:
> 
>     emacs -Q -fn fixed --eval "(buffer-face-set 'variable-pitch)"
> 
> Although the default font is specified on the command line as "fixed"
> (from the misc-fixed X11 family of fonts), and the variable-pitch font
> selected is not in the misc-fixed family, having "fixed" as the
> default font affects the font selected for the "variable-pitch" face.
> Which font it selects changed recently.
> 
> This bug seems to have been introduced by commit 6b1ed2f2c9,
> "Fix antialias face attribute when text is scaled", 27 August.
> 
> In an Emacs built before that commit, the above command displays text
> with a pretty sans-serif font.

The above works fine on my system: I get a "fixed" font without the
"--eval" argument, and variable-pitch font with it, as expected.

It sounds like so do you, but you don't like the variable-pitch font
you get, is that so?  If so, then the problem is with the fonts you
have installed, not necessarily with what Emacs produces.  That is,
assuming you do get variable-pitch font in the buffer, why are you
saying that Emacs doesn't perform correctly?  The font being "ugly" is
something only you know.

I'm guessing that what you see is due to some attributes of the
"fixed" font used as the default face's font.  The commit you
identified as the culprit preserves the attributes of the original
font which the face doesn't override, instead of throwing them away.
So I think the pretty sans-serif font you wanted to see is rejected
because it doesn't fit some of those attributes of the default face's
font.  Can you try figuring out which attribute(s) are those?  It
could be the :weight attribute, for example (the "medium" vs "regular"
issue, which is new in Emacs 29), or maybe :size.

> That commit undoes some of the changes that addressed bug#17973.
> Bug#17973 fixed display of misc-fixed fonts.  With that recent commit,
> using misc-fixed fonts again breaks some faces.

Please tell the details: which faces are broken and how?

P.S. And why isn't this reported as a bug to the bug-tracker, instead
of here?



      reply	other threads:[~2022-10-15  7:32 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-10-15  3:40 anti-alias fix results in ugly scaled fonts Stephen Gildea
2022-10-15  7:32 ` Eli Zaretskii [this message]

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=83k0517d9t.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=stepheng+emacs@gildea.com \
    /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).