-
-
Notifications
You must be signed in to change notification settings - Fork 4.7k
Fix main bundle on IE11 #21725
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
Fix main bundle on IE11 #21725
Conversation
Signed-off-by: Julius Härtl <[email protected]>
Signed-off-by: Julius Härtl <[email protected]>
|
/backport 5aef97c to stable19 |
rullzer
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🐘 ❤️
does that work? |
nickvergessen
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Confirmed it fixes the login on IE11
MorrisJobke
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Checke the output and it looks good 👍
Fixes #21316
Make sure that properties are always quoted on production builds during terser optimization as unicode properties from our js dependencies are otherwise breaking the main bundle on IE11.