Skip to content

Commit 01ad6ce

Browse files
committed
Remove previous text/icon classes when updating buttons.
Fixes #5567 - button text, hide then show again.
1 parent 888725a commit 01ad6ce

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

ui/jquery.ui.button.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@
1515

1616
var lastActive,
1717
baseClasses = "ui-button ui-widget ui-state-default ui-corner-all",
18-
otherClasses = "ui-state-hover ui-state-active " +
19-
"ui-button-icons-only ui-button-icon-only ui-button-text-icons ui-button-text-icon ui-button-text-only",
18+
stateClasses = "ui-state-hover ui-state-active ",
19+
typeClasses = "ui-button-icons-only ui-button-icon-only ui-button-text-icons ui-button-text-icon ui-button-text-only",
2020
formResetHandler = function( event ) {
2121
$( ":ui-button", event.target.form ).each(function() {
2222
var inst = $( this ).data( "button" );
@@ -217,7 +217,7 @@ $.widget( "ui.button", {
217217
this.element
218218
.removeClass( "ui-helper-hidden-accessible" );
219219
this.buttonElement
220-
.removeClass( baseClasses + " " + otherClasses )
220+
.removeClass( baseClasses + " " + stateClasses + " " + typeClasses )
221221
.removeAttr( "role" )
222222
.removeAttr( "aria-pressed" )
223223
.html( this.buttonElement.find(".ui-button-text").html() );
@@ -278,7 +278,7 @@ $.widget( "ui.button", {
278278
}
279279
return;
280280
}
281-
var buttonElement = this.buttonElement,
281+
var buttonElement = this.buttonElement.removeClass( typeClasses ),
282282
buttonText = $( "<span></span>" )
283283
.addClass( "ui-button-text" )
284284
.html( this.options.label )

0 commit comments

Comments
 (0)