From f4b37d89820535d6c7503925aa8872645681a865 Mon Sep 17 00:00:00 2001 From: Dave Methvin Date: Mon, 3 Mar 2014 20:55:30 -0500 Subject: CSS: Return values should be numbers Fixes #14792 --- src/css.js | 4 ++-- test/unit/css.js | 3 ++- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/src/css.js b/src/css.js index 30342a230..4d6838a4e 100644 --- a/src/css.js +++ b/src/css.js @@ -29,8 +29,8 @@ var cssShow = { position: "absolute", visibility: "hidden", display: "block" }, cssNormalTransform = { - letterSpacing: 0, - fontWeight: 400 + letterSpacing: "0", + fontWeight: "400" }, cssPrefixes = [ "Webkit", "O", "Moz", "ms" ]; diff --git a/test/unit/css.js b/test/unit/css.js index 67bd332a4..09839f185 100644 --- a/test/unit/css.js +++ b/test/unit/css.js @@ -813,12 +813,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("
test
").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 -- cgit v1.2.3