Re: [notmuch] [PATCH] Make notmuch-show 'X' (and 'x') commands remove inbox (and...

diff --git a/80/a4519942c70e64c0c66c12d8a00e2e8f080c9b b/80/a4519942c70e64c0c66c12d8a00e2e8f080c9b
new file mode 100644 (file)
index 0000000..02065e7
--- /dev/null
+++ b/80/a4519942c70e64c0c66c12d8a00e2e8f080c9b
@@ -0,0 +1,60 @@
+Return-Path: <cworth@cworth.org>\r
+X-Original-To: notmuch@notmuchmail.org\r
+Delivered-To: notmuch@notmuchmail.org\r
+Received: from localhost (localhost [127.0.0.1])\r
+       by olra.theworths.org (Postfix) with ESMTP id 4E892431FC0;\r
+       Sun, 22 Nov 2009 15:19:45 -0800 (PST)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+Received: from olra.theworths.org ([127.0.0.1])\r
+       by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
+       with ESMTP id CL9b6SVoeyRW; Sun, 22 Nov 2009 15:19:44 -0800 (PST)\r
+Received: from cworth.org (localhost [127.0.0.1])\r
+       by olra.theworths.org (Postfix) with ESMTP id 4ECD7431FAE;\r
+       Sun, 22 Nov 2009 15:19:44 -0800 (PST)\r
+From: Carl Worth <cworth@cworth.org>\r
+To: Keith Packard <keithp@keithp.com>, notmuch@notmuchmail.org\r
+In-Reply-To: <1258493565-13508-1-git-send-email-keithp@keithp.com>\r
+References: <1258493565-13508-1-git-send-email-keithp@keithp.com>\r
+Date: Mon, 23 Nov 2009 00:19:29 +0100\r
+Message-ID: <87d43axi4e.fsf@yoom.home.cworth.org>\r
+MIME-Version: 1.0\r
+Content-Type: text/plain; charset=us-ascii\r
+Subject: Re: [notmuch] [PATCH] Make notmuch-show 'X' (and 'x') commands\r
+ remove inbox (and unread) tags\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.12\r
+Precedence: list\r
+List-Id: "Use and development of the notmuch mail system."\r
+       <notmuch.notmuchmail.org>\r
+List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
+List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
+List-Post: <mailto:notmuch@notmuchmail.org>\r
+List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
+List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
+X-List-Received-Date: Sun, 22 Nov 2009 23:19:45 -0000\r
+\r
+On Tue, 17 Nov 2009 13:32:45 -0800, Keith Packard <keithp@keithp.com> wrote:\r
+> When closing a thread view, mark the thread as archived by removing\r
+> the "inbox" tag, and for the 'x' variant, the "unread" tag as well,\r
+> then kill the buffer and update the search window view as well.\r
+\r
+I'm starting to come around to this patch, Keith. :-)\r
+\r
+I think one thing we're going to really need is a better help message\r
+for presenting the keybindings. The function names are all really\r
+confusing, (the "notmuch-show" prefix doesn't help). So what I'd really\r
+like is something that displays the keybinding and the first line of the\r
+documentation of a function, rather than the function name itself.\r
+\r
+> This makes 'x' much the same as 'a', but instead of taking you to the\r
+> next message, it takes you back to the search window instead.\r
+\r
+Except that the patch also reverses the current behavior of "a" and\r
+"A". That should be a separate patch anyway, but I'm also not sure that\r
+I like it. I like that the command that takes the extra effort of\r
+pressing shift is the command that does *more*, (removing both "unread"\r
+and "inbox"), rather than the command that does *less*.\r
+\r
+-Carl\r