From: "Drew Adams" <drew.adams@oracle.com>
To: "'Antoine Levitt'" <antoine.levitt@gmail.com>, <emacs-devel@gnu.org>
Subject: RE: Box attribute in mode-line and scrolling
Date: Sun, 27 Mar 2011 10:39:07 -0700 [thread overview]
Message-ID: <BE4150E4F57146A58EE9BEC22A9D2278@us.oracle.com> (raw)
In-Reply-To: <87tyeospbb.fsf@gmail.com>
>> >> There seems to be a conflict between scrolling and the box
>> >> attribute in mode-line. I have in my config a ":box t"
>> >> attribute in mode-line (from the theme I use, zenburn.el).
>> >> With this setting on, C-v M-v produces a net displacement
>> >> of one line up, which is annoying. This seems to be
>> >> irrespective of settings such as scroll-conservatively or
>> >> scroll-preserve-screen-position.
>> >>
>> >> This is a pretty old bug. It might (or might not) be
>> >> related to
>> >> http://lists.gnu.org/archive/html/emacs-devel/2010-10/msg00339.html
>> >
>> > That doesn't seem like a bug, but the fact that adding the box
>> > enlarges the size of the modeline, and likely the last line of the
>> > window isn't fully displayed.
>>
>> Yes, that seems to be what's happening. But how is that not a
>> bug? Even if the last line isn't fully displayed, can't emacs
>> try and figure out a way to respect the semantics of C-v and
>> M-v cancelling each other out in the middle of a file?
>>
>> > Does it also happen if, instead of ":box t" you use ":box
>> > '(:line-width -1 :style released-button)"?
>>
>> Yup, that fixes it, and even if I don't use :style released-button.
>>
>> > This gotcha should be pointed out more clearly in the doc,
>> > since the default behavior increases the height of the
>> > current line, which can throw off various things.
>> >
>> > [And/or let's] change the default :line-width to -1 instead of 1.
>
> Right. That seems to be a different issue though. Even with a larger
> mode line, why do the scrolling commands behave this way?
My guess is that it is not a different issue. As you guessed in citing the
October message (see e.g. the last message in that thread), as Juanma indicated
is likely, and as you seem to have confirmed using the fix he suggested, the
problem for the scrolling commands is also (I'm guessing) the increased
(mode-)line height.
Documenting this unfortunate side effect better, and changing the default so as
to not increase the line height, would (if our guesses are correct) directly
address your issue.
next prev parent reply other threads:[~2011-03-27 17:39 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-03-27 11:43 Box attribute in mode-line and scrolling Antoine Levitt
2011-03-27 12:15 ` Juanma Barranquero
2011-03-27 12:35 ` Antoine Levitt
2011-03-27 12:47 ` Antoine Levitt
2011-03-27 15:06 ` Drew Adams
2011-03-27 17:12 ` Antoine Levitt
2011-03-27 17:39 ` Drew Adams [this message]
2011-03-27 13:24 ` Eli Zaretskii
2011-03-27 13:43 ` Antoine Levitt
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=BE4150E4F57146A58EE9BEC22A9D2278@us.oracle.com \
--to=drew.adams@oracle.com \
--cc=antoine.levitt@gmail.com \
--cc=emacs-devel@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).