Skip to content
Browse files

[checkboxradio] Use .buttonMarkup() to update icon instead of class m…

…anipulation
  • Loading branch information...
1 parent 2e2d3dc commit bb49223cd009ed743ea5e7882ff6d04130676efb @gabrielschulhof gabrielschulhof committed
Showing with 5 additions and 8 deletions.
  1. +5 −8 js/widgets/forms/checkboxradio.js
View
13 js/widgets/forms/checkboxradio.js
@@ -37,8 +37,8 @@ $.widget( "mobile.checkboxradio", $.mobile.widget, {
activeBtn = icon ? "" : " " + $.mobile.activeBtnClass,
checkedClass = "ui-" + checkedState + activeBtn,
uncheckedClass = "ui-" + uncheckedState,
- checkedicon = "ui-icon-" + checkedState,
- uncheckedicon = "ui-icon-" + uncheckedState;
+ checkedicon = checkedState,
+ uncheckedicon = uncheckedState;
if ( inputtype !== "checkbox" && inputtype !== "radio" ) {
return;
@@ -179,15 +179,12 @@ $.widget( "mobile.checkboxradio", $.mobile.widget, {
refresh: function() {
var input = this.element[0],
- label = this.label,
- icon = label.find( ".ui-icon" );
+ label = this.label;
if ( input.checked ) {
- label.addClass( this.checkedClass ).removeClass( this.uncheckedClass );
- icon.addClass( this.checkedicon ).removeClass( this.uncheckedicon );
+ label.addClass( this.checkedClass ).removeClass( this.uncheckedClass ).buttonMarkup( { icon: this.checkedicon } );
} else {
- label.removeClass( this.checkedClass ).addClass( this.uncheckedClass );
- icon.removeClass( this.checkedicon ).addClass( this.uncheckedicon );
+ label.removeClass( this.checkedClass ).addClass( this.uncheckedClass ).buttonMarkup( { icon: this.uncheckedicon } );;
}
if ( input.disabled ) {

0 comments on commit bb49223

Please sign in to comment.
Something went wrong with that request. Please try again.