From: Eli Zaretskii <eliz@gnu.org>
To: martin rudalics <rudalics@gmx.at>
Cc: 14825@debbugs.gnu.org
Subject: bug#14825: 24.3.50; split-window-below miscounts window lines
Date: Tue, 09 Jul 2013 19:12:31 +0300 [thread overview]
Message-ID: <83bo6bww40.fsf@gnu.org> (raw)
In-Reply-To: <51DBD33D.4000307@gmx.at>
> Date: Tue, 09 Jul 2013 11:09:17 +0200
> From: martin rudalics <rudalics@gmx.at>
> CC: 14825@debbugs.gnu.org
>
> > The reason for this seems to be that window.el and window.c do all
> > calculations in canonical lines, which is incorrect when the font of the
> > default face changes. Moreover, the documentation barely hints on the
> > fact that "lines" actually means "canonical-height lines" in almost all
> > window-* functions that deal with vertical dimensions.
>
> Currently, all window and frame sizing functions are based on canonical
> character sizes. This will change when we do resizing pixelwise.
Resizing is only part of the problem. There are functions that are
unrelated to resizing, which report dimensions in canonical units.
They should at least document this fact in the doc string, if not fix
it. (The fix is not too hard, btw; see window-screen-lines for an
example.)
> However, your example seems contrived: You don't change the default face
> but the buffer's default face. In many cases, split-window is followed
> by displaying another buffer in the new window so this buffer's default
> might not be appropriate for the other buffer anyway.
We have this problem already: one could split the window, and _then_
change the default face in one of the child windows.
next prev parent reply other threads:[~2013-07-09 16:12 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-08 17:52 bug#14825: 24.3.50; split-window-below miscounts window lines Eli Zaretskii
2013-07-09 9:09 ` martin rudalics
2013-07-09 16:12 ` Eli Zaretskii [this message]
2013-07-10 7:20 ` martin rudalics
2013-07-10 15:42 ` Eli Zaretskii
2013-07-11 6:27 ` martin rudalics
2013-07-11 16:52 ` Eli Zaretskii
2013-07-12 8:21 ` martin rudalics
2013-07-12 9:09 ` Eli Zaretskii
2013-07-12 10:12 ` martin rudalics
2013-07-12 13:29 ` Eli Zaretskii
2013-07-12 14:57 ` Juanma Barranquero
2013-07-13 11:10 ` martin rudalics
2013-07-13 11:54 ` Eli Zaretskii
2013-07-13 13:56 ` martin rudalics
2013-07-13 14:38 ` Eli Zaretskii
2019-10-15 17:38 ` martin rudalics
2020-09-21 12:38 ` Lars Ingebrigtsen
2020-09-21 14:29 ` Eli Zaretskii
2020-09-21 17:46 ` martin rudalics
2020-09-21 17:48 ` Eli Zaretskii
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=83bo6bww40.fsf@gnu.org \
--to=eliz@gnu.org \
--cc=14825@debbugs.gnu.org \
--cc=rudalics@gmx.at \
/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.