From 7eb0f768802440a9e8c57045a8dc72e9fa7ed090 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Scott=20Gonz=C3=A1lez?= Date: Tue, 3 Mar 2015 10:54:49 -0500 Subject: [PATCH] [fixup]: Clean up from gh-1324 --- tests/unit/resizable/resizable_options.js | 6 +-- ui/resizable.js | 55 ++++++++++------------- 2 files changed, 27 insertions(+), 34 deletions(-) diff --git a/tests/unit/resizable/resizable_options.js b/tests/unit/resizable/resizable_options.js index e5cb1c71309..6e3a93a2f85 100644 --- a/tests/unit/resizable/resizable_options.js +++ b/tests/unit/resizable/resizable_options.js @@ -435,21 +435,21 @@ test( "alsoResize + containment", function() { test( "alsoResize + multiple selection", function() { expect( 6 ); var other1 = $( "
" ) - .addClass("other") + .addClass( "other" ) .css({ width: 50, height: 50 }) .appendTo( "body" ), other2 = $( "
" ) - .addClass("other") + .addClass( "other" ) .css({ width: 50, height: 50 }) .appendTo( "body"), element = $( "#resizable1" ).resizable({ - alsoResize: $.merge(other1, other2), + alsoResize: other1.add( other2 ), containment: "#container" }); diff --git a/ui/resizable.js b/ui/resizable.js index d44370779b5..09efc9a37d6 100644 --- a/ui/resizable.js +++ b/ui/resizable.js @@ -991,17 +991,15 @@ $.ui.plugin.add("resizable", "alsoResize", { start: function() { var that = $(this).resizable( "instance" ), - o = that.options, - _store = function(exp) { - $(exp).each(function() { - var el = $(this); - el.data("ui-resizable-alsoresize", { - width: parseInt(el.width(), 10), height: parseInt(el.height(), 10), - left: parseInt(el.css("left"), 10), top: parseInt(el.css("top"), 10) - }); - }); - }; - _store(o.alsoResize); + o = that.options; + + $(o.alsoResize).each(function() { + var el = $(this); + el.data("ui-resizable-alsoresize", { + width: parseInt(el.width(), 10), height: parseInt(el.height(), 10), + left: parseInt(el.css("left"), 10), top: parseInt(el.css("top"), 10) + }); + }); }, resize: function(event, ui) { @@ -1014,28 +1012,23 @@ $.ui.plugin.add("resizable", "alsoResize", { width: (that.size.width - os.width) || 0, top: (that.position.top - op.top) || 0, left: (that.position.left - op.left) || 0 - }, + }; + + $(o.alsoResize).each(function() { + var el = $(this), start = $(this).data("ui-resizable-alsoresize"), style = {}, + css = el.parents(ui.originalElement[0]).length ? + [ "width", "height" ] : + [ "width", "height", "top", "left" ]; - _alsoResize = function(exp, c) { - $(exp).each(function() { - var el = $(this), start = $(this).data("ui-resizable-alsoresize"), style = {}, - css = c && c.length ? - c : - el.parents(ui.originalElement[0]).length ? - [ "width", "height" ] : - [ "width", "height", "top", "left" ]; - - $.each(css, function(i, prop) { - var sum = (start[prop] || 0) + (delta[prop] || 0); - if (sum && sum >= 0) { - style[prop] = sum || null; - } - }); - - el.css(style); + $.each(css, function(i, prop) { + var sum = (start[prop] || 0) + (delta[prop] || 0); + if (sum && sum >= 0) { + style[prop] = sum || null; + } }); - }; - _alsoResize(o.alsoResize); + + el.css(style); + }); }, stop: function() {