Re: user32: If the listbox loses focus while holding capture, release it by essentially simulating a button up event.

[Nikolay Sivov]

On 2/20/2014 15:35, Huw Davies wrote:
> ---
>   dlls/user32/listbox.c       |  1 +
>   dlls/user32/tests/listbox.c | 30 ++++++++++++++++++++++++++++++
>   2 files changed, 31 insertions(+)
>
> case WM_KILLFOCUS:
> +        LISTBOX_HandleLButtonUp( descr ); /* Release capture if we have it */
>           descr->in_focus = FALSE;
>           descr->wheel_remain = 0;
> This isn't totally correct. It's fine for when you press and hold left 
> button and then change focus with Tab for example, LBN_SELCHANGE does 
> indeed happen in this case. But when you just Tab away from a list with 
> no mouse commands you shouldn't send LBN_SELCHANGE.

user32: If the listbox loses focus while holding capture, release it by essentially simulating a button up event.

[Huw Davies]

On Thu, Feb 20, 2014 at 03:46:26PM +0400, Nikolay Sivov wrote:
> On 2/20/2014 15:35, Huw Davies wrote:
> > ---
> >  dlls/user32/listbox.c       |  1 +
> >  dlls/user32/tests/listbox.c | 30 ++++++++++++++++++++++++++++++
> >  2 files changed, 31 insertions(+)
> >
> > case WM_KILLFOCUS:
> > +        LISTBOX_HandleLButtonUp( descr ); /* Release capture if we have it */
> >          descr->in_focus = FALSE;
> >          descr->wheel_remain = 0;
> > This isn't totally correct. It's fine for when you press and hold
> > left button and then change focus with Tab for example,
> > LBN_SELCHANGE does indeed happen in this case. But when you just Tab
> > away from a list with no mouse commands you shouldn't send
> > LBN_SELCHANGE.

Right, LBN_SELCHANGE should only get sent if we release the capture.
Isn't that what this does?

Huw.

user32: If the listbox loses focus while holding capture, release it by essentially simulating a button up event.

[Nikolay Sivov]

On 2/20/2014 16:01, Huw Davies wrote:
> On Thu, Feb 20, 2014 at 03:46:26PM +0400, Nikolay Sivov wrote:
> > On 2/20/2014 15:35, Huw Davies wrote:
> > > ---
> > >   dlls/user32/listbox.c       |  1 +
> > >   dlls/user32/tests/listbox.c | 30 ++++++++++++++++++++++++++++++
> > >   2 files changed, 31 insertions(+)
> > >
> > > case WM_KILLFOCUS:
> > > +        LISTBOX_HandleLButtonUp( descr ); /* Release capture if we have it */
> > >           descr->in_focus = FALSE;
> > >           descr->wheel_remain = 0;
> > > This isn't totally correct. It's fine for when you press and hold
> > > left button and then change focus with Tab for example,
> > > LBN_SELCHANGE does indeed happen in this case. But when you just Tab
> > > away from a list with no mouse commands you shouldn't send
> > > LBN_SELCHANGE.
> > > Right, LBN_SELCHANGE should only get sent if we release the capture.
> > > Isn't that what this does?
> > > Yes, I tested it, looks fine. Sorry for a noise.
>
> Huw.