all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: help-gnu-emacs@gnu.org
Subject: Re: Set-window-vscroll sometimes doesn't work
Date: Thu, 22 Oct 2020 19:05:01 +0300	[thread overview]
Message-ID: <83k0vil00i.fsf@gnu.org> (raw)
In-Reply-To: <8C545387-7A95-4B4E-A71E-05949E8A4641@gmail.com> (message from Yuan Fu on Wed, 21 Oct 2020 15:05:26 -0400)

> From: Yuan Fu <casouri@gmail.com>
> Date: Wed, 21 Oct 2020 15:05:26 -0400
> Cc: help-gnu-emacs@gnu.org
> 
>  But my point was that you shouldn't set window-start if you need to
>  use a non-zero vscroll.
> 
> It does: mwheel-scroll calls scroll-down/up, which calls window_scroll_pixel_based, which sets w->start
> (and it sets vscroll for tall image).

If you examine the code in window_scroll_pixel_based carefully, you
will see that it either sets vscroll or w->start, but never both.

>  next-line and friends only vscroll when scroll-conservatively=0 and ARG=1, which allows it to get
>  away from the problem I face (set-window-start and set-window-vscroll in the same time). I either
>  implements smooth scrolling to only work when ARG=1 or I need to set both value at the same
>  time.
> 
>  I don't understand why you say this.  First, C-n works on tall images
>  with arguments greater than 1.  And second, I still don't understand
>  why you need to set window-start, because vscroll should be enough for
>  your needs, as far as I understand.
> 
> C-n doesn’t smooth scroll on tall images when ARG > 1: The code checks for ARG = 1, if not, it doesn’t do
> line-move-partial. I also tried C-2 C-n, no smooth scrolling.

What do you mean by "smooth scrolling" here?  If I invoke C-2 C-n" on
a large image, the image gets scrolled by the amount of pixels that is
equivalent to 2 screen lines.  Isn't that what you see?

> I thought vscroll is a relative measure and is measured against window-start. And it kind of makes sense to
> set a window start as the basis and set vscroll based on that. So I kind of assumed that window-start is
> always at least partially visible and vscroll is always less than the height of the first line. Now, if I throw away
> window-start and scrolls entirely by vscroll, will there be any problems? Like, does anyone relies on
> window-start being visible? I felt like that’s what people uses to get the first line displayed in the window.

When vscroll is so large that the window-start point is entirely not
visible, you should move window-start so it becomes visible.  But that
shouldn't be a problem, because it means you've scrolled the entire
image off-screen.



      parent reply	other threads:[~2020-10-22 16:05 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-18  0:35 Set-window-vscroll sometimes doesn't work Yuan Fu
2020-10-18 14:34 ` Eli Zaretskii
2020-10-18 17:24   ` Yuan Fu
2020-10-18 17:31     ` Eli Zaretskii
2020-10-18 17:51       ` Yuan Fu
2020-10-18 17:55         ` Eli Zaretskii
2020-10-18 18:02           ` Yuan Fu
2020-10-18 19:12           ` Yuan Fu
2020-10-18 19:19             ` Eli Zaretskii
2020-10-18 20:06               ` Yuan Fu
2020-10-19  2:23                 ` Eli Zaretskii
     [not found]                   ` <B1D235FE-49B8-4F0A-9C02-78B7E3244C47@gmail.com>
2020-10-19  6:00                     ` Yuan Fu
2020-10-19 16:24                     ` Eli Zaretskii
2020-10-19 16:56                       ` Yuan Fu
2020-10-19 17:23                         ` Eli Zaretskii
2020-10-19 20:39                           ` Yuan Fu
2020-10-20 16:04                             ` Eli Zaretskii
2020-10-20 18:00                               ` Yuan Fu
2020-10-20 18:19                               ` Yuan Fu
2020-10-21 16:25                                 ` Eli Zaretskii
2020-10-21 19:05                                   ` Yuan Fu
2020-10-22  4:16                                     ` Yuan Fu
2020-10-22 16:08                                       ` Eli Zaretskii
2020-10-22 16:50                                         ` Yuan Fu
2020-10-22 17:09                                           ` Eli Zaretskii
2020-10-22 19:33                                             ` Yuan Fu
2020-10-23  6:05                                               ` Eli Zaretskii
2020-10-23 16:48                                                 ` Yuan Fu
2020-10-23 18:11                                                   ` Eli Zaretskii
2020-10-23 20:30                                                     ` Yuan Fu
2020-10-24  7:17                                                       ` Eli Zaretskii
2020-10-26  8:38                                                         ` Jens C. Jensen
2020-10-26 15:18                                                           ` Eli Zaretskii
2020-10-27 18:10                                                         ` Yuan Fu
2020-10-27 18:32                                                           ` Eli Zaretskii
2020-10-22 16:05                                     ` Eli Zaretskii [this message]

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=83k0vil00i.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=help-gnu-emacs@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 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.