From: Alexander Schmitz Date: Fri, 11 Sep 2015 14:48:51 +0000 (-0400) Subject: Controlgroup: Style updates X-Git-Tag: 1.12.0-beta.1~75 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=3a30eef1c1e4c2608db8b621f2c3fb947380a703;p=jquery-ui.git Controlgroup: Style updates --- diff --git a/tests/unit/controlgroup/common.js b/tests/unit/controlgroup/common.js index 4dc86afbd..e072535d2 100644 --- a/tests/unit/controlgroup/common.js +++ b/tests/unit/controlgroup/common.js @@ -23,6 +23,6 @@ common.testWidget( "controlgroup", { // Callbacks create: null } -}); +} ); } ); diff --git a/tests/unit/controlgroup/core.js b/tests/unit/controlgroup/core.js index 385e0cfd4..43acdd665 100644 --- a/tests/unit/controlgroup/core.js +++ b/tests/unit/controlgroup/core.js @@ -64,6 +64,6 @@ test( "selectmenu: open/close corners", function( assert ) { selects.eq( 2 ).selectmenu( "close" ); assert.hasClasses( selectButton, "ui-corner-bottom", "vertical: Last selectmenu gets ui-corner-bottom when closed" ); -}); +} ); } ); diff --git a/tests/unit/controlgroup/methods.js b/tests/unit/controlgroup/methods.js index de1d56f9d..b0d711ee7 100644 --- a/tests/unit/controlgroup/methods.js +++ b/tests/unit/controlgroup/methods.js @@ -12,8 +12,8 @@ test( "destroy", function( assert ) { expect( 1 ); assert.domEqual( ".controlgroup", function() { $( ".controlgroup" ).controlgroup().controlgroup( "destroy" ); - }); -}); + } ); +} ); test( "disable", function( assert ) { expect( 2 ); var element = $( ".controlgroup" ).controlgroup().controlgroup( "disable" ); @@ -21,7 +21,7 @@ test( "disable", function( assert ) { "The widget does not get the disabled class, because we disable each child widget" ); strictEqual( element.find( ".ui-state-disabled" ).length, 6, "Child widgets are disabled" ); -}); +} ); test( "enable", function( assert ) { expect( 2 ); @@ -30,7 +30,7 @@ test( "enable", function( assert ) { "ui-state-disabled is not present on widget after enabling" ); strictEqual( element.find( "ui-state-disabled" ).length, 0, "Child widgets are disabled" ); -}); +} ); var tests = { "checkboxradio": "", @@ -63,9 +63,9 @@ $.each( tests, function( widget, html ) { var i, control, currentClasses, controls = [], - element = $( "
" ).controlgroup({ + element = $( "
" ).controlgroup( { direction: name - }).appendTo( "body" ); + } ).appendTo( "body" ); // checks the elements with in the controlgroup against the expected class list function checkCornerClasses( classList ) { @@ -143,8 +143,8 @@ $.each( tests, function( widget, html ) { // remove the controlgroup before we start the next set element.remove(); - }); - }); -}); + } ); + } ); +} ); } ); diff --git a/tests/unit/controlgroup/options.js b/tests/unit/controlgroup/options.js index e2e8f4ee7..90a57e093 100644 --- a/tests/unit/controlgroup/options.js +++ b/tests/unit/controlgroup/options.js @@ -18,66 +18,66 @@ test( "disabled", function( assert ) { assert.lacksClasses( element, "ui-state-disabled" ); strictEqual( element.find( ".ui-state-disabled" ).length, 0, "Child widgets are not disabled" ); -}); +} ); test( "items - null", function() { expect( 2 ); - var element = $( ".controlgroup" ).controlgroup({ + var element = $( ".controlgroup" ).controlgroup( { items: { "button": null, "selectmenu": null, "checkboxradio": null } - }); + } ); strictEqual( element.children( ".ui-button" ).length, 0, "Child widgets are not called when selector is null" ); element.controlgroup( "option", "items", { "button": "button" - }); + } ); strictEqual( element.children( ".ui-button" ).length, 2, "Correct child widgets are called when selector is updated" ); -}); +} ); test( "items: custom selector", function() { expect( 1 ); - var element = $( ".controlgroup" ).controlgroup({ + var element = $( ".controlgroup" ).controlgroup( { items: { "button": ".button" } - }); + } ); strictEqual( element.children( ".ui-button" ).length, 4, "Correct child widgets are called when custom selector used" ); -}); +} ); $.widget( "ui.test", { - _create: function (){ + _create: function() { this.element.addClass( "ui-test ui-button" ); }, // Controlgroup requires a refresh method to exist refresh: $.noop -}); +} ); test( "items: custom widget", function() { expect( 2 ); - var element = $( ".controlgroup" ).controlgroup({ + var element = $( ".controlgroup" ).controlgroup( { items: { "test": ".test" } - }); + } ); strictEqual( element.children( ".ui-button" ).length, 7, "Correct child widgets are called when custom selector used" ); strictEqual( element.children( ".ui-test" ).length, 1, "Custom widget called" ); -}); +} ); test( "excludeInvisible", function( assert ) { expect( 4 ); - var element = $( ".controlgroup" ).controlgroup({ + var element = $( ".controlgroup" ).controlgroup( { excludeInvisible: false - }), + } ), buttons = element.children( ".ui-button" ); assert.lacksClassStart( buttons.eq( 1 ), "ui-corner" ); @@ -88,7 +88,7 @@ test( "excludeInvisible", function( assert ) { assert.lacksClassStart( buttons.eq( 0 ), "ui-corner" ); assert.hasClasses( buttons.eq( 1 ), "ui-corner-left", "ExcludeInvisible: true: First button is hidden second button get corner class" ); -}); +} ); test( "direction", function( assert ) { expect( 6 ); @@ -103,6 +103,6 @@ test( "direction", function( assert ) { assert.hasClasses( element, "ui-controlgroup-vertical" ); assert.hasClasses( buttons.first(), "ui-corner-top" ); assert.hasClasses( buttons.last(), "ui-corner-bottom" ); -}); +} ); } );