Skip to content

Commit fb9adb9

Browse files
committed
CSS: Don't workaround the IE 11 iframe-in-fullscreen sizing issues
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 ff1a082 Fixes jquerygh-3041 Refs jquerygh-1764 Refs jquerygh-2401 Refs 90d828b
1 parent 49f830a commit fb9adb9

File tree

1 file changed

+0
-13
lines changed

1 file changed

+0
-13
lines changed

src/css.js

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -123,19 +123,6 @@ function getWidthOrHeight( elem, name, extra ) {
123123
styles = getStyles( elem ),
124124
isBorderBox = jQuery.css( elem, "boxSizing", false, styles ) === "border-box";
125125

126-
// Support: IE11 only
127-
// In IE 11 fullscreen elements inside of an iframe have
128-
// 100x too small dimensions (gh-1764).
129-
if ( document.msFullscreenElement && window.top !== window ) {
130-
131-
// Support: IE11 only
132-
// Running getBoundingClientRect on a disconnected node
133-
// in IE throws an error.
134-
if ( elem.getClientRects().length ) {
135-
val = Math.round( elem.getBoundingClientRect()[ name ] * 100 );
136-
}
137-
}
138-
139126
// Some non-html elements return undefined for offsetWidth, so check for null/undefined
140127
// svg - https://bugzilla.mozilla.org/show_bug.cgi?id=649285
141128
// MathML - https://bugzilla.mozilla.org/show_bug.cgi?id=491668

0 commit comments

Comments
 (0)