Skip to content
Permalink
Browse files

Sortable: Added a variable to track if initialization is complete. Fi…

…xes #4759 - a major optimization is possible in sortable().

(cherry picked from commit ba6916f)
  • Loading branch information
IanSimpson authored and scottgonzalez committed Jan 20, 2012
1 parent 9e4455f commit b00faa95d0d372f345e24f9abe9d16a2b67ca258
Showing with 5 additions and 1 deletion.
  1. +5 −1 ui/jquery.ui.sortable.js
@@ -16,6 +16,7 @@

$.widget("ui.sortable", $.ui.mouse, {
widgetEventPrefix: "sort",
ready: false,
options: {
appendTo: "parent",
axis: false,
@@ -57,6 +58,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--){

0 comments on commit b00faa9

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