From 739e71911e262d2e653216e5d33fc486c39f59ec Mon Sep 17 00:00:00 2001 From: Alexander Schmitz Date: Fri, 14 Aug 2015 10:16:05 -0400 Subject: [PATCH] Spinner: Update markup for buttons --- ui/widgets/spinner.js | 27 +++++++++++++-------------- 1 file changed, 13 insertions(+), 14 deletions(-) diff --git a/ui/widgets/spinner.js b/ui/widgets/spinner.js index 3a0b6deef..1c7e9b364 100644 --- a/ui/widgets/spinner.js +++ b/ui/widgets/spinner.js @@ -230,12 +230,8 @@ $.widget( "ui.spinner", { // Add buttons .append( - "" + - "" + - "" + - "" + - "" + - "" + "" + + "" ); }, @@ -250,6 +246,7 @@ $.widget( "ui.spinner", { // Button bindings this.buttons = this.uiSpinner.children( "a" ) .attr( "tabIndex", -1 ) + .attr( "aria-hidden", true ) .button( { classes: { "ui-button": "" @@ -261,8 +258,14 @@ $.widget( "ui.spinner", { this._addClass( this.buttons.first(), "ui-spinner-button ui-spinner-up" ); this._addClass( this.buttons.last(), "ui-spinner-button ui-spinner-down" ); - this.buttons.first().button( "option", "icon", this.options.icons.up ); - this.buttons.last().button( "option", "icon", this.options.icons.down ); + this.buttons.first().button( { + "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 // unless the wrapper has an explicit height @@ -562,12 +565,8 @@ if ( $.uiBackCompat !== false ) { _buttonHtml: function() { return "" + - "" + - "" + - "" + - "" + - "" + - ""; + "" + + ""; } } ); } -- 2.39.5