Fix regression for "-200px" in animations; closes gh-822.

Also removes fake IIFEs ffrom effects.

diff --git a/src/effects.js b/src/effects.js
index 7aae8a414508a396a04b6af28fef42e223e74428..f5c1f8d47166e843d1e6128b4c97e89d9d817139 100644 (file)
--- a/src/effects.js
+++ b/src/effects.js
@@ -1,8 +1,6 @@
-(function( jQuery ) {
-
 var fxNow, timerId,
        rfxtypes = /^(?:toggle|show|hide)$/,
-       rfxnum = /^([\-+]=)?((?:\d*\.)?\d+)([a-z%]*)$/i,
+       rfxnum = /^([+\-]=)?([\d+.\-]+)([a-z%]*)$/i,
        rrun = /queueHooks$/,
        animationPrefilters = [ defaultPrefilter ],
        tweeners = {
@@ -633,5 +631,3 @@ if ( jQuery.expr && jQuery.expr.filters ) {
                }).length;
        };
 }
-
-})( jQuery );

diff --git a/test/unit/effects.js b/test/unit/effects.js
index 48793e234e30f35b2e53eb4d03ea0caed0e7aede..729ba29bc2e38132a2ab465faf0fd383cea09e5d 100644 (file)
--- a/test/unit/effects.js
+++ b/test/unit/effects.js
@@ -197,6 +197,15 @@ test("animate negative margin", function() {
        });
 });
 
+test("animate negative margin with px", function() {
+       expect(1);
+       stop();
+       jQuery("#foo").animate({ marginTop: "-100px" }, 100, function() {
+               equal( jQuery(this).css("marginTop"), "-100px", "Verify margin." );
+               start();
+       });
+});
+
 test("animate negative padding", function() {
        expect(1);
        stop();