From: SpoonNZ Date: Fri, 20 Jan 2012 03:54:47 +0000 (+1300) Subject: Sortable: Added a variable to track if initialization is complete. Fixes #4759 -... X-Git-Tag: 1.9m7~99 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=ba6916f22ac3fac993975abc0f86d6cb0bf9c08d;p=jquery-ui.git Sortable: Added a variable to track if initialization is complete. Fixes #4759 - a major optimization is possible in sortable(). --- diff --git a/ui/jquery.ui.sortable.js b/ui/jquery.ui.sortable.js index 62d227a3d..600569451 100644 --- a/ui/jquery.ui.sortable.js +++ b/ui/jquery.ui.sortable.js @@ -17,6 +17,7 @@ $.widget("ui.sortable", $.ui.mouse, { version: "@VERSION", widgetEventPrefix: "sort", + ready: false, options: { appendTo: "parent", axis: false, @@ -58,6 +59,9 @@ $.widget("ui.sortable", $.ui.mouse, { //Initialize mouse events for interaction this._mouseInit(); + + //We're ready to go + this.ready = true }, @@ -571,7 +575,7 @@ $.widget("ui.sortable", $.ui.mouse, { var queries = [[$.isFunction(this.options.items) ? this.options.items.call(this.element[0], event, { item: this.currentItem }) : $(this.options.items, this.element), this]]; var connectWith = this._connectWith(); - if(connectWith) { + if(connectWith && this.ready) { //Shouldn't be run the first time through due to massive slow-down for (var i = connectWith.length - 1; i >= 0; i--){ var cur = $(connectWith[i]); for (var j = cur.length - 1; j >= 0; j--){