Re: [PATCH] remove message archiving from show-advance-and-archive

Subject: Re: [PATCH] remove message archiving from show-advance-and-archive

Date: Wed, 09 Jun 2010 10:54:57 -0700

To: Jameson Rollins, David Edmondson, Notmuch Mail

Cc:

From: Carl Worth


On Wed, 09 Jun 2010 11:55:49 -0400, Jameson Rollins <jrollins@finestructure.net> wrote:
> I actually submitted this patch because there was noise on #notmuch
> about people (including from cworth) not liking the default behavior
> where the "inbox" tag is removed by the advance function.

For the record, I wrote the current behavior for my own use, so I've
been pretty happy with it.

But I have seen enough people complain about it that I've been convinced
that something is wrong about it. I'm not yet sure what the right answer
is. But if we at least have separate commands for separate actions, then
hopefully it wouldn't be too difficult for users to combine those
actions.

When we have one command that does several different things, then it's
much harder for users to tweak.

I suppose one option here would be a customizable variable that controls
what happens when trying to advance "past" the last message, (do
nothing, advance to next thread, archive current thread and advance to
next thread)?

-Carl

-- 
carl.d.worth@intel.com
part-000.sig (application/pgp-signature)

Thread: