Skip to content
Permalink
Browse files

sortable: all events of the inner nodes of the sortable item were unb…

…ound after the sort (this is related to remove() in jQuery, bugfix is not to use remove()), fixes #3488
  • Loading branch information
Paul Bakaus
Paul Bakaus committed Nov 3, 2008
1 parent 9be9c9f commit 0e16fcf1e865a6ea553265e202ed54b6935c54f4
Showing with 4 additions and 2 deletions.
  1. +4 −2 ui/ui.sortable.js
@@ -790,8 +790,10 @@ $.widget("ui.sortable", $.extend({}, $.ui.mouse, {
}

this._propagate("beforeStop", e, null, noPropagation);

this.placeholder.remove();

//$(this.placeholder[0]).remove(); would have been the jQuery way - unfortunately, it unbinds ALL events from the original node!
this.placeholder[0].parentNode.removeChild(this.placeholder[0]);

if(this.options.helper != "original") this.helper.remove(); this.helper = null;
this._propagate("stop", e, null, noPropagation);

0 comments on commit 0e16fcf

Please sign in to comment.
You can’t perform that action at this time.