From a79799e3b39f1468a40583ae1573f40a8929400c Mon Sep 17 00:00:00 2001
From: John Bender <john.m.bender@gmail.com>
Date: Wed, 2 Feb 2011 23:46:20 -0800
Subject: [PATCH] added step tests for keypress, removed data-step attribute
 check in favor of html5 range step attribute

---
 js/jquery.mobile.forms.slider.js   |  2 +-
 tests/unit/slider/index.html       |  4 ++++
 tests/unit/slider/slider_events.js | 18 +++++++++++++++++-
 3 files changed, 22 insertions(+), 2 deletions(-)

diff --git a/js/jquery.mobile.forms.slider.js b/js/jquery.mobile.forms.slider.js
index 9cac74b4..61fac56f 100644
--- a/js/jquery.mobile.forms.slider.js
+++ b/js/jquery.mobile.forms.slider.js
@@ -34,7 +34,7 @@ $.widget( "mobile.slider", $.mobile.widget, {
 			},
 			min = (cType == 'input') ? parseFloat(control.attr('min')) : 0,
 			max = (cType == 'input') ? parseFloat(control.attr('max')) : control.find('option').length-1,
-			step = window.parseFloat(control.attr('data-step') || 1),
+			step = window.parseFloat(control.attr('step') || 1),
 			slider = $('<div class="ui-slider '+ selectClass +' ui-btn-down-'+ trackTheme+' ui-btn-corner-all" role="application"></div>'),
 			handle = $('<a href="#" class="ui-slider-handle"></a>')
 				.appendTo(slider)
diff --git a/tests/unit/slider/index.html b/tests/unit/slider/index.html
index 02787f98..a8fc034d 100644
--- a/tests/unit/slider/index.html
+++ b/tests/unit/slider/index.html
@@ -77,6 +77,10 @@
 		<input type="range" name="slider-1" id="range-slider-end" value="15" min="0" max="100" data-theme="b" data-track-theme="a" />
 	</div>
 
+	<div data-role="fieldcontain">
+		<input type="range" name="stepped" id="stepped" value="15" min="0" max="100" step="10" data-theme="b" data-track-theme="a" />
+	</div>
+
 	<div data-role="fieldcontain">
 		<select name="slider" id="slider-switch" data-role="slider">
 			<option value="off">Off</option>
diff --git a/tests/unit/slider/slider_events.js b/tests/unit/slider/slider_events.js
index 5241f094..4069752e 100644
--- a/tests/unit/slider/slider_events.js
+++ b/tests/unit/slider/slider_events.js
@@ -19,7 +19,7 @@
 			handle.trigger('keydown');
 
 			val += opts.increment;
-			same(val, window.parseFloat(slider.val(), 10), "new value is one larger");
+			same(val, window.parseFloat(slider.val(), 10), "new value is " + opts.increment + " different");
 		});
 	};
 
@@ -62,6 +62,22 @@
 		});
 	});
 
+	test( "slider should move positive by steps on keypress", function(){
+		keypressTest({
+			selector: "#stepped",
+			keyCodes: ['RIGHT'],
+			increment: 10
+		});
+	});
+
+	test( "slider should move negative by steps on keypress", function(){
+		keypressTest({
+			selector: "#stepped",
+			keyCodes: ['LEFT'],
+			increment: -10
+		});
+	});
+
 	test( "slider should validate input value on blur", function(){
 		var slider = $("#range-slider-up");
 		slider.focus();