Re: comctl32: restore focus if it was lost during the processing of PROPSHEET_SetWizButtons

[Alexandre Julliard]

Aric Stewart <aric(a)codeweavers.com> writes:
> commit 87f9053ec43e0272e27280e1e7ab9674fc79b109
> Author: Aric Stewart <aric(a)codeweavers.com>
> Date:   Fri Jun 18 13:21:56 2010 +0900
>
> comctl32: restore focus if it was lost during the processing of PROPSHEET_SetWizButtons

It would probably be better to not disable buttons that don't need
it. What is this fixing?

-- 
Alexandre Julliard
julliard(a)winehq.org

comctl32: restore focus if it was lost during the processing of PROPSHEET_SetWizButtons

[Aric Stewart]

Alexandre Julliard wrote:
> Aric Stewart <aric(a)codeweavers.com> writes:
> > commit 87f9053ec43e0272e27280e1e7ab9674fc79b109
> > Author: Aric Stewart <aric(a)codeweavers.com>
> > Date:   Fri Jun 18 13:21:56 2010 +0900
> >
> > comctl32: restore focus if it was lost during the processing of PROPSHEET_SetWizButtons
>
> It would probably be better to not disable buttons that don't need
> it. What is this fixing?

I can modify the patch to do that.

This is triggering a bug on the mac that causes the window manager to 
loose track of where to send messages causing the resulting dialog to be 
unable to get mouse events.

-aric

comctl32: restore focus if it was lost during the processing of PROPSHEET_SetWizButtons

[Alexandre Julliard]

Aric Stewart <aric(a)codeweavers.com> writes:
> This is triggering a bug on the mac that causes the window manager to
> loose track of where to send messages causing the resulting dialog to
> be unable to get mouse events.

I find that surprising, the button focus should not be visible to the
window manager.

-- 
Alexandre Julliard
julliard(a)winehq.org