From eb3d95543870447d1194f6e9e7c43c6d2637a35f Mon Sep 17 00:00:00 2001 From: Giovanni Giacobbi Date: Sun, 20 Mar 2011 03:19:36 +0100 Subject: [PATCH] Restructured if/else's for better readability (cherry picked from commit 3ce8429f4fce08176c8bc06d4ff4bf8830f8c916) --- ui/jquery.ui.button.js | 18 +++++++----------- 1 file changed, 7 insertions(+), 11 deletions(-) diff --git a/ui/jquery.ui.button.js b/ui/jquery.ui.button.js index bddedead6..a98e96f46 100644 --- a/ui/jquery.ui.button.js +++ b/ui/jquery.ui.button.js @@ -182,21 +182,17 @@ $.widget( "ui.button", { }, _determineButtonType: function() { - + if ( this.element.is(":checkbox") ) { this.type = "checkbox"; + } else if ( this.element.is(":radio") ) { + this.type = "radio"; + } else if ( this.element.is("input") ) { + this.type = "input"; } else { - if ( this.element.is(":radio") ) { - this.type = "radio"; - } else { - if ( this.element.is("input") ) { - this.type = "input"; - } else { - this.type = "button"; - } - } + this.type = "button"; } - + if ( this.type === "checkbox" || this.type === "radio" ) { // we don't search against the document in case the element // is disconnected from the DOM