From: Juri Linkov <juri@jurta.org>
To: Juanma Barranquero <lekktu@gmail.com>
Cc: Emacs developers <emacs-devel@gnu.org>
Subject: Re: [Emacs-diffs] /srv/bzr/emacs/trunk r99831: Scrolling commands which does not signal errors at top/bottom.
Date: Thu, 08 Apr 2010 02:19:03 +0300 [thread overview]
Message-ID: <87tyrmetgw.fsf@mail.jurta.org> (raw)
In-Reply-To: <x2of7ccd24b1004071516i825f3c81i46a30ad2f037a570@mail.gmail.com> (Juanma Barranquero's message of "Thu, 8 Apr 2010 00:16:04 +0200")
>> `mwheel-scroll' disobeys `scroll-preserve-screen-position' too.
>
> I would consider that a bug, too. The docstring for
> `scroll-preserve-screen-position' does talk about "scroll commands".
> The reason the option exists, I think, is because some of us find the
> non-preserving behavior quite unbearable (I used Mikael Sjödin's
> pager.el for years because I found the standard scroll-(up|down)
> commands almost unusable).
I have a 1997-year copy of pager.el (not sure if later versions exist).
It contains the variable `pager-keep-column-commands' that defines
a list of scrolling commands that keep the current column.
Maybe `scroll-up-command' and `scroll-down-command' should use a similar
list. And then `mwheel-scroll' and scroll-(up|down)-line should call
`scroll-(up|down)-command' instead of `scroll-(up|down)'.
>> I think
>> it would be wrong to add all possible scrolling commands to this code:
>
> Why? Perhaps it should be implemented differently, but that's not the
> same as being "wrong".
By "wrong" I meant that window.c is the wrong place to implement this.
--
Juri Linkov
http://www.jurta.org/emacs/
next prev parent reply other threads:[~2010-04-07 23:19 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <E1NywTs-00007U-Lf@internal.in.savannah.gnu.org>
2010-04-06 0:51 ` [Emacs-diffs] /srv/bzr/emacs/trunk r99831: Scrolling commands which does not signal errors at top/bottom Juanma Barranquero
2010-04-06 0:59 ` Juri Linkov
2010-04-06 1:58 ` Juanma Barranquero
2010-04-06 16:12 ` Juri Linkov
2010-04-06 18:26 ` Juanma Barranquero
2010-04-06 20:19 ` Juri Linkov
2010-04-06 20:49 ` Juanma Barranquero
2010-04-06 21:06 ` Juri Linkov
2010-04-06 21:39 ` Juanma Barranquero
2010-04-06 21:47 ` Juanma Barranquero
2010-04-07 20:17 ` Juri Linkov
2010-04-07 22:16 ` Juanma Barranquero
2010-04-07 23:19 ` Juri Linkov [this message]
2010-04-07 23:35 ` Juanma Barranquero
2010-04-09 15:29 ` Stefan Monnier
2010-04-09 22:46 ` Juri Linkov
2010-04-10 14:51 ` Stefan Monnier
2010-04-14 16:49 ` Juri Linkov
2010-04-15 4:41 ` Stefan Monnier
2010-04-15 23:56 ` Juri Linkov
2010-04-16 0:41 ` Stefan Monnier
2010-04-16 1:27 ` Juri Linkov
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=87tyrmetgw.fsf@mail.jurta.org \
--to=juri@jurta.org \
--cc=emacs-devel@gnu.org \
--cc=lekktu@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.