0f/55361de26340ff45923e8a225db5132eebb07e

   1 Return-Path: <awg@xvx.ca>\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 46F26431FD0\r
   6         for <notmuch@notmuchmail.org>; Mon,  9 Jan 2012 17:53: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: -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 JyM78WbgLOCI for <notmuch@notmuchmail.org>;\r
  16         Mon,  9 Jan 2012 17:53:32 -0800 (PST)\r
  17 Received: from mail-bk0-f53.google.com (mail-bk0-f53.google.com\r
  18         [209.85.214.53]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 4E46C431FB6\r
  21         for <notmuch@notmuchmail.org>; Mon,  9 Jan 2012 17:53:32 -0800 (PST)\r
  22 Received: by bkat2 with SMTP id t2so93876bka.26\r
  23         for <notmuch@notmuchmail.org>; Mon, 09 Jan 2012 17:53:30 -0800 (PST)\r
  24 MIME-Version: 1.0\r
  25 Received: by 10.205.120.11 with SMTP id fw11mr8048335bkc.89.1326160410609;\r
  26         Mon, 09 Jan 2012 17:53:30 -0800 (PST)\r
  27 Received: by 10.205.33.73 with HTTP; Mon, 9 Jan 2012 17:53:30 -0800 (PST)\r
  28 X-Originating-IP: [12.50.241.130]\r
  29 In-Reply-To: <m2r4z9irmp.fsf@gmail.com>\r
  30 References: <1326009162-19524-1-git-send-email-awg+notmuch@xvx.ca>\r
  31         <m2r4z9irmp.fsf@gmail.com>\r
  32 Date: Mon, 9 Jan 2012 18:53:30 -0700\r
  33 Message-ID:\r
  34  <CAMoJFUupEQofCnqaaDLEaH+_QcO4b7hd-f8=HX8jJUCFke40Qw@mail.gmail.com>\r
  35 Subject: Re: [PATCH 0/4] Quoting HTML-only emails in replies redux\r
  36 From: Adam Wolfe Gordon <awg@xvx.ca>\r
  37 To: Aaron Ecay <aaronecay@gmail.com>\r
  38 Content-Type: text/plain; charset=windows-1252\r
  39 Content-Transfer-Encoding: quoted-printable\r
  40 X-Mailman-Approved-At: Wed, 11 Jan 2012 18:03:20 -0800\r
  41 Cc: notmuch@notmuchmail.org\r
  42 X-BeenThere: notmuch@notmuchmail.org\r
  43 X-Mailman-Version: 2.1.13\r
  44 Precedence: list\r
  45 List-Id: "Use and development of the notmuch mail system."\r
  46         <notmuch.notmuchmail.org>\r
  47 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  48         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  49 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  50 List-Post: <mailto:notmuch@notmuchmail.org>\r
  51 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  52 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  53         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  54 X-List-Received-Date: Tue, 10 Jan 2012 01:53:33 -0000\r
  55 \r
  56 Thanks for the suggestions.  Specific comments inline:\r
  57 \r
  58 On Sun, Jan 8, 2012 at 18:36, Aaron Ecay <aaronecay@gmail.com> wrote:\r
  59 >> There should probably be some customize variables for this in emacs, to =\r
  60 control\r
  61 >> (for example) whether to quote HTML parts and whether to prefer HTML or\r
  62 >> plaintext parts for quoting. Any suggestions for what should be customiz=\r
  63 able\r
  64 >> would be appreciated.\r
  65 >\r
  66 > I think two variables should suffice: one (boolean) to control whether\r
  67 > to quote standalone text/html parts, and one to control which parts of a\r
  68 > multipart/alternative part to quote. =A0The latter should take possible\r
  69 > values 'text, 'html, and 'both. =A0This requires the emacs reply\r
  70 > functionality to distinguish between html parts that are part of a\r
  71 > multipart/alternative and those which are not, which (AFAICT) the\r
  72 > current code doesn=92t do.\r
  73 \r
  74 The first one I think is obvious, and easy to add.  My previous\r
  75 version actually had such an option, but I didn't bother with it this\r
  76 time in the interest of getting the patch out.  I'll add this in and\r
  77 send a new version.\r
  78 \r
  79 Regarding the second suggestion, you're right that the current code\r
  80 doesn't differentiate between text parts that are part of a\r
  81 multipart/alternative and ones that aren't.  However, it does only\r
  82 include parts with inline disposition, so it shouldn't end up quoting\r
  83 stuff intended as attachments.  My thinking in the current operation\r
  84 was to keep most of the complexity (i.e. going through the MIME parts\r
  85 to pick out the relevant ones) in the C code, so that it doesn't have\r
  86 to be implemented in each client - and also so I didn't have to\r
  87 implement it in emacs lisp ;-).  Do you have some ideas for a JSON\r
  88 format that's more complex than the current one (to include the\r
  89 information necessary for making decisions about what to quote), but\r
  90 less complex than something like notmuch show --format=3Djson (which\r
  91 would require the client to descend the MIME tree completely to create\r
  92 a reply)?\r
  93 \r
  94 A third customization option I was thinking about is a way to control\r
  95 the format of the first line of the body (On $date, $person wrote).  I\r
  96 think it would be fairly simple to let people specify an arbitrary\r
  97 format using any of the headers of the original message, so you would\r
  98 set it to something like "On %date%, %from% wrote", or "Quoth %from%\r
  99 (%date%)".  This might be particularly useful to users who correspond\r
 100 normally in a language other than English.  Anyone have thoughts on\r
 101 whether this is worth implementing?\r
 102 \r
 103 > I haven=92t tested the patch yet, but it looks very promising. =A0Thanks!\r
 104 \r
 105 Thanks for the suggestions - let me know if you have a chance to try it out=\r
 106 .\r
 107 \r
 108 --=20\r
 109 Adam Wolfe Gordon\r