[Webkit-unassigned] [Bug 80425] [Qt] REGRESSION(r109575) Broke tap-to-zoom gesture

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Mar 7 05:22:16 PST 2012


https://bugs.webkit.org/show_bug.cgi?id=80425





--- Comment #4 from Dinu Jacob <dinu.jacob at Nokia.com>  2012-03-07 05:22:16 PST ---
(In reply to comment #3)
> (From update of attachment 130504 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=130504&action=review
> 
> > Tools/MiniBrowser/qt/MiniBrowserApplication.cpp:196
> > +        touchPoint.setStartScenePos(m_startScreenPos);
> 
> You're right, the double tap recognizer uses the start screen pos, but there's a typo in this line where the start_screen_ pos is set on the start_Scene_ pos property. Please fix before landing :)

That was intentional. As canvas translates the touch events we need to set the screen position as the scene position. I can add a comment there.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list