-
Notifications
You must be signed in to change notification settings - Fork 170
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
Remove support for IE11 #993
Conversation
Thanks for looking into this! Also, it looks like a good idea to remove the polyfills as well. I've been through the list and noted that they polyfill IE only missing feature (in other words these features were implemented circa 2015 or before) Also, what is the consequence of removing buble? Is there any kind of transpilation yet? What will be the final code compatibility target? |
Compare bundle sizes
The bubble compiler seems to exist to support ES6 syntax in IE11. Browsers that don't support ES6 are currently deprecated, so it should provide the same compatibility as before. |
@papillot Could you please review this PR? |
@HHongSeungWoo I'll make my best to finalize this in the upcoming days. |
f080ab0
to
bf69539
Compare
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.
Thanks for the improvements.
Will do a release with these changes soon.
IE11 is deprecated.
This change will save a lot of bundle size.