-
Notifications
You must be signed in to change notification settings - Fork 4
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
Revisit polyfilling #62
Comments
I wonder if it would be possible to ship polyfilled and non-polyfilled versions. 🤔 |
note: Look into babel-preset-env, which is supposedly able to auto-polyfill source code by injecting require statements where needed. |
Still no easy solution for this (w3ctag/polyfills#6) but shipping a polyfilled / |
UMD support will probably need to be dropped, as automatic polyfilling causes some weird issues with it. (cjs, esm are both fine) |
It might worth reconsidering the polyfilling issue.
Originally I shipped all necessary polyfills to make it easier to adapt the library, but that comes with some cons:
Using something like polyfill.io, or just a separate polyfill js file made by the app author should be an easy enough thing to do, since I'm already assuming a lot of knowledge with these modules anyway.
Anyway, needs some more research into polyfilling best practices.
The text was updated successfully, but these errors were encountered: