Ie8iframe #215

Merged
merged 3 commits into from Sep 10, 2014

Conversation

Projects
None yet
2 participants
Contributor

davegurnell commented Mar 7, 2014

Fixes #214 - iframe transport under IE8.

davegurnell added some commits Mar 7, 2014

Start fixing #214 - workaround for bug in IE8 iframe onload implement…
…ation

IE8 doesn't honour onload listeners attached using Javascript. See:

    http://stackoverflow.com/questions/12910534/why-doesnt-ie8-handle-iframe-onload-events

This commit attaches the handler using HTML instead.

A further commit will be needed to reorder the code to avoid a race condition
where the iframe is initialized before the handler is setup.
Finish fixing #214 - workaround for bug in IE8 iframe onload implemen…
…tation

Reorder the code that sets up the iframe onload handler to ensure
the handler is defined before the iframe is added to the window.

@RubaXa RubaXa merged commit dcf042c into mailru:master Sep 10, 2014

RubaXa added a commit that referenced this pull request Sep 10, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment