From f378e34789ce616384ddc25704b7f58528099864 Mon Sep 17 00:00:00 2001 From: Alexander Schmitz Date: Fri, 21 Aug 2015 00:07:28 -0400 Subject: [PATCH] Resizable: Style updates Ref #14246 Ref gh-1588 --- tests/unit/resizable/common.js | 2 +- tests/unit/resizable/events.js | 6 +++--- tests/unit/resizable/options.js | 10 +++++----- ui/widgets/resizable.js | 6 +++--- 4 files changed, 12 insertions(+), 12 deletions(-) diff --git a/tests/unit/resizable/common.js b/tests/unit/resizable/common.js index 6096446f3..3f1f82bce 100644 --- a/tests/unit/resizable/common.js +++ b/tests/unit/resizable/common.js @@ -29,7 +29,7 @@ common.testWidget( "resizable", { minWidth: 10, zIndex: 90, - // callbacks + // Callbacks create: null, resize: null, start: null, diff --git a/tests/unit/resizable/events.js b/tests/unit/resizable/events.js index eb64400c6..b77130741 100644 --- a/tests/unit/resizable/events.js +++ b/tests/unit/resizable/events.js @@ -203,7 +203,7 @@ test( "resize (containment) works with parent with negative offset", function() fixedContainer = absoluteContainer.wrap( "
" ).parent(), increaseWidthBy = 50; - // position fixed container in window top left + // Position fixed container in window top left fixedContainer.css({ width: 400, height: 100, @@ -212,7 +212,7 @@ test( "resize (containment) works with parent with negative offset", function() left: 0 }); - // position absolute container within fixed on slightly outside window + // Position absolute container within fixed on slightly outside window absoluteContainer.css({ width: 400, height: 100, @@ -221,7 +221,7 @@ test( "resize (containment) works with parent with negative offset", function() left: -50 }); - // set up resizable to be contained within absolute container + // Set up resizable to be contained within absolute container target.resizable({ handles: "all", containment: "parent" diff --git a/tests/unit/resizable/options.js b/tests/unit/resizable/options.js index 1e643a576..234348309 100644 --- a/tests/unit/resizable/options.js +++ b/tests/unit/resizable/options.js @@ -126,7 +126,7 @@ test("aspectRatio: 'preserve' (ne)", function() { test( "aspectRatio: Resizing can move objects", function() { expect( 7 ); - // http://bugs.jqueryui.com/ticket/7018 - Resizing can move objects + // Http://bugs.jqueryui.com/ticket/7018 - Resizing can move objects var handleW = ".ui-resizable-w", handleNW = ".ui-resizable-nw", target = $( "#resizable1" ).resizable({ @@ -143,7 +143,7 @@ test( "aspectRatio: Resizing can move objects", function() { equal( target.height(), 100, "compare height - no size change" ); equal( target.position().left, 75, "compare left - no movement" ); - // http://bugs.jqueryui.com/ticket/9107 - aspectRatio and containment not handled correctly + // Http://bugs.jqueryui.com/ticket/9107 - aspectRatio and containment not handled correctly $( "#container" ).css({ width: 200, height: 300, position: "absolute", left: 100, top: 100 }); $( "#resizable1" ).css({ width: 100, height: 100, left: 0, top: 0 }); @@ -173,7 +173,7 @@ test( "containment", function() { test( "containment - not immediate parent", function() { expect( 4 ); - // http://bugs.jqueryui.com/ticket/7485 - Resizable: Containment calculation is wrong + // Http://bugs.jqueryui.com/ticket/7485 - Resizable: Containment calculation is wrong // when containment element is not the immediate parent var element = $( "#child" ).resizable({ containment: "#container2", @@ -205,7 +205,7 @@ test( "containment - not immediate parent", function() { test( "containment - immediate parent", function() { expect( 4 ); - // http://bugs.jqueryui.com/ticket/10140 - Resizable: Width calculation is wrong when containment element is "position: relative" + // Http://bugs.jqueryui.com/ticket/10140 - Resizable: Width calculation is wrong when containment element is "position: relative" // when containment element is immediate parent var element = $( "#child" ).resizable({ containment: "parent", @@ -302,7 +302,7 @@ test( "grid - Resizable: can be moved when grid option is set (#9611)", function test( "grid - maintains grid with padding and border when approaching no dimensions", function() { expect( 2 ); - // http://bugs.jqueryui.com/ticket/10437 - Resizable: border with grid option working wrong + // Http://bugs.jqueryui.com/ticket/10437 - Resizable: border with grid option working wrong var handle = ".ui-resizable-nw", target = $( "#resizable1" ).css({ padding: 5, diff --git a/ui/widgets/resizable.js b/ui/widgets/resizable.js index 95cf918d9..cc6f9f186 100644 --- a/ui/widgets/resizable.js +++ b/ui/widgets/resizable.js @@ -60,7 +60,7 @@ $.widget("ui.resizable", $.ui.mouse, { // See #7960 zIndex: 90, - // callbacks + // Callbacks resize: null, start: null, stop: null @@ -153,7 +153,7 @@ $.widget("ui.resizable", $.ui.mouse, { display: "block" }) ); - // support: IE9 + // Support: IE9 // avoid IE jump (hard set the margin) this.originalElement.css({ margin: this.originalElement.css("margin") }); @@ -796,7 +796,7 @@ $.ui.plugin.add("resizable", "animate", { $(pr[0]).css({ width: data.width, height: data.height }); } - // propagating resize, and updating values for each animation step + // Propagating resize, and updating values for each animation step that._updateCache(data); that._propagate("resize", event); -- 2.39.5