From 92451a02876fa845ab1f1cd558821b3551817fa2 Mon Sep 17 00:00:00 2001 From: Rick Waldron Date: Tue, 28 Jun 2011 20:15:43 -0400 Subject: [PATCH] jQuery core style guide conformance: jquery.mobile.forms.button.js --- js/jquery.mobile.forms.button.js | 74 ++++++++++++++++++-------------- 1 file changed, 41 insertions(+), 33 deletions(-) diff --git a/js/jquery.mobile.forms.button.js b/js/jquery.mobile.forms.button.js index b296884f..d6cde2c2 100644 --- a/js/jquery.mobile.forms.button.js +++ b/js/jquery.mobile.forms.button.js @@ -3,11 +3,13 @@ * Copyright (c) jQuery Project * Dual licensed under the MIT or GPL Version 2 licenses. * http://jquery.org/license -*/ -(function($, undefined ) { +*/ + +(function( $, undefined ) { + $.widget( "mobile.button", $.mobile.widget, { options: { - theme: null, + theme: null, icon: null, iconpos: null, inline: null, @@ -15,15 +17,16 @@ $.widget( "mobile.button", $.mobile.widget, { shadow: true, iconshadow: true }, - _create: function(){ + _create: function() { var $el = this.element, - o = this.options; - - //add ARIA role + o = this.options, + type; + + // Add ARIA role this.button = $( "
" ) .text( $el.text() || $el.val() ) .buttonMarkup({ - theme: o.theme, + theme: o.theme, icon: o.icon, iconpos: o.iconpos, inline: o.inline, @@ -32,43 +35,48 @@ $.widget( "mobile.button", $.mobile.widget, { iconshadow: o.iconshadow }) .insertBefore( $el ) - .append( $el.addClass('ui-btn-hidden') ); - - //add hidden input during submit - var type = $el.attr('type'); - if( type !== 'button' && type !== 'reset' ){ - $el.bind("vclick", function(){ - var $buttonPlaceholder = $("", - {type: "hidden", name: $el.attr("name"), value: $el.attr("value")}) - .insertBefore($el); - - //bind to doc to remove after submit handling - $(document).submit(function(){ + .append( $el.addClass( "ui-btn-hidden" ) ); + + // Add hidden input during submit + type = $el.attr( "type" ); + + if ( type !== "button" && type !== "reset" ) { + + $el.bind( "vclick", function() { + + var $buttonPlaceholder = $( "", { + type: "hidden", + name: $el.attr( "name" ), + value: $el.attr( "value" ) + }) + .insertBefore( $el ); + + // Bind to doc to remove after submit handling + $( document ).submit(function(){ $buttonPlaceholder.remove(); }); }); } + this.refresh(); - }, - enable: function(){ - this.element.attr("disabled", false); - this.button.removeClass("ui-disabled").attr("aria-disabled", false); - return this._setOption("disabled", false); + enable: function() { + this.element.attr( "disabled", false ); + this.button.removeClass( "ui-disabled" ).attr( "aria-disabled", false ); + return this._setOption( "disabled", false ); }, - disable: function(){ - this.element.attr("disabled", true); - this.button.addClass("ui-disabled").attr("aria-disabled", true); - return this._setOption("disabled", true); + disable: function() { + this.element.attr( "disabled", true ); + this.button.addClass( "ui-disabled" ).attr( "aria-disabled", true ); + return this._setOption( "disabled", true ); }, - refresh: function(){ - if( this.element.attr('disabled') ){ + refresh: function() { + if ( this.element.attr( "disabled" ) ) { this.disable(); - } - else{ + } else { this.enable(); } }