From 5eef5917fdb399ace2698154c4cd7bbd24f13182 Mon Sep 17 00:00:00 2001 From: rwldrn Date: Tue, 14 Jun 2011 15:59:22 -0400 Subject: [PATCH] Landing pull request 409. Adds fillOpacity to internal cssNumber. Fixes #9548. More Details: - https://github.com/jquery/jquery/pull/409 - http://bugs.jquery.com/ticket/9548 --- src/css.js | 9 +++++---- test/unit/css.js | 10 ++++++++++ 2 files changed, 15 insertions(+), 4 deletions(-) diff --git a/src/css.js b/src/css.js index fcc784d2..c60bcdde 100644 --- a/src/css.js +++ b/src/css.js @@ -50,13 +50,14 @@ jQuery.extend({ // Exclude the following css properties to add px cssNumber: { - "zIndex": true, + "fillOpacity": true, "fontWeight": true, - "opacity": true, - "zoom": true, "lineHeight": true, + "opacity": true, + "orphans": true, "widows": true, - "orphans": true + "zIndex": true, + "zoom": true }, // Add in properties whose names you wish to fix before diff --git a/test/unit/css.js b/test/unit/css.js index e631f496..f421f7bd 100644 --- a/test/unit/css.js +++ b/test/unit/css.js @@ -475,3 +475,13 @@ test("widows & orphans #8936", function () { $p.remove(); }); + +test("Do not append px to 'fill-opacity' #9548", 1, function() { + + var $div = jQuery("
").appendTo("#qunit-fixture"); + + $div.css("fill-opacity", 0).animate({ "fill-opacity": 1.0 }, 0, function () { + equal( jQuery(this).css("fill-opacity"), 1, "Do not append px to 'fill-opacity'"); + }); + +});