Can one of the admins verify this patch?

run ci

I'm sorry, I don't follow. What is CI?

Hey @TheOriginalJosh,

I've just ran (run ci command) our local CI (continues integration) to check if everything is running properly with your pull request. :)

@enchev Cool, if you have any problems or questions. Don't hesitate to ask.

Hey @TheOriginalJosh - thanx for the PR,
However, I have concerns about it.
The loop variable i is not used inside the loop. The idea of the method is to get the center of the gesture - which is median point of all the current pointers. That's why the original code sums the coordinates of all the pointers and the takes the average of them. The code in the PR will take into account only one of the pointers - so if you are panning with more than one finger it will not work correctly.

Actually - the getRawX() methods will return screen coordinates. That said, I agree that using the getRaw methods will be more stable, especially if you are changing the layout of the element that is being panned.

Closing the in favor of #2118 which includes the multi-touch scenario.

@vakrilov Np, when I tested it on my phone, which is a Nexus 5x with Android Marshmellow, I was able to use multi-touch without a problem. Changing the layout is where I had the problems, not so much problems but just occasional inconsistency with the delta coordinates that end up causing a jumpiness​ that i wasn't getting on ios.

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.