From: storm@cua.dk (Kim F. Storm)
To: rms@gnu.org
Cc: emacs-devel@gnu.org, sdl.web@gmail.com, Kenichi Handa <handa@m17n.org>
Subject: Re: A bug in tetris
Date: Tue, 21 Aug 2007 10:32:11 +0200 [thread overview]
Message-ID: <87sl6dp9tg.fsf@kfs-lx.testafd.dk> (raw)
In-Reply-To: <E1INC1Y-0000no-OA@fencepost.gnu.org> (Richard Stallman's message of "Mon\, 20 Aug 2007 14\:30\:48 -0400")
Richard Stallman <rms@gnu.org> writes:
> > Maybe the newlines in the gamegrid must be marked to not increase
> > line height by its own:
>
> > (propertize "\n" 'line-height t)
>
> Isn't it better to make that the default behaviour if a line
> is not empty?
>
> Did we discuss this question when that feature was added?
I don't remember.
> Was there any particular reason to handle this the way we did?
I answered (vaguely) yesterday, but I now recall the reasoning
for not making it standard:
In most cases, I would assume that if you put a face
(with a smaller line height) on the text on a whole line,
you also put that face on the newline if you want the whole
line to have the smaller line height.
If you leave the default face on the newline, you (probably)
want to have the default line spacing for the line even if the
text on the line is in a smaller face.
So this is the current default behaviour (as it was in 21.x)
-- which IMO is correct for normal use cases.
In the rare case where a line is in the default face, but only
contains (small height) images (e.g. in a game grid or a sliced image)
do you get into a case where the height of the newline may get into
the way -- that is why I made a special line-height=t property for the
newline to handle such unusual cases.
The insert-sliced-image function uses this property.
Game grid functions should do that too.
Conclusion: I think we should keep the current default.
--
Kim F. Storm <storm@cua.dk> http://www.cua.dk
next prev parent reply other threads:[~2007-08-21 8:32 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-08-18 18:45 A bug in tetris Leo
2007-08-19 15:07 ` Johan Bockgård
2007-08-19 15:22 ` Mathias Dahl
2007-08-19 15:37 ` Johan Bockgård
2007-08-19 16:00 ` Mathias Dahl
2007-08-19 18:02 ` Leo
2007-08-19 18:27 ` Sven Joachim
2007-08-19 18:52 ` Leo
2007-08-19 22:27 ` Kim F. Storm
2007-08-20 1:30 ` Kenichi Handa
2007-08-20 10:37 ` Kim F. Storm
2007-08-20 18:30 ` Richard Stallman
2007-08-21 8:32 ` Kim F. Storm [this message]
2007-08-21 12:08 ` Kenichi Handa
2007-08-23 12:17 ` Kenichi Handa
2007-08-23 12:38 ` Leo
2007-08-21 23:24 ` Richard Stallman
2007-08-20 15:16 ` Richard Stallman
2007-08-20 17:25 ` Leo
2007-08-19 22:30 ` Richard Stallman
2007-08-19 17:58 ` Leo
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=87sl6dp9tg.fsf@kfs-lx.testafd.dk \
--to=storm@cua.dk \
--cc=emacs-devel@gnu.org \
--cc=handa@m17n.org \
--cc=rms@gnu.org \
--cc=sdl.web@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.