74/c60ea37e3327a53c729631792739709575e18f

   1 Return-Path: <m.walters@qmul.ac.uk>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5         by olra.theworths.org (Postfix) with ESMTP id A0BE9431FB6\r
   6         for <notmuch@notmuchmail.org>; Mon, 10 Dec 2012 08:05:33 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 1.401\r
  10 X-Spam-Level: *\r
  11 X-Spam-Status: No, score=1.401 tagged_above=-999 required=5\r
  12         tests=[DKIM_ADSP_CUSTOM_MED=0.001, FREEMAIL_FROM=0.001,\r
  13         FREEMAIL_REPLY=2.499, NML_ADSP_CUSTOM_MED=1.2, RCVD_IN_DNSWL_MED=-2.3]\r
  14         autolearn=disabled\r
  15 Received: from olra.theworths.org ([127.0.0.1])\r
  16         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  17         with ESMTP id pK9Py4VZVCoG for <notmuch@notmuchmail.org>;\r
  18         Mon, 10 Dec 2012 08:05:33 -0800 (PST)\r
  19 Received: from mail2.qmul.ac.uk (mail2.qmul.ac.uk [138.37.6.6])\r
  20         (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\r
  21         (No client certificate requested)\r
  22         by olra.theworths.org (Postfix) with ESMTPS id DBAAA431FAF\r
  23         for <notmuch@notmuchmail.org>; Mon, 10 Dec 2012 08:05:32 -0800 (PST)\r
  24 Received: from smtp.qmul.ac.uk ([138.37.6.40])\r
  25         by mail2.qmul.ac.uk with esmtp (Exim 4.71)\r
  26         (envelope-from <m.walters@qmul.ac.uk>)\r
  27         id 1Ti5r7-0006ln-Oj; Mon, 10 Dec 2012 16:05:29 +0000\r
  28 Received: from 93-97-24-31.zone5.bethere.co.uk ([93.97.24.31] helo=localhost)\r
  29         by smtp.qmul.ac.uk with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.69)\r
  30         (envelope-from <m.walters@qmul.ac.uk>)\r
  31         id 1Ti5r7-0000NV-As; Mon, 10 Dec 2012 16:05:21 +0000\r
  32 From: Mark Walters <markwalters1009@gmail.com>\r
  33 To: Damien Cassou <damien.cassou@gmail.com>, notmuch@notmuchmail.org\r
  34 Subject: Re: [PATCH v3] emacs: display tags in notmuch-show with links\r
  35 In-Reply-To: <1355149964-27905-1-git-send-email-damien.cassou@gmail.com>\r
  36 References: <1355149964-27905-1-git-send-email-damien.cassou@gmail.com>\r
  37 User-Agent: Notmuch/0.14+155~g7edfdc3 (http://notmuchmail.org) Emacs/23.4.1\r
  38         (x86_64-pc-linux-gnu)\r
  39 Date: Mon, 10 Dec 2012 16:05:23 +0000\r
  40 Message-ID: <87boe1yjss.fsf@qmul.ac.uk>\r
  41 MIME-Version: 1.0\r
  42 Content-Type: text/plain; charset=us-ascii\r
  43 X-Sender-Host-Address: 93.97.24.31\r
  44 X-QM-SPAM-Info: Sender has good ham record.  :)\r
  45 X-QM-Body-MD5: 0878f369ea640389694f568bba45616d (of first 20000 bytes)\r
  46 X-SpamAssassin-Score: -1.2\r
  47 X-SpamAssassin-SpamBar: -\r
  48 X-SpamAssassin-Report: The QM spam filters have analysed this message to\r
  49         determine if it is\r
  50         spam. We require at least 5.0 points to mark a message as spam.\r
  51         This message scored -1.2 points.\r
  52         Summary of the scoring: \r
  53         * -2.3 RCVD_IN_DNSWL_MED RBL: Sender listed at http://www.dnswl.org/,\r
  54         *      medium trust\r
  55         *      [138.37.6.40 listed in list.dnswl.org]\r
  56         * 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\r
  57         provider *      (markwalters1009[at]gmail.com)\r
  58         *  1.0 FREEMAIL_REPLY From and body contain different freemails\r
  59         *  0.1 AWL AWL: From: address is in the auto white-list\r
  60 X-QM-Scan-Virus: ClamAV says the message is clean\r
  61 X-BeenThere: notmuch@notmuchmail.org\r
  62 X-Mailman-Version: 2.1.13\r
  63 Precedence: list\r
  64 List-Id: "Use and development of the notmuch mail system."\r
  65         <notmuch.notmuchmail.org>\r
  66 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  67         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  68 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  69 List-Post: <mailto:notmuch@notmuchmail.org>\r
  70 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  71 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  72         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  73 X-List-Received-Date: Mon, 10 Dec 2012 16:05:33 -0000\r
  74 \r
  75 \r
  76 On Mon, 10 Dec 2012, Damien Cassou <damien.cassou@gmail.com> wrote:\r
  77 > This patch obsoletes:\r
  78 > id:1353266322-20318-1-git-send-email-damien.cassou@gmail.com\r
  79 >\r
  80 > [PATCH 1/4] emacs: Add a thread's tags to notmuch-show header-line\r
  81 > [PATCH 2/4] emacs: Make tags in notmuch-show header-line clickable\r
  82 > [PATCH 3/4] emacs: Make all tags in `notmuch-show' clickable\r
  83 > [PATCH 4/4] emacs: Add unit-tests for clickable tags\r
  84 >\r
  85 > This patch makes clickable all tags that appear in notmuch-show\r
  86 > buffers. Each tag is a link to open a new notmuch-search buffer for\r
  87 > this tag. Additionally, the buffer's header-line now shows the\r
  88 > thread's tags (clickable only if the `header-button' library is loaded\r
  89 > or loadable).\r
  90 >\r
  91 > This patch is the first one of an upcoming series whose goal is to\r
  92 > integrate notmuch-labeler into notmuch. See the following for more\r
  93 > details:\r
  94 > https://github.com/DamienCassou/notmuch-labeler\r
  95 \r
  96 Hi \r
  97 \r
  98 I like this series but there are two (related) problems that I missed on\r
  99 earlier review. \r
 100 \r
 101 The first is that the headerline tags are not updated when the user tags\r
 102 a message, and the second is that the tags for the message itself (ie in\r
 103 the emacs buffer not the headerline) are updated but cease to be\r
 104 buttons.\r
 105 \r
 106 I think the function notmuch-show-update-tags is the one that does the\r
 107 updating of the display. (Incidentally that seems to break if the user\r
 108 likes tags containing brackets).\r
 109 \r
 110 This is probably the place to link to as there are lots of ways messages\r
 111 get tagged.\r
 112 \r
 113 Best wishes (and sorry for not catching this before)\r
 114 \r
 115 Mark\r
 116 \r
 117 \r
 118 \r
 119 \r
 120 >\r
 121 > With respect to v2, I took care of the comments you made:\r
 122 > - moved notmuch-tagger-separate-elems to notmuch-lib\r
 123 > - renamed a few methods\r
 124 > - changed some comments to better reflect the method behavior\r
 125 > - changed links in the body so that TAB won't stop at them\r
 126 > - changed miscellaneous small things\r
 127 > _______________________________________________\r
 128 > notmuch mailing list\r
 129 > notmuch@notmuchmail.org\r
 130 > http://notmuchmail.org/mailman/listinfo/notmuch\r