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.

Sign up for GitHub

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

Jump to bottom

Browser support? #10

Open
Finesse opened this issue Feb 6, 2018 · 4 comments
Open

Browser support? #10

Finesse opened this issue Feb 6, 2018 · 4 comments

Comments

@Finesse
Copy link

Finesse commented Feb 6, 2018

What browsers does this library support or what JS features requires?
@Finesse
Copy link
Author

Finesse commented Feb 6, 2018

I've made a quick research and I've found that it uses Array.prototype.forEach in the wrappy dependency, so it doesn't support IE≤8.

@Finesse
Copy link
Author

Finesse commented Feb 6, 2018

No browser support because of #4 (comment). Maybe a polyfill will help but I don't want to make things complicated and spoil the global namespace.

@sorribas
Copy link
Owner

sorribas commented Feb 6, 2018

It should work in the browser when compiled with browserify or a similar bundler. Do you have any specific issues you're facing?

@Finesse
Copy link
Author

Finesse commented Feb 6, 2018

@sorribas I compiled it with Rollup and Babel and got a code with process.nextTick which is not defined in browser.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sorribas @Finesse