unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Jim Porter <jporterbugs@gmail.com>
To: Stefan Kangas <stefankangas@gmail.com>
Cc: emacs-devel@gnu.org
Subject: Re: Increase default `line-spacing' to 0.05, 0.10 or 0.15 [proposal]
Date: Thu, 6 May 2021 16:17:17 -0700	[thread overview]
Message-ID: <CANh=_JF0wj6sPOE-TSLca-T_sfRL1ez3M-BnooKzRAgW=+1YHw@mail.gmail.com> (raw)
In-Reply-To: <CADwFkm=+-ir-za1pXuutx30eQ4u1jMZUfo7Yk1yonAit50mgJQ@mail.gmail.com>

On Thu, May 6, 2021 at 1:21 PM Stefan Kangas <stefankangas@gmail.com> wrote:
>
> Jim Porter <jporterbugs@gmail.com> writes:
>
> > On 5/4/2021 9:12 AM, Óscar Fuentes wrote:
> >> There are some negative consequences, though, like the "line" of
> >> display-fill-column-indicator-mode losing any apparience of continuity.
>
> Given how that mode is implemented, by setting
> `display-fill-column-indicator' to ?\u2502, I don't see how it could be
> improved without a complete redesign.  It doesn't look like this was
> implemented with `line-spacing' in mind.

Yeah, Emacs' reliance on using characters to draw lines would probably
lead me to set `line-spacing' back to 0 if this were changed. It would
be interesting if Emacs were able to handle these issues, e.g. by
adding some special handling when it sees box-drawing characters, but
that would probably be a lot of work.

> > I'm not sure if there's an easy way to ensure the fill line looks
> > continuous in this case, but if that could be improved, it would help in
> > a few other areas even if line-spacing weren't increased. For example,
> > with `org-prettify-entities' set to t, subscripts add a bit to the line
> > height, causing the fill line to appear discontinuous. Likewise, I
> > believe overlines add a bit to the line height too.
>
> Do you have a recipe to reproduce this?

Sure, here it is. This might depend on the font used, though:

  emacs -Q
  C-x C-f file.org RET
  x_subscript RET
  foo RET
  bar RET
  M-x display-fill-column-indicator-mode
  ;; The fill column line should be continuous for most fonts
  M-x org-toggle-pretty-entities
  ;; The fill column line should be discontinuous between lines 1 and 2

Even on fonts where the fill column indicator is always discontinuous
(e.g. Deja Vu Sans Mono for me), `M-x org-toggle-pretty-entities'
makes the gap larger.

- Jim



  reply	other threads:[~2021-05-06 23:17 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-05-04 15:57 Increase default `line-spacing' to 0.05, 0.10 or 0.15 [proposal] Stefan Kangas
2021-05-04 16:12 ` Óscar Fuentes
2021-05-04 16:59   ` Jim Porter
2021-05-05  7:08     ` Augusto Stoffel
2021-05-05  8:51       ` Daniel Mendler
2021-05-05 19:47       ` Stefan Kangas
2021-05-06  9:26         ` Augusto Stoffel
2021-05-06 10:10           ` Eli Zaretskii
2021-05-06 11:47             ` Augusto Stoffel
2021-05-06 11:57               ` Eli Zaretskii
2021-05-06 12:27                 ` Augusto Stoffel
2021-05-06 15:21                   ` Eli Zaretskii
2021-05-06 15:46                     ` Augusto Stoffel
2021-05-06 16:16                       ` Eli Zaretskii
2021-05-06 12:30                 ` Gregory Heytings
2021-05-06 15:22                   ` Eli Zaretskii
2021-05-06 16:21                     ` Gregory Heytings
2021-05-06 16:29                       ` Eli Zaretskii
2021-05-06 16:57                         ` Daniele Nicolodi
2021-05-06 17:53                           ` Eli Zaretskii
2021-05-06 17:57                             ` Eli Zaretskii
2021-05-06 20:24                             ` Daniele Nicolodi
2021-05-06 17:01                         ` Gregory Heytings
2021-05-06 17:34                           ` Eli Zaretskii
2021-05-06 18:15                             ` Gregory Heytings
2021-05-06 20:22           ` Stefan Kangas
2021-05-06 20:21     ` Stefan Kangas
2021-05-06 23:17       ` Jim Porter [this message]
2021-05-07  6:03         ` Yuri Khan
2021-05-07  4:05       ` Eli Zaretskii
2021-05-04 16:18 ` Eli Zaretskii
2021-05-04 21:29   ` Stefan Kangas
2021-05-05  2:28     ` Eli Zaretskii
2021-05-05  5:14 ` Richard Stallman
2021-05-05 19:16   ` Stefan Kangas
2021-05-06 20:21     ` Stefan Kangas
2021-05-07  4:03       ` Eli Zaretskii
2021-05-07 18:43         ` Stefan Kangas
2021-05-08  6:19           ` Eli Zaretskii
2021-05-08  7:51             ` Daniele Nicolodi
2021-05-08  8:06               ` Eli Zaretskii
2021-05-08  9:40                 ` Daniele Nicolodi
2021-05-05 12:18 ` Daniele Nicolodi
2021-05-05 19:17   ` Stefan Kangas

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='CANh=_JF0wj6sPOE-TSLca-T_sfRL1ez3M-BnooKzRAgW=+1YHw@mail.gmail.com' \
    --to=jporterbugs@gmail.com \
    --cc=emacs-devel@gnu.org \
    --cc=stefankangas@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 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).