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

Framework: Use @babel/runtime universally #9641

Closed
wants to merge 1 commit into from

Conversation

aduth
Copy link
Member

@aduth aduth commented Sep 5, 2018

Fixes #9640
Related: #8580, #8586
Alternative to: #9643

This pull request seeks to remove usage of @babel/runtime-corejs2 in an effort to resolve issues with loading the editor in Internet Explorer. This work is largely speculative, though is verified as a fix to the issue described in #9640. I'm not entirely clear on the context provided in #8580 and #8586, but as best I can tell the use of @babel/runtime-corejs2 was specific to how packages were built for npm consumption. Since, as far as I'm aware, this is no longer the case as of #9171, there should not be expected a regression on #8580.

Testing instructions:

Repeat Steps to Reproduce described in #9640 both in Internet Explorer and your preferred browser, verifying that the editor loads as expected.

@aduth aduth added [Type] Bug An existing feature does not function as intended Browser Issues Issues or PRs that are related to browser specific problems labels Sep 5, 2018
@aduth aduth added this to the 3.8 milestone Sep 5, 2018
@gziolo
Copy link
Member

gziolo commented Sep 6, 2018

Closing this one in favor of #9643.

@gziolo gziolo closed this Sep 6, 2018
@gziolo gziolo deleted the fix/ie11-babel-runtime-corejs branch September 6, 2018 07:39
@gziolo gziolo removed this from the 3.8 milestone Sep 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Browser Issues Issues or PRs that are related to browser specific problems [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Editor: Editor won't load in Internet Explorer
2 participants