Re: [PATCH v2] emacs: add function to toggle display of all multipart/alternative parts

Subject: Re: [PATCH v2] emacs: add function to toggle display of all multipart/alternative parts

Date: Fri, 22 Jun 2012 19:34:06 -0300

To: Jani Nikula, notmuch@notmuchmail.org

Cc:

From: David Bremner

Jani Nikula <jani@nikula.org> writes:

>
> A small wrinkle is that in order to make the variable survive
> notmuch-show-refresh-view (which is required for expanding/collapsing
> the parts, but ends up calling kill-all-local-variables through
> notmuch-show-mode) it is necessary to give it the permanent-local
> property.

The code looks simple enough; should we apply this patch while we wait
for something fancier? I don't really know how to evaluate the
permanent-local bit.

d

• Previous message (by thread): Re: [PATCH 1/2] test: add emacs show mode test for toggling display of multipart/alternative

Thread:

• Jani Nikula—[PATCH] emacs: add function to toggle display of all multipart/alternative parts [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Jameson Graef Rollins—Re: [PATCH] emacs: add function to toggle display of all multipart/alternative parts [inbox, signed, unread]
    • Mark Walters—Re: [PATCH] emacs: add function to toggle display of all multipart/alternative parts [inbox, unread]
      • Jameson Graef Rollins—Re: [PATCH] emacs: add function to toggle display of all multipart/alternative parts [inbox, signed, unread]
        • Mark Walters—[PATCH v2] emacs: add function to toggle display of all multipart/alternative parts [inbox, notmuch::obsolete, notmuch::patch, unread]
          • Jani Nikula—Re: [PATCH v2] emacs: add function to toggle display of all multipart/alternative parts [inbox, unread]
            • Jameson Graef Rollins—Re: [PATCH v2] emacs: add function to toggle display of all multipart/alternative parts [inbox, signed, unread]
              • Jani Nikula—Re: [PATCH v2] emacs: add function to toggle display of all multipart/alternative parts [inbox, unread]
                • Mark Walters—Re: [PATCH v2] emacs: add function to toggle display of all multipart/alternative parts [inbox, unread]
                  • Jameson Graef Rollins—Re: [PATCH v2] emacs: add function to toggle display of all multipart/alternative parts [inbox, signed, unread]
          • Jameson Graef Rollins—[PATCH 1/2] test: add emacs show mode test for toggling display of multipart/alternative [inbox, notmuch::patch, notmuch::stale, notmuch::test, unread]
            • Jameson Graef Rollins—[PATCH 2/2] test: move all emacs show tests to emacs-show test script [inbox, notmuch::patch, notmuch::stale, unread]
              • David Bremner—Re: [PATCH 2/2] test: move all emacs show tests to emacs-show test script [inbox, unread]
            • Ethan Glasser-Camp—Re: [PATCH 1/2] test: add emacs show mode test for toggling display of multipart/alternative [inbox, unread]
          • Jeremy Nickurak—Re: [PATCH v2] emacs: add function to toggle display of all multipart/alternative parts [inbox, unread]
          • Ethan Glasser-Camp—Re: [PATCH v2] emacs: add function to toggle display of all multipart/alternative parts [inbox, unread]
    • Jani Nikula—Re: [PATCH] emacs: add function to toggle display of all multipart/alternative parts [inbox, unread]
  • Jani Nikula—[PATCH v2] emacs: add function to toggle display of all multipart/alternative parts [inbox, notmuch::patch, notmuch::wip, unread]
    • Jani Nikula—Re: [PATCH v2] emacs: add function to toggle display of all multipart/alternative parts [inbox, unread]
    • David Bremner—Re: [PATCH v2] emacs: add function to toggle display of all multipart/alternative parts [inbox, unread]
      • Jani Nikula—Re: [PATCH v2] emacs: add function to toggle display of all multipart/alternative parts [inbox, unread]
    • Mark Walters—Re: [PATCH v2] emacs: add function to toggle display of all multipart/alternative parts [inbox, notmuch::review, unread]
      • Ethan Glasser-Camp—[PATCH] Add NEWS item for multipart/alternative toggle [inbox, notmuch::patch, notmuch::wip, unread]