Re: [PATCH v3 1/2] emacs: User-defined sections in notmuch-hello
authorDaniel Schoepe <daniel.schoepe@googlemail.com>
Thu, 7 Jul 2011 18:25:37 +0000 (20:25 +0200)
committerW. Trevor King <wking@tremily.us>
Fri, 7 Nov 2014 17:39:00 +0000 (09:39 -0800)
33/3226fe8541d211c99c7d84eae286137a099693 [new file with mode: 0644]

diff --git a/33/3226fe8541d211c99c7d84eae286137a099693 b/33/3226fe8541d211c99c7d84eae286137a099693
new file mode 100644 (file)
index 0000000..ada4bca
--- /dev/null
@@ -0,0 +1,162 @@
+Return-Path: <daniel.schoepe@googlemail.com>\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 0DDE0429E25\r
+       for <notmuch@notmuchmail.org>; Thu,  7 Jul 2011 11:25:50 -0700 (PDT)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: -0.789\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=-0.789 tagged_above=-999 required=5\r
+       tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
+       FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7, T_MIME_NO_TEXT=0.01]\r
+       autolearn=disabled\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 RT9CbNv9EyxG for <notmuch@notmuchmail.org>;\r
+       Thu,  7 Jul 2011 11:25:49 -0700 (PDT)\r
+Received: from mail-fx0-f46.google.com (mail-fx0-f46.google.com\r
+       [209.85.161.46]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
+       (No client certificate requested)\r
+       by olra.theworths.org (Postfix) with ESMTPS id A7F71431FD0\r
+       for <notmuch@notmuchmail.org>; Thu,  7 Jul 2011 11:25:48 -0700 (PDT)\r
+Received: by fxh19 with SMTP id 19so1384562fxh.19\r
+       for <notmuch@notmuchmail.org>; Thu, 07 Jul 2011 11:25:47 -0700 (PDT)\r
+DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
+       d=googlemail.com; s=gamma;\r
+       h=from:to:subject:in-reply-to:references:user-agent:date:message-id\r
+       :mime-version:content-type;\r
+       bh=z/uldayyH7PDdNujGUUfgNmg0N3gxuX9wBSAr7RPWmM=;\r
+       b=Iklo0FcAcX/4NATyxqQyjOv1e+lzugGbK+5mnEISbFhjgYM06WSWsNChFCwcRKxTzZ\r
+       r3znqrdHA+8gvdiuQ67LsY/LTD4G/l5GtWqLnTJ8j+C0Ran4FGsC9dMa2naRyvXU8ZBq\r
+       S9Wv4H8OmeHGAn1t1nAyEUdIjIeX+HuzZBIrc=\r
+Received: by 10.223.144.136 with SMTP id z8mr1688271fau.31.1310063147188;\r
+       Thu, 07 Jul 2011 11:25:47 -0700 (PDT)\r
+Received: from localhost (dslb-088-069-138-212.pools.arcor-ip.net\r
+       [88.69.138.212])\r
+       by mx.google.com with ESMTPS id m6sm4877273fac.3.2011.07.07.11.25.44\r
+       (version=TLSv1/SSLv3 cipher=OTHER);\r
+       Thu, 07 Jul 2011 11:25:45 -0700 (PDT)\r
+From: Daniel Schoepe <daniel.schoepe@googlemail.com>\r
+To: Michal Sojka <sojkam1@fel.cvut.cz>, notmuch@notmuchmail.org\r
+Subject: Re: [PATCH v3 1/2] emacs: User-defined sections in notmuch-hello\r
+In-Reply-To: <871uy25d3y.fsf@steelpick.2x.cz>\r
+References: <1309379221-5617-1-git-send-email-daniel.schoepe@googlemail.com>\r
+       <1309883030-28899-1-git-send-email-daniel.schoepe@googlemail.com>\r
+       <1309883030-28899-2-git-send-email-daniel.schoepe@googlemail.com>\r
+       <87fwmjabii.fsf@steelpick.2x.cz>\r
+       <87oc17r38a.fsf@tredergarh.home.box>\r
+       <871uy25d3y.fsf@steelpick.2x.cz>\r
+User-Agent: Notmuch/0.5-319-g20a5437 (http://notmuchmail.org) Emacs/23.3.1\r
+       (x86_64-pc-linux-gnu)\r
+Date: Thu, 07 Jul 2011 20:25:37 +0200\r
+Message-ID: <87wrfuc5i6.fsf@gilead.invalid>\r
+MIME-Version: 1.0\r
+Content-Type: multipart/signed; boundary="=-=-=";\r
+       micalg=pgp-sha1; protocol="application/pgp-signature"\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.13\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: Thu, 07 Jul 2011 18:25:50 -0000\r
+\r
+--=-=-=\r
+Content-Transfer-Encoding: quoted-printable\r
+\r
+On Thu, 07 Jul 2011 17:23:13 +0200, Michal Sojka <sojkam1@fel.cvut.cz> wrot=\r
+e:\r
+> This wouldn't work for me. My all-tags section covers almost entire\r
+> screen and finding non-zero entries there is not very convenient. I find\r
+> much more useful to have a section saying: "Hey, you have unread\r
+> messages only for these three tags". Moreover, it wouldn't help me to see\r
+> non-zero number of unread messages and when I click the button I would\r
+> see all the messages, not only the unread ones. It simply seems very\r
+> confusing to me.\r
+\r
+I agree with you, personally, but I don't think this particular bit\r
+(:hide-empty-searches) increases code complexity that much (and all of\r
+it is localized to the functions generating the sections, which possibly\r
+should be moved to a separate file). I also find it plausible that a\r
+user might want behavior like this (as, e.g. an overview, if he doesn't\r
+want an all tags section, but still see what tags he has set).\r
+\r
+Hence I don't see why we should not provide this option.\r
+\r
+> > If we decide to keep this functionality, it should be inverted though,\r
+> > i.e. one has to explicitly specify :show-empty-searches to get them.\r
+>=20\r
+> >=20\r
+> > About the counts: I introduced this because Austin Clements says he\r
+> > finds it useful in his comment here:\r
+> >=20\r
+> > id:"BANLkTi=3D729DWai4q57iBSfz1wDhBXsmndQ@mail.gmail.com"\r
+>=20\r
+> I agree that it is useful to see unread counts, but it is not useful to\r
+> see all messages when I click the button.\r
+\r
+As I said, this is not my preference either, but personal taste is not\r
+necessarily the best argument for _not including_ something, especially\r
+for the UI of a mail client. And in this case there even is someone\r
+saying that behavior would be useful to him.\r
+\r
+> > A user might still want to have the section collapsed when starting the\r
+> > notmuch UI and only have it shown when he needs it. (I use that for a\r
+> > section that displays unread counts for each tag).\r
+>=20\r
+> You are right. I use emacs --daemon so I actually initialize notmuch UI\r
+> only when emacs crashes or when I run out of battery power ;-)\r
+\r
+Yes, me too, but I like seeing only sections that I care about every\r
+time I open notmuch while still having more information available in a\r
+convenient way. But I'm not too attached to this feature, so if the\r
+consensus is that that bit should be removed, I'll bow to public opinion. :)\r
+\r
+> If I understand this variable correctly, it stores the label (text) of\r
+> the button you have your point at. This allows you to stay at the same\r
+> button after reloading of notmuch-hello even if the layout changes,\r
+> right? Then having the same named button in multiple sections results in\r
+> moving the first (or last) occurrence of this button when notmuch-hello\r
+> is reloaded.\r
+\r
+Ah, I misunderstood what you were referring to earlier. That was already\r
+case before my patch though, except that now it's more likely to have\r
+buttons with the same name. Anyway, I think this is just a minor issue\r
+and can be addressed in a different patch.\r
+\r
+In any event, this definitely needs better documentation.\r
+\r
+Thanks for all your input on this!\r
+\r
+Cheers,\r
+Daniel\r
+\r
+--=-=-=\r
+Content-Type: application/pgp-signature\r
+\r
+-----BEGIN PGP SIGNATURE-----\r
+Version: GnuPG v1.4.11 (GNU/Linux)\r
+\r
+iQIcBAEBAgAGBQJOFfoiAAoJEIaTAtce+Z+JdO0QALZH8JFgm6aUZniQK0INCWAk\r
+71rME1nUG7YFoJZbVRg7ugQykoSXGEWct37fIIz+K0tghL1qegG+EJwKivpQqPa2\r
+cHsTjxBUuamyEVrvf96pzHMk8UVUmhYOx3XMtzUt/gJmGxHatph+F7JQFx4whLvy\r
+cZYuYeAIg+dZJx8ylkMrgTtwgOjNLiwwGds4RWwCt2iwVccP6TwrWRiXkOiqcFD7\r
+dRQd474W2noVxE5Mn8TOh6LpCmtJhrysooKvSU4yat2mYZUN1Nm7t9CvCMEgEU9c\r
+HYJOR+zoyiVQKM9hhjO5Ts4HboJzsFbyJA9jXAJ8J09m2sphg0KCd1wCa2JaG94/\r
+SwGdPFjW/4usEWj+szRcCc4XFPmc3pyo/eSJ8BTjmfRtWoO56vF+w3Y3dkYeG09s\r
+03r7oG9tixmut8wNH5QYimME8qxcQCrOMUfjd2vPQ5Qgx892dwpmy3ZP3Wuv9SHA\r
+cJdxzUB7GclMYggXHjMDH41d7oSZd9MMOQCWB7U3kTCoTjy4HJ3iG49KH1mblld0\r
+desbjIrZ3qd96Mb9IVRk4RrFlEIuS1QW9P+zJqCoJAXvdOtnSXddF2sHDSgqqucC\r
+lopsc/cxnU21JYbzrdNuIjyOF5Z5qXg9bW0YjzJ0HZLDvEV0Ac7Igi+0NtMsV3uT\r
+/vDpiofVKBsU/fP1UyJH\r
+=lBAa\r
+-----END PGP SIGNATURE-----\r
+--=-=-=--\r