-
Notifications
You must be signed in to change notification settings - Fork 147
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
ES6 Symbol breaks < IE Edge #41
Comments
Oh yeah. I polyfill Symbol, normally, but you're right, I probably shouldn't assume it's existence here. This code replaces symbols with strings (via .toString when setting the symbol property). It's not a polyfill, but it will get things working in IE:
|
Works like a charm! Thanks! |
@stephband i'm still getting this problem and my app is not running on IE11. could you release a solution as a commit? i am using jquery.event.move 2.0.0 and jquery 3.3.1. |
Adding this effectively disables vertical scroll in both Internet Explorer 11 Mobile and Edge Mobile (that is, the complete Windows Phone and Windows Mobile device family). I'll never understand that obsession with the implementation of ES 2015 (and newer) functions with the only goal of rendering scripts unusable in older browsers. Please remember that not everyone can acquire a 1300 USD iPhone device to use the methods from ECMA Script 2020... it's the other way around: scripts must work in older devices whenever possible. Do not take this as a personal attack to the plugin developers, this is just a suggestion. |
Where to place that code? |
Hi Stephen,
great work on this project. I just noticed the usage of ES6 Symbols in the source: https://github.com/stephband/jquery.event.move/blob/master/js/jquery.event.move.js#L83
As you might know no IE version except Edge supports this. Any chance on some more IE support or should I just use an ES6-shim?
Cheers!
Arno
The text was updated successfully, but these errors were encountered: