emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
From: Michael Heerdegen <michael_heerdegen@web.de>
To: emacs-orgmode@gnu.org
Subject: Re: [BUG] org-agenda-later sometimes creates a void display [9.7.11 (release_9.7.11 @ /usr/local/share/emacs/31.0.50/lisp/org/)]
Date: Sun, 12 Jan 2025 01:20:34 +0100	[thread overview]
Message-ID: <878qrgzxv1.fsf@web.de> (raw)
In-Reply-To: <8734hp1kjk.fsf@localhost> (Ihor Radchenko's message of "Sat, 11 Jan 2025 14:39:59 +0000")

Ihor Radchenko <yantar92@posteo.net> writes:

> +    (unless (< (point) wstart) ; do no scroll past the point
> +      (set-window-start nil wstart))))

Thanks.

Hmmm - don't we have the same problem when (point) > (window-end)?  And
this case is nastier, since exact window-end calculation needs a
redisplay.  We might introduce a flicker when trying to fix it.

At the risk of annoying you even more (I'm really sorry about that) -
are you open to a small redesign?

Since line lengths differ, we are currently setting window-start to a
random line, to some degree.  (And I think window-start should better be
set to the beginning of line).

One step back: What problem does this hack solve?  Org already remembers
the unit number of the time period point is in, and restores that: with
point in a Wednesday, the new view will have point set to the beginning
of the next Wednesday.  The display engine ensures that point is
made visible.

So that window-start thing does something different.  I guess that the
purpose of this is to make the switch visually more appealing, so that
the eye is presented a similar perspective.  But then, shouldn't we at
least use a line based approach?

Then, what the code is currently agnostic to is point at being at eob.
This always annoyed me.

How about something like this?

 - If point is at bob, we ensure that point is restored at bob - and
   likewise for eob.

 - When bob was visible in the prior view, we use window-start = 1
   for the new view, too

 - But when bob was not visible in the last view, we try to restore the
   visible line number containing the window point, so that hitting f
   will show the cursor at the same vertical position as in the last
   view.  Or maybe better: restore the vertical position of the
   beginning of the weekday point was in.

If you are open to the approach I could give it a try.


Thanks,

Michael.


  reply	other threads:[~2025-01-12  0:20 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-01-05  0:29 [BUG] org-agenda-later sometimes creates a void display [9.7.11 (release_9.7.11 @ /usr/local/share/emacs/31.0.50/lisp/org/)] Michael Heerdegen
2025-01-05  8:52 ` Ihor Radchenko
2025-01-05 18:18   ` Michael Heerdegen
2025-01-05 18:31     ` Ihor Radchenko
2025-01-05 20:12       ` Michael Heerdegen
2025-01-11 14:39         ` Ihor Radchenko
2025-01-12  0:20           ` Michael Heerdegen [this message]
2025-01-12  7:55             ` Ihor Radchenko

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.orgmode.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=878qrgzxv1.fsf@web.de \
    --to=michael_heerdegen@web.de \
    --cc=emacs-orgmode@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/org-mode.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).