diff options
author | Scott González <scott.gonzalez@gmail.com> | 2010-03-11 20:10:43 +0000 |
---|---|---|
committer | Scott González <scott.gonzalez@gmail.com> | 2010-03-11 20:10:43 +0000 |
commit | 057e156e626a4cfd332eaa30ef56b41b6f47d507 (patch) | |
tree | 5421879d472a21936d1550ded33cc6219f98e0d7 /ui | |
parent | d3cb190377782b5f952d5f213101a8399f5908ff (diff) | |
download | jquery-ui-057e156e626a4cfd332eaa30ef56b41b6f47d507.tar.gz jquery-ui-057e156e626a4cfd332eaa30ef56b41b6f47d507.zip |
Button: Conform to coding standards.
Diffstat (limited to 'ui')
-rw-r--r-- | ui/jquery.ui.button.js | 58 |
1 files changed, 30 insertions, 28 deletions
diff --git a/ui/jquery.ui.button.js b/ui/jquery.ui.button.js index 392f96e4e..ce4c86a94 100644 --- a/ui/jquery.ui.button.js +++ b/ui/jquery.ui.button.js @@ -17,18 +17,18 @@ var lastActive, baseClasses = "ui-button ui-widget ui-state-default ui-corner-all", otherClasses = "ui-state-hover ui-state-active " + "ui-button-icons-only ui-button-icon-only ui-button-text-icons ui-button-text-icon ui-button-text-only", - formResetHandler = function(event) { - $(':ui-button', event.target.form).each(function() { - var inst = $(this).data('button'); + formResetHandler = function( event ) { + $( ":ui-button", event.target.form ).each(function() { + var inst = $( this ).data( "button" ); setTimeout(function() { - inst.refresh() - }, 1); + inst.refresh(); + }, 1 ); }); }, - radioGroup = function(radio) { + radioGroup = function( radio ) { var name = radio.name, form = radio.form, - radios = $([]); + radios = $( [] ); if ( name ) { if ( form ) { radios = $( form ).find( "[name='" + name + "']" ); @@ -52,8 +52,10 @@ $.widget( "ui.button", { } }, _create: function() { - this.element.closest('form').unbind('reset.button').bind('reset.button', formResetHandler); - + this.element.closest( "form" ) + .unbind( "reset.button" ) + .bind( "reset.button", formResetHandler ); + this._determineButtonType(); this.hasTitle = !!this.buttonElement.attr( "title" ); @@ -92,10 +94,9 @@ $.widget( "ui.button", { .bind( "blur.button", function() { $( this ).removeClass( focusClass ); }); - + if ( toggleButton ) { - var self = this; - this.element.bind('change.button', function() { + this.element.bind( "change.button", function() { self.refresh(); }); } @@ -151,11 +152,12 @@ $.widget( "ui.button", { .bind( "keyup.button", function() { $( this ).removeClass( "ui-state-active" ); }); - if (this.buttonElement.is("a")) { + + if ( this.buttonElement.is("a") ) { this.buttonElement.keyup(function(event) { - if (event.keyCode == $.ui.keyCode.SPACE) { + if ( event.keyCode === $.ui.keyCode.SPACE ) { // TODO pass through original event correctly (just as 2nd argument doesn't work) - $(this).trigger("click"); + $( this ).click(); } }); } @@ -220,29 +222,29 @@ $.widget( "ui.button", { $.Widget.prototype._setOption.apply( this, arguments ); this._resetButton(); }, - + refresh: function() { if ( this.type === "radio" ) { radioGroup( this.element[0] ).each(function() { - if ( $(this).is(':checked') ) { - $(this).button('widget') - .addClass('ui-state-active') - .attr('aria-pressed', true); + if ( $( this ).is( ":checked" ) ) { + $( this ).button( "widget" ) + .addClass( "ui-state-active" ) + .attr( "aria-pressed", true ); } else { - $(this).button('widget') - .removeClass('ui-state-active') - .attr('aria-pressed', false); + $( this ).button( "widget" ) + .removeClass( "ui-state-active" ) + .attr( "aria-pressed", false ); } }); } else if ( this.type === "checkbox" ) { - if ( this.element.is(':checked') ) { + if ( this.element.is( ":checked" ) ) { this.buttonElement - .addClass('ui-state-active') - .attr('aria-pressed', true); + .addClass( "ui-state-active" ) + .attr( "aria-pressed", true ); } else { this.buttonElement - .removeClass('ui-state-active') - .attr('aria-pressed', false); + .removeClass( "ui-state-active" ) + .attr( "aria-pressed", false ); } } }, |