all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "Francesc Rocher" <francesc.rocher@gmail.com>
To: "David Kastrup" <dak@gnu.org>
Cc: lennart.borgman@gmail.com, emacs-devel@gnu.org
Subject: Re: Terrible underline
Date: Mon, 10 Mar 2008 22:15:41 +0100	[thread overview]
Message-ID: <ade856a30803101415oe373e06j5ed8a74db30f1873@mail.gmail.com> (raw)
In-Reply-To: <85myp9kjy6.fsf@lola.goethe.zz>

On Sat, Mar 8, 2008 at 10:02 PM, David Kastrup <dak@gnu.org> wrote:
> "Francesc Rocher" <francesc.rocher@gmail.com> writes:
>
>  >> "Lennart Borgman (gmail)" <address@hidden> writes:
>  >>> David Kastrup wrote:
>  >>>>    "Lennart Borgman (gmail)" <address@hidden> writes:
>  >>>>
>  >>>>        I get terrible underlines with CVS Emacs 23, see the attached
>  >>>>        picture. I guess this has something to do with the new font handling,
>  >>>>        or?
>  >>>
>  >>>    They look fine to me.  Have you looked at the screenshot?  Maybe it
>  >>>    captures the way things should look rather than how they actually do?
>  >>>
>  >> Thanks, yes I looked at the screen shot. There is no space
>  >> between the characters and the underline. To me that is nearly
>  >> unreadable.
>  >>
>  >> It looked much better before the merge I think.
>  >
>  > Yes, you're right. It's time to get support for the variable
>  > 'x-underline-at-descent-line' under win32. Before the merge it
>  > was not necessary, but now this patch should be installed:
>
>  [...]
>
>
>  > Then,
>  >
>  >    (setq x-underline-at-descent-line t)
>  >
>  > should make the underlining readable (it should look as before).
>
>  Why would one want to have an option then?  Can you think of a use case
>  where people would want to have it set to nil?
>
>  --
>  David Kastrup, Kriemhildstr. 15, 44793 Bochum
>

This variable was introduced to force the drawing of the underline
line at the descent line. Please note that the variable
'x-use-underline-position-properties' can make use of the font
properties, but by default the underline line is drawn at the base
line (don't try to use it under win32, it isn't supported yet). I
think the better would be to use the font properties and, probably, to
remove both variables. The question is whether all fonts have reliable
values about the position and width of the underline line.


-- 
Francesc Rocher




  reply	other threads:[~2008-03-10 21:15 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-08 18:57 Terrible underline Francesc Rocher
2008-03-08 21:02 ` David Kastrup
2008-03-10 21:15   ` Francesc Rocher [this message]
  -- strict thread matches above, loose matches on Subject: below --
2008-03-06 15:46 Lennart Borgman (gmail)
2008-03-06 21:35 ` David Kastrup
2008-03-06 23:20   ` Lennart Borgman (gmail)
2008-03-06 23:31   ` David De La Harpe Golden
2008-03-06 23:44     ` Lennart Borgman (gmail)

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

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

  git send-email \
    --in-reply-to=ade856a30803101415oe373e06j5ed8a74db30f1873@mail.gmail.com \
    --to=francesc.rocher@gmail.com \
    --cc=dak@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=lennart.borgman@gmail.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 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.