Skip to content
Permalink
Browse files

CSS: Return values should be numbers

Fixes #14792
  • Loading branch information...
dmethvin committed Mar 4, 2014
1 parent 06adf7c commit f4b37d89820535d6c7503925aa8872645681a865
Showing with 4 additions and 3 deletions.
  1. +2 −2 src/css.js
  2. +2 −1 test/unit/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" ];
@@ -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("<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

0 comments on commit f4b37d8

Please sign in to comment.
You can’t perform that action at this time.