Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Preserve script load/execution order #48465

Merged
merged 2 commits into from
Oct 17, 2019

Conversation

tylersmalley
Copy link
Contributor

We need to either disable async loading, or defer execution. In IE11, we're running into an issue where the logout page is executing before the vendors file.

Fixes #46943

current defer async=false
Load 590 ms 518 ms 490 ms
Finish 7.89 s 7.85 s 7.87 s

The above was tested averaging out five requests. Cache was disabled using a 40 ms, 28 Mb/s down, 11 Mb/s up network throttling profile.

We need to either disable async loading, or defer exection. In IE11,
we're running into an issue where the logout page is executing before
the vendors file.

Signed-off-by: Tyler Smalley <[email protected]>
@tylersmalley tylersmalley added release_note:skip Skip the PR/issue when compiling release notes Team:Operations Team label for Operations Team v7.5.0 v7.6.0 v8.0.0 labels Oct 16, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@LeeDr
Copy link
Contributor

LeeDr commented Oct 16, 2019

I patched my local 7.5.0 snapshot build with the change from this PR and it worked. I logged in and out on IE11 5 times with no problem.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Member

@jbudz jbudz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works for me.

@joshdover did you have any concerns here?

Signed-off-by: Tyler Smalley <[email protected]>
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@tylersmalley tylersmalley merged commit 57d5afb into elastic:master Oct 17, 2019
tylersmalley pushed a commit to tylersmalley/kibana that referenced this pull request Oct 17, 2019
We need to either disable async loading, or defer exection. In IE11,
we're running into an issue where the logout page is executing before
the vendors file.

Signed-off-by: Tyler Smalley <[email protected]>
tylersmalley pushed a commit to tylersmalley/kibana that referenced this pull request Oct 17, 2019
We need to either disable async loading, or defer exection. In IE11,
we're running into an issue where the logout page is executing before
the vendors file.

Signed-off-by: Tyler Smalley <[email protected]>
tylersmalley pushed a commit to tylersmalley/kibana that referenced this pull request Oct 17, 2019
We need to either disable async loading, or defer exection. In IE11,
we're running into an issue where the logout page is executing before
the vendors file.

Signed-off-by: Tyler Smalley <[email protected]>
tylersmalley pushed a commit that referenced this pull request Oct 17, 2019
We need to either disable async loading, or defer exection. In IE11,
we're running into an issue where the logout page is executing before
the vendors file.

Signed-off-by: Tyler Smalley <[email protected]>
tylersmalley pushed a commit that referenced this pull request Oct 17, 2019
We need to either disable async loading, or defer exection. In IE11,
we're running into an issue where the logout page is executing before
the vendors file.

Signed-off-by: Tyler Smalley <[email protected]>
tylersmalley pushed a commit that referenced this pull request Oct 17, 2019
We need to either disable async loading, or defer exection. In IE11,
we're running into an issue where the logout page is executing before
the vendors file.

Signed-off-by: Tyler Smalley <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes Team:Operations Team label for Operations Team v7.4.1 v7.5.0 v7.6.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Kibana breaks on IE11 on logout, 'vendors' is undefined
5 participants