7b/77d0657ef8094cc9befb4f3b060d9dbf232dd9

   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 1426F431FC0\r
   6         for <notmuch@notmuchmail.org>; Sun,  1 Sep 2013 10:14:15 -0700 (PDT)\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 ktuiO3+Sz0q8 for <notmuch@notmuchmail.org>;\r
  18         Sun,  1 Sep 2013 10:14:07 -0700 (PDT)\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 D5F54431FAF\r
  23         for <notmuch@notmuchmail.org>; Sun,  1 Sep 2013 10:14:06 -0700 (PDT)\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 1VGBDu-0001xs-OC; Sun, 01 Sep 2013 18:14:05 +0100\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.71)\r
  30         (envelope-from <m.walters@qmul.ac.uk>)\r
  31         id 1VGBDt-00078v-H4; Sun, 01 Sep 2013 18:14:01 +0100\r
  32 From: Mark Walters <markwalters1009@gmail.com>\r
  33 To: Istvan Marko <notmuch@kismala.com>, notmuch@notmuchmail.org\r
  34 Subject: Re: [PATCH] emacs: add buttons for all multipart/related parts\r
  35 In-Reply-To: <m3r4dtgz9k.fsf@zsu.kismala.com>\r
  36 References: <m3r4dtgz9k.fsf@zsu.kismala.com>\r
  37 User-Agent: Notmuch/0.15.2+269~g01f5508 (http://notmuchmail.org) Emacs/23.4.1\r
  38         (x86_64-pc-linux-gnu)\r
  39 Date: Sun, 01 Sep 2013 18:13:59 +0100\r
  40 Message-ID: <871u58lb1k.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: b43aa68e48acbff414177ce82b8a0f9e (of first 20000 bytes)\r
  46 X-SpamAssassin-Score: 0.6\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 0.6 points. Summary of the scoring: \r
  52         * 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\r
  53         provider *      (markwalters1009[at]gmail.com)\r
  54         *  1.0 FREEMAIL_REPLY From and body contain different freemails\r
  55         * -0.4 AWL AWL: From: address is in the auto white-list\r
  56 X-QM-Scan-Virus: ClamAV says the message is clean\r
  57 X-BeenThere: notmuch@notmuchmail.org\r
  58 X-Mailman-Version: 2.1.13\r
  59 Precedence: list\r
  60 List-Id: "Use and development of the notmuch mail system."\r
  61         <notmuch.notmuchmail.org>\r
  62 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  63         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  64 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  65 List-Post: <mailto:notmuch@notmuchmail.org>\r
  66 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  67 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  68         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  69 X-List-Received-Date: Sun, 01 Sep 2013 17:14:15 -0000\r
  70 \r
  71 \r
  72 On Fri, 16 Aug 2013, Istvan Marko <notmuch@kismala.com> wrote:\r
  73 > When text/html parts include images as multipart/related and the\r
  74 > text/plain alternative is used these images can be completely hidden\r
  75 > with no easy way to access them or even find out that they are there.\r
  76 >\r
  77 > Make notmuch-show-insert-part-multipart/related add buttons for all\r
  78 > parts, the first one visible the rest hidden.\r
  79 \r
  80 Hello\r
  81 \r
  82 This patch LGTM +1.\r
  83 \r
  84 It is a clear improvement anyway but would benefit from my fix for the\r
  85 bug this exposes\r
  86 id:1377246875-7784-1-git-send-email-markwalters1009@gmail.com (or some\r
  87 alternative). I don't know if we want to push the two together or just\r
  88 push each as it gets reviewed.\r
  89 \r
  90 Best wishes\r
  91 \r
  92 Mark\r
  93 \r
  94 \r
  95 \r
  96 \r
  97 \r
  98 \r
  99 > ---\r
 100 >  emacs/notmuch-show.el | 4 ++++\r
 101 >  1 file changed, 4 insertions(+)\r
 102 >\r
 103 > diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el\r
 104 > index 82b70ba..20844f0 100644\r
 105 > --- a/emacs/notmuch-show.el\r
 106 > +++ b/emacs/notmuch-show.el\r
 107 > @@ -621,6 +621,10 @@ message at DEPTH in the current thread."\r
 108 >  \r
 109 >      ;; Render the primary part.\r
 110 >      (notmuch-show-insert-bodypart msg (car inner-parts) depth)\r
 111 > +    ;; Add hidden buttons for the rest\r
 112 > +    (mapc (lambda (inner-part)\r
 113 > +         (notmuch-show-insert-bodypart msg inner-part depth t))\r
 114 > +       (cdr inner-parts))\r
 115 >  \r
 116 >      (when notmuch-show-indent-multipart\r
 117 >        (indent-rigidly start (point) 1)))\r
 118 > -- \r
 119 > 1.8.1.4\r
 120 >\r
 121 >\r
 122 >\r
 123 >\r
 124 > -- \r
 125 >       Istvan\r
 126 > _______________________________________________\r
 127 > notmuch mailing list\r
 128 > notmuch@notmuchmail.org\r
 129 > http://notmuchmail.org/mailman/listinfo/notmuch\r