Spinner: Update markup for buttons

This commit is contained in:
Alexander Schmitz 2015-08-14 10:16:05 -04:00
parent eae5671dc5
commit 739e71911e

View File

@ -230,12 +230,8 @@ $.widget( "ui.spinner", {
// Add buttons // Add buttons
.append( .append(
"<a>" + "<a></a>" +
"<span>&#9650;</span>" + "<a></a>"
"</a>" +
"<a>" +
"<span>&#9660;</span>" +
"</a>"
); );
}, },
@ -250,6 +246,7 @@ $.widget( "ui.spinner", {
// Button bindings // Button bindings
this.buttons = this.uiSpinner.children( "a" ) this.buttons = this.uiSpinner.children( "a" )
.attr( "tabIndex", -1 ) .attr( "tabIndex", -1 )
.attr( "aria-hidden", true )
.button( { .button( {
classes: { classes: {
"ui-button": "" "ui-button": ""
@ -261,8 +258,14 @@ $.widget( "ui.spinner", {
this._addClass( this.buttons.first(), "ui-spinner-button ui-spinner-up" ); this._addClass( this.buttons.first(), "ui-spinner-button ui-spinner-up" );
this._addClass( this.buttons.last(), "ui-spinner-button ui-spinner-down" ); this._addClass( this.buttons.last(), "ui-spinner-button ui-spinner-down" );
this.buttons.first().button( "option", "icon", this.options.icons.up ); this.buttons.first().button( {
this.buttons.last().button( "option", "icon", this.options.icons.down ); "icon": this.options.icons.up,
"showLabel": false
} );
this.buttons.last().button({
"icon": this.options.icons.down,
"showLabel": false
} );
// IE 6 doesn't understand height: 50% for the buttons // IE 6 doesn't understand height: 50% for the buttons
// unless the wrapper has an explicit height // unless the wrapper has an explicit height
@ -562,12 +565,8 @@ if ( $.uiBackCompat !== false ) {
_buttonHtml: function() { _buttonHtml: function() {
return "" + return "" +
"<a>" + "<a></a>" +
"<span>&#9650;</span>" + "<a></a>";
"</a>" +
"<a>" +
"<span>&#9660;</span>" +
"</a>";
} }
} ); } );
} }