-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Hover feature is disabled unwantedly #328
Comments
Same Problem Here! Even :active doesn't work :( |
I tried the above in all modern versions of Firefox, Chrome, and Safari on Mac and iPhone and it seems to work as expected. What's your testing environment? |
My smartphone runs on Android 8. And the browser is Chrome 103.0.0.0. |
Remember that this is a very old version of touch punch which is never updated. Please use: And see if that fixes the issue |
Thanks. However, I just tested it and the issue remains. |
Interesting - yes, I can confirm on my android phone, the button does not display. However, it is NOT an issue with touch-punch per se - I think that to hover on a touch device, you are supposed to hold your finger down on the element for a couple of seconds. However, the problem is that jquery.ui sortable takes this as meaning you want to sort the element. Remark out the line $('ul').sortable(); in your codepen and the delete button appears when you hold your finger down on the elements. I had a similar application which I seem to recall I ended up having two buttons - a delete and a move button to overcome this. |
Here is a working demo.
In the demo, a delete
button
is supposed to appear when its parent element (li
, drag handle) is hovered/touched. Now the delete buttons can no longer appear because the hover feature of the drag handles are disabled by the plugin.The text was updated successfully, but these errors were encountered: