1d/5141601295b117ea1fe2b7b5f239220a5e5426

   1 Return-Path: <jrollins@finestructure.net>\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 A814D431FC0\r
   6         for <notmuch@notmuchmail.org>; Tue, 20 Aug 2013 08:18:08 -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: -2.3\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.3 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3] autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id kZEaRMFaNJQ5 for <notmuch@notmuchmail.org>;\r
  16         Tue, 20 Aug 2013 08:18:04 -0700 (PDT)\r
  17 Received: from outgoing-mail.its.caltech.edu (outgoing-mail.its.caltech.edu\r
  18         [131.215.239.19])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 444B0431FB6\r
  20         for <notmuch@notmuchmail.org>; Tue, 20 Aug 2013 08:18:04 -0700 (PDT)\r
  21 Received: from fire-doxen.imss.caltech.edu (localhost [127.0.0.1])\r
  22         by fire-doxen-postvirus (Postfix) with ESMTP id AC1A732802C;\r
  23         Tue, 20 Aug 2013 08:18:01 -0700 (PDT)\r
  24 X-Spam-Scanned: at Caltech-IMSS on fire-doxen by amavisd-new\r
  25 Received: from finestructure.net (cpe-76-173-75-27.socal.res.rr.com\r
  26         [76.173.75.27]) (Authenticated sender: jrollins)\r
  27         by fire-doxen-submit (Postfix) with ESMTP id DADAE2E50D38;\r
  28         Tue, 20 Aug 2013 08:17:57 -0700 (PDT)\r
  29 Received: by finestructure.net (Postfix, from userid 1000)\r
  30         id 627D761730; Tue, 20 Aug 2013 08:17:57 -0700 (PDT)\r
  31 From: Jameson Graef Rollins <jrollins@finestructure.net>\r
  32 To: Istvan Marko <notmuch@kismala.com>, notmuch@notmuchmail.org\r
  33 Subject: Re: [PATCH] emacs: add buttons for all multipart/related parts\r
  34 In-Reply-To: <m3r4dtgz9k.fsf@zsu.kismala.com>\r
  35 References: <m3r4dtgz9k.fsf@zsu.kismala.com>\r
  36 User-Agent: Notmuch/0.16+1~g9f861e2 (http://notmuchmail.org) Emacs/24.3.1\r
  37         (x86_64-pc-linux-gnu)\r
  38 Date: Tue, 20 Aug 2013 08:17:54 -0700\r
  39 Message-ID: <87fvu4fl25.fsf@servo.finestructure.net>\r
  40 MIME-Version: 1.0\r
  41 Content-Type: multipart/signed; boundary="=-=-=";\r
  42         micalg=pgp-sha256; protocol="application/pgp-signature"\r
  43 X-BeenThere: notmuch@notmuchmail.org\r
  44 X-Mailman-Version: 2.1.13\r
  45 Precedence: list\r
  46 List-Id: "Use and development of the notmuch mail system."\r
  47         <notmuch.notmuchmail.org>\r
  48 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  49         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  50 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  51 List-Post: <mailto:notmuch@notmuchmail.org>\r
  52 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  53 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  54         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  55 X-List-Received-Date: Tue, 20 Aug 2013 15:18:08 -0000\r
  56 \r
  57 --=-=-=\r
  58 Content-Type: text/plain\r
  59 Content-Transfer-Encoding: quoted-printable\r
  60 \r
  61 On Fri, Aug 16 2013, Istvan Marko <notmuch@kismala.com> wrote:\r
  62 > When text/html parts include images as multipart/related and the\r
  63 > text/plain alternative is used these images can be completely hidden\r
  64 > with no easy way to access them or even find out that they are there.\r
  65 >\r
  66 > Make notmuch-show-insert-part-multipart/related add buttons for all\r
  67 > parts, the first one visible the rest hidden.\r
  68 \r
  69 Hey, Istvan.  Thanks so much for this patch.  It seems to be addressing\r
  70 the same issue I reported in id:87ob8u8l6k.fsf@servo.finestructure.net.\r
  71 \r
  72 However, the behavior of the part button that now appears seems to be a\r
  73 bit strange.  Clicking/hitting enter on the part attempts to save it\r
  74 rather than open it.  I wouldn't be surprised if has nothing to do with\r
  75 the correctness of this patch, though.  So is it possible that the\r
  76 button is not inserted in the most ideal way, or is there some other\r
  77 issue preventing it from behaving properly (could it have to do with the\r
  78 part being "inline"?)?\r
  79 \r
  80 jamie.\r
  81 \r
  82 \r
  83 >  emacs/notmuch-show.el | 4 ++++\r
  84 >  1 file changed, 4 insertions(+)\r
  85 >\r
  86 > diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el\r
  87 > index 82b70ba..20844f0 100644\r
  88 > --- a/emacs/notmuch-show.el\r
  89 > +++ b/emacs/notmuch-show.el\r
  90 > @@ -621,6 +621,10 @@ message at DEPTH in the current thread."\r
  91 >=20=20\r
  92 >      ;; Render the primary part.\r
  93 >      (notmuch-show-insert-bodypart msg (car inner-parts) depth)\r
  94 > +    ;; Add hidden buttons for the rest\r
  95 > +    (mapc (lambda (inner-part)\r
  96 > +         (notmuch-show-insert-bodypart msg inner-part depth t))\r
  97 > +       (cdr inner-parts))\r
  98 >=20=20\r
  99 >      (when notmuch-show-indent-multipart\r
 100 >        (indent-rigidly start (point) 1)))\r
 101 > --=20\r
 102 > 1.8.1.4\r
 103 \r
 104 --=-=-=\r
 105 Content-Type: application/pgp-signature\r
 106 \r
 107 -----BEGIN PGP SIGNATURE-----\r
 108 Version: GnuPG v1.4.14 (GNU/Linux)\r
 109 \r
 110 iQIcBAEBCAAGBQJSE4ijAAoJEO00zqvie6q86CYP/ioM/6FNf947Gni2n2EpSgCs\r
 111 csO1rxQrEHuLvcTljUDBfwP0R8NC7y22mGCl1iBx6mVcXKjPTQa97WVqBPRSYsJR\r
 112 FcIycTW0MSBp5rOX8ZvsJG4DxlF8I+NLctne35uwWawE0LQqYz2QHERdr59YB438\r
 113 qVNdZkcakEPUaK44Upk9FIEo7efEtIjSBkCFIDEMgiTVOz+dqW79SuDRx1oJjeit\r
 114 0eyKDtLdqXvfUjvI0xD/zcdYU36W2BI175EjmuAnr1Ow50Q73AQFzjaF+z4DtOjr\r
 115 +saXVPiRiuO45LgvW0JAVc11sY8t1+caFjx2zFnRy/8viGyWEvnLmucoTf/yWAvR\r
 116 L1PnI4Gb9qykGl2SS+npfuTrZdVgicun8S/H8n0lt5HEI03Ku2GzfF38kXB4w3dj\r
 117 CBsZrCKAKCErSfhGp/ZvT4O6qCaZ08BRzZmXebXTPI5PKPTwEDBL7M9doCPT/ZvV\r
 118 XspKqS9bKXfERos28sWUWAJxgxo7SCEiYqZgX4mKFpUk5jkOTY5oIWDWs+2C1EZa\r
 119 HOI5ZQxCm7RjLbfz//o4zA4ukCTerMFRREwFfi8eju34Hdv1tBB0Zyo8k7vpv3Eu\r
 120 cTmdxmOcmgTWOvNrKUx6cfoLZmFx42z7pHVVHU62GxvLHUIvLEuVDV2MPjlLgRev\r
 121 5TnpD03LLoaZx4pM3rUB\r
 122 =MEVP\r
 123 -----END PGP SIGNATURE-----\r
 124 --=-=-=--\r