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

2012-10-19 Thread Ethan Glasser-Camp
Mark Walters writes: > Some messages are sent as multipart/alternative but the alternatives > contain different information. This allows the user to cycle which > part to view. By default this is bound to 'W'. > --- > > This version at least uses the notmuch escaping for message-id which > makes

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

2012-10-19 Thread Ethan Glasser-Camp
Mark Walters markwalters1...@gmail.com writes: Some messages are sent as multipart/alternative but the alternatives contain different information. This allows the user to cycle which part to view. By default this is bound to 'W'. --- This version at least uses the notmuch escaping for

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

2012-10-09 Thread Jeremy Nickurak
On Fri, Aug 10, 2012 at 1:10 AM, Mark Walters wrote: > > Some messages are sent as multipart/alternative but the alternatives > contain different information. This allows the user to cycle which > part to view. By default this is bound to 'W'. I've started using this, and quite like it. Only

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

2012-10-09 Thread Jeremy Nickurak
On Fri, Aug 10, 2012 at 1:10 AM, Mark Walters markwalters1...@gmail.com wrote: Some messages are sent as multipart/alternative but the alternatives contain different information. This allows the user to cycle which part to view. By default this is bound to 'W'. I've started using this, and

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

2012-08-10 Thread Jani Nikula
On Aug 10, 2012 7:18 PM, "Jameson Graef Rollins" wrote: > > On Fri, Aug 10 2012, Jani Nikula wrote: > > How would this work together with something like [1] (rationale in [2])? > > > > [1] id:" ab777cf0fa83778d3399ac52094df9230738819d.1328798471.git.jani at nikula.org" > > [2]

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

2012-08-10 Thread Mark Walters
On Fri, 10 Aug 2012, Jani Nikula wrote: > On Aug 10, 2012 7:18 PM, "Jameson Graef Rollins" finestructure.net> > wrote: >> >> On Fri, Aug 10 2012, Jani Nikula wrote: >> > How would this work together with something like [1] (rationale in [2])? >> > >> > [1] id:" >

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

2012-08-10 Thread Jani Nikula
On Fri, 10 Aug 2012, Mark Walters wrote: > Some messages are sent as multipart/alternative but the alternatives > contain different information. This allows the user to cycle which > part to view. By default this is bound to 'W'. > --- > > On Thu, 09 Aug 2012, Jameson Graef Rollins > wrote: >>

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

2012-08-10 Thread Jameson Graef Rollins
On Fri, Aug 10 2012, Jani Nikula wrote: > How would this work together with something like [1] (rationale in [2])? > > [1] id:"ab777cf0fa83778d3399ac52094df9230738819d.1328798471.git.jani at > nikula.org" > [2] id:"cover.1328719309.git.jani at nikula.org" > > If you introduce a mechanism to

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

2012-08-10 Thread Mark Walters
Some messages are sent as multipart/alternative but the alternatives contain different information. This allows the user to cycle which part to view. By default this is bound to 'W'. --- On Thu, 09 Aug 2012, Jameson Graef Rollins wrote: > On Mon, Jun 18 2012, Mark Walters wrote: >> I agree

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

2012-08-10 Thread Mark Walters
Some messages are sent as multipart/alternative but the alternatives contain different information. This allows the user to cycle which part to view. By default this is bound to 'W'. --- On Thu, 09 Aug 2012, Jameson Graef Rollins jroll...@finestructure.net wrote: On Mon, Jun 18 2012, Mark

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

2012-08-10 Thread Jani Nikula
On Fri, 10 Aug 2012, Mark Walters markwalters1...@gmail.com wrote: Some messages are sent as multipart/alternative but the alternatives contain different information. This allows the user to cycle which part to view. By default this is bound to 'W'. --- On Thu, 09 Aug 2012, Jameson Graef

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

2012-08-10 Thread Jameson Graef Rollins
On Fri, Aug 10 2012, Jani Nikula j...@nikula.org wrote: How would this work together with something like [1] (rationale in [2])? [1] id:ab777cf0fa83778d3399ac52094df9230738819d.1328798471.git.j...@nikula.org [2] id:cover.1328719309.git.j...@nikula.org If you introduce a mechanism to store

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

2012-08-10 Thread Jani Nikula
On Aug 10, 2012 7:18 PM, Jameson Graef Rollins jroll...@finestructure.net wrote: On Fri, Aug 10 2012, Jani Nikula j...@nikula.org wrote: How would this work together with something like [1] (rationale in [2])? [1] id: ab777cf0fa83778d3399ac52094df9230738819d.1328798471.git.j...@nikula.org

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

2012-08-10 Thread Mark Walters
On Fri, 10 Aug 2012, Jani Nikula j...@nikula.org wrote: On Aug 10, 2012 7:18 PM, Jameson Graef Rollins jroll...@finestructure.net wrote: On Fri, Aug 10 2012, Jani Nikula j...@nikula.org wrote: How would this work together with something like [1] (rationale in [2])? [1] id:

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

2012-08-10 Thread Jameson Graef Rollins
On Fri, Aug 10 2012, Mark Walters markwalters1...@gmail.com wrote: I am not sure I agree: I think maybe toggling parts is better. Either the parts contain the same information and then the current behaviour is probably fine, or they are not in which case we might want to see both at once If

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

2012-06-24 Thread Mark Walters
On Tue, 19 Jun 2012, Jani Nikula wrote: > Add function notmuch-show-toggle-multipart-alternative to toggle the > value of notmuch-show-all-multipart/alternative-parts variable in the > buffer, and redisplay the thread with either all or preferred > alternative parts expanded. > > A small wrinkle

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

2012-06-24 Thread Jani Nikula
On Jun 23, 2012 1:34 AM, David Bremner da...@tethera.net wrote: Jani Nikula j...@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

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

2012-06-24 Thread Mark Walters
On Tue, 19 Jun 2012, Jani Nikula j...@nikula.org wrote: Add function notmuch-show-toggle-multipart-alternative to toggle the value of notmuch-show-all-multipart/alternative-parts variable in the buffer, and redisplay the thread with either all or preferred alternative parts expanded. A

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

2012-06-22 Thread David Bremner
Jani Nikula 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 >

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

2012-06-22 Thread David Bremner
Jani Nikula j...@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

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

2012-06-20 Thread Jani Nikula
On Tue, 19 Jun 2012, Jani Nikula wrote: > Add function notmuch-show-toggle-multipart-alternative to toggle the > value of notmuch-show-all-multipart/alternative-parts variable in the > buffer, and redisplay the thread with either all or preferred > alternative parts expanded. I see that Mark has

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

2012-06-20 Thread Jani Nikula
Add function notmuch-show-toggle-multipart-alternative to toggle the value of notmuch-show-all-multipart/alternative-parts variable in the buffer, and redisplay the thread with either all or preferred alternative parts expanded. A small wrinkle is that in order to make the variable survive

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

2012-06-19 Thread Jani Nikula
On Tue, 19 Jun 2012, Jani Nikula j...@nikula.org wrote: Add function notmuch-show-toggle-multipart-alternative to toggle the value of notmuch-show-all-multipart/alternative-parts variable in the buffer, and redisplay the thread with either all or preferred alternative parts expanded. I see