Re: [notmuch] [PATCH] add notmuch-show-delete keybinding 'd'

Subject: Re: [notmuch] [PATCH] add notmuch-show-delete keybinding 'd'

Date: Wed, 24 Feb 2010 10:53:50 -0800

To: racin@free.fr, Sebastian Spaeth

Cc: notmuch

From: Carl Worth


On Wed, 20 Jan 2010 11:56:07 +0100 (CET), racin@free.fr wrote:
> I posted a similar patch a while ago, that also did not show deleted
> messages by default. Don't know if Carl wants to integrate this though

OK. When two people are independently contributing similar
functionality, it's more than clear that I'm far too behind on patch
review.

Matthieu, I've just posted my review of your original patch. I think I
like the name "deleted" for the tag rather than "delete", (it's
consistent with "unread" at least that way). And I like the support for
excluding deleted results in that patch as well.

But this patch does have two good ideas not in the other patch, (both of
which I mentioned in the review):

1. It adds a keybinding to the notmuch-show mode

2. It removes the "inbox" and "unread" tags while adding the tag to
   indicate deletion.

-Carl
part-000.sig (application/pgp-signature)

Thread: