-
Notifications
You must be signed in to change notification settings - Fork 745
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
It is unclear what the compatibility is of these samples #24
Comments
Ps: I did run the babel transpilation, but that doesn't help much. |
IE11 does not support ES6 classes, so without transpiliing you might have to move to edge anyway if you want native support. I would be very cautious with moving these samples to production. The code is written with BEM in mind. BEM is awesome for prototyping, but might not be really what you might want in your production code. I think that the motivation behind these elements is more to demonstrate techniques and features supported by the next generation web, than it is to craft production-ready "elements". |
Also TouchEvents are not supported by edge. |
Right exactly what @DanielVoogsgerd said. The code is here for reference because they are live-coded, but I will definitely add a note to the README so that it's setting expectations more clearly. |
@DanielVoogsgerd @0xR @paullewis @dglazkov @sethladd For anyone willing to debug on their own lovly OS X and Safari, just add some browser polyfills:
|
I am considering of using the techniques shown in these samples, but in order to do so in production I have certain compatibility requirements.
I just tried the samples in IE9 and nothing works.
It would help if the README would show in which browsers these examples would work and whether it is possible to use the techniques shown in older browsers with some tweaking.
The text was updated successfully, but these errors were encountered: