Landing tweak from 'haruka' that fixes non-pixel fontSize values in IE. Fixes #760.
authorJohn Resig <jeresig@gmail.com>
Wed, 11 Nov 2009 19:17:16 +0000 (14:17 -0500)
committerJohn Resig <jeresig@gmail.com>
Wed, 11 Nov 2009 19:17:16 +0000 (14:17 -0500)
src/css.js
test/data/testsuite.css
test/unit/css.js

index 6abcc95..ddd42b4 100644 (file)
@@ -167,7 +167,7 @@ jQuery.extend({
 
                                // Put in the new values to get a computed value out
                                elem.runtimeStyle.left = elem.currentStyle.left;
-                               style.left = ret || 0;
+                               style.left = camelCase === "fontSize" ? "1em" : (ret || 0);
                                ret = style.pixelLeft + "px";
 
                                // Revert the changed values
@@ -212,4 +212,4 @@ if ( jQuery.expr && jQuery.expr.filters ) {
        jQuery.expr.filters.visible = function(elem){
                return !jQuery.expr.filters.hidden(elem);
        };
-}
\ No newline at end of file
+}
index 7a16243..5cc7828 100644 (file)
@@ -103,3 +103,7 @@ div.chain.out { background: green; }
 div.chain.out div { background: red; display: none; }
 
 div#show-tests * { display: none; }
+
+#nothiddendiv { font-size: 16px; }
+#nothiddendivchild.em { font-size: 2em; }
+#nothiddendivchild.prct { font-size: 150%; }
index 649db59..beac04a 100644 (file)
@@ -1,7 +1,7 @@
 module("css");
 
 test("css(String|Hash)", function() {
-       expect(23);
+       expect(27);
 
        equals( jQuery('#main').css("display"), 'none', 'Check for css property "display"');
 
@@ -38,6 +38,17 @@ test("css(String|Hash)", function() {
        equals( jQuery('#empty').css('opacity'), '0', "Assert opacity is accessible via filter property set in stylesheet in IE" );
        jQuery('#empty').css({ opacity: '1' });
        equals( jQuery('#empty').css('opacity'), '1', "Assert opacity is taken from style attribute when set vs stylesheet in IE with filters" );
+
+       var div = jQuery('#nothiddendiv'), child = jQuery('#nothiddendivchild');
+
+       equals( parseInt(div.css("fontSize")), 16, "Verify fontSize px set." );
+       equals( parseInt(child.css("fontSize")), 16, "Verify fontSize px set." );
+
+       child.attr("class", "em");
+       equals( parseInt(child.css("fontSize")), 32, "Verify fontSize em set." );
+
+       child.attr("class", "prct");
+       equals( parseInt(child.css("fontSize")), 24, "Verify fontSize % set." );
 });
 
 test("css(String, Object)", function() {