Skip to content
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

Another polyfill to add for older browsers #34

Closed
brentrobbins opened this issue Jan 28, 2020 · 2 comments · Fixed by #36
Closed

Another polyfill to add for older browsers #34

brentrobbins opened this issue Jan 28, 2020 · 2 comments · Fixed by #36

Comments

@brentrobbins
Copy link

Thanks for this create library. It works great!

I am using it for a project that requires IE11 support, so following your note I added the Intersection Observer polyfill. That did fix the Intersection Observer, however IE11 was also triggering an error for the CustomEvent. So I found a nice polyfill for that (https://github.com/kumarharsh/custom-event-polyfill) which did the trick. So I propose adding another note for this polyfill to help support older browsers.

@mciastek
Copy link
Owner

mciastek commented Feb 4, 2020

Looks like a great idea. I think that the best place to include information about polyfill is Events section in docs. What do you think?

@brentrobbins
Copy link
Author

Hi @mciastek that's exactly where I was thinking too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants