]> source.dussan.org Git - jquery-ui.git/commitdiff
Checkboxradio: Typos and minor style fixes
authorAlexander Schmitz <arschmitz@gmail.com>
Wed, 30 Sep 2015 15:47:06 +0000 (11:47 -0400)
committerAlexander Schmitz <arschmitz@gmail.com>
Thu, 8 Oct 2015 18:02:58 +0000 (14:02 -0400)
ui/widgets/checkboxradio.js

index 13ba9dadf0ba48cdfcc4c32508553866c1f0107e..88b2f784d6496523f7fe8cac36211c19264b92d9 100644 (file)
@@ -12,7 +12,9 @@
 //>>description: Enhances a form with multiple themeable checkboxes or radio buttons.
 //>>docs: http://api.jqueryui.com/checkboxradio/
 //>>demos: http://jqueryui.com/checkboxradio/
-//>>css.structure: ../themes/base/core.css, ../themes/base/button.css, ../themes/base/checkboxradio.css
+//>>css.structure: ../themes/base/core.css
+//>>css.structure: ../themes/base/button.css
+//>>css.structure: ../themes/base/checkboxradio.css
 //>>css.theme: ../themes/base/theme.css
 
 ( function( factory ) {
@@ -46,9 +48,9 @@ $.widget( "ui.checkboxradio", [ $.ui.formResetMixin, {
        },
 
        _getCreateOptions: function() {
-               var disabled, labels,
-                       that = this,
-                       options = this._super() || {};
+               var disabled, labels;
+               var that = this;
+               var options = this._super() || {};
 
                // We read the type here, because it makes more sense to throw a element type error first,
                // rather then the error for lack of a label. Often if its the wrong type, it
@@ -152,14 +154,13 @@ $.widget( "ui.checkboxradio", [ $.ui.formResetMixin, {
                var formParent = this.formParent[ 0 ];
                var radios = $( [] );
 
-               if ( name ) {
-                       name = $.ui.escapeSelector( name );
-                       radios = this.formParent.find( "[name='" + $.ui.escapeSelector( name ) + "']" ).filter( function() {
-                               var form = $( this ).form();
-                               return ( form.length ? form : $( "body" ) )[ 0 ] === formParent;
-                       } );
-               }
-               return radios.not( this.element );
+               if ( !name ) {
+                       return $( [] );
+               }``
+               return this.formParent.find( "[name='" + $.ui.escapeSelector( name ) + "']" ).filter( function() {
+                       var form = $( this ).form();
+                       return ( form.length ? form : $( "body" ) )[ 0 ] === formParent;
+               } ).not( this.element );
        },
 
        _toggleClasses: function() {
@@ -207,7 +208,7 @@ $.widget( "ui.checkboxradio", [ $.ui.formResetMixin, {
                        this._toggleClass( this.label, null, "ui-state-disabled", value );
                        this.element[ 0 ].disabled = value;
 
-                       // Don't refresh if disabled
+                       // Don't refresh when setting disabled
                        return;
                }
                this.refresh();