Fixed bug #14 and wrote a test case for it.

EmKayDK · Sep 19, 2015 · 273f28a · 273f28a
1 parent 6fb296c
commit 273f28a
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/jquery.jstepper.js b/jquery.jstepper.js
@@ -1,4 +1,4 @@
-// jStepper 1.5.2
+// jStepper 1.5.3
 
 // A jQuery plugin by EmKay usable for making a numeric textfield value easy to increase or decrease.
 
@@ -75,9 +75,11 @@
 
 				if (o.overflowMode === 'ignore') {
 
-					if (o.maxValue) {
+					var objValueToCheck = $(this).val().indexOf("-") === 0 ? o.minValue : o.maxValue;
 
-						if ($(this).val().length >= o.maxValue.toString().length) {
+					if (objValueToCheck) {
+
+						if ($(this).val().length >= objValueToCheck.toString().length) {
 
 							if (
 								((key >= 48 && key <= 57) || (key >= 96 && key <= 105)) &&

diff --git a/website/scripts/tests.js b/website/scripts/tests.js
@@ -278,7 +278,7 @@ function RunQUnit() {
 
 	test('Validating value of field', function() {
 
-		expect(3);
+		expect(4);
 
 		$('#txtQUnit').unbind();
 
@@ -295,6 +295,12 @@ function RunQUnit() {
 		$('#txtQUnit').trigger(objBlurEvent);
 		equal($('#txtQUnit').val(), '123456', 'Pasting in to many dashes');
 
+		$('#txtQUnit').unbind();
+		$('#txtQUnit').jStepper({ minValue: -1111, maxValue: 20, overflowMode: 'ignore' });
+		$('#txtQUnit').val('-1');
+		syn.type('1', $('#txtQUnit'));
+		equal($('#txtQUnit').val(), '-11', 'Writing a negative number that is too long compared to maxValue');
+
 	});
 
 	// Hiding QUnit fields after testing is complete