[PATCH] Fix default client template

2012-05-27 Thread Romain Francoise
list-clients/choose-client are broken in current svn because the new template is not syntactically correct, see following patch. While we're on the subject, is it a feature that choose-window now uses the same template as list-windows? The old choose-window included the title of the active pane wh

Re: [PATCH] Fix default client template

2012-05-27 Thread Thomas Adam
On 27 May 2012 13:49, Romain Francoise wrote: > While we're on the subject, is it a feature that choose-window now uses > the same template as list-windows? The old choose-window included the > title of the active pane which was a lot more useful than, say, the > layout. Yes. And hey, guess what

Re: [PATCH] Fix default client template

2012-05-27 Thread Romain Francoise
Thomas Adam writes: > Yes. And hey, guess what? You're now free to change it with -F. choose-window is a very important command, using an inferior default just to factorize some code is a disservice to our users. -- L

Re: [PATCH] Fix default client template

2012-05-27 Thread Nicholas Marriott
I don't mind having DEFAULT_CHOOSE_* and DEFAULT_LIST_* separately, I don't much like having the layout there, but it has to be in list-windows output. On Sun, May 27, 2012 at 04:08:01PM +0200, Romain Francoise wrote: > Thomas Adam writes: > > > Yes. And hey, guess what? You're now free to ch

Re: [PATCH] Fix default client template

2012-05-27 Thread Nicholas Marriott
Oh and I applied your fix for the client ones, thanks. On Sun, May 27, 2012 at 10:45:00PM +0100, Nicholas Marriott wrote: > I don't mind having DEFAULT_CHOOSE_* and DEFAULT_LIST_* separately, I > don't much like having the layout there, but it has to be in > list-windows output. > > > On Sun,

Re: [PATCH] Fix default client template

2012-05-27 Thread Thomas Adam
Hi, On Sun, May 27, 2012 at 10:45:00PM +0100, Nicholas Marriott wrote: > I don't mind having DEFAULT_CHOOSE_* and DEFAULT_LIST_* separately, I > don't much like having the layout there, but it has to be in > list-windows output. Well we've already set a precedent for adding information when assig