Ticket #760 (closed bug: fixed)
.css('fontSize') returns different value in Internet Explorer
| Reported by: | kswedberg | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | 1.4 |
| Component: | core | Version: | 1.3.2 |
| Keywords: | css, style IE | Cc: | |
| Blocking: | Blocked by: |
Description (last modified by john) (diff)
The .css() method returns a different font-size value for Firefox and Internet Explorer if the stylesheet has set the size with any unit other than pixel.
As Dave Methvin explains*, this has something to do with Firefox and other browsers using getComputedStyle (which computes the size in px) while IE uses currentStyle (which gets the size from the style rule).
The difference can be problematic in some cases.
Attachments
Change History
comment:2 Changed 6 years ago by lukelutman
IE returns values for all properties (not just font-size) in whatever units were specified in the stylesheet (em, in, cm, etc...).
I've been working on a hack/fix that boils down to:
- insert a temporary element into the dom as a sibling of the target element
- set the temp element's font-size to match the target element's font-size
- set the temp element's width to the non-px value
- measure the temp element using offsetWidth (which is always in pixels)
Here's an example:
(function($){
var tmp = $('<span style="border: 0; display: block; padding: 0; position: absolute; overflow: hidden;" />')[0];
$.px = function(elem, val) {
if(elem && elem.parentNode && /(cm|em|in|pt)$/.test(val)) {
var style = (document.defaultView && document.defaultView.getComputedStyle && document.defaultView.getComputedStyle(elem, null)) || elem.currentStyle;
tmp.style.fontSize = (style.getPropertyValue && style.getPropertyValue('font-size')) || style.fontSize;
tmp.style.width = val;
elem.parentNode.insertBefore(tmp, elem);
val = tmp.offsetWidth + 'px';
tmp.parentNode.removeChild(tmp);
return val;
};
})(jQuery);
Which can be used like so:
var elem = $('p')[0];
var value = $(elem).css('font-size');
// px returns undefined if the value can't be converted, so default to the original value.
var fixed = $.px(elem, value) || value;
comment:3 Changed 6 years ago by john
- need set to Test Case
- Version set to 1.1.2
- Milestone set to 1.1.3
comment:4 Changed 6 years ago by john
- Status changed from new to closed
- Version changed from 1.1.2 to 1.1.4
- Resolution set to fixed
- Description modified (diff)
- Milestone changed from 1.1.3 to 1.2
Fixed in SVN rev [3132].
comment:5 Changed 6 years ago by gudoy
- Status changed from closed to reopened
- Resolution fixed deleted
It seems that the bug reappeared in 1.2.1 release.
comment:6 Changed 6 years ago by bruno1378
Yes, I am seeing this using the latest release of jQuery (1.2.1).
var fontSize = $('body').css('font-size');
In IE it's returning 795px.
My CSS on the body is
font: 68.75%/1.366 Verdana, Tahoma, Arial, Helvetica, sans-serif;
comment:7 Changed 6 years ago by bruno1378
actually i've noticed
.css('font-size')
works different than
.css('fontSize')
which returns the same cross-browser...just not the computed style. This works fine for my application at the moment...
comment:8 Changed 5 years ago by kswedberg
Hmm. I'm not seeing any difference between .css('fontSize') and .css('font-size') using 1.2.6.
Looks like the problem now might have to do with the value being computed up the DOM tree. When font size is set to a percentage, it's affected by the window width.
See example here (jQuery should say "18px" for all of them): http://test.learningjquery.com/font-size.html
comment:11 in reply to: ↑ description ; follow-ups: ↓ 12 ↓ 13 Changed 4 years ago by binarypusher
Doing a bit of testing reveals the discrepancy has an approximated formula for the returned ie value (based on a em value from css):
n2 x 12 = ie value in pixels.
so css: 1.0em, firefox: 12px, ie:12px css: 2.0em, firefox: 24px, ie:48px css: 3.0em, firefox: 36px, ie:108px
to fetch an approximated px value in ie:
square root of (the iepx value / 12)
so parseint(Math.sqrt($(obj).css("font-size")/12),10);
comment:12 in reply to: ↑ 11 Changed 4 years ago by binarypusher
n2 x 12 = ie value in pixels.
hmm the page didn't like my carat. it's n(to the power of)2 x 12
comment:13 in reply to: ↑ 11 Changed 4 years ago by binarypusher
Replying to binarypusher:
to fetch an approximated px value in ie: parseint(Math.sqrt($(obj).css("font-size")/12),10);
should have checked this before hand. sorry for the multiposts ... can't see how to edit an entry. here's the working version:
(Math.sqrt(parseInt($(this).css("font-size"),10).toFixed(2)/12)*12).toFixed(1);
Changed 4 years ago by haruka
-
attachment
jquery-css-fontsize-ie.patch
added
patch: fix css("font-size") in IE
comment:14 Changed 4 years ago by haruka
In IE, when font-size is set to "2.0em" in CSS, you get '2.0 * 2.0 em' size in pixels with css("font-size"). Because the size is calculated "2.0em" relational size of "the element's font size (..is 2.0em)".
And when font-size is set to "80%" is CSS, css("font-size") is calculated "80%" relational size of "the element's width".
The patch (jquery-css-fontsize-ie.patch) will fix it. after patched, curCSS method always calculates "1em" of "the element's font size" when css("font-size") called.
Changed 4 years ago by haruka
-
attachment
jquery-css-fontsize-ie-proof.html
added
this is a proof of the patch's concept.
comment:15 Changed 4 years ago by trixi
The fix by haruka orks well. I´m using a similiar approach to fix this odd behaviour in my clip-Animation-Plugin ( http://plugins.jquery.com/files/clip.js.txt). But I would suggest, that you use a 100em base instead of 1em, to get information about "subpixels". getComputedStyle also returns floating point pixel-values and its important, if you want to multiply the fontsize with another value.
If you are fixing this, it would be great, if you also fix backgroundPosition and clip.
comment:16 Changed 4 years ago by john
- Status changed from reopened to closed
- Version changed from 1.1.4 to 1.3.2
- Resolution set to fixed
- Milestone changed from 1.2 to 1.4
Landed fix for this: http://github.com/jquery/jquery/commit/449e099b97d823ed0252d8821880bc0e471701ea
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
