Re: [PATCH v2 0/5] Use invisibility to toggle display of all parts including multipart

Subject: Re: [PATCH v2 0/5] Use invisibility to toggle display of all parts including multipart

Date: Sat, 15 Dec 2012 00:52:08 -0500

To: Mark Walters, notmuch@notmuchmail.org

Cc:

From: Austin Clements


On Thu, 13 Dec 2012, Mark Walters <markwalters1009@gmail.com> wrote:
> This is version 2 of the patch series (version 1 at
> id:1354663662-20524-1-git-send-email-markwalters1009@gmail.com)
>
> Almost all of the changes and fixes are in response to Austin's
> review. I have changed things as he suggested.
>
> There are two small extra tiny patches in this version: I set the
> default value of notmuch-show-all-multipart/alternative-parts to nil.
>
> The second is to fix a bug reported by Jani: that fake-patch parts (as
> provided by notmuch-wash) did not get hidden when the whole message
> was hidden.  This was due to notmuch-wash-convert-inline-patch-to-part
> not passing a msg on down to notmuch-show-insert-bodypart. Previously
> this did not matter but with with this series msg is needed to
> construct the correct overlays.
>
> Best wishes
>
> Mark

This series LGTM other than the few little things I pointed out.

Thread: