Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Spinner: Remove ui-state- classes from spinner, along with the backgr…
…ound:none TR override. Fixes #8654 - Spinner background-color
  • Loading branch information
jzaefferer committed Oct 14, 2012
1 parent 025266e commit 112185e
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 5 deletions.
1 change: 0 additions & 1 deletion themes/base/jquery.ui.spinner.css
Expand Up @@ -17,7 +17,6 @@
.ui-spinner-down { bottom: 0; }

/* TR overrides */
span.ui-spinner { background: none; }
.ui-spinner .ui-icon-triangle-1-s {
/* need to fix icons sprite */
background-position:-65px -16px;
Expand Down
5 changes: 1 addition & 4 deletions ui/jquery.ui.spinner.js
Expand Up @@ -94,7 +94,6 @@ $.widget( "ui.spinner", {
},
keyup: "_stop",
focus: function() {
this.uiSpinner.addClass( "ui-state-active" );
this.previous = this.element.val();
},
blur: function( event ) {
Expand All @@ -104,7 +103,6 @@ $.widget( "ui.spinner", {
}

this._refresh();
this.uiSpinner.removeClass( "ui-state-active" );
if ( this.previous !== this.element.val() ) {
this._trigger( "change", event );
}
Expand Down Expand Up @@ -196,7 +194,6 @@ $.widget( "ui.spinner", {
.parent()
// add buttons
.append( this._buttonHtml() );
this._hoverable( uiSpinner );

this.element.attr( "role", "spinbutton" );

Expand Down Expand Up @@ -242,7 +239,7 @@ $.widget( "ui.spinner", {
},

_uiSpinnerHtml: function() {
return "<span class='ui-spinner ui-state-default ui-widget ui-widget-content ui-corner-all'></span>";
return "<span class='ui-spinner ui-widget ui-widget-content ui-corner-all'></span>";
},

_buttonHtml: function() {
Expand Down

0 comments on commit 112185e

Please sign in to comment.