]> source.dussan.org Git - jquery.git/commitdiff
CSS: Don't workaround the IE 11 iframe-in-fullscreen sizing issues
authorMichał Gołębiowski <m.goleb@gmail.com>
Tue, 26 Apr 2016 19:25:30 +0000 (21:25 +0200)
committerMichał Gołębiowski <m.goleb@gmail.com>
Tue, 26 Apr 2016 19:25:30 +0000 (21:25 +0200)
IE 11 used to have an issue where if an element inside an iframe was put
in fullscreen mode, the element dimensions started being 100 times too small;
we've added a workaround that would multiply them by 100. However, the IE 11
issue has been unexpectedly fixed and since our detection was really detecting
the browser and not a bug, we've started breaking the browser instead of fixing
it.

Since there's no good way to detect if the bug exists, we have to back the
workaround out completely.

Refs ff1a0822f72d2b39fac691dfcceab6ede5623b90
Fixes gh-3041
Refs gh-1764
Refs gh-2401
Refs 90d828bad0d6d318d73d6cf6209d9dc7ac13878c

src/css.js

index ec55a24115a9ee8181459f33873f35b46370cc6b..bd131399a3afc8eb24e1fbda9a9500277fc78f34 100644 (file)
@@ -123,19 +123,6 @@ function getWidthOrHeight( elem, name, extra ) {
                styles = getStyles( elem ),
                isBorderBox = jQuery.css( elem, "boxSizing", false, styles ) === "border-box";
 
-       // Support: IE11 only
-       // In IE 11 fullscreen elements inside of an iframe have
-       // 100x too small dimensions (gh-1764).
-       if ( document.msFullscreenElement && window.top !== window ) {
-
-               // Support: IE11 only
-               // Running getBoundingClientRect on a disconnected node
-               // in IE throws an error.
-               if ( elem.getClientRects().length ) {
-                       val = Math.round( elem.getBoundingClientRect()[ name ] * 100 );
-               }
-       }
-
        // Some non-html elements return undefined for offsetWidth, so check for null/undefined
        // svg - https://bugzilla.mozilla.org/show_bug.cgi?id=649285
        // MathML - https://bugzilla.mozilla.org/show_bug.cgi?id=491668