]> source.dussan.org Git - jquery.git/commitdiff
CSS: Return values should be numbers
authorDave Methvin <dave.methvin@gmail.com>
Tue, 4 Mar 2014 01:55:30 +0000 (20:55 -0500)
committerDave Methvin <dave.methvin@gmail.com>
Wed, 5 Mar 2014 02:50:42 +0000 (21:50 -0500)
Fixes #14792
(cherry picked from commit f4b37d89820535d6c7503925aa8872645681a865)

src/css.js
test/unit/css.js

index 0f90f0b6a0ad9aa2d0d45946b2fbb827c1927c92..4b1f9144085c3f7adc5d832b3ffaf44ae4027fdc 100644 (file)
@@ -32,8 +32,8 @@ var
 
        cssShow = { position: "absolute", visibility: "hidden", display: "block" },
        cssNormalTransform = {
-               letterSpacing: 0,
-               fontWeight: 400
+               letterSpacing: "0",
+               fontWeight: "400"
        },
 
        cssPrefixes = [ "Webkit", "O", "Moz", "ms" ];
index 2209f1a40ea04efc9ba89fdc33cfb327c883830e..e084ebf60016d659e233b511a721712f89f03c11 100644 (file)
@@ -871,12 +871,13 @@ testIframeWithCallback( "css('width') should work correctly before document read
 );
 
 test("certain css values of 'normal' should be convertable to a number, see #8627", function() {
-       expect ( 2 );
+       expect ( 3 );
 
        var el = jQuery("<div style='letter-spacing:normal;font-weight:normal;'>test</div>").appendTo("#qunit-fixture");
 
        ok( jQuery.isNumeric( parseFloat( el.css("letterSpacing") ) ), "css('letterSpacing') not convertable to number, see #8627" );
        ok( jQuery.isNumeric( parseFloat( el.css("fontWeight") ) ), "css('fontWeight') not convertable to number, see #8627" );
+       equal( typeof el.css( "fontWeight" ), "string", ".css() returns a string" );
 });
 
 // only run this test in IE9