From 21bf01acda2185922e9d5c05c51574e50409878a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Scott=20Gonz=C3=A1lez?= <scott.gonzalez@gmail.com>
Date: Tue, 6 Sep 2016 14:24:36 -0400
Subject: [PATCH] Slider: Apply `tabIndex` to custom handles

---
 tests/unit/slider/core.js     | 12 ++++++++++++
 tests/unit/slider/slider.html |  4 ++--
 ui/widgets/slider.js          |  4 +++-
 3 files changed, 17 insertions(+), 3 deletions(-)

diff --git a/tests/unit/slider/core.js b/tests/unit/slider/core.js
index 57affb0d1..8031367f0 100644
--- a/tests/unit/slider/core.js
+++ b/tests/unit/slider/core.js
@@ -25,6 +25,18 @@ QUnit.test( "markup structure", function( assert ) {
 	assert.hasClasses( handle[ 1 ], "ui-slider-handle" );
 } );
 
+QUnit.test( "custom handle", function( assert ) {
+	assert.expect( 3 );
+
+	var element = $( "#slider-custom-handle" ).slider();
+	var customHandle = $( ".custom-handle" );
+	var sliderHandles = element.find( ".ui-slider-handle" );
+
+	assert.equal( sliderHandles.length, 1, "Only one handle" );
+	assert.strictEqual( sliderHandles[ 0 ], customHandle[ 0 ], "Correct handle" );
+	assert.equal( customHandle.attr( "tabIndex" ), 0, "tabIndex" );
+} );
+
 QUnit.test( "keydown HOME on handle sets value to min", function( assert ) {
 	assert.expect( 2 );
 	element = $( "<div></div>" );
diff --git a/tests/unit/slider/slider.html b/tests/unit/slider/slider.html
index 39bcd05c3..2afbeafde 100644
--- a/tests/unit/slider/slider.html
+++ b/tests/unit/slider/slider.html
@@ -25,8 +25,8 @@
 
 <div id="slider1"></div>
 <div id="slider2"></div>
-<div id="slider3" style="position: relative; margin: 40px; width: 217px; height: 28px;">
-	<div class="ui-slider-handle" style="position: absolute; height: 21px; left: 0px; bottom: 0px; width: 17px;"></div>
+<div id="slider-custom-handle">
+	<div class="ui-slider-handle custom-handle"></div>
 </div>
 
 </div>
diff --git a/ui/widgets/slider.js b/ui/widgets/slider.js
index 4c1bdc05e..8b0f907f5 100644
--- a/ui/widgets/slider.js
+++ b/ui/widgets/slider.js
@@ -117,7 +117,9 @@ return $.widget( "ui.slider", $.ui.mouse, {
 		this.handle = this.handles.eq( 0 );
 
 		this.handles.each( function( i ) {
-			$( this ).data( "ui-slider-handle-index", i );
+			$( this )
+				.data( "ui-slider-handle-index", i )
+				.attr( "tabIndex", 0 );
 		} );
 	},