]> source.dussan.org Git - jquery-ui.git/commitdiff
Dev: Make sortable pass JSHint with only the loopfunc setting override
authorMike Sherov <mike.sherov@gmail.com>
Sat, 17 Nov 2012 03:51:24 +0000 (22:51 -0500)
committerMike Sherov <mike.sherov@gmail.com>
Sat, 17 Nov 2012 19:24:44 +0000 (14:24 -0500)
ui/jquery.ui.sortable.js

index a98e46632102f981acb46c6a4b7d017a3d380bbd..b1fb642fee4a9f5d7e5c9df091575bb0d164e678 100644 (file)
@@ -15,7 +15,8 @@
  */
 (function( $, undefined ) {
 
-/*jshint onevar: false, curly: false, eqeqeq: false, laxbreak: true, shadow: true, loopfunc: true */
+/*jshint loopfunc: true */
+
 $.widget("ui.sortable", $.ui.mouse, {
        version: "@VERSION",
        widgetEventPrefix: "sort",
@@ -72,8 +73,9 @@ $.widget("ui.sortable", $.ui.mouse, {
                        .removeClass("ui-sortable ui-sortable-disabled");
                this._mouseDestroy();
 
-               for ( var i = this.items.length - 1; i >= 0; i-- )
+               for ( var i = this.items.length - 1; i >= 0; i-- ) {
                        this.items[i].item.removeData(this.widgetName + "-item");
+               }
 
                return this;
        },
@@ -90,34 +92,44 @@ $.widget("ui.sortable", $.ui.mouse, {
        },
 
        _mouseCapture: function(event, overrideHandle) {
-               var that = this;
+               var currentItem = null,
+                       validHandle = false,
+                       that = this;
 
                if (this.reverting) {
                        return false;
                }
 
-               if(this.options.disabled || this.options.type == 'static') return false;
+               if(this.options.disabled || this.options.type === 'static') {
+                       return false;
+               }
 
                //We have to refresh the items data once first
                this._refreshItems(event);
 
                //Find out if the clicked node (or one of its parents) is a actual item in this.items
-               var currentItem = null;
-
                $(event.target).parents().each(function() {
-                       if($.data(this, that.widgetName + '-item') == that) {
+                       if($.data(this, that.widgetName + '-item') === that) {
                                currentItem = $(this);
                                return false;
                        }
                });
-               if($.data(event.target, that.widgetName + '-item') == that) currentItem = $(event.target);
+               if($.data(event.target, that.widgetName + '-item') === that) {
+                       currentItem = $(event.target);
+               }
 
-               if(!currentItem) return false;
+               if(!currentItem) {
+                       return false;
+               }
                if(this.options.handle && !overrideHandle) {
-                       var validHandle = false;
-
-                       $(this.options.handle, currentItem).find("*").andSelf().each(function() { if(this == event.target) validHandle = true; });
-                       if(!validHandle) return false;
+                       $(this.options.handle, currentItem).find("*").andSelf().each(function() {
+                               if(this === event.target) {
+                                       validHandle = true;
+                               }
+                       });
+                       if(!validHandle) {
+                               return false;
+                       }
                }
 
                this.currentItem = currentItem;
@@ -128,7 +140,9 @@ $.widget("ui.sortable", $.ui.mouse, {
 
        _mouseStart: function(event, overrideHandle, noActivation) {
 
-               var o = this.options;
+               var i,
+                       o = this.options;
+
                this.currentContainer = this;
 
                //We only need to call refreshPositions, because the refreshItems call has been moved to mouseCapture
@@ -184,7 +198,7 @@ $.widget("ui.sortable", $.ui.mouse, {
                this.domPosition = { prev: this.currentItem.prev()[0], parent: this.currentItem.parent()[0] };
 
                //If the helper is not the original, hide the original so it's not playing any role during the drag, won't cause anything bad this way
-               if(this.helper[0] != this.currentItem[0]) {
+               if(this.helper[0] !== this.currentItem[0]) {
                        this.currentItem.hide();
                }
 
@@ -192,47 +206,60 @@ $.widget("ui.sortable", $.ui.mouse, {
                this._createPlaceholder();
 
                //Set a containment if given in the options
-               if(o.containment)
+               if(o.containment) {
                        this._setContainment();
+               }
 
                if(o.cursor) { // cursor option
-                       if ($('body').css("cursor")) this._storedCursor = $('body').css("cursor");
+                       if ($('body').css("cursor")) {
+                               this._storedCursor = $('body').css("cursor");
+                       }
                        $('body').css("cursor", o.cursor);
                }
 
                if(o.opacity) { // opacity option
-                       if (this.helper.css("opacity")) this._storedOpacity = this.helper.css("opacity");
+                       if (this.helper.css("opacity")) {
+                               this._storedOpacity = this.helper.css("opacity");
+                       }
                        this.helper.css("opacity", o.opacity);
                }
 
                if(o.zIndex) { // zIndex option
-                       if (this.helper.css("zIndex")) this._storedZIndex = this.helper.css("zIndex");
+                       if (this.helper.css("zIndex")) {
+                               this._storedZIndex = this.helper.css("zIndex");
+                       }
                        this.helper.css("zIndex", o.zIndex);
                }
 
                //Prepare scrolling
-               if(this.scrollParent[0] != document && this.scrollParent[0].tagName != 'HTML')
+               if(this.scrollParent[0] !== document && this.scrollParent[0].tagName !== 'HTML') {
                        this.overflowOffset = this.scrollParent.offset();
+               }
 
                //Call callbacks
                this._trigger("start", event, this._uiHash());
 
                //Recache the helper size
-               if(!this._preserveHelperProportions)
+               if(!this._preserveHelperProportions) {
                        this._cacheHelperProportions();
+               }
 
 
                //Post 'activate' events to possible containers
-               if(!noActivation) {
-                       for (var i = this.containers.length - 1; i >= 0; i--) { this.containers[i]._trigger("activate", event, this._uiHash(this)); }
+               if( !noActivation ) {
+                       for ( i = this.containers.length - 1; i >= 0; i-- ) {
+                               this.containers[ i ]._trigger( "activate", event, this._uiHash( this ) );
+                       }
                }
 
                //Prepare possible droppables
-               if($.ui.ddmanager)
+               if($.ui.ddmanager) {
                        $.ui.ddmanager.current = this;
+               }
 
-               if ($.ui.ddmanager && !o.dropBehaviour)
+               if ($.ui.ddmanager && !o.dropBehaviour) {
                        $.ui.ddmanager.prepareOffsets(this, event);
+               }
 
                this.dragging = true;
 
@@ -243,6 +270,9 @@ $.widget("ui.sortable", $.ui.mouse, {
        },
 
        _mouseDrag: function(event) {
+               var i, item, itemElement, intersection,
+                       o = this.options,
+                       scrolled = false;
 
                //Compute the helpers position
                this.position = this._generatePosition(event);
@@ -254,50 +284,62 @@ $.widget("ui.sortable", $.ui.mouse, {
 
                //Do scrolling
                if(this.options.scroll) {
-                       var o = this.options, scrolled = false;
-                       if(this.scrollParent[0] != document && this.scrollParent[0].tagName != 'HTML') {
+                       if(this.scrollParent[0] !== document && this.scrollParent[0].tagName !== 'HTML') {
 
-                               if((this.overflowOffset.top + this.scrollParent[0].offsetHeight) - event.pageY < o.scrollSensitivity)
+                               if((this.overflowOffset.top + this.scrollParent[0].offsetHeight) - event.pageY < o.scrollSensitivity) {
                                        this.scrollParent[0].scrollTop = scrolled = this.scrollParent[0].scrollTop + o.scrollSpeed;
-                               else if(event.pageY - this.overflowOffset.top < o.scrollSensitivity)
+                               } else if(event.pageY - this.overflowOffset.top < o.scrollSensitivity) {
                                        this.scrollParent[0].scrollTop = scrolled = this.scrollParent[0].scrollTop - o.scrollSpeed;
+                               }
 
-                               if((this.overflowOffset.left + this.scrollParent[0].offsetWidth) - event.pageX < o.scrollSensitivity)
+                               if((this.overflowOffset.left + this.scrollParent[0].offsetWidth) - event.pageX < o.scrollSensitivity) {
                                        this.scrollParent[0].scrollLeft = scrolled = this.scrollParent[0].scrollLeft + o.scrollSpeed;
-                               else if(event.pageX - this.overflowOffset.left < o.scrollSensitivity)
+                               } else if(event.pageX - this.overflowOffset.left < o.scrollSensitivity) {
                                        this.scrollParent[0].scrollLeft = scrolled = this.scrollParent[0].scrollLeft - o.scrollSpeed;
+                               }
 
                        } else {
 
-                               if(event.pageY - $(document).scrollTop() < o.scrollSensitivity)
+                               if(event.pageY - $(document).scrollTop() < o.scrollSensitivity) {
                                        scrolled = $(document).scrollTop($(document).scrollTop() - o.scrollSpeed);
-                               else if($(window).height() - (event.pageY - $(document).scrollTop()) < o.scrollSensitivity)
+                               } else if($(window).height() - (event.pageY - $(document).scrollTop()) < o.scrollSensitivity) {
                                        scrolled = $(document).scrollTop($(document).scrollTop() + o.scrollSpeed);
+                               }
 
-                               if(event.pageX - $(document).scrollLeft() < o.scrollSensitivity)
+                               if(event.pageX - $(document).scrollLeft() < o.scrollSensitivity) {
                                        scrolled = $(document).scrollLeft($(document).scrollLeft() - o.scrollSpeed);
-                               else if($(window).width() - (event.pageX - $(document).scrollLeft()) < o.scrollSensitivity)
+                               } else if($(window).width() - (event.pageX - $(document).scrollLeft()) < o.scrollSensitivity) {
                                        scrolled = $(document).scrollLeft($(document).scrollLeft() + o.scrollSpeed);
+                               }
 
                        }
 
-                       if(scrolled !== false && $.ui.ddmanager && !o.dropBehaviour)
+                       if(scrolled !== false && $.ui.ddmanager && !o.dropBehaviour) {
                                $.ui.ddmanager.prepareOffsets(this, event);
+                       }
                }
 
                //Regenerate the absolute position used for position checks
                this.positionAbs = this._convertPositionTo("absolute");
 
                //Set the helper position
-               if(!this.options.axis || this.options.axis != "y") this.helper[0].style.left = this.position.left+'px';
-               if(!this.options.axis || this.options.axis != "x") this.helper[0].style.top = this.position.top+'px';
+               if(!this.options.axis || this.options.axis !== "y") {
+                       this.helper[0].style.left = this.position.left+'px';
+               }
+               if(!this.options.axis || this.options.axis !== "x") {
+                       this.helper[0].style.top = this.position.top+'px';
+               }
 
                //Rearrange
-               for (var i = this.items.length - 1; i >= 0; i--) {
+               for (i = this.items.length - 1; i >= 0; i--) {
 
                        //Cache variables and intersection, continue if no intersection
-                       var item = this.items[i], itemElement = item.item[0], intersection = this._intersectsWithPointer(item);
-                       if (!intersection) continue;
+                       item = this.items[i];
+                       itemElement = item.item[0];
+                       intersection = this._intersectsWithPointer(item);
+                       if (!intersection) {
+                               continue;
+                       }
 
                        // Only put the placeholder inside the current Container, skip all
                        // items form other containers. This works because when moving
@@ -306,18 +348,22 @@ $.widget("ui.sortable", $.ui.mouse, {
                        //
                        // Without this moving items in "sub-sortables" can cause the placeholder to jitter
                        // beetween the outer and inner container.
-                       if (item.instance !== this.currentContainer) continue;
+                       if (item.instance !== this.currentContainer) {
+                               continue;
+                       }
 
-                       if (itemElement != this.currentItem[0] //cannot intersect with itself
-                               &&      this.placeholder[intersection == 1 ? "next" : "prev"]()[0] != itemElement //no useless actions that have been done before
-                               &&      !$.contains(this.placeholder[0], itemElement) //no action if the item moved is the parent of the item checked
-                               && (this.options.type == 'semi-dynamic' ? !$.contains(this.element[0], itemElement) : true)
-                               //&& itemElement.parentNode == this.placeholder[0].parentNode // only rearrange items within the same container
+                       // cannot intersect with itself
+                       // no useless actions that have been done before
+                       // no action if the item moved is the parent of the item checked
+                       if (itemElement !== this.currentItem[0] &&
+                               this.placeholder[intersection === 1 ? "next" : "prev"]()[0] !== itemElement &&
+                               !$.contains(this.placeholder[0], itemElement) &&
+                               (this.options.type === 'semi-dynamic' ? !$.contains(this.element[0], itemElement) : true)
                        ) {
 
-                               this.direction = intersection == 1 ? "down" : "up";
+                               this.direction = intersection === 1 ? "down" : "up";
 
-                               if (this.options.tolerance == "pointer" || this._intersectsWithSides(item)) {
+                               if (this.options.tolerance === "pointer" || this._intersectsWithSides(item)) {
                                        this._rearrange(event, item);
                                } else {
                                        break;
@@ -332,7 +378,9 @@ $.widget("ui.sortable", $.ui.mouse, {
                this._contactContainers(event);
 
                //Interconnect with droppables
-               if($.ui.ddmanager) $.ui.ddmanager.drag(this, event);
+               if($.ui.ddmanager) {
+                       $.ui.ddmanager.drag(this, event);
+               }
 
                //Call callbacks
                this._trigger('sort', event, this._uiHash());
@@ -344,21 +392,24 @@ $.widget("ui.sortable", $.ui.mouse, {
 
        _mouseStop: function(event, noPropagation) {
 
-               if(!event) return;
+               if(!event) {
+                       return;
+               }
 
                //If we are using droppables, inform the manager about the drop
-               if ($.ui.ddmanager && !this.options.dropBehaviour)
+               if ($.ui.ddmanager && !this.options.dropBehaviour) {
                        $.ui.ddmanager.drop(this, event);
+               }
 
                if(this.options.revert) {
-                       var that = this;
-                       var cur = this.placeholder.offset();
+                       var that = this,
+                               cur = this.placeholder.offset();
 
                        this.reverting = true;
 
                        $(this.helper).animate({
-                               left: cur.left - this.offset.parent.left - this.margins.left + (this.offsetParent[0] == document.body ? 0 : this.offsetParent[0].scrollLeft),
-                               top: cur.top - this.offset.parent.top - this.margins.top + (this.offsetParent[0] == document.body ? 0 : this.offsetParent[0].scrollTop)
+                               left: cur.left - this.offset.parent.left - this.margins.left + (this.offsetParent[0] === document.body ? 0 : this.offsetParent[0].scrollLeft),
+                               top: cur.top - this.offset.parent.top - this.margins.top + (this.offsetParent[0] === document.body ? 0 : this.offsetParent[0].scrollTop)
                        }, parseInt(this.options.revert, 10) || 500, function() {
                                that._clear(event);
                        });
@@ -376,10 +427,11 @@ $.widget("ui.sortable", $.ui.mouse, {
 
                        this._mouseUp({ target: null });
 
-                       if(this.options.helper == "original")
+                       if(this.options.helper === "original") {
                                this.currentItem.css(this._storedCSS).removeClass("ui-sortable-helper");
-                       else
+                       } else {
                                this.currentItem.show();
+                       }
 
                        //Post deactivating events to containers
                        for (var i = this.containers.length - 1; i >= 0; i--){
@@ -394,8 +446,12 @@ $.widget("ui.sortable", $.ui.mouse, {
 
                if (this.placeholder) {
                        //$(this.placeholder[0]).remove(); would have been the jQuery way - unfortunately, it unbinds ALL events from the original node!
-                       if(this.placeholder[0].parentNode) this.placeholder[0].parentNode.removeChild(this.placeholder[0]);
-                       if(this.options.helper != "original" && this.helper && this.helper[0].parentNode) this.helper.remove();
+                       if(this.placeholder[0].parentNode) {
+                               this.placeholder[0].parentNode.removeChild(this.placeholder[0]);
+                       }
+                       if(this.options.helper !== "original" && this.helper && this.helper[0].parentNode) {
+                               this.helper.remove();
+                       }
 
                        $.extend(this, {
                                helper: null,
@@ -417,12 +473,15 @@ $.widget("ui.sortable", $.ui.mouse, {
 
        serialize: function(o) {
 
-               var items = this._getItemsAsjQuery(o && o.connected);
-               var str = []; o = o || {};
+               var items = this._getItemsAsjQuery(o && o.connected),
+                       str = [];
+               o = o || {};
 
                $(items).each(function() {
                        var res = ($(o.item || this).attr(o.attribute || 'id') || '').match(o.expression || (/(.+)[\-=_](.+)/));
-                       if(res) str.push((o.key || res[1]+'[]')+'='+(o.key && o.expression ? res[1] : res[2]));
+                       if (res) {
+                               str.push((o.key || res[1]+'[]')+'='+(o.key && o.expression ? res[1] : res[2]));
+                       }
                });
 
                if(!str.length && o.key) {
@@ -435,8 +494,10 @@ $.widget("ui.sortable", $.ui.mouse, {
 
        toArray: function(o) {
 
-               var items = this._getItemsAsjQuery(o && o.connected);
-               var ret = []; o = o || {};
+               var items = this._getItemsAsjQuery(o && o.connected),
+                       ret = [];
+
+               o = o || {};
 
                items.each(function() { ret.push($(o.item || this).attr(o.attribute || 'id') || ''); });
                return ret;
@@ -449,29 +510,26 @@ $.widget("ui.sortable", $.ui.mouse, {
                var x1 = this.positionAbs.left,
                        x2 = x1 + this.helperProportions.width,
                        y1 = this.positionAbs.top,
-                       y2 = y1 + this.helperProportions.height;
-
-               var l = item.left,
+                       y2 = y1 + this.helperProportions.height,
+                       l = item.left,
                        r = l + item.width,
                        t = item.top,
-                       b = t + item.height;
-
-               var dyClick = this.offset.click.top,
-                       dxClick = this.offset.click.left;
-
-               var isOverElement = (y1 + dyClick) > t && (y1 + dyClick) < b && (x1 + dxClick) > l && (x1 + dxClick) < r;
-
-               if (this.options.tolerance == "pointer"
-                       || this.options.forcePointerForContainers
-                       || (this.options.tolerance != "pointer" && this.helperProportions[this.floating ? 'width' : 'height'] > item[this.floating ? 'width' : 'height'])
+                       b = t + item.height,
+                       dyClick = this.offset.click.top,
+                       dxClick = this.offset.click.left,
+                       isOverElement = (y1 + dyClick) > t && (y1 + dyClick) < b && (x1 + dxClick) > l && (x1 + dxClick) < r;
+
+               if ( this.options.tolerance === "pointer" ||
+                       this.options.forcePointerForContainers ||
+                       (this.options.tolerance !== "pointer" && this.helperProportions[this.floating ? 'width' : 'height'] > item[this.floating ? 'width' : 'height'])
                ) {
                        return isOverElement;
                } else {
 
-                       return (l < x1 + (this.helperProportions.width / 2) // Right Half
-                               && x2 - (this.helperProportions.width / 2) < r // Left Half
-                               && t < y1 + (this.helperProportions.height / 2) // Bottom Half
-                               && y2 - (this.helperProportions.height / 2) < b ); // Top Half
+                       return (l < x1 + (this.helperProportions.width / 2) && // Right Half
+                               x2 - (this.helperProportions.width / 2) < r && // Left Half
+                               t < y1 + (this.helperProportions.height / 2) && // Bottom Half
+                               y2 - (this.helperProportions.height / 2) < b ); // Top Half
 
                }
        },
@@ -484,12 +542,13 @@ $.widget("ui.sortable", $.ui.mouse, {
                        verticalDirection = this._getDragVerticalDirection(),
                        horizontalDirection = this._getDragHorizontalDirection();
 
-               if (!isOverElement)
+               if (!isOverElement) {
                        return false;
+               }
 
                return this.floating ?
-                       ( ((horizontalDirection && horizontalDirection == "right") || verticalDirection == "down") ? 2 : 1 )
-                       : ( verticalDirection && (verticalDirection == "down" ? 2 : 1) );
+                       ( ((horizontalDirection && horizontalDirection === "right") || verticalDirection === "down") ? 2 : 1 )
+                       : ( verticalDirection && (verticalDirection === "down" ? 2 : 1) );
 
        },
 
@@ -501,9 +560,9 @@ $.widget("ui.sortable", $.ui.mouse, {
                        horizontalDirection = this._getDragHorizontalDirection();
 
                if (this.floating && horizontalDirection) {
-                       return ((horizontalDirection == "right" && isOverRightHalf) || (horizontalDirection == "left" && !isOverRightHalf));
+                       return ((horizontalDirection === "right" && isOverRightHalf) || (horizontalDirection === "left" && !isOverRightHalf));
                } else {
-                       return verticalDirection && ((verticalDirection == "down" && isOverBottomHalf) || (verticalDirection == "up" && !isOverBottomHalf));
+                       return verticalDirection && ((verticalDirection === "down" && isOverBottomHalf) || (verticalDirection === "up" && !isOverBottomHalf));
                }
 
        },
@@ -526,23 +585,22 @@ $.widget("ui.sortable", $.ui.mouse, {
 
        _connectWith: function() {
                var options = this.options;
-               return options.connectWith.constructor == String
-                       ? [options.connectWith]
-                       : options.connectWith;
+               return options.connectWith.constructor === String ? [options.connectWith] : options.connectWith;
        },
 
        _getItemsAsjQuery: function(connected) {
 
-               var items = [];
-               var queries = [];
-               var connectWith = this._connectWith();
+               var i, j, cur, inst,
+                       items = [],
+                       queries = [],
+                       connectWith = this._connectWith();
 
                if(connectWith && connected) {
-                       for (var i = connectWith.length - 1; i >= 0; i--){
-                               var cur = $(connectWith[i]);
-                               for (var j = cur.length - 1; j >= 0; j--){
-                                       var inst = $.data(cur[j], this.widgetFullName);
-                                       if(inst && inst != this && !inst.options.disabled) {
+                       for (i = connectWith.length - 1; i >= 0; i--){
+                               cur = $(connectWith[i]);
+                               for ( j = cur.length - 1; j >= 0; j--){
+                                       inst = $.data(cur[j], this.widgetFullName);
+                                       if(inst && inst !== this && !inst.options.disabled) {
                                                queries.push([$.isFunction(inst.options.items) ? inst.options.items.call(inst.element) : $(inst.options.items, inst.element).not(".ui-sortable-helper").not('.ui-sortable-placeholder'), inst]);
                                        }
                                }
@@ -551,7 +609,7 @@ $.widget("ui.sortable", $.ui.mouse, {
 
                queries.push([$.isFunction(this.options.items) ? this.options.items.call(this.element, null, { options: this.options, item: this.currentItem }) : $(this.options.items, this.element).not(".ui-sortable-helper").not('.ui-sortable-placeholder'), this]);
 
-               for (var i = queries.length - 1; i >= 0; i--){
+               for (i = queries.length - 1; i >= 0; i--){
                        queries[i][0].each(function() {
                                items.push(this);
                        });
@@ -567,8 +625,9 @@ $.widget("ui.sortable", $.ui.mouse, {
 
                this.items = $.grep(this.items, function (item) {
                        for (var j=0; j < list.length; j++) {
-                               if(list[j] == item.item[0])
+                               if(list[j] === item.item[0]) {
                                        return false;
+                               }
                        }
                        return true;
                });
@@ -579,16 +638,18 @@ $.widget("ui.sortable", $.ui.mouse, {
 
                this.items = [];
                this.containers = [this];
-               var items = this.items;
-               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();
+
+               var i, j, cur, inst, targetData, _queries, item, queriesLength,
+                       items = this.items,
+                       queries = [[$.isFunction(this.options.items) ? this.options.items.call(this.element[0], event, { item: this.currentItem }) : $(this.options.items, this.element), this]],
+                       connectWith = this._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--){
-                                       var inst = $.data(cur[j], this.widgetFullName);
-                                       if(inst && inst != this && !inst.options.disabled) {
+                       for (i = connectWith.length - 1; i >= 0; i--){
+                               cur = $(connectWith[i]);
+                               for (j = cur.length - 1; j >= 0; j--){
+                                       inst = $.data(cur[j], this.widgetFullName);
+                                       if(inst && inst !== this && !inst.options.disabled) {
                                                queries.push([$.isFunction(inst.options.items) ? inst.options.items.call(inst.element[0], event, { item: this.currentItem }) : $(inst.options.items, inst.element), inst]);
                                                this.containers.push(inst);
                                        }
@@ -596,12 +657,12 @@ $.widget("ui.sortable", $.ui.mouse, {
                        }
                }
 
-               for (var i = queries.length - 1; i >= 0; i--) {
-                       var targetData = queries[i][1];
-                       var _queries = queries[i][0];
+               for (i = queries.length - 1; i >= 0; i--) {
+                       targetData = queries[i][1];
+                       _queries = queries[i][0];
 
-                       for (var j=0, queriesLength = _queries.length; j < queriesLength; j++) {
-                               var item = $(_queries[j]);
+                       for (j=0, queriesLength = _queries.length; j < queriesLength; j++) {
+                               item = $(_queries[j]);
 
                                item.data(this.widgetName + '-item', targetData); // Data for target checking (mouse manager)
 
@@ -623,21 +684,24 @@ $.widget("ui.sortable", $.ui.mouse, {
                        this.offset.parent = this._getParentOffset();
                }
 
-               for (var i = this.items.length - 1; i >= 0; i--){
-                       var item = this.items[i];
+               var i, item, t, p;
+
+               for (i = this.items.length - 1; i >= 0; i--){
+                       item = this.items[i];
 
                        //We ignore calculating positions of all connected containers when we're not over them
-                       if(item.instance != this.currentContainer && this.currentContainer && item.item[0] != this.currentItem[0])
+                       if(item.instance !== this.currentContainer && this.currentContainer && item.item[0] !== this.currentItem[0]) {
                                continue;
+                       }
 
-                       var t = this.options.toleranceElement ? $(this.options.toleranceElement, item.item) : item.item;
+                       t = this.options.toleranceElement ? $(this.options.toleranceElement, item.item) : item.item;
 
                        if (!fast) {
                                item.width = t.outerWidth();
                                item.height = t.outerHeight();
                        }
 
-                       var p = t.offset();
+                       p = t.offset();
                        item.left = p.left;
                        item.top = p.top;
                }
@@ -645,8 +709,8 @@ $.widget("ui.sortable", $.ui.mouse, {
                if(this.options.custom && this.options.custom.refreshContainers) {
                        this.options.custom.refreshContainers.call(this);
                } else {
-                       for (var i = this.containers.length - 1; i >= 0; i--){
-                               var p = this.containers[i].element.offset();
+                       for (i = this.containers.length - 1; i >= 0; i--){
+                               p = this.containers[i].element.offset();
                                this.containers[i].containerCache.left = p.left;
                                this.containers[i].containerCache.top = p.top;
                                this.containers[i].containerCache.width = this.containers[i].element.outerWidth();
@@ -659,10 +723,11 @@ $.widget("ui.sortable", $.ui.mouse, {
 
        _createPlaceholder: function(that) {
                that = that || this;
-               var o = that.options;
+               var className,
+                       o = that.options;
 
-               if(!o.placeholder || o.placeholder.constructor == String) {
-                       var className = o.placeholder;
+               if(!o.placeholder || o.placeholder.constructor === String) {
+                       className = o.placeholder;
                        o.placeholder = {
                                element: function() {
 
@@ -670,8 +735,9 @@ $.widget("ui.sortable", $.ui.mouse, {
                                                .addClass(className || that.currentItem[0].className+" ui-sortable-placeholder")
                                                .removeClass("ui-sortable-helper")[0];
 
-                                       if(!className)
+                                       if(!className) {
                                                el.style.visibility = "hidden";
+                                       }
 
                                        return el;
                                },
@@ -679,7 +745,9 @@ $.widget("ui.sortable", $.ui.mouse, {
 
                                        // 1. If a className is set as 'placeholder option, we don't force sizes - the class is responsible for that
                                        // 2. The option 'forcePlaceholderSize can be enabled to force it even if a class name is specified
-                                       if(className && !o.forcePlaceholderSize) return;
+                                       if(className && !o.forcePlaceholderSize) {
+                                               return;
+                                       }
 
                                        //If the element doesn't have a actual height by itself (without styles coming from a stylesheet), it receives the inline height from the dragged item
                                        if(!p.height()) { p.height(that.currentItem.innerHeight() - parseInt(that.currentItem.css('paddingTop')||0, 10) - parseInt(that.currentItem.css('paddingBottom')||0, 10)); }
@@ -700,22 +768,24 @@ $.widget("ui.sortable", $.ui.mouse, {
        },
 
        _contactContainers: function(event) {
+               var i, j, dist, itemWithLeastDistance, posProperty, sizeProperty, base, cur, nearBottom,
+                       innermostContainer = null,
+                       innermostIndex = null;
 
                // get innermost container that intersects with item
-               var innermostContainer = null, innermostIndex = null;
-
-
-               for (var i = this.containers.length - 1; i >= 0; i--){
+               for (i = this.containers.length - 1; i >= 0; i--) {
 
                        // never consider a container that's located within the item itself
-                       if($.contains(this.currentItem[0], this.containers[i].element[0]))
+                       if($.contains(this.currentItem[0], this.containers[i].element[0])) {
                                continue;
+                       }
 
                        if(this._intersectsWith(this.containers[i].containerCache)) {
 
                                // if we've already found a container and it's more "inner" than this, then continue
-                               if(innermostContainer && $.contains(this.containers[i].element[0], innermostContainer.element[0]))
+                               if(innermostContainer && $.contains(this.containers[i].element[0], innermostContainer.element[0])) {
                                        continue;
+                               }
 
                                innermostContainer = this.containers[i];
                                innermostIndex = i;
@@ -731,7 +801,9 @@ $.widget("ui.sortable", $.ui.mouse, {
                }
 
                // if no intersecting containers found, return
-               if(!innermostContainer) return;
+               if(!innermostContainer) {
+                       return;
+               }
 
                // move the item into the container if it's not there already
                if(this.containers.length === 1) {
@@ -740,15 +812,20 @@ $.widget("ui.sortable", $.ui.mouse, {
                } else {
 
                        //When entering a new container, we will find the item with the least distance and append our item near it
-                       var dist = 10000; var itemWithLeastDistance = null;
-                       var posProperty = this.containers[innermostIndex].floating ? 'left' : 'top';
-                       var sizeProperty = this.containers[innermostIndex].floating ? 'width' : 'height';
-                       var base = this.positionAbs[posProperty] + this.offset.click[posProperty];
-                       for (var j = this.items.length - 1; j >= 0; j--) {
-                               if(!$.contains(this.containers[innermostIndex].element[0], this.items[j].item[0])) continue;
-                               if(this.items[j].item[0] == this.currentItem[0]) continue;
-                               var cur = this.items[j].item.offset()[posProperty];
-                               var nearBottom = false;
+                       dist = 10000;
+                       itemWithLeastDistance = null;
+                       posProperty = this.containers[innermostIndex].floating ? 'left' : 'top';
+                       sizeProperty = this.containers[innermostIndex].floating ? 'width' : 'height';
+                       base = this.positionAbs[posProperty] + this.offset.click[posProperty];
+                       for (j = this.items.length - 1; j >= 0; j--) {
+                               if(!$.contains(this.containers[innermostIndex].element[0], this.items[j].item[0])) {
+                                       continue;
+                               }
+                               if(this.items[j].item[0] === this.currentItem[0]) {
+                                       continue;
+                               }
+                               cur = this.items[j].item.offset()[posProperty];
+                               nearBottom = false;
                                if(Math.abs(cur - base) > Math.abs(cur + this.items[j][sizeProperty] - base)){
                                        nearBottom = true;
                                        cur += this.items[j][sizeProperty];
@@ -760,8 +837,10 @@ $.widget("ui.sortable", $.ui.mouse, {
                                }
                        }
 
-                       if(!itemWithLeastDistance && !this.options.dropOnEmpty) //Check if dropOnEmpty is enabled
+                       //Check if dropOnEmpty is enabled
+                       if(!itemWithLeastDistance && !this.options.dropOnEmpty) {
                                return;
+                       }
 
                        this.currentContainer = this.containers[innermostIndex];
                        itemWithLeastDistance ? this._rearrange(event, itemWithLeastDistance, null, true) : this._rearrange(event, null, this.containers[innermostIndex].element, true);
@@ -780,24 +859,31 @@ $.widget("ui.sortable", $.ui.mouse, {
 
        _createHelper: function(event) {
 
-               var o = this.options;
-               var helper = $.isFunction(o.helper) ? $(o.helper.apply(this.element[0], [event, this.currentItem])) : (o.helper == 'clone' ? this.currentItem.clone() : this.currentItem);
+               var o = this.options,
+                       helper = $.isFunction(o.helper) ? $(o.helper.apply(this.element[0], [event, this.currentItem])) : (o.helper === 'clone' ? this.currentItem.clone() : this.currentItem);
 
-               if(!helper.parents('body').length) //Add the helper to the DOM if that didn't happen already
-                       $(o.appendTo != 'parent' ? o.appendTo : this.currentItem[0].parentNode)[0].appendChild(helper[0]);
+               //Add the helper to the DOM if that didn't happen already
+               if(!helper.parents('body').length) {
+                       $(o.appendTo !== 'parent' ? o.appendTo : this.currentItem[0].parentNode)[0].appendChild(helper[0]);
+               }
 
-               if(helper[0] == this.currentItem[0])
+               if(helper[0] === this.currentItem[0]) {
                        this._storedCSS = { width: this.currentItem[0].style.width, height: this.currentItem[0].style.height, position: this.currentItem.css("position"), top: this.currentItem.css("top"), left: this.currentItem.css("left") };
+               }
 
-               if(!helper[0].style.width || o.forceHelperSize) helper.width(this.currentItem.width());
-               if(!helper[0].style.height || o.forceHelperSize) helper.height(this.currentItem.height());
+               if(!helper[0].style.width || o.forceHelperSize) {
+                       helper.width(this.currentItem.width());
+               }
+               if(!helper[0].style.height || o.forceHelperSize) {
+                       helper.height(this.currentItem.height());
+               }
 
                return helper;
 
        },
 
        _adjustOffsetFromHelper: function(obj) {
-               if (typeof obj == 'string') {
+               if (typeof obj === 'string') {
                        obj = obj.split(' ');
                }
                if ($.isArray(obj)) {
@@ -828,14 +914,16 @@ $.widget("ui.sortable", $.ui.mouse, {
                // 1. The position of the helper is absolute, so it's position is calculated based on the next positioned parent
                // 2. The actual offset parent is a child of the scroll parent, and the scroll parent isn't the document, which means that
                //    the scroll is included in the initial calculation of the offset of the parent, and never recalculated upon drag
-               if(this.cssPosition == 'absolute' && this.scrollParent[0] != document && $.contains(this.scrollParent[0], this.offsetParent[0])) {
+               if(this.cssPosition === 'absolute' && this.scrollParent[0] !== document && $.contains(this.scrollParent[0], this.offsetParent[0])) {
                        po.left += this.scrollParent.scrollLeft();
                        po.top += this.scrollParent.scrollTop();
                }
 
-               if((this.offsetParent[0] == document.body) //This needs to be actually done for all browsers, since pageX/pageY includes this information
-               || (this.offsetParent[0].tagName && this.offsetParent[0].tagName.toLowerCase() == 'html' && $.ui.ie)) //Ugly IE fix
+               // This needs to be actually done for all browsers, since pageX/pageY includes this information
+               // with an ugly IE fix
+               if( this.offsetParent[0] === document.body || (this.offsetParent[0].tagName && this.offsetParent[0].tagName.toLowerCase() === 'html' && $.ui.ie)) {
                        po = { top: 0, left: 0 };
+               }
 
                return {
                        top: po.top + (parseInt(this.offsetParent.css("borderTopWidth"),10) || 0),
@@ -846,7 +934,7 @@ $.widget("ui.sortable", $.ui.mouse, {
 
        _getRelativeOffset: function() {
 
-               if(this.cssPosition == "relative") {
+               if(this.cssPosition === "relative") {
                        var p = this.currentItem.position();
                        return {
                                top: p.top - (parseInt(this.helper.css("top"),10) || 0) + this.scrollParent.scrollTop(),
@@ -874,19 +962,24 @@ $.widget("ui.sortable", $.ui.mouse, {
 
        _setContainment: function() {
 
-               var o = this.options;
-               if(o.containment == 'parent') o.containment = this.helper[0].parentNode;
-               if(o.containment == 'document' || o.containment == 'window') this.containment = [
-                       0 - this.offset.relative.left - this.offset.parent.left,
-                       0 - this.offset.relative.top - this.offset.parent.top,
-                       $(o.containment == 'document' ? document : window).width() - this.helperProportions.width - this.margins.left,
-                       ($(o.containment == 'document' ? document : window).height() || document.body.parentNode.scrollHeight) - this.helperProportions.height - this.margins.top
-               ];
+               var ce, co, over,
+                       o = this.options;
+               if(o.containment === 'parent') {
+                       o.containment = this.helper[0].parentNode;
+               }
+               if(o.containment === 'document' || o.containment === 'window') {
+                       this.containment = [
+                               0 - this.offset.relative.left - this.offset.parent.left,
+                               0 - this.offset.relative.top - this.offset.parent.top,
+                               $(o.containment === 'document' ? document : window).width() - this.helperProportions.width - this.margins.left,
+                               ($(o.containment === 'document' ? document : window).height() || document.body.parentNode.scrollHeight) - this.helperProportions.height - this.margins.top
+                       ];
+               }
 
                if(!(/^(document|window|parent)$/).test(o.containment)) {
-                       var ce = $(o.containment)[0];
-                       var co = $(o.containment).offset();
-                       var over = ($(ce).css("overflow") != 'hidden');
+                       ce = $(o.containment)[0];
+                       co = $(o.containment).offset();
+                       over = ($(ce).css("overflow") !== 'hidden');
 
                        this.containment = [
                                co.left + (parseInt($(ce).css("borderLeftWidth"),10) || 0) + (parseInt($(ce).css("paddingLeft"),10) || 0) - this.margins.left,
@@ -900,22 +993,25 @@ $.widget("ui.sortable", $.ui.mouse, {
 
        _convertPositionTo: function(d, pos) {
 
-               if(!pos) pos = this.position;
-               var mod = d == "absolute" ? 1 : -1;
-               var scroll = this.cssPosition == 'absolute' && !(this.scrollParent[0] != document && $.contains(this.scrollParent[0], this.offsetParent[0])) ? this.offsetParent : this.scrollParent, scrollIsRootNode = (/(html|body)/i).test(scroll[0].tagName);
+               if(!pos) {
+                       pos = this.position;
+               }
+               var mod = d === "absolute" ? 1 : -1,
+                       scroll = this.cssPosition === 'absolute' && !(this.scrollParent[0] !== document && $.contains(this.scrollParent[0], this.offsetParent[0])) ? this.offsetParent : this.scrollParent,
+                       scrollIsRootNode = (/(html|body)/i).test(scroll[0].tagName);
 
                return {
                        top: (
-                               pos.top                                                                                                                                 // The absolute mouse position
-                               + this.offset.relative.top * mod                                                                                // Only for relative positioned nodes: Relative offset from element to offset parent
-                               + this.offset.parent.top * mod                                                                                  // The offsetParent's offset without borders (offset + border)
-                               - ( ( this.cssPosition == 'fixed' ? -this.scrollParent.scrollTop() : ( scrollIsRootNode ? 0 : scroll.scrollTop() ) ) * mod)
+                               pos.top +                                                                                                                               // The absolute mouse position
+                               this.offset.relative.top * mod +                                                                                // Only for relative positioned nodes: Relative offset from element to offset parent
+                               this.offset.parent.top * mod -                                                                                  // The offsetParent's offset without borders (offset + border)
+                               ( ( this.cssPosition === 'fixed' ? -this.scrollParent.scrollTop() : ( scrollIsRootNode ? 0 : scroll.scrollTop() ) ) * mod)
                        ),
                        left: (
-                               pos.left                                                                                                                                // The absolute mouse position
-                               + this.offset.relative.left * mod                                                                               // Only for relative positioned nodes: Relative offset from element to offset parent
-                               + this.offset.parent.left * mod                                                                                 // The offsetParent's offset without borders (offset + border)
-                               - ( ( this.cssPosition == 'fixed' ? -this.scrollParent.scrollLeft() : scrollIsRootNode ? 0 : scroll.scrollLeft() ) * mod)
+                               pos.left +                                                                                                                              // The absolute mouse position
+                               this.offset.relative.left * mod +                                                                               // Only for relative positioned nodes: Relative offset from element to offset parent
+                               this.offset.parent.left * mod   -                                                                               // The offsetParent's offset without borders (offset + border)
+                               ( ( this.cssPosition === 'fixed' ? -this.scrollParent.scrollLeft() : scrollIsRootNode ? 0 : scroll.scrollLeft() ) * mod)
                        )
                };
 
@@ -923,19 +1019,20 @@ $.widget("ui.sortable", $.ui.mouse, {
 
        _generatePosition: function(event) {
 
-               var o = this.options, scroll = this.cssPosition == 'absolute' && !(this.scrollParent[0] != document && $.contains(this.scrollParent[0], this.offsetParent[0])) ? this.offsetParent : this.scrollParent, scrollIsRootNode = (/(html|body)/i).test(scroll[0].tagName);
+               var top, left,
+                       o = this.options,
+                       pageX = event.pageX,
+                       pageY = event.pageY,
+                       scroll = this.cssPosition === 'absolute' && !(this.scrollParent[0] !== document && $.contains(this.scrollParent[0], this.offsetParent[0])) ? this.offsetParent : this.scrollParent, scrollIsRootNode = (/(html|body)/i).test(scroll[0].tagName);
 
                // This is another very weird special case that only happens for relative elements:
                // 1. If the css position is relative
                // 2. and the scroll parent is the document or similar to the offset parent
                // we have to refresh the relative offset during the scroll so there are no jumps
-               if(this.cssPosition == 'relative' && !(this.scrollParent[0] != document && this.scrollParent[0] != this.offsetParent[0])) {
+               if(this.cssPosition === 'relative' && !(this.scrollParent[0] !== document && this.scrollParent[0] !== this.offsetParent[0])) {
                        this.offset.relative = this._getRelativeOffset();
                }
 
-               var pageX = event.pageX;
-               var pageY = event.pageY;
-
                /*
                 * - Position constraining -
                 * Constrain the position to a mix of grid, containment.
@@ -944,17 +1041,25 @@ $.widget("ui.sortable", $.ui.mouse, {
                if(this.originalPosition) { //If we are not dragging yet, we won't check for options
 
                        if(this.containment) {
-                               if(event.pageX - this.offset.click.left < this.containment[0]) pageX = this.containment[0] + this.offset.click.left;
-                               if(event.pageY - this.offset.click.top < this.containment[1]) pageY = this.containment[1] + this.offset.click.top;
-                               if(event.pageX - this.offset.click.left > this.containment[2]) pageX = this.containment[2] + this.offset.click.left;
-                               if(event.pageY - this.offset.click.top > this.containment[3]) pageY = this.containment[3] + this.offset.click.top;
+                               if(event.pageX - this.offset.click.left < this.containment[0]) {
+                                       pageX = this.containment[0] + this.offset.click.left;
+                               }
+                               if(event.pageY - this.offset.click.top < this.containment[1]) {
+                                       pageY = this.containment[1] + this.offset.click.top;
+                               }
+                               if(event.pageX - this.offset.click.left > this.containment[2]) {
+                                       pageX = this.containment[2] + this.offset.click.left;
+                               }
+                               if(event.pageY - this.offset.click.top > this.containment[3]) {
+                                       pageY = this.containment[3] + this.offset.click.top;
+                               }
                        }
 
                        if(o.grid) {
-                               var top = this.originalPageY + Math.round((pageY - this.originalPageY) / o.grid[1]) * o.grid[1];
+                               top = this.originalPageY + Math.round((pageY - this.originalPageY) / o.grid[1]) * o.grid[1];
                                pageY = this.containment ? ( (top - this.offset.click.top >= this.containment[1] && top - this.offset.click.top <= this.containment[3]) ? top : ((top - this.offset.click.top >= this.containment[1]) ? top - o.grid[1] : top + o.grid[1])) : top;
 
-                               var left = this.originalPageX + Math.round((pageX - this.originalPageX) / o.grid[0]) * o.grid[0];
+                               left = this.originalPageX + Math.round((pageX - this.originalPageX) / o.grid[0]) * o.grid[0];
                                pageX = this.containment ? ( (left - this.offset.click.left >= this.containment[0] && left - this.offset.click.left <= this.containment[2]) ? left : ((left - this.offset.click.left >= this.containment[0]) ? left - o.grid[0] : left + o.grid[0])) : left;
                        }
 
@@ -962,18 +1067,18 @@ $.widget("ui.sortable", $.ui.mouse, {
 
                return {
                        top: (
-                               pageY                                                                                                                           // The absolute mouse position
-                               - this.offset.click.top                                                                                                 // Click offset (relative to the element)
-                               - this.offset.relative.top                                                                                              // Only for relative positioned nodes: Relative offset from element to offset parent
-                               - this.offset.parent.top                                                                                                // The offsetParent's offset without borders (offset + border)
-                               + ( ( this.cssPosition == 'fixed' ? -this.scrollParent.scrollTop() : ( scrollIsRootNode ? 0 : scroll.scrollTop() ) ))
+                               pageY -                                                                                                                         // The absolute mouse position
+                               this.offset.click.top -                                                                                                 // Click offset (relative to the element)
+                               this.offset.relative.top        -                                                                                       // Only for relative positioned nodes: Relative offset from element to offset parent
+                               this.offset.parent.top +                                                                                                // The offsetParent's offset without borders (offset + border)
+                               ( ( this.cssPosition === 'fixed' ? -this.scrollParent.scrollTop() : ( scrollIsRootNode ? 0 : scroll.scrollTop() ) ))
                        ),
                        left: (
-                               pageX                                                                                                                           // The absolute mouse position
-                               - this.offset.click.left                                                                                                // Click offset (relative to the element)
-                               - this.offset.relative.left                                                                                             // Only for relative positioned nodes: Relative offset from element to offset parent
-                               - this.offset.parent.left                                                                                               // The offsetParent's offset without borders (offset + border)
-                               + ( ( this.cssPosition == 'fixed' ? -this.scrollParent.scrollLeft() : scrollIsRootNode ? 0 : scroll.scrollLeft() ))
+                               pageX -                                                                                                                         // The absolute mouse position
+                               this.offset.click.left -                                                                                                // Click offset (relative to the element)
+                               this.offset.relative.left       -                                                                                       // Only for relative positioned nodes: Relative offset from element to offset parent
+                               this.offset.parent.left +                                                                                               // The offsetParent's offset without borders (offset + border)
+                               ( ( this.cssPosition === 'fixed' ? -this.scrollParent.scrollLeft() : scrollIsRootNode ? 0 : scroll.scrollLeft() ))
                        )
                };
 
@@ -981,7 +1086,7 @@ $.widget("ui.sortable", $.ui.mouse, {
 
        _rearrange: function(event, i, a, hardRefresh) {
 
-               a ? a[0].appendChild(this.placeholder[0]) : i.item[0].parentNode.insertBefore(this.placeholder[0], (this.direction == 'down' ? i.item[0] : i.item[0].nextSibling));
+               a ? a[0].appendChild(this.placeholder[0]) : i.item[0].parentNode.insertBefore(this.placeholder[0], (this.direction === 'down' ? i.item[0] : i.item[0].nextSibling));
 
                //Various things done here to improve the performance:
                // 1. we create a setTimeout, that calls refreshPositions
@@ -992,7 +1097,9 @@ $.widget("ui.sortable", $.ui.mouse, {
                var counter = this.counter;
 
                this._delay(function() {
-                       if(counter == this.counter) this.refreshPositions(!hardRefresh); //Precompute after each DOM insertion, NOT on mousemove
+                       if(counter === this.counter) {
+                               this.refreshPositions(!hardRefresh); //Precompute after each DOM insertion, NOT on mousemove
+                       }
                });
 
        },
@@ -1002,24 +1109,33 @@ $.widget("ui.sortable", $.ui.mouse, {
                this.reverting = false;
                // We delay all events that have to be triggered to after the point where the placeholder has been removed and
                // everything else normalized again
-               var delayedTriggers = [];
+               var i,
+                       delayedTriggers = [];
 
                // We first have to update the dom position of the actual currentItem
                // Note: don't do it if the current item is already removed (by a user), or it gets reappended (see #4088)
-               if(!this._noFinalSort && this.currentItem.parent().length) this.placeholder.before(this.currentItem);
+               if(!this._noFinalSort && this.currentItem.parent().length) {
+                       this.placeholder.before(this.currentItem);
+               }
                this._noFinalSort = null;
 
-               if(this.helper[0] == this.currentItem[0]) {
-                       for(var i in this._storedCSS) {
-                               if(this._storedCSS[i] == 'auto' || this._storedCSS[i] == 'static') this._storedCSS[i] = '';
+               if(this.helper[0] === this.currentItem[0]) {
+                       for(i in this._storedCSS) {
+                               if(this._storedCSS[i] === 'auto' || this._storedCSS[i] === 'static') {
+                                       this._storedCSS[i] = '';
+                               }
                        }
                        this.currentItem.css(this._storedCSS).removeClass("ui-sortable-helper");
                } else {
                        this.currentItem.show();
                }
 
-               if(this.fromOutside && !noPropagation) delayedTriggers.push(function(event) { this._trigger("receive", event, this._uiHash(this.fromOutside)); });
-               if((this.fromOutside || this.domPosition.prev != this.currentItem.prev().not(".ui-sortable-helper")[0] || this.domPosition.parent != this.currentItem.parent()[0]) && !noPropagation) delayedTriggers.push(function(event) { this._trigger("update", event, this._uiHash()); }); //Trigger update callback if the DOM position has changed
+               if(this.fromOutside && !noPropagation) {
+                       delayedTriggers.push(function(event) { this._trigger("receive", event, this._uiHash(this.fromOutside)); });
+               }
+               if((this.fromOutside || this.domPosition.prev !== this.currentItem.prev().not(".ui-sortable-helper")[0] || this.domPosition.parent !== this.currentItem.parent()[0]) && !noPropagation) {
+                       delayedTriggers.push(function(event) { this._trigger("update", event, this._uiHash()); }); //Trigger update callback if the DOM position has changed
+               }
 
                // Check if the items Container has Changed and trigger appropriate
                // events.
@@ -1033,8 +1149,10 @@ $.widget("ui.sortable", $.ui.mouse, {
 
 
                //Post events to containers
-               for (var i = this.containers.length - 1; i >= 0; i--){
-                       if(!noPropagation) delayedTriggers.push((function(c) { return function(event) { c._trigger("deactivate", event, this._uiHash(this)); };  }).call(this, this.containers[i]));
+               for (i = this.containers.length - 1; i >= 0; i--){
+                       if(!noPropagation) {
+                               delayedTriggers.push((function(c) { return function(event) { c._trigger("deactivate", event, this._uiHash(this)); };  }).call(this, this.containers[i]));
+                       }
                        if(this.containers[i].containerCache.over) {
                                delayedTriggers.push((function(c) { return function(event) { c._trigger("out", event, this._uiHash(this)); };  }).call(this, this.containers[i]));
                                this.containers[i].containerCache.over = 0;
@@ -1042,15 +1160,21 @@ $.widget("ui.sortable", $.ui.mouse, {
                }
 
                //Do what was originally in plugins
-               if(this._storedCursor) $('body').css("cursor", this._storedCursor); //Reset cursor
-               if(this._storedOpacity) this.helper.css("opacity", this._storedOpacity); //Reset opacity
-               if(this._storedZIndex) this.helper.css("zIndex", this._storedZIndex == 'auto' ? '' : this._storedZIndex); //Reset z-index
+               if(this._storedCursor) {
+                       $('body').css("cursor", this._storedCursor);
+               }
+               if(this._storedOpacity) {
+                       this.helper.css("opacity", this._storedOpacity);
+               }
+               if(this._storedZIndex) {
+                       this.helper.css("zIndex", this._storedZIndex === 'auto' ? '' : this._storedZIndex);
+               }
 
                this.dragging = false;
                if(this.cancelHelperRemoval) {
                        if(!noPropagation) {
                                this._trigger("beforeStop", event, this._uiHash());
-                               for (var i=0; i < delayedTriggers.length; i++) {
+                               for (i=0; i < delayedTriggers.length; i++) {
                                        delayedTriggers[i].call(this, event);
                                } //Trigger all delayed events
                                this._trigger("stop", event, this._uiHash());
@@ -1060,15 +1184,20 @@ $.widget("ui.sortable", $.ui.mouse, {
                        return false;
                }
 
-               if(!noPropagation) this._trigger("beforeStop", event, this._uiHash());
+               if(!noPropagation) {
+                       this._trigger("beforeStop", event, this._uiHash());
+               }
 
                //$(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.helper[0] != this.currentItem[0]) this.helper.remove(); this.helper = null;
+               if(this.helper[0] !== this.currentItem[0]) {
+                       this.helper.remove();
+               }
+               this.helper = null;
 
                if(!noPropagation) {
-                       for (var i=0; i < delayedTriggers.length; i++) {
+                       for (i=0; i < delayedTriggers.length; i++) {
                                delayedTriggers[i].call(this, event);
                        } //Trigger all delayed events
                        this._trigger("stop", event, this._uiHash());