Opened 14 years ago

Closed 14 years ago

Last modified 13 years ago

#3541 closed bug (fixed)

Events removed on sortable elements

Reported by: antin Owned by: paul
Priority: critical Milestone: 1.7
Component: ui.sortable Version: 1.6rc2
Keywords: sortable event lost removed Cc:
Blocked by: Blocking:


This bug is related to ticket #3023 which has been marked as closed. I have also attached the description to that ticket.

The issue is that once a sortable element has been dragged it removes all events that were bound via jquery for all children of the sorted element.

For example with the following html:


The issue only arises when the events are bound to any child of the items being sorted (in the example above if you use jquery to bind a click event to the li then the event will be preserved after sort, however if you bind a click event to the a tag then it will be lost). This bug was introduced into jquery ui when the createPlaceholder function was reworked.

The issue is in the createPlaceholder function (line 306 in my version):

el.innerHTML = self.currentItem[0].innerHTML;

Previous versions of the createPlaceholder function merely placed an empty div, whereas the new version copies all the nested elements across. The issue arises once the element is dragged, when the clear function is called and the placeholder is removed from the dom (line 733):


The issue is that the 'expando' attribute that jquery uses to tag elements gets copied across to the temporary placeholder element, and then when remove is called jquery finds that an event was bound for the same id and mistakenly deletes it from the real element.

Commenting out the offending line in createPlaceholder fixes the issue, alternatively when the innerHTML is copied all expando attributes could be stripped.

Change History (3)

comment:1 Changed 14 years ago by Scott González

Component: ui.coreui.sortable
Milestone: TBD1.6
Priority: majorcritical

comment:2 Changed 14 years ago by paul

The issue can only be reproduced in IE - it seems to copy all DOM attributes as raw html attributes when using innerHTML :/

comment:3 Changed 14 years ago by paul

Resolution: fixed
Status: newclosed

Fixed in r1002 by an ugly regex that removes the expando attribute.

Note: See TracTickets for help on using tickets.