Thanks! I've checked in the patch.

I also uploaded an updated exe for the Windows users with the recent fixes,
http://leozide.com/~leo/leocad.zip if anyone needs it.

On Mon, Feb 9, 2015 at 3:15 PM, Sylvain L. Sauvage <
[email protected]> wrote:

> Le lundi 9 février 2015, 14:11:02 Leonardo Zide a écrit :
> >   The function that was causing problems because it was
> > deprecated is QWheelEvent::orientation(), I was checking if
> > the orientation is vertical and it didn't return that
> > correctly after 5.2.
>
>   Oh yes, I missed the deleted lines in r.1765!
> (Darn, svn diff is very really slow!
>  I’m too used to git’s speed ;oP)
>
>   I understand it better now: orientation() is an ORed value of
> Horizontal and Vertical. With the new angleDelta() function
> returning both H and V deltas, orientation() now always returns
> Horizontal|Vertical (not a value you might expect with actual
> wheels).
>
> >   I'll do some tests on Qt5 when I have a chance but if your
> > patch works I'll add something like that to the code (the
> > accumulator can't be static because there are multiple
> > widgets).
>
>   Of course. static’s really not good OO coding.  I used it to
> quickly test the idea.  Not that adding a member would have been
> a lot less quick...  Indeed, attached: a little bit better patch
> (just providing it to not let the crappy one be my last word:
> that’s not rocket science and just rewriting it in your own
> style will take you more time than writing it from scratch :o).
>
> Thanks,
> --
>  Sylvain Sauvage
> _______________________________________________
> Leocad mailing list
> [email protected]
> https://list.gerf.org/listinfo/leocad
>
>
_______________________________________________
Leocad mailing list
[email protected]
https://list.gerf.org/listinfo/leocad

Reply via email to