[PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-26 Thread David Edmondson
On Sat, 24 Apr 2010 05:42:54 -0700, Carl Worth wrote: > On Fri, 23 Apr 2010 21:08:48 +0100, David Edmondson wrote: > > I like the current behaviour, but changing the default would be fine. > > Which parts of it do you like? Being able to toggle the header back and > forth? Or just that the

[PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-26 Thread Dirk Hohndel
On Mon, 26 Apr 2010 11:55:33 +0100, David Edmondson wrote: > On Sat, 24 Apr 2010 05:42:54 -0700, Carl Worth wrote: > > On Fri, 23 Apr 2010 21:08:48 +0100, David Edmondson wrote: > > > I like the current behaviour, but changing the default would be fine. > > > > Which parts of it do you like?

Re: [PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-26 Thread David Edmondson
On Sat, 24 Apr 2010 05:42:54 -0700, Carl Worth cwo...@cworth.org wrote: On Fri, 23 Apr 2010 21:08:48 +0100, David Edmondson d...@dme.org wrote: I like the current behaviour, but changing the default would be fine. Which parts of it do you like? Being able to toggle the header back and

Re: [PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-26 Thread Dirk Hohndel
On Mon, 26 Apr 2010 11:55:33 +0100, David Edmondson d...@dme.org wrote: On Sat, 24 Apr 2010 05:42:54 -0700, Carl Worth cwo...@cworth.org wrote: On Fri, 23 Apr 2010 21:08:48 +0100, David Edmondson d...@dme.org wrote: I like the current behaviour, but changing the default would be fine.

[PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-24 Thread Carl Worth
On Sat, 24 Apr 2010 05:42:54 -0700, Carl Worth wrote: > I think I'll accept the patch, change the default, then remove From from > the list of headers inserted. I've done this now. I've also exported the list of headers to display as another nice customizable option in "M-x customize-group"

[PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-24 Thread Carl Worth
On Fri, 23 Apr 2010 21:08:48 +0100, David Edmondson wrote: > I like the current behaviour, but changing the default would be fine. Which parts of it do you like? Being able to toggle the header back and forth? Or just that the hidden headers take up so little vertical space? I think that I

Re: [PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-24 Thread Carl Worth
On Fri, 23 Apr 2010 21:08:48 +0100, David Edmondson d...@dme.org wrote: I like the current behaviour, but changing the default would be fine. Which parts of it do you like? Being able to toggle the header back and forth? Or just that the hidden headers take up so little vertical space? I think

Re: [PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-24 Thread Carl Worth
On Sat, 24 Apr 2010 05:42:54 -0700, Carl Worth cwo...@cworth.org wrote: I think I'll accept the patch, change the default, then remove From from the list of headers inserted. I've done this now. I've also exported the list of headers to display as another nice customizable option in M-x

[PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread David Edmondson
I like the current behaviour, but changing the default would be fine. On Friday, April 23, 2010, Carl Worth wrote: > On Fri, 23 Apr 2010 12:54:21 +0100, David Edmondson wrote: >> Add `notmuch-show-headers-visible' which, when set `t', causes headers >> to be shown by default. > > A nice

[PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread Jameson Rollins
On Fri, 23 Apr 2010 12:26:43 -0700, Carl Worth wrote: > A nice improvement, definitely. But I don't hear anyone actually wanting > a configuration value here. > > Would anyone complain if I just made these all visible by default? > > Would anyone complain if I removed the code to allow for th

[PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread Servilio Afre Puentes
On 23 April 2010 13:13, Dirk Hohndel wrote: > On Fri, 23 Apr 2010 12:54:21 +0100, David Edmondson wrote: >> Add `notmuch-show-headers-visible' which, when set `t', causes headers >> to be shown by default. > > Excellent - Carl, can you pull this into 0.3, please? +1 Servilio

[PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread David Edmondson
Add `notmuch-show-headers-visible' which, when set `t', causes headers to be shown by default. --- emacs/notmuch-show.el |9 +++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index cd859f0..3ea07c8 100644 ---

[PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread Carl Worth
On Fri, 23 Apr 2010 12:54:21 +0100, David Edmondson wrote: > Add `notmuch-show-headers-visible' which, when set `t', causes headers > to be shown by default. A nice improvement, definitely. But I don't hear anyone actually wanting a configuration value here. Would anyone complain if I just made

[PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread Dirk Hohndel
On Fri, 23 Apr 2010 12:54:21 +0100, David Edmondson wrote: > Add `notmuch-show-headers-visible' which, when set `t', causes headers > to be shown by default. Excellent - Carl, can you pull this into 0.3, please? /D -- Dirk Hohndel Intel Open Source Technology Center

Re: [PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread Dirk Hohndel
On Fri, 23 Apr 2010 12:54:21 +0100, David Edmondson d...@dme.org wrote: Add `notmuch-show-headers-visible' which, when set `t', causes headers to be shown by default. Excellent - Carl, can you pull this into 0.3, please? /D -- Dirk Hohndel Intel Open Source Technology Center

Re: [PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread Jameson Rollins
On Fri, 23 Apr 2010 10:13:37 -0700, Dirk Hohndel hohn...@infradead.org wrote: On Fri, 23 Apr 2010 12:54:21 +0100, David Edmondson d...@dme.org wrote: Add `notmuch-show-headers-visible' which, when set `t', causes headers to be shown by default. Excellent - Carl, can you pull this into 0.3,

Re: [PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread Servilio Afre Puentes
On 23 April 2010 13:13, Dirk Hohndel hohn...@infradead.org wrote: On Fri, 23 Apr 2010 12:54:21 +0100, David Edmondson d...@dme.org wrote: Add `notmuch-show-headers-visible' which, when set `t', causes headers to be shown by default. Excellent - Carl, can you pull this into 0.3, please? +1

Re: [PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread Carl Worth
On Fri, 23 Apr 2010 12:54:21 +0100, David Edmondson d...@dme.org wrote: Add `notmuch-show-headers-visible' which, when set `t', causes headers to be shown by default. A nice improvement, definitely. But I don't hear anyone actually wanting a configuration value here. Would anyone complain if I

Re: [PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread Jameson Rollins
On Fri, 23 Apr 2010 12:26:43 -0700, Carl Worth cwo...@cworth.org wrote: A nice improvement, definitely. But I don't hear anyone actually wanting a configuration value here. Would anyone complain if I just made these all visible by default? Would anyone complain if I removed the code to

Re: [PATCH] emacs: Allow headers to be shown by default in show mode

2010-04-23 Thread David Edmondson
I like the current behaviour, but changing the default would be fine. On Friday, April 23, 2010, Carl Worth cwo...@cworth.org wrote: On Fri, 23 Apr 2010 12:54:21 +0100, David Edmondson d...@dme.org wrote: Add `notmuch-show-headers-visible' which, when set `t', causes headers to be shown by