mbraak / jqTree

Tree widget for jQuery
https://mbraak.github.io/jqTree/
Apache License 2.0
1.02k stars 177 forks source link

onIsMoveHandle is broken on dev branch #28

Closed rosenfeld closed 12 years ago

rosenfeld commented 12 years ago

I've just noticed that I can't move my nodes anymore in the dev branch. Can you replicate this?

rosenfeld commented 12 years ago

Seems to be fixed if you use mousedown, mousemove and mouseup events without the 'mousewidget.' prefix.

mbraak commented 12 years ago

I will look into it.

rosenfeld commented 12 years ago

This worked for me: https://github.com/rosenfeld/jqTree/tree/fixed-drag-and-drop

mbraak commented 12 years ago

Ouch. It seems that the fix for issue 24 breaked drag and drop. This is now fixed in the dev branch.

The namespace of the mouse events was set incorrectly. The correct syntax is 'mousedown.mousewidget'.