e8/8e26c4d974e6faecc9ebf097172b3a31d4dfd1

   1 Return-Path: <cworth@cworth.org>\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 D53A14196F2\r
   6         for <notmuch@notmuchmail.org>; Sat, 24 Apr 2010 08:14:13 -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.89\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.89 tagged_above=-999 required=5\r
  12         tests=[ALL_TRUSTED=-1, BAYES_00=-1.9, T_MIME_NO_TEXT=0.01]\r
  13         autolearn=ham\r
  14 Received: from olra.theworths.org ([127.0.0.1])\r
  15         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16         with ESMTP id PavO4KJvrPtQ; Sat, 24 Apr 2010 08:14:12 -0700 (PDT)\r
  17 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  18         by olra.theworths.org (Postfix) with ESMTP id BB1E8431FC1;\r
  19         Sat, 24 Apr 2010 08:14:12 -0700 (PDT)\r
  20 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  21         id 61A4F568DEC; Sat, 24 Apr 2010 08:14:12 -0700 (PDT)\r
  22 From: Carl Worth <cworth@cworth.org>\r
  23 To: David Edmondson <dme@dme.org>, notmuch@notmuchmail.org\r
  24 Subject: Re: [PATCH] emacs: Add more functions to clean up text/plain parts\r
  25 In-Reply-To: <1271939166-24100-1-git-send-email-dme@dme.org>\r
  26 References: <1271939166-24100-1-git-send-email-dme@dme.org>\r
  27 User-Agent: Notmuch/0.2-188-g9757095 (http://notmuchmail.org) Emacs/23.1.1\r
  28         (i486-pc-linux-gnu)\r
  29 Date: Sat, 24 Apr 2010 08:14:12 -0700\r
  30 Message-ID: <87tyr0hohn.fsf@yoom.home.cworth.org>\r
  31 MIME-Version: 1.0\r
  32 Content-Type: multipart/signed; boundary="=-=-=";\r
  33         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  34 X-BeenThere: notmuch@notmuchmail.org\r
  35 X-Mailman-Version: 2.1.13\r
  36 Precedence: list\r
  37 List-Id: "Use and development of the notmuch mail system."\r
  38         <notmuch.notmuchmail.org>\r
  39 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  40         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  41 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  42 List-Post: <mailto:notmuch@notmuchmail.org>\r
  43 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  44 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  45         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  46 X-List-Received-Date: Sat, 24 Apr 2010 15:14:14 -0000\r
  47 \r
  48 --=-=-=\r
  49 Content-Transfer-Encoding: quoted-printable\r
  50 \r
  51 On Thu, 22 Apr 2010 13:26:06 +0100, David Edmondson <dme@dme.org> wrote:\r
  52 > This is a small variant on the previous version of the patch. The\r
  53 > wrapping of long lines is not enabled by default - it's simply an\r
  54 > option in the customise interface.\r
  55 \r
  56 This is really close now. I especially like that the various wash\r
  57 options are as simple as just checkboxes in the customize\r
  58 interface. (That might not even be new in this case, but I at least\r
  59 didn't find it before.)\r
  60 \r
  61 And that again goes to my point. I don't think we should enable this\r
  62 washing by default since it can appear as silent corruption to the user,\r
  63 without any indication that it happened nor how to turn it off.\r
  64 \r
  65 Notmuch has always done some modification of the message with things\r
  66 like hiding long citations, etc. But those at least provide\r
  67 self-documenting buttons on how to make them disappear.\r
  68 \r
  69 Here's what I see in the customize buffer with the latest patch, along\r
  70 with some review. Some of this review applies to documentation already\r
  71 in notmuch---I'm just getting pickier as things appear in\r
  72 customize because I think we need to hold our documentation there to a\r
  73 higher standard. (Previously, one would practically have to dive into\r
  74 the source to find the documentation, and that suggests the reader has\r
  75 more experience and definitely means the user gsts a lot more context).\r
  76 \r
  77 Here, I'm trying to review these options from the point of view of a new\r
  78 user who just started using notmuch, wants to tweak a few things, and is\r
  79 looking at the customize buffer to figure out what tweaks are possible.\r
  80 \r
  81         Notmuch Show Hook:=20\r
  82         [X] notmuch-show-pretty-hook\r
  83         [ ] notmuch-show-turn-off-word-wrap\r
  84         INS\r
  85            State: STANDARD.\r
  86            A list of functions called after populating a More\r
  87 \r
  88 What does "pretty hook" mean? What information would a user need to\r
  89 determine whether to turn this on or off?\r
  90 \r
  91 With "turn off word wrap", we have on option to disable wrapping here,\r
  92 and another option later to turn some wrapping on again. How is a user\r
  93 expected to figure out which combination of options does what they want?\r
  94 \r
  95         Notmuch Show Insert Text/Plain Hook: Hide Value\r
  96         [ ] notmuch-wash-wrap-long-lines\r
  97                 Wrap text in the region whilst maintaining the correct prefix.\r
  98         [X] notmuch-wash-tidy-citations\r
  99                 Clean up citations.\r
 100         [X] notmuch-wash-compress-blanks\r
 101                 Compress successive blank lines into one blank line. Remove More\r
 102         [X] notmuch-wash-markup-citations\r
 103                 Markup citations, and up to one signature in the buffer.\r
 104 \r
 105 "Tidy", "Clean up", and "Markup" citations are all too vague. What do\r
 106 each of these actually do?\r
 107 \r
 108 Also, think about line breaks so that something like "Remove more"\r
 109 doesn't appear there by default.\r
 110 \r
 111 Thanks,\r
 112 \r
 113 =2DCarl\r
 114 \r
 115 --=-=-=\r
 116 Content-Type: application/pgp-signature\r
 117 \r
 118 -----BEGIN PGP SIGNATURE-----\r
 119 Version: GnuPG v1.4.10 (GNU/Linux)\r
 120 \r
 121 iD8DBQFL0wrE6JDdNq8qSWgRAkBFAKCLHnzisSaucm51jUNocz/sV6kPygCghPsW\r
 122 Z6FHt0sQaIQudrec8gq1hm4=\r
 123 =XFeA\r
 124 -----END PGP SIGNATURE-----\r
 125 --=-=-=--\r