Re: find-window behavior on multiple matches in the same window

2012-04-01 Thread Nicholas Marriott
Applied, thanks. On Wed, Mar 28, 2012 at 10:37:23AM -0700, Jonathan Daugherty wrote: > > I reckon each window should only appear in the list once regardless > > with the first match, so this is a bug. > > In that case, see attached for CVS diff. > > -- > Jonathan Daugherty > Software Engine

Re: [PATCH] Allow using mouse to scroll down in copy mode

2012-04-01 Thread Nicholas Marriott
Hi If this isn't the default already it's a bug, copy mode should be exited when you scroll off the bottom of the history entirely not off the bottom of the current visible screen. So no need for the option... On Sun, Apr 01, 2012 at 02:50:20AM -0400, James Nylen wrote: >The attached patch

Re: [PATCH] Allow using mouse to scroll down in copy mode

2012-04-01 Thread James Nylen
I think there's a bug then. The attached patch fixes the issue. On Sun, Apr 1, 2012 at 4:29 AM, Nicholas Marriott < nicholas.marri...@gmail.com> wrote: > Hi > > If this isn't the default already it's a bug, copy mode should be exited > when you scroll off the bottom of the history entirely not o

Re: [PATCH] Allow using mouse to scroll down in copy mode

2012-04-01 Thread James Nylen
Updated patch with a now-unused variable removed. On Sun, Apr 1, 2012 at 5:06 AM, James Nylen wrote: > I think there's a bug then. The attached patch fixes the issue. > > > On Sun, Apr 1, 2012 at 4:29 AM, Nicholas Marriott < > nicholas.marri...@gmail.com> wrote: > >> Hi >> >> If this isn't the

Re: [PATCH] Allow using mouse to scroll down in copy mode

2012-04-01 Thread Nicholas Marriott
Applied, thanks. On Sun, Apr 01, 2012 at 05:10:10AM -0400, James Nylen wrote: >Updated patch with a now-unused variable removed. > >On Sun, Apr 1, 2012 at 5:06 AM, James Nylen <[1]jny...@gmail.com> wrote: > > I think there's a bug then.* The attached patch fixes the issue. > >