43/0bf2fe9010d6a4d2512e20f13e02cabe221026

   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 E7DA8431FD4\r
   6         for <notmuch@notmuchmail.org>; Mon, 16 Jan 2012 12:12:10 -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 t4wJKQFE4PNC for <notmuch@notmuchmail.org>;\r
  16         Mon, 16 Jan 2012 12:12:10 -0800 (PST)\r
  17 Received: from dmz-mailsec-scanner-4.mit.edu (DMZ-MAILSEC-SCANNER-4.MIT.EDU\r
  18         [18.9.25.15])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 5D644431FB6\r
  20         for <notmuch@notmuchmail.org>; Mon, 16 Jan 2012 12:12:10 -0800 (PST)\r
  21 X-AuditID: 1209190f-b7f8a6d000000914-af-4f1484995b3e\r
  22 Received: from mailhub-auth-3.mit.edu ( [18.9.21.43])\r
  23         by dmz-mailsec-scanner-4.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id AC.F6.02324.994841F4; Mon, 16 Jan 2012 15:12:09 -0500 (EST)\r
  25 Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
  26         by mailhub-auth-3.mit.edu (8.13.8/8.9.2) with ESMTP id q0GKC8cE023611; \r
  27         Mon, 16 Jan 2012 15:12:09 -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 q0GKC7dQ005377\r
  32         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  33         Mon, 16 Jan 2012 15:12:08 -0500 (EST)\r
  34 Received: from amthrax by awakening.csail.mit.edu with local (Exim 4.77)\r
  35         (envelope-from <amdragon@mit.edu>)\r
  36         id 1RmsuN-00065p-3t; Mon, 16 Jan 2012 15:11:59 -0500\r
  37 Date: Mon, 16 Jan 2012 15:11:59 -0500\r
  38 From: Austin Clements <amdragon@MIT.EDU>\r
  39 To: Jameson Graef Rollins <jrollins@finestructure.net>\r
  40 Subject: Re: [PATCH 1/1] Make buttons for attachments allow viewing as well\r
  41         as saving\r
  42 Message-ID: <20120116201159.GD16740@mit.edu>\r
  43 References: <1326629796-11436-1-git-send-email-markwalters1009@gmail.com>\r
  44         <1326629796-11436-2-git-send-email-markwalters1009@gmail.com>\r
  45         <87wr8r5trv.fsf@servo.finestructure.net>\r
  46 MIME-Version: 1.0\r
  47 Content-Type: text/plain; charset=us-ascii\r
  48 Content-Disposition: inline\r
  49 In-Reply-To: <87wr8r5trv.fsf@servo.finestructure.net>\r
  50 User-Agent: Mutt/1.5.21 (2010-09-15)\r
  51 X-Brightmail-Tracker:\r
  52  H4sIAAAAAAAAA+NgFuphleLIzCtJLcpLzFFi42IR4hTV1p3ZIuJvcHCGicWefV4Wq+fyWFy/\r
  53         OZPZgdnj7mkuj52z7rJ7PFt1izmAOYrLJiU1J7MstUjfLoErY8O/h8wFl4QquleuZWxg/MLb\r
  54         xcjJISFgInFs32dWCFtM4sK99WxdjFwcQgL7GCUWdL5hhnA2MEq8Or2LFcI5ySQxb/MVqLIl\r
  55         jBKTb3cyg/SzCKhK7H62mwnEZhPQkNi2fzkjiC0iYCbR8+UPmM0s4Cox48IusBphgXCJO3cW\r
  56         sIPYvAI6EvdOb4HasAVo6J9XjBAJQYmTM5+wQDRrSdz49xKomQPIlpZY/o8DJMwpYCpx/VIj\r
  57         WImogIrElJPb2CYwCs1C0j0LSfcshO4FjMyrGGVTcqt0cxMzc4pTk3WLkxPz8lKLdE30cjNL\r
  58         9FJTSjcxggNdkn8H47eDSocYBTgYlXh4b3qL+AuxJpYVV+YeYpTkYFIS5Y1tBArxJeWnVGYk\r
  59         FmfEF5XmpBYfYpTgYFYS4RX0AcrxpiRWVqUW5cOkpDlYlMR51bTe+QkJpCeWpGanphakFsFk\r
  60         ZTg4lCR4fzQDNQoWpaanVqRl5pQgpJk4OEGG8wAN/wNSw1tckJhbnJkOkT/FqCglznsIJCEA\r
  61         ksgozYPrhSWiV4ziQK8I834AqeIBJjG47ldAg5mABue0CoEMLklESEk1MGa92ea13iiogn8D\r
  62         e+zvzb92d4RW8Olqr5Y6cO6vdZ/y0Zt7j1zzn3O2/KWk5tzVv/sUrhhH5V0N2yLC22LzPT7E\r
  63         tJrZZ42NWSO3x+4JUyWfpJ948Jl5V8m/ld/r3hpfOdgusMZ6aonNjlvVhgYadc9uzMhU+R53\r
  64         9jCXhAPDlDn/K/2KOOd371NiKc5INNRiLipOBACs4oeCHwMAAA==\r
  65 Cc: notmuch@notmuchmail.org\r
  66 X-BeenThere: notmuch@notmuchmail.org\r
  67 X-Mailman-Version: 2.1.13\r
  68 Precedence: list\r
  69 List-Id: "Use and development of the notmuch mail system."\r
  70         <notmuch.notmuchmail.org>\r
  71 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  72         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  73 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  74 List-Post: <mailto:notmuch@notmuchmail.org>\r
  75 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  76 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  77         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  78 X-List-Received-Date: Mon, 16 Jan 2012 20:12:11 -0000\r
  79 \r
  80 Quoth Jameson Graef Rollins on Jan 16 at 11:31 am:\r
  81 > On Sun, 15 Jan 2012 12:16:36 +0000, Mark Walters <markwalters1009@gmail.com> wrote:\r
  82 > > Define a keymap for attachment buttons to allow multiple actions.\r
  83 > > Define 3 possible actions:\r
  84 > >     save attachment: exactly as currently,\r
  85 > >     view attachment: uses mailcap entry,\r
  86 > >     view attachment with user chosen program\r
  87 > \r
  88 > Great improvement, Mark!  Thanks for this.  I've been wanting this kind\r
  89 > of functionality for a while, actually, and this is a really great\r
  90 > implementation.  It works like a charm, and the code looks good to me,\r
  91 > modulo a couple small comments below.\r
  92 > \r
  93 > > Keymap on a button is: s for save, v for view and o for view with\r
  94 > > other program. Default (i.e. enter or mouse button) is save but is\r
  95 > > easily configurable e.g. set to view with\r
  96 > > (setq notmuch-show-part-button-default-action 'notmuch-show-part-view-action)\r
  97 > \r
  98 > Actually, this should really be a defcustom.  Maybe something like this:\r
  99 > \r
 100 > (defcustom notmuch-show-part-button-default-action 'notmuch-show-part-button-save\r
 101 >   "Default part header button action (on ENTER or mouse click)."\r
 102 >   :group 'notmuch\r
 103 >   :type '(choice (function :tag "Save part"\r
 104 >                           :value notmuch-show-part-button-save)\r
 105 >                 (function :tag "View part"\r
 106 >                           :value notmuch-show-part-button-view)\r
 107 >                 (function :tag "View interactively"\r
 108 >                           :value notmuch-show-part-button-interactively-view))\r
 109 > \r
 110 > Unfortunately this isn't quite working right, since it's not setting the\r
 111 > default properly, but if someone can help me figure out what I'm doing\r
 112 > wrong, I think this is at least the right idea.\r
 113 \r
 114 Jamie's defcustom doesn't work for me either (apparently it works even\r
 115 less for me than it does for Jamie), but the following works for me\r
 116 \r
 117 (defcustom notmuch-show-part-button-default-action 'notmuch-show-part-button-save\r
 118   "Default part header button action (on ENTER or mouse click)."\r
 119   :group 'notmuch\r
 120   :type '(choice (const :tag "Save part"\r
 121                         notmuch-show-part-button-save)\r
 122                  (const :tag "View part"\r
 123                         notmuch-show-part-button-view)\r
 124                  (const :tag "View interactively"\r
 125                         notmuch-show-part-button-interactively-view)))\r