link event firing on sortable with connect list

[rambat]

If you have 2 sortable lists populated with links inside the unordered list elements dropping the element onto a connected list fires the click event.

Dropping on the same list does not fire the event.

Tested with 1.7.2.

You can duplicate this error by injecting a link into a list item on the demo page ( http://jqueryui.com/demos/sortable/#connect-lists)

Then drag that item into the connected list.

[joern.zaefferer]

See also #4845.

[jessicah]

I just discovered this today as well, definitely a connectWith issue.

Haven't had it happen in IE8 yet, happens in Chrome and Firefox mostly consistently; occasionally it doesn't fire the link.

[awgy]

I ran into this issue as well. The click event isn't being prevented when the sorted item moves to a connected sortable. If the sorted item remains in its original list, the click event is properly prevented.

I tracked this down to the _preventClickEvent property used by ui.mouse. It's set to true in _mouseUp(), but it's set for the original container-- which is fine, unless the container changed (which is the case when the item moves to a connected sortable). I've pushed a patch to a GitHub fork that changes the click event prevention to be tied to the event target, not the container.

 See GitHub fork commit

[awgy]

Correction: Please use the following GitHub commit link:

 GitHub commit 15d1a0d

[scott.gonzalez]

Fixed in  e2a693b.