diff options
| author | Paul Bakaus <paul.bakaus@googlemail.com> | 2008-11-03 13:23:15 +0000 |
|---|---|---|
| committer | Paul Bakaus <paul.bakaus@googlemail.com> | 2008-11-03 13:23:15 +0000 |
| commit | 0e16fcf1e865a6ea553265e202ed54b6935c54f4 (patch) | |
| tree | 17e9b73761336cdd95045f1be4cc098c85753ee7 | |
| parent | 9be9c9f699611f18a229efca381ec223a8bd14ec (diff) | |
| download | jquery-ui-0e16fcf1e865a6ea553265e202ed54b6935c54f4.tar.gz jquery-ui-0e16fcf1e865a6ea553265e202ed54b6935c54f4.zip | |
sortable: all events of the inner nodes of the sortable item were unbound after the sort (this is related to remove() in jQuery, bugfix is not to use remove()), fixes #3488
| -rw-r--r-- | ui/ui.sortable.js | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/ui/ui.sortable.js b/ui/ui.sortable.js index af48588d5..f9777ddc5 100644 --- a/ui/ui.sortable.js +++ b/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); |