Droppable: Style updates

Ref #14246

7 files changed, 89 insertions, 87 deletions

diff --git a/tests/unit/droppable/common-deprecated.js b/tests/unit/droppable/common-deprecated.js
index ecbc9b1a1..fc086757f 100644
--- a/tests/unit/droppable/common-deprecated.js
+++ b/tests/unit/droppable/common-deprecated.js
@@ -23,6 +23,6 @@ common.testWidget( "droppable", {
 		out: null,
 		over: null
 	}
-});
+} );
 
 } );
diff --git a/tests/unit/droppable/common.js b/tests/unit/droppable/common.js
index 5d0e55d6f..d4bdb9dc0 100644
--- a/tests/unit/droppable/common.js
+++ b/tests/unit/droppable/common.js
@@ -21,6 +21,6 @@ common.testWidget( "droppable", {
 		out: null,
 		over: null
 	}
-});
+} );
 
 } );
diff --git a/tests/unit/droppable/core.js b/tests/unit/droppable/core.js
index 75951c1bc..c4ce0378c 100644
--- a/tests/unit/droppable/core.js
+++ b/tests/unit/droppable/core.js
@@ -4,26 +4,26 @@ define( [
 	"ui/widgets/droppable"
 ], function( $, testHelper ) {
 
-module("droppable: core");
+module( "droppable: core" );
 
-test("element types", function() {
-	var typeNames = ("p,h1,h2,h3,h4,h5,h6,blockquote,ol,ul,dl,div,form" +
+test( "element types", function() {
+	var typeNames = ( "p,h1,h2,h3,h4,h5,h6,blockquote,ol,ul,dl,div,form" +
 		",table,fieldset,address,ins,del,em,strong,q,cite,dfn,abbr" +
 		",acronym,code,samp,kbd,var,img,hr" +
-		",input,button,label,select,iframe").split(",");
+		",input,button,label,select,iframe" ).split( "," );
 
 	expect( typeNames.length );
 
-	$.each(typeNames, function(i) {
-		var typeName = typeNames[i],
-			el = $(document.createElement(typeName)).appendTo("body");
+	$.each( typeNames, function( i ) {
+		var typeName = typeNames[ i ],
+			el = $( document.createElement( typeName ) ).appendTo( "body" );
 
-		(typeName === "table" && el.append("<tr><td>content</td></tr>"));
+		( typeName === "table" && el.append( "<tr><td>content</td></tr>" ) );
 		el.droppable();
 		testHelper.shouldDrop();
-		el.droppable("destroy");
+		el.droppable( "destroy" );
 		el.remove();
-	});
-});
+	} );
+} );
 
 } );
diff --git a/tests/unit/droppable/events.js b/tests/unit/droppable/events.js
index 9c84b099f..68df6e506 100644
--- a/tests/unit/droppable/events.js
+++ b/tests/unit/droppable/events.js
@@ -13,7 +13,7 @@ test( "droppable destruction/recreation on drop event", function( assert ) {
 			drop: function() {
 				var element = $( this ),
 					newDroppable = $( "<div>" )
-						.css({ width: 100, height: 100 })
+						.css( { width: 100, height: 100 } )
 						.text( "Droppable" );
 				element.after( newDroppable );
 				element.remove();
@@ -33,10 +33,10 @@ test( "droppable destruction/recreation on drop event", function( assert ) {
 	draggable.simulate( "drag", {
 		dx: dx,
 		dy: dy
-	});
+	} );
 
 	assert.lacksClasses( droppable2, "active", "subsequent droppable no longer active" );
-});
+} );
 
 // todo: comment the following in when ready to actually test
 /*
diff --git a/tests/unit/droppable/helper.js b/tests/unit/droppable/helper.js
index 460fa1aa6..b9bc5d6fb 100644
--- a/tests/unit/droppable/helper.js
+++ b/tests/unit/droppable/helper.js
@@ -5,13 +5,15 @@ define( [
 
 return $.extend( helper, {
 	shouldDrop: function() {
+
 		// todo: actually implement this
-		ok(true, "missing test - untested code is broken code");
+		ok( true, "missing test - untested code is broken code" );
 	},
 
 	shouldNotDrop: function() {
+
 		// todo: actually implement this
-		ok(true, "missing test - untested code is broken code");
+		ok( true, "missing test - untested code is broken code" );
 	}
 } );
 
diff --git a/tests/unit/droppable/methods.js b/tests/unit/droppable/methods.js
index 10ac48499..c159be56f 100644
--- a/tests/unit/droppable/methods.js
+++ b/tests/unit/droppable/methods.js
@@ -4,78 +4,78 @@ define( [
 	"ui/widgets/droppable"
 ], function( $, testHelper ) {
 
-module("droppable: methods");
+module( "droppable: methods" );
 
-test("init", function() {
+test( "init", function() {
 	expect( 5 );
 
-	$("<div></div>").appendTo("body").droppable().remove();
-	ok(true, ".droppable() called on element");
+	$( "<div></div>" ).appendTo( "body" ).droppable().remove();
+	ok( true, ".droppable() called on element" );
 
-	$([]).droppable();
-	ok(true, ".droppable() called on empty collection");
+	$( [] ).droppable();
+	ok( true, ".droppable() called on empty collection" );
 
-	$("<div></div>").droppable();
-	ok(true, ".droppable() called on disconnected DOMElement");
+	$( "<div></div>" ).droppable();
+	ok( true, ".droppable() called on disconnected DOMElement" );
 
-	$("<div></div>").droppable().droppable("option", "foo");
-	ok(true, "arbitrary option getter after init");
+	$( "<div></div>" ).droppable().droppable( "option", "foo" );
+	ok( true, "arbitrary option getter after init" );
 
-	$("<div></div>").droppable().droppable("option", "foo", "bar");
-	ok(true, "arbitrary option setter after init");
-});
+	$( "<div></div>" ).droppable().droppable( "option", "foo", "bar" );
+	ok( true, "arbitrary option setter after init" );
+} );
 
-test("destroy", function() {
+test( "destroy", function() {
 	expect( 4 );
 
-	$("<div></div>").appendTo("body").droppable().droppable("destroy").remove();
-	ok(true, ".droppable('destroy') called on element");
+	$( "<div></div>" ).appendTo( "body" ).droppable().droppable( "destroy" ).remove();
+	ok( true, ".droppable('destroy') called on element" );
 
-	$([]).droppable().droppable("destroy");
-	ok(true, ".droppable('destroy') called on empty collection");
+	$( [] ).droppable().droppable( "destroy" );
+	ok( true, ".droppable('destroy') called on empty collection" );
 
-	$("<div></div>").droppable().droppable("destroy");
-	ok(true, ".droppable('destroy') called on disconnected DOMElement");
+	$( "<div></div>" ).droppable().droppable( "destroy" );
+	ok( true, ".droppable('destroy') called on disconnected DOMElement" );
 
-	var expected = $("<div></div>").droppable(),
-		actual = expected.droppable("destroy");
-	equal(actual, expected, "destroy is chainable");
-});
+	var expected = $( "<div></div>" ).droppable(),
+		actual = expected.droppable( "destroy" );
+	equal( actual, expected, "destroy is chainable" );
+} );
 
-test("enable", function() {
-	expect(7);
+test( "enable", function() {
+	expect( 7 );
 
 	var el, expected, actual;
 
-	el = $("#droppable1").droppable({ disabled: true });
+	el = $( "#droppable1" ).droppable( { disabled: true } );
 	testHelper.shouldNotDrop();
-	el.droppable("enable");
+	el.droppable( "enable" );
 	testHelper.shouldDrop();
-	equal(el.droppable("option", "disabled"), false, "disabled option getter");
-	el.droppable("destroy");
-	el.droppable({ disabled: true });
+	equal( el.droppable( "option", "disabled" ), false, "disabled option getter" );
+	el.droppable( "destroy" );
+	el.droppable( { disabled: true } );
 	testHelper.shouldNotDrop();
-	el.droppable("option", "disabled", false);
-	equal(el.droppable("option", "disabled"), false, "disabled option setter");
+	el.droppable( "option", "disabled", false );
+	equal( el.droppable( "option", "disabled" ), false, "disabled option setter" );
 	testHelper.shouldDrop();
 
-	expected = $("<div></div>").droppable(),
-	actual = expected.droppable("enable");
-	equal(actual, expected, "enable is chainable");
-});
+	expected = $( "<div></div>" ).droppable(),
+	actual = expected.droppable( "enable" );
+	equal( actual, expected, "enable is chainable" );
+} );
 
 test( "disable", function( assert ) {
 	expect( 10 );
 
 	var actual, expected,
-		element = $( "#droppable1" ).droppable({ disabled: false });
+		element = $( "#droppable1" ).droppable( { disabled: false } );
 
 	testHelper.shouldDrop();
 	element.droppable( "disable" );
 	testHelper.shouldNotDrop();
 	equal( element.droppable( "option", "disabled" ), true, "disabled option getter" );
 	element.droppable( "destroy" );
-	element.droppable({ disabled: false });
+	element.droppable( { disabled: false } );
 	testHelper.shouldDrop();
 	element.droppable( "option", "disabled", true );
 	assert.lacksClasses( element.droppable( "widget" ), "ui-state-disabled" );
@@ -87,6 +87,6 @@ test( "disable", function( assert ) {
 	expected = $( "<div></div>" ).droppable();
 	actual = expected.droppable( "disable" );
 	equal( actual, expected, "disable is chainable" );
-});
+} );
 
 } );
diff --git a/tests/unit/droppable/options.js b/tests/unit/droppable/options.js
index c2035ce8c..856c2d215 100644
--- a/tests/unit/droppable/options.js
+++ b/tests/unit/droppable/options.js
@@ -24,23 +24,23 @@ test( "activeClass", function() {
 */
 test( "{ addClasses: true }, default", function( assert ) {
 	expect( 1 );
-	var el = $( "<div />" ).droppable({ addClasses: true });
+	var el = $( "<div />" ).droppable( { addClasses: true } );
 	assert.hasClasses( el, "ui-droppable" );
 	el.droppable( "destroy" );
-});
+} );
 
 test( "{ addClasses: false }", function( assert ) {
 	expect( 1 );
-	var el = $( "<div />" ).droppable({ addClasses: false });
+	var el = $( "<div />" ).droppable( { addClasses: false } );
 
 	assert.lacksClasses( el, "ui-droppable" );
 	el.droppable( "destroy" );
-});
+} );
 
 test( "scope", function() {
 	expect( 4 );
 	var droppableOffset, draggableOffset, oldDraggableOffset, dx, dy,
-		draggable1 = $( "<div />" ).appendTo( "#qunit-fixture" ).draggable({ revert: "invalid" }),
+		draggable1 = $( "<div />" ).appendTo( "#qunit-fixture" ).draggable( { revert: "invalid" } ),
 		draggable2 = $( "<div />" ).appendTo( "#qunit-fixture" ).droppable(),
 		droppable = $( "<div />" ).appendTo( "#qunit-fixture" ).droppable(),
 		newScope = "test";
@@ -57,7 +57,7 @@ test( "scope", function() {
 	draggable1.simulate( "drag", {
 		dx: dx,
 		dy: dy
-	});
+	} );
 
 	draggableOffset = draggable1.offset();
 	equal( draggableOffset.left, droppableOffset.left );
@@ -72,12 +72,12 @@ test( "scope", function() {
 	draggable2.simulate( "drag", {
 		dx: dx,
 		dy: dy
-	});
+	} );
 
 	draggableOffset = draggable2.offset();
 	equal( draggableOffset.left, oldDraggableOffset.left );
 	equal( draggableOffset.top, oldDraggableOffset.top );
-});
+} );
 /*
 test( "greedy", function() {
 	ok(false, 'missing test - untested code is broken code');
@@ -109,7 +109,7 @@ test( "tolerance, intersect", function() {
 
 	draggable = $( "<div />" )
 		.appendTo( "#qunit-fixture" )
-		.css({
+		.css( {
 			width: 10,
 			height: 10,
 			position: "absolute",
@@ -118,32 +118,32 @@ test( "tolerance, intersect", function() {
 			// Droppable: droppable region is offset by draggables margin
 			marginTop: 3,
 			marginLeft: 3
-		})
+		} )
 		.draggable();
 
 	droppable = $( "<div />" )
 		.appendTo( "#qunit-fixture" )
-		.css({ width: 10, height: 10, position: "absolute", top: 13, left: 13 })
-		.droppable({ tolerance: "intersect" });
+		.css( { width: 10, height: 10, position: "absolute", top: 13, left: 13 } )
+		.droppable( { tolerance: "intersect" } );
 
 	$.each( dataset, function() {
 		var data = this;
 
-		draggable.css({
+		draggable.css( {
 			top: 0,
 			left: 0
-		});
+		} );
 
 		droppable.off( "drop" ).on( "drop", function() {
 			equal( true, data[ 2 ], data[ 3 ] );
-		});
+		} );
 
 		$( draggable ).simulate( "drag", {
 			dx: data[ 0 ],
 			dy: data[ 1 ]
-		});
-	});
-});
+		} );
+	} );
+} );
 
 test( "tolerance, pointer", function() {
 	expect( 3 );
@@ -162,40 +162,40 @@ test( "tolerance, pointer", function() {
 
 	draggable = $( "<div />" )
 		.appendTo( "#qunit-fixture" )
-		.css({ width: 10, height: 10, position: "absolute" })
+		.css( { width: 10, height: 10, position: "absolute" } )
 		.draggable();
 
 	droppable = $( "<div />" )
 		.appendTo( "#qunit-fixture" )
-		.css({ width: 10, height: 10, position: "absolute", top: 5, left: 5 })
-		.droppable({ tolerance: "pointer" });
+		.css( { width: 10, height: 10, position: "absolute", top: 5, left: 5 } )
+		.droppable( { tolerance: "pointer" } );
 
 	$.each( dataset, function() {
 		var data = this;
 
 		droppable.off( "drop" ).on( "drop", function() {
 			equal( true, data[ 2 ], data[ 3 ] );
-		});
+		} );
 
 		$( draggable ).simulate( "drag", {
 			dx: ( data[ 0 ] - $( draggable ).position().left ),
 			dy: ( data[ 1 ] - $( draggable ).position().top )
-		});
-	});
+		} );
+	} );
 
 	// Http://bugs.jqueryui.com/ticket/4977 - tolerance, pointer - bug when pointer outside draggable
-	draggable.css({ top: 0, left: 0 }).draggable( "option", "axis", "x" );
-	droppable.css({ top: 15, left: 15 });
+	draggable.css( { top: 0, left: 0 } ).draggable( "option", "axis", "x" );
+	droppable.css( { top: 15, left: 15 } );
 
 	droppable.off( "drop" ).on( "drop", function() {
 		ok( true, "drop fires as long as pointer is within droppable" );
-	});
+	} );
 
 	$( draggable ).simulate( "drag", {
 		dx: 10,
 		dy: 10
-	});
-});
+	} );
+} );
 
 /*
 test( "tolerance, touch", function() {