a4/7838133bd2fd5b14a7e99d71457d339ab27b9a

   1 Return-Path: <jani@nikula.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 ABC2B431FAF\r
   6         for <notmuch@notmuchmail.org>; Sun,  2 Dec 2012 11:48:12 -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 3yFHoChwDjQa for <notmuch@notmuchmail.org>;\r
  16         Sun,  2 Dec 2012 11:48:08 -0800 (PST)\r
  17 Received: from mail-lb0-f181.google.com (mail-lb0-f181.google.com\r
  18         [209.85.217.181]) (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 69B0E431FAE\r
  21         for <notmuch@notmuchmail.org>; Sun,  2 Dec 2012 11:48:08 -0800 (PST)\r
  22 Received: by mail-lb0-f181.google.com with SMTP id ge1so1909606lbb.26\r
  23         for <notmuch@notmuchmail.org>; Sun, 02 Dec 2012 11:48:04 -0800 (PST)\r
  24 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
  25         d=google.com; s=20120113;\r
  26         h=from:to:subject:in-reply-to:references:user-agent:date:message-id\r
  27         :mime-version:content-type:x-gm-message-state;\r
  28         bh=0LAAsEJfAnSg9rUi36Z94yfasLyK1kL7zwFnCRy5Lg0=;\r
  29         b=Z5IxaYYtS2FLhgvVkYwkIyo/vH1ByGQmVZ1d1EGcpaTP8YxEflyEaaM+zeTrzI0vgj\r
  30         b6gXrnn0o5DKUvF72bVdGuzi1h479BY3HfC0Z4zHerWvEXAmujYWE6zgGxpxhtx8aiEo\r
  31         CMcjU0JR9bwP/h5McR1XCrnZNBFNo01tGMIIorP4YwnLyZwcGHgtQqgvqBIJIhMDT4um\r
  32         Xgy/xfcysIo/P5OLJLl3K5KcuCNPb0O+H3n4by5SI3hnnu3KZkWWTNnrMpIUhITLAqqM\r
  33         FaQYdzF5wHNrOhYC8GSjXDBt2ed5lXZEzHXzZ4uhZzoO7GmKWiNcXACZRk0WLBqzoMAD\r
  34         kuJQ==\r
  35 Received: by 10.112.10.98 with SMTP id h2mr3352804lbb.127.1354477684064;\r
  36         Sun, 02 Dec 2012 11:48:04 -0800 (PST)\r
  37 Received: from localhost (dsl-hkibrasgw4-fe51df00-27.dhcp.inet.fi.\r
  38         [80.223.81.27])\r
  39         by mx.google.com with ESMTPS id gr12sm4333716lab.3.2012.12.02.11.48.01\r
  40         (version=SSLv3 cipher=OTHER); Sun, 02 Dec 2012 11:48:02 -0800 (PST)\r
  41 From: Jani Nikula <jani@nikula.org>\r
  42 To: Mark Walters <markwalters1009@gmail.com>, notmuch@notmuchmail.org\r
  43 Subject: Re: [PATCH (draft) 0/2] Allow emacs to toggle display of all parts\r
  44         including multipart\r
  45 In-Reply-To: <1351152563-27277-1-git-send-email-markwalters1009@gmail.com>\r
  46 References: <1351152563-27277-1-git-send-email-markwalters1009@gmail.com>\r
  47 User-Agent: Notmuch/0.14+128~g6dd5220 (http://notmuchmail.org) Emacs/23.4.1\r
  48         (i686-pc-linux-gnu)\r
  49 Date: Sun, 02 Dec 2012 21:48:00 +0200\r
  50 Message-ID: <87mwxwme1b.fsf@nikula.org>\r
  51 MIME-Version: 1.0\r
  52 Content-Type: text/plain; charset=us-ascii\r
  53 X-Gm-Message-State:\r
  54  ALoCoQkSu8u2fpSNDzZiYstDveYIwgZ/jvRfNgFPqHf4r4lpuuBVb2GTbEBbgJDu1iLmNdt4UWO2\r
  55 X-BeenThere: notmuch@notmuchmail.org\r
  56 X-Mailman-Version: 2.1.13\r
  57 Precedence: list\r
  58 List-Id: "Use and development of the notmuch mail system."\r
  59         <notmuch.notmuchmail.org>\r
  60 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  61         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  62 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  63 List-Post: <mailto:notmuch@notmuchmail.org>\r
  64 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  65 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  66         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  67 X-List-Received-Date: Sun, 02 Dec 2012 19:48:12 -0000\r
  68 \r
  69 On Thu, 25 Oct 2012, Mark Walters <markwalters1009@gmail.com> wrote:\r
  70 > This patch series adds a function to toggle the display of any emacs\r
  71 > part in the show buffer. This is particularly useful for\r
  72 > multipart/alternative where the parts have different information.\r
  73 \r
  74 I didn't look at the code, but I tried it. And I like it. There are some\r
  75 annoyances and some things I'll still want to customize for myself, but\r
  76 I really like it.\r
  77 \r
  78 > The first patch binds this toggle to 't' on the part button. This\r
  79 > patch works by itself.\r
  80 \r
  81 I'd like to rebind RET to do the toggle by default. I don't think it's\r
  82 very useful to be able to save multipart/alternative parts by default.\r
  83 \r
  84 > The second patch makes it that "viewing" (ie v on the part-button) a\r
  85 > "not shown" part displays the part in the buffer. Note this is not a\r
  86 > toggle since having displayed the part in the buffer the user may wish\r
  87 > to view the part externally (eg if it is a text/html part).\r
  88 >\r
  89 > Caveats: \r
  90 >        the patches work by reloading the whole buffer: this may mean\r
  91 >        extra messages appear in the thread. This is not ideal but is\r
  92 >        the same as toggling indentation etc.\r
  93 \r
  94 It's all right.\r
  95 \r
  96 >        the reload saves state as normal but this means the view\r
  97 >        returns to the top of the current message. Since we know\r
  98 >        exactly where the user was (ie on the button) this should be\r
  99 >        restored.\r
 100 \r
 101 This is annoying, and should be fixed eventually (but this can come\r
 102 afterwards). One option would be to just take a line count from the\r
 103 beginning of the message to the button, and return point to the button\r
 104 after refresh. This should work even if new messages show up.\r
 105 \r
 106 >        for technical reasons the "view" part functionality in the\r
 107 >        second patch only works for view called explicitly: not if\r
 108 >        the user has customised the default action to be view and\r
 109 >        calls that. This is easy to fix but the most natural way\r
 110 >        would break existing customisation of this action.\r
 111 >\r
 112 >        reloading the buffer without keeping state does not currently\r
 113 >        reset the toggled parts. Again this is easy to fix but lets\r
 114 >        see if people like the general approach first.\r
 115 \r
 116 IMHO can be fixed later.\r
 117 \r
 118 One more wishlist item, which also doesn't need to be part of this\r
 119 series, or by you: I'd like to be able to specify *which*\r
 120 multipart/alternative parts should be displayed in addition to\r
 121 preferred. A list of (regexps matching) mime types to always open,\r
 122 whether they're preferred or not. I've previously sent patches to this\r
 123 effect, but they were never merged (I can repost if you want).\r
 124 \r
 125 BR,\r
 126 Jani.\r
 127 \r
 128 \r
 129 >\r
 130 > Best wishes\r
 131 >\r
 132 > Mark\r
 133 >\r
 134 >\r
 135 > Mark Walters (2):\r
 136 >   emacs: allow the user to toggle the visibility of\r
 137 >     multipart/alternative parts\r
 138 >   emacs: show: make "view part" show hidden parts\r
 139 >\r
 140 >  emacs/notmuch-show.el |   53 ++++++++++++++++++++++++++++++++++++++++--------\r
 141 >  1 files changed, 44 insertions(+), 9 deletions(-)\r
 142 >\r
 143 > -- \r
 144 > 1.7.9.1\r
 145 >\r
 146 > _______________________________________________\r
 147 > notmuch mailing list\r
 148 > notmuch@notmuchmail.org\r
 149 > http://notmuchmail.org/mailman/listinfo/notmuch\r