-
Notifications
You must be signed in to change notification settings - Fork 439
Touch support is broken. #170
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
I put an explanation of my changes in the pull request. Feel free to turn some of those into comments in the code. |
@collinstocks, though I haven't used touch support yet, I just wanted to let you know I appreciate you producing a fix. this removes a barrier for me in adopting this plugin. |
I think some people are still having issues with iOS touch support. Unfortunately, I don't have an iOS device to test this on. I also didn't end up using this plugin, so I'm not continuing work on this fix, but if you'd like to take it from here, feel free to do so. |
It is still an issue on iPad 2 iOS 8. Using jquery-sortable.js v0.9.13 which should include the above fix. |
Does anyone tried this http://touchpunch.furf.com/ ? |
Here is a patch to fix it:
#169
The text was updated successfully, but these errors were encountered: