[PATCH 0/3] Make notmuch-show-refresh-view retain state by default

Austin Clements amdragon at MIT.EDU
Sun Feb 19 10:03:29 PST 2012


Quoth Dmitry Kurochkin on Feb 19 at  9:29 pm:
> On Sun, 19 Feb 2012 12:25:41 -0500, Austin Clements <amdragon at MIT.EDU> wrote:
> > Quoth Tomi Ollila on Feb 19 at 11:43 am:
> > > On Sun, 19 Feb 2012 01:22:10 -0500, Austin Clements <amdragon at MIT.EDU> wrote:
> > > > Based on the thread at id:"20120213152858.GO27039 at mit.edu" it seems
> > > > like people want show refresh to retain message state by default (I
> > > > certainly do), rather than reset it by default.  As a nice bonus, this
> > > > fixes a broken test.
> > > 
> > > Hmm
> > > 
> > > Every '=' keypress in a thread removes 'unread' tag from next unread 
> > > message :o
> > 
> > Oh dear.  It's slightly subtler than that, though.  Refreshing will
> > remove the unread tag from the first message matching the query.  (I
> > suspect you had tag:unread in your query?)  This is particularly
> > confusing for state-retaining refresh because that message might not
> > even be open.
> > 
> > > That is probably desirable feature when refreshing view without
> > > retaining state. Also, probably notmuch-show-refresh-view should
> > > be fixed not to mark next message unread when retaining state ?
> > 
> > I would argue that refresh should never have side-effects, so neither
> > case should mark anything read.  I'll send a v2.
> 
> I agree.  But this looks like a separate issue, no?  So why v2 instead
> of a separate patch/series?

I wanted to slip it in before inverting the default behavior of
refresh.


More information about the notmuch mailing list