e4/261b0dae8a98b73973f722a2ed3c06253aedc9

   1 Return-Path: <amdragon@mit.edu>\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 09969431FAF\r
   6         for <notmuch@notmuchmail.org>; Mon, 10 Dec 2012 20:07:55 -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: -0.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] 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 n7HfM+xQAeM3 for <notmuch@notmuchmail.org>;\r
  16         Mon, 10 Dec 2012 20:07:54 -0800 (PST)\r
  17 Received: from dmz-mailsec-scanner-3.mit.edu (DMZ-MAILSEC-SCANNER-3.MIT.EDU\r
  18         [18.9.25.14])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 882A2431FAE\r
  20         for <notmuch@notmuchmail.org>; Mon, 10 Dec 2012 20:07:54 -0800 (PST)\r
  21 X-AuditID: 1209190e-b7f516d0000008e4-9d-50c6b19ad62c\r
  22 Received: from mailhub-auth-2.mit.edu ( [18.7.62.36])\r
  23         by dmz-mailsec-scanner-3.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id 9E.D6.02276.A91B6C05; Mon, 10 Dec 2012 23:07:54 -0500 (EST)\r
  25 Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
  26         by mailhub-auth-2.mit.edu (8.13.8/8.9.2) with ESMTP id qBB47rdv003793; \r
  27         Mon, 10 Dec 2012 23:07:54 -0500\r
  28 Received: from awakening.csail.mit.edu (awakening.csail.mit.edu [18.26.4.91])\r
  29         (authenticated bits=0)\r
  30         (User authenticated as amdragon@ATHENA.MIT.EDU)\r
  31         by outgoing.mit.edu (8.13.6/8.12.4) with ESMTP id qBB47qhh017737\r
  32         (version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NOT);\r
  33         Mon, 10 Dec 2012 23:07:53 -0500 (EST)\r
  34 Received: from amthrax by awakening.csail.mit.edu with local (Exim 4.80)\r
  35         (envelope-from <amdragon@mit.edu>)\r
  36         id 1TiH8K-0003ct-5p; Mon, 10 Dec 2012 23:07:52 -0500\r
  37 From: Austin Clements <aclements@csail.mit.edu>\r
  38 To: Mark Walters <markwalters1009@gmail.com>, notmuch@notmuchmail.org\r
  39 Subject: Re: [PATCH] emacs: show: make RET always toggle parts where plausible\r
  40 In-Reply-To: <1354700480-5170-1-git-send-email-markwalters1009@gmail.com>\r
  41 References: <1354663662-20524-4-git-send-email-markwalters1009@gmail.com>\r
  42         <1354700480-5170-1-git-send-email-markwalters1009@gmail.com>\r
  43 User-Agent: Notmuch/0.14+159~g6895fee (http://notmuchmail.org) Emacs/23.4.1\r
  44         (i486-pc-linux-gnu)\r
  45 Date: Mon, 10 Dec 2012 23:07:52 -0500\r
  46 Message-ID: <87obi1nsdj.fsf@awakening.csail.mit.edu>\r
  47 MIME-Version: 1.0\r
  48 Content-Type: text/plain; charset=us-ascii\r
  49 X-Brightmail-Tracker:\r
  50  H4sIAAAAAAAAA+NgFnrAIsWRmVeSWpSXmKPExsUixG6nojtr47EAg02fuSxWz+WxuH5zJrMD\r
  51         k8fOWXfZPZ6tusUcwBTFZZOSmpNZllqkb5fAlbH69m/mgq08FRc2XGFpYJzC1cXIySEhYCLR\r
  52         ffARK4QtJnHh3nq2LkYuDiGBfYwSk+a+YQNJCAlsYJR49TwbInGRSWLavG3MEM4SRolNc3Yx\r
  53         glSxCehLrFg7CWyUiICrxNNvn5lBbGEBP4n/1xrBJnEKeErcOHyOFaK5jVFibc9uoCIODlGB\r
  54         eInZ53xAalgEVCW+NGxlAbF5gc67s3ImlC0ocXLmEzCbWUBL4sa/l0wTGAVmIUnNQpJawMi0\r
  55         ilE2JbdKNzcxM6c4NVm3ODkxLy+1SNdYLzezRC81pXQTIyggOSX5djB+Pah0iFGAg1GJh7dC\r
  56         9ViAEGtiWXFl7iFGSQ4mJVFe3+VAIb6k/JTKjMTijPii0pzU4kOMEhzMSiK8pblAOd6UxMqq\r
  57         1KJ8mJQ0B4uSOO+VlJv+QgLpiSWp2ampBalFMFkZDg4lCd7dG4AaBYtS01Mr0jJzShDSTByc\r
  58         IMN5gIYXgdTwFhck5hZnpkPkTzEqSonzLgNJCIAkMkrz4HphCeMVozjQK8K8PSBVPMBkA9f9\r
  59         CmgwE9Dgk4KHQQaXJCKkpBoYk4U73qhkXn5k7jFFbME720muG1dMr/Tu+dUzO7LJfOM5AWn+\r
  60         vY95Zpzl+utY/+67HVtB1rl30SLWexnrSsIuJ7m5K6n052ZzFt/6taZD4fsvVRnznj9PuoP8\r
  61         3N71TlCccO/6JYUArUXe4XNCWctdpux6/bB9Rd/j113Sy9Z4Lp1rwb0yKjpciaU4I9FQi7mo\r
  62         OBEAZA2Q5PMCAAA=\r
  63 X-BeenThere: notmuch@notmuchmail.org\r
  64 X-Mailman-Version: 2.1.13\r
  65 Precedence: list\r
  66 List-Id: "Use and development of the notmuch mail system."\r
  67         <notmuch.notmuchmail.org>\r
  68 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  69         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  70 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  71 List-Post: <mailto:notmuch@notmuchmail.org>\r
  72 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  73 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  74         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  75 X-List-Received-Date: Tue, 11 Dec 2012 04:07:55 -0000\r
  76 \r
  77 I would just fold this patch in with the rest of the series.  This is\r
  78 definitely the right way to toggle.\r
  79 \r
  80 On Wed, 05 Dec 2012, Mark Walters <markwalters1009@gmail.com> wrote:\r
  81 > This makes RET toggle the visibility of any part which has a component\r
  82 > that can be displayed in the buffer. This included text parts (plain/\r
  83 > html/ x-tex etc) and images. Parts which cannot be displayed (eg pdf)\r
  84 > RET acts as before and saves/views etc.\r
  85 >\r
  86 > Definite actions can always be accessed with s,v and o (eg v to view a\r
  87 > text/html part in a browser).\r
  88 >\r
  89 > ---\r
  90 >\r
  91 > This is a slight tweak at Jani's suggestion on irc. Applies on top of\r
  92 > the previous series.\r
  93 >\r
  94 >\r
  95 >\r
  96 >  emacs/notmuch-show.el |    2 +-\r
  97 >  1 files changed, 1 insertions(+), 1 deletions(-)\r
  98 >\r
  99 > diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el\r
 100 > index a4daff8..e319e5c 100644\r
 101 > --- a/emacs/notmuch-show.el\r
 102 > +++ b/emacs/notmuch-show.el\r
 103 > @@ -2028,7 +2028,7 @@ the user (see `notmuch-show-stash-mlarchive-link-alist')."\r
 104 >  (defun notmuch-show-part-button-default (&optional button)\r
 105 >    (interactive)\r
 106 >    (let ((button (or button (button-at (point)))))\r
 107 > -    (if (invisible-p (button-get button 'invisibility-spec))\r
 108 > +    (if (button-get button 'invisibility-spec)\r
 109 >       (notmuch-show-toggle-invisible-part-action button)\r
 110 >        (notmuch-show-part-button-internal button notmuch-show-part-button-default-action))))\r
 111 >  \r
 112 > -- \r
 113 > 1.7.9.1\r
 114 >\r
 115 > _______________________________________________\r
 116 > notmuch mailing list\r
 117 > notmuch@notmuchmail.org\r
 118 > http://notmuchmail.org/mailman/listinfo/notmuch\r