On Mon, Feb 10 2025, Damien Cassou wrote:

> Hi,
>
> is anyone interested in reviewing this? I've been using it for a month
> with no truble.

Maeby too detailed story this time, could have conveyed the intention
with less words (but the intention, is clear, at least) :D

Anyway, the behavior does not differ just in not doing that 
(notmuch-show-message-adjust), so explaining if (not) there is difference
in interactive behavior would be nice. We could tolerate the current
commit message if it is close enough of the final understanding of
this change. 
Kill-yanked notmuch-show-previous-message from my copy of the sources...

    (defun notmuch-show-previous-message ()
      "Show the previous message or the start of the current message."
      (interactive)
      (if (= (point) (notmuch-show-message-top))
          (notmuch-show-goto-message-previous)
        (notmuch-show-move-to-message-top))
      (notmuch-show-message-adjust))

Tomi

>
> Thank you
>
> -- 
> Damien Cassou
>
> "Success is the ability to go from one failure to another without
> losing enthusiasm." --Winston Churchill
_______________________________________________
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-leave@notmuchmail.org