unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Sebastian Wiesner <swiesner@lunaryorn.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: 18077@debbugs.gnu.org
Subject: bug#18077: 24.4.50; Info-quoted face should inherit default
Date: Fri, 25 Jul 2014 16:03:20 +0200	[thread overview]
Message-ID: <2877876.NhWE4son0T@lunaryorn> (raw)
In-Reply-To: <8338dpk58h.fsf@gnu.org>

[-- Attachment #1: Type: text/plain, Size: 3317 bytes --]

Am Freitag, 25. Juli 2014, 15:11:42 schrieb Eli Zaretskii:
> > From: Sebastian Wiesner <swiesner@lunaryorn.com>
> > Cc: 18077@debbugs.gnu.org
> > Date: Fri, 25 Jul 2014 12:25:47 +0200
> > 
> > > Would you like to suggest a patch for fixing this problem?  Please
> > > note that the underlying issue here is that the font should provide
> > > nice-looking glyphs for the Unicode quote characters.
> > 
> > I do not think that I am in a position to suggest a patch that could
> > generally fix this problem.  I use Emacs only in two different
> > environments, and have no idea of what limits and constraints Emacs must
> > obey with regards to fonts to remain portable across various platforms.
> > 
> > I can only say that for my case it would be entirely sufficient to simply
> > make Info-quoted inherit from default, i.e. use '((:inherit default)) as
> > spec in defface.
> > 
> > But I only use a GUI Emacs, and only on very recent and modern systems
> > (Arch Linux with KDE, and OS X 10.9), where Unicode fonts are less of a
> > problem.  On these systems Emacs typically picks a font with good unicode
> > coverage for the default face automatically, e.g. DejaVu Sans Mono or a
> > variant thereof.
>
> Character coverage is not the problem.  The problem is that some
> widely used fonts that cover these characters have ugly or barely
> visible glyphs for them.

I see.  I doubt, though, that "courier" has generally better quote glyphs than 
those  decent modern fonts that are typically used for the default face, 
either automatically or by user customization.

Take a look at the attached images, which show the quote characters of various 
fonts in Emacs on both of my systems:  

On OS X, the quote characters of Courier aren't any better than that of the 
standard monospace faces Menlo and Monaco.  On Linux, Courier isn't even anti-
aliased.  And in both cases the quote characters of Courier are worse than 
those of Source Code Pro and Inconsolata, which are popular 3rd party fonts.

Now, I understand that my setup is just some example and by no means 
authoritative, and that Courier may look significantly different and much 
better on your Linux system.

But I'm trying to demonstrate with these examples that Courier is by no means 
*guaranteed* to have good quote characters, and can in fact be significantly 
worse than other fonts.  

IOW, there's a chance that Courier gets it right, and there's also a good 
chance that Courier gets it wrong.  And that goes for the default face as 
well:  It can either have good quotes, or bad ones. 

However, the default fonts of modern systems are typically not that bad, and 
users typically don't choose very poor fonts for their "default" face, so I 
think it's less likely for "default" to get it wrong that for "Info-quoted".  
And the latter can fail really bad, as seen in my Arch Linux example.

 
> > inheriting from default seems to be the standard among other
> > built-in faces.
> 
> I would trust Stefan (who introduced this face) that he knows that for
> quite some time, and had his reasons for not doing that.

I'd trust Stefan to the end of the universe, for sure, but he might just have 
been lucky to have a proper Courier variant installed on his own systems.  
Even he can't test a face on each and every font setup out there :)

[-- Attachment #2: OS X.png --]
[-- Type: image/png, Size: 13674 bytes --]

[-- Attachment #3: Arch Linux.png --]
[-- Type: image/png, Size: 9662 bytes --]

  reply	other threads:[~2014-07-25 14:03 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-22  8:15 bug#18077: 24.4.50; Info-quoted face should inherit default swiesner
2014-07-25  8:25 ` Eli Zaretskii
2014-07-25 10:25   ` Sebastian Wiesner
2014-07-25 12:11     ` Eli Zaretskii
2014-07-25 14:03       ` Sebastian Wiesner [this message]
2014-07-27 23:30         ` Juri Linkov
2014-07-28  8:12           ` Stefan Monnier
2021-04-03 23:58             ` Stefan Kangas
2021-04-04  7:33               ` Eli Zaretskii
2014-07-25 22:37 ` Stefan Monnier
2014-07-25 22:55   ` Sebastian Wiesner
2014-07-26  8:03     ` Stefan Monnier
2014-07-26  8:58       ` Sebastian Wiesner
2014-07-27 10:16         ` Stefan Monnier
2014-07-27 20:04           ` Sebastian Wiesner
2014-07-27 23:23             ` Stefan Monnier
2014-07-28  8:33               ` Sebastian Wiesner
2014-07-26  7:47   ` Andreas Schwab

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=2877876.NhWE4son0T@lunaryorn \
    --to=swiesner@lunaryorn.com \
    --cc=18077@debbugs.gnu.org \
    --cc=eliz@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).