-
Notifications
You must be signed in to change notification settings - Fork 111
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
react-hotkeys-hook js distributions don't support IE. #212
Comments
@Wallysburg Thx for creating the issue. I will dig into that. At first glance it seems that pika pack is ignoring the target of the tsconfig, but I need to look into that. |
There is no config to output es2015 or es5 with |
Workaround - With webpack and babel this could be the solution: {
test: /\.js$/,
exclude: /node_modules(?!(\/react-hotkeys-hook))/,
} |
👍 upvote for fixing this transpiling issue, thanks |
@ypresto Yeah you are right. There is unfortunately not much I can do about. I'd have to replace the whole building and publishing system. |
@Wallysburg @reidphilpot For the time being I removed the usage of default parameters to fix this issue. Please check if 2.1.2 works for you in IE and let me know if there are any issues. |
I'm using v2.1.3 and I'm still getting webpack errors thrown in IE11. Has anyone else had better luck? |
@joshwreford can you pinpoint what is causing the error? |
@joshwreford Can you check out v2.3.0 and try it in IE11? |
Hello!
First of all, thanks for creating this useful package.
Unfortunately, default parameters aren't supported by any version of IE, so any project that is consuming this project won't be able to support IE. While I dream of a world where web devs will no longer have to support IE, this is sadly not going to be a reality for years to come.
The problem is that the current build for react-hotkeys-hook is targeting es2018 which outputs a javascript dist that contains the usage of default parameters:
Fortunately, the fix here would be to simply change the target parameter within the tsconfig.json to target es2015 instead. This would ensure that the distributed js would be supported by more antiquated browsers like IE.
The text was updated successfully, but these errors were encountered: