Re: [PATCH V2 1/1] Add expand/collapse functionality in choose-tree

2012-10-02 Thread Thomas Adam
Hi, On 2 October 2012 08:54, Nicholas Marriott wrote: > I'd like explicit expand/collapse as well so added them back, also with OK. I just assumed they'd be redundant if we had a toggle, but sure, they can go back in. :) > some trivial renaming of a couple of things I don't like the look of a

Re: [PATCH V2 1/1] Add expand/collapse functionality in choose-tree

2012-10-02 Thread Nicholas Marriott
Applied now. On Tue, Oct 02, 2012 at 09:13:22AM +0100, Thomas Adam wrote: > Hi, > > On 2 October 2012 08:54, Nicholas Marriott > wrote: > > I'd like explicit expand/collapse as well so added them back, also with > > OK. I just assumed they'd be redundant if we had a toggle, but sure, > they

Re: [PATCH V2 1/1] Add expand/collapse functionality in choose-tree

2012-10-02 Thread Nicholas Marriott
I'd like explicit expand/collapse as well so added them back, also with some trivial renaming of a couple of things I don't like the look of and a nuclear reformat of the mode-key.c tables. Look ok? Index: mode-key.c === RCS file: /c

Re: [PATCH V2 1/1] Add expand/collapse functionality in choose-tree

2012-10-01 Thread Romain Francoise
Thomas Adam writes: > I've still left the bindings for expand_all and collapse_all unchanged -- > if they need to change, feel free to do so. I suggest using '+' and '-'. Mnemonic and easy to reach. -- Got visibility?

[PATCH V2 1/1] Add expand/collapse functionality in choose-tree

2012-10-01 Thread Thomas Adam
This patch adds the ability to expand/collapse a single session using the spacebar. Likewise, C-Left and C-Right will collapse and expand all sessions, respectively. --- This is the second version of the original patch, which takes Romain's suggestion. Now, spacebar will expand/collapse a singl