On Sun, Aug 07, 2016 at 09:57:04AM +0200, Eike Rathke wrote:
> On Saturday, 2016-08-06 18:03:53 -0700, Kevin J. McCarthy wrote:
> 
> > Attached is the revised patch.  This version doesn't restrict the
> > operations where the pager position is saved.  Instead, it just makes
> > sure the position is cleared when we return to the index menu.
> 
> But this means that returning to the index and then viewing the same
> message again will display it from top instead of the last position,
> doesn't it?

Hi Eike,

It depends how you return to the index.  Right now, if you hit <exit>,
and then re-open the message, Mutt does not remember the position.

It only remembers the position if you hit up,down,tab, etc (a keymap
that intends to open *another* message but fails), and then re-open the
same message again.

This is what I meant when I said "I consider the original behavior a
mistake".  Mutt is purposely clearing the position when you deliberately
exit the message.  All other operations are saving the position, but
this was meant to be just for those redirecting back to the same message
(sort, sync, toggle headers, break/link thread, edit type, and
pgp-check-traditional).

This isn't something I would bother fixing, except that it causes a bug.
Once in a while, a message happens to share the same address as the last
saved header pointer, and the pager tries to open the new message and
scroll down to the position in the old message.

-- 
Kevin J. McCarthy
GPG Fingerprint: 8975 A9B3 3AA3 7910 385C  5308 ADEF 7684 8031 6BDA

Attachment: signature.asc
Description: PGP signature

Reply via email to