Opened 10 years ago
Closed 10 years ago
#9539 closed bug (invalid)
1.11.0 (git) -- Recent ui.mouse change on git breaks ui.draggable on iPad
Reported by: | johnnyshields | Owned by: | johnnyshields |
---|---|---|---|
Priority: | minor | Milestone: | none |
Component: | ui.mouse | Version: | git (not yet released) |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
This commit:
in ui.mouse.js line 117:
Iframe mouseup check - mouseup occurred in another document else if ( !event.which ) {
return this._mouseUp( event );
}
These added lines break ui.draggable on iPad (I'm using touch punch). Previously, dragging worked as expected on iPad. After adding these lines, when I touch a draggable element, draggable appears to start but then immediately stop.
Removing the lines listed above fixes the issue.
Change History (7)
comment:2 Changed 10 years ago by
Owner: | set to johnnyshields |
---|---|
Status: | new → pending |
To be clear here, you're not seeing an issue with jQuery UI itself, just that the changes made in git have broke touch punch - correct?
comment:3 Changed 10 years ago by
Status: | pending → new |
---|
I have not observed an issue on non-mobile platforms, but have not tested extensively. Regardless, touch-punch is widely used and this should be fixed before release. It's likely that these lines will impact similar mobile frameworks.
comment:4 Changed 10 years ago by
Status: | new → pending |
---|
I'm not seeing this issue on iOS6 - http://jsfiddle.net/tj_vantoll/38UFh/.
Can you please alter my test case to show the issue you're experiencing?
comment:5 Changed 10 years ago by
Status: | pending → new |
---|
Huh I played around with it but couldn't reproduce on JS Fiddle. Bizarre...
I'll have to take deeper look at what's going on in my app, will need awhile for this.
comment:6 Changed 10 years ago by
Status: | new → pending |
---|
Ok, if you can get a make reproducible test case we can investigate.
comment:7 Changed 10 years ago by
Resolution: | → invalid |
---|---|
Status: | pending → closed |
Because we get so many tickets, we often need to return them to the initial reporter for more information. If that person does not reply within 14 days, the ticket will automatically be closed, and that has happened in this case. If you still are interested in pursuing this issue, feel free to add a comment with the requested information and we will be happy to reopen the ticket if it is still valid. Thanks!
Code above is formatted incorrectly. The affected lines are: