[vlc-devel] [PATCH] Fix QToolButtonExt edge cases

[Jean-Baptiste Kempf]

On Sat, Dec 17, 2011 at 09:37:33PM +0000, Cheng Sun wrote :
> Firstly, once the user presses down on a QToolButtonExt there is no
> way to cancel the action; either a short or a long click will be
> generated. Compare this to a normal button, which can be cancelled by
> releasing the mouse outside of the button area.

Agreed.

> Secondly, with the mouse button held down, moving the mouse in and out
> of the button area will generate multiple short click events, when no
> event at all is desired.

Agreed too.

> -    bool longClick;
> +    bool releasedIsDown;
> +    bool ignoreNextClick;

Very badly chosen variables, if I could say.

Best regards,

-- 
Jean-Baptiste Kempf
http://www.jbkempf.com/ - +33 672 704 734
Sent from my Electronic Device