Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Cleaning up "draggable"

See original GitHub issue

Hi Bernardo, nice work!

I have one issue: If you go to: http://sslweb.azurewebsites.net/

click the bottom left icon (sort)
drag and drop a list-item to manually sort the list
click again the bottom left icon to disable sorting

Result on phones (tested on android and windows phone)

still possible to drag an item
scrolling on the list is broken

I guess there is something wrong with the event cleanup. In the list you can find items like <div class="win-itembox" draggable="true">. The draggable attribute will be removed by clicking the ‘disable sorting’ icon.

How can I access the ‘DragDropTouch’ object on the ‘.win-itembox’ element to do some cleanup?

Best Regards, George

Issue Analytics

State:
Created 7 years ago
Comments:5 (3 by maintainers)

Top GitHub Comments

1reaction

Bernardo-Castilhocommented, Jul 18, 2016

Hi George

Glad you like the library.

I inspected your sample app on my Google Nexus and I think I see what the problem is. The DragDropTouch code is looking for the presence of the “draggable” attribute, not to its value. This is a bug. I thought draggable worked like “disabled” or “required”, but it does not. The attribute can be set to true or to false.

I will review the code and apply a fix ASAP.

Thanks a lot for reporting this issue.

In the meantime, if you don’t want to wait, perhaps you could try removing the draggable attribute from the elements instead of setting it to false. That should fix the problem. Hopefully…

0reactions

Bernardo-Castilhocommented, Jul 18, 2016

Thank you for helping me improve the library.