Re: [PATCH v5 1/4] emacs: show: modify insert-part-header to save the button text

Subject: Re: [PATCH v5 1/4] emacs: show: modify insert-part-header to save the button text

Date: Fri, 21 Dec 2012 10:10:49 -0400

To: Mark Walters, notmuch@notmuchmail.org

Cc:

From: David Bremner

Mark Walters <markwalters1009@gmail.com> writes:

> This just make notmuch-show-insert-part-header save the basic button
> text for parts as an attribute. This makes it simpler for the button
> action (added in a later patch) to reword the label as appropriate (eg
> append "(not shown)" or not as appropriate).

Pushed. At a quick try, it seems very nice. 

I had to rebase patch 2, please double check I didn't mangle something.

Also, definitely NEWS-worthy...

d

• Previous message (by thread): Re: [PATCH v5 1/4] emacs: show: modify insert-part-header to save the button text

Thread:

• Mark Walters—[PATCH v5 0/4] Use invisibility to toggle display of all parts including multipart [inbox, unread]
  • Mark Walters—[PATCH v5 1/4] emacs: show: modify insert-part-header to save the button text [inbox, notmuch::patch, notmuch::pushed, unread]
    • David Bremner—Re: [PATCH v5 1/4] emacs: show: modify insert-part-header to save the button text [inbox, unread]
  • Mark Walters—[PATCH v5 2/4] emacs: show: add overlays for each part [inbox, notmuch::patch, notmuch::pushed, unread]
  • Mark Walters—[PATCH v5 3/4] emacs: show: add invisibility button action [inbox, notmuch::patch, notmuch::pushed, unread]
  • Mark Walters—[PATCH v5 4/4] emacs: show: set default show-all-multipart/alternatives to nil [inbox, notmuch::patch, notmuch::pushed, unread]
  • Austin Clements—Re: [PATCH v5 0/4] Use invisibility to toggle display of all parts including multipart [inbox, notmuch::review, unread]