From: Damien Cassou <damien@cassou.me>
Date: Sun, 16 Feb 2025 21:22:23 +0000 (+0100)
Subject: notmuch-show-imenu-prev-index-position-function: Don't recenter
X-Git-Tag: archive/debian/0.39_rc0-1~10
X-Git-Url: https://git.notmuchmail.org/git?a=commitdiff_plain;h=0e10ca3a625c25c0238ecca2767aab7035b88a22;p=notmuch

notmuch-show-imenu-prev-index-position-function: Don't recenter

The previous version of
notmuch-show-imenu-prev-index-position-function was calling
notmuch-show-previous-message which was recentering the window. This
is a problem when which-func-mode is active because which-func-mode is
called each time the user moves point around.

Because this function is only meant to be executed by imenu, there is
no reason to recenter the window. For the same reason, the new
implementation doesn't have to follow strictly how
notmuch-show-previous-message is implemented because imenu only places
point at very specific locations before calling
notmuch-show-imenu-prev-index-position-function.
---

diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el
index 14e3c698..921c0ef1 100644
--- a/emacs/notmuch-show.el
+++ b/emacs/notmuch-show.el
@@ -2686,7 +2686,9 @@ This function is used as a value for
 `imenu-prev-index-position-function'."
   (if (bobp)
       nil
-    (notmuch-show-previous-message)
+    (if (eobp)
+	(notmuch-show-move-to-message-top)
+      (notmuch-show-goto-message-previous))
     t))
 
 (defun notmuch-show-imenu-extract-index-name-function ()