Skip to content

Fix issue with seamless iframe support#2

Merged
jeremyabbott merged 1 commit intomasterfrom
iframeFix
Jan 15, 2016
Merged

Fix issue with seamless iframe support#2
jeremyabbott merged 1 commit intomasterfrom
iframeFix

Conversation

@erikaleslie
Copy link

Updated code for supporting seamless iframes to check if the iframe is
actually seamless before applying the logic. When the iframe is not
seamless, the logic should be the same as when the page is not in an
iframe at all.

These changes assume that seamless iframes will have scrolling="no" or
the HTML5 seamless attribute.

I also fixed some of the tabs and spaces in the file.

@erikaleslie erikaleslie changed the title Fix issue with seamless iframe support in non-seamless iframes Fix issue with seamless iframe support Jan 11, 2016
Updated code for supporting seamless iframes to check if the iframe is
actually seamless before applying the logic. When the iframe is not
seamless, the logic should be the same as when the page is not in an
iframe at all.

These changes assume that seamless iframes will have scrolling="no" or
the HTML5 seamless attribute.
jeremyabbott added a commit that referenced this pull request Jan 15, 2016
Fix issue with seamless iframe support
@jeremyabbott jeremyabbott merged commit 1cb98a2 into master Jan 15, 2016
@erikaleslie erikaleslie deleted the iframeFix branch January 18, 2016 04:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants