aboutsummaryrefslogtreecommitdiffstats
path: root/ui
diff options
context:
space:
mode:
authorScott González <scott.gonzalez@gmail.com>2010-03-11 20:10:43 +0000
committerScott González <scott.gonzalez@gmail.com>2010-03-11 20:10:43 +0000
commit057e156e626a4cfd332eaa30ef56b41b6f47d507 (patch)
tree5421879d472a21936d1550ded33cc6219f98e0d7 /ui
parentd3cb190377782b5f952d5f213101a8399f5908ff (diff)
downloadjquery-ui-057e156e626a4cfd332eaa30ef56b41b6f47d507.tar.gz
jquery-ui-057e156e626a4cfd332eaa30ef56b41b6f47d507.zip
Button: Conform to coding standards.
Diffstat (limited to 'ui')
-rw-r--r--ui/jquery.ui.button.js58
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 );
}
}
},