Skip to content

Commit

Permalink
CSS: Don't workaround the IE 11 iframe-in-fullscreen sizing issues
Browse files Browse the repository at this point in the history
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.

Fixes jquerygh-3041
Refs jquerygh-1764
Refs jquerygh-2401
Refs 90d828b
  • Loading branch information
mgol committed Apr 26, 2016
1 parent f496182 commit ff1a082
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions src/css.js
Original file line number Diff line number Diff line change
Expand Up @@ -128,13 +128,6 @@ function getWidthOrHeight( elem, name, extra ) {
val = elem.getBoundingClientRect()[ name ];
}

// Support: IE 11 only
// In IE 11 fullscreen elements inside of an iframe have
// 100x too small dimensions (gh-1764).
if ( document.msFullscreenElement && window.top !== window ) {
val *= 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
Expand Down

0 comments on commit ff1a082

Please sign in to comment.