Remove patch for very early versions of Opera 9 that made it impossib…

…le to animate values smaller than -10000. Fixes #7193. Thanks to igorw for initial patch & test case.
jquery · Dec 30, 2010 · 4eeae8b0bc789ad525e98568cffb37b04b27ae84 · 4eeae8b
1 parent 3e0cc81
commit 4eeae8b0bc789ad525e98568cffb37b04b27ae84
Unified Split

Showing with 18 additions and 1 deletion.

+1 −1 src/effects.js

+17 −0 test/unit/effects.js
diff --git a/src/effects.js b/src/effects.js
@@ -337,7 +337,7 @@ jQuery.fx.prototype = {
 		}
 		var r = parseFloat( jQuery.css( this.elem, this.prop ) );
-  
-		return r && r > -10000 ? r : 0;
+  
+		return r || 0;
 	},
 	// Start an animation from one number to another

diff --git a/test/unit/effects.js b/test/unit/effects.js
@@ -560,6 +560,23 @@ jQuery.checkOverflowDisplay = function(){
 	start();
 }
+  
+test("support negative values < -10000 (bug #7193)", function () {
+  
+	expect(1);
+  
+	stop();
+  
+
+  
+	jQuery.extend(jQuery.fx.step, {
+  
+		"marginBottom": function(fx) {
+  
+			equals( fx.cur(), -11000, "Element has margin-bottom of -11000" );
+  
+			delete jQuery.fx.step.marginBottom;
+  
+		}
+  
+    });
+  
+
+  
+	jQuery("#main").css("marginBottom", "-11000px").animate({ marginBottom: "-11001px" }, {
+  
+		duration: 1,
+  
+		complete: start
+  
+	});
+  
+});
+  
+
 test("JS Overflow and Display", function() {
 	expect(2);
 	stop();