From: Eli Zaretskii <eliz@gnu.org>
To: emacs-devel@gnu.org, Po Lu <luangruo@yahoo.com>
Cc: mwd@md5i.com
Subject: Re: Redisplay resets vscroll when window start changes
Date: Wed, 17 Nov 2021 09:45:24 +0200 [thread overview]
Message-ID: <4C42D18C-5A5A-4611-8F54-CF5294F453B1@gnu.org> (raw)
In-Reply-To: <87sfvvs8rf.fsf@yahoo.com>
On November 17, 2021 6:31:32 AM GMT+02:00, Po Lu <luangruo@yahoo.com> wrote:
> Eli Zaretskii <eliz@gnu.org> writes:
>
> > My suggestion is to implement equivalents of C-n/C-p that do what you
> > want, instead of asking for previous/next-line to be able to preserve
> > the vscroll, because those commands aren't supposed to support the use
> > case you want to support.
>
> Yes, I understand that. But the problem is any command that can call
> `line-move' will end up resetting the vscroll, which most people don't
> expect. Since making `line-move' not reset the vscroll is not an
> option, a better solution will have to be found, which replacing
> previous-line/next-line is not.
My point is precisely that line-move should not be used for this purpose, because it and the other 2 functions it calls don't support this use case, and OTOH include a lot of code logic which implicitly assumes a very different use case. It would be much easier and cleaner to implement what you need from scratch.
next prev parent reply other threads:[~2021-11-17 7:45 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <87k0h9pp7t.fsf.ref@yahoo.com>
2021-11-15 12:39 ` Redisplay resets vscroll when window start changes Po Lu
2021-11-15 12:44 ` Po Lu
2021-11-15 14:18 ` Eli Zaretskii
2021-11-16 0:02 ` Po Lu
2021-11-16 12:51 ` Eli Zaretskii
2021-11-16 12:54 ` Po Lu
2021-11-16 13:01 ` Po Lu
2021-11-16 13:34 ` Eli Zaretskii
2021-11-16 13:38 ` Po Lu
2021-11-16 13:45 ` Eli Zaretskii
2021-11-16 17:12 ` Michael Welsh Duggan
2021-11-16 18:31 ` Eli Zaretskii
2021-11-17 3:24 ` Stefan Monnier
2021-11-17 13:34 ` Eli Zaretskii
2021-11-17 0:26 ` Po Lu
2021-11-17 3:31 ` Eli Zaretskii
2021-11-17 3:40 ` Stefan Monnier
2021-11-17 13:40 ` Eli Zaretskii
2021-11-17 20:40 ` Stefan Monnier
2021-11-18 6:36 ` Eli Zaretskii
2021-11-19 3:17 ` Stefan Monnier
2021-11-19 7:13 ` Eli Zaretskii
2021-11-19 18:07 ` Stefan Monnier
2021-11-19 18:43 ` Eli Zaretskii
2021-11-17 4:31 ` Po Lu
2021-11-17 7:45 ` Eli Zaretskii [this message]
2021-11-17 0:21 ` Po Lu
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=4C42D18C-5A5A-4611-8F54-CF5294F453B1@gnu.org \
--to=eliz@gnu.org \
--cc=emacs-devel@gnu.org \
--cc=luangruo@yahoo.com \
--cc=mwd@md5i.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.