Skip to content

/ react-vis

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

Jump to bottom

[Request] List of polyfills #1081

Open
bhutanigautam opened this issue Jan 7, 2019 · 2 comments
Open

[Request] List of polyfills #1081

bhutanigautam opened this issue Jan 7, 2019 · 2 comments
Labels
docs enhancement

Comments

@bhutanigautam
Copy link

@bhutanigautam bhutanigautam commented Jan 7, 2019

Hello Team,

Firstly, thank you for open sourcing this library - Awesome!

Now to my request -

Can you document the polyfills that are required for this project?

react-vis makes use of ES6 array methods isn't very descriptive and I don't want to include everything that babel-polyfills brings in; instead, I would rather just polyfill what's required.

Per #663 - will most likely have to add polyfills sooner or later. Even if the project is not going to polyfill it's requirements, a list would be great!
@lzhuor
Copy link
Contributor

@lzhuor lzhuor commented Mar 11, 2019

I like your idea and have encountered a similar situation. I think we need to set baseline version of browser first (particularly IE).
@bhutanigautam
Copy link
Author

@bhutanigautam bhutanigautam commented Mar 21, 2019

Agreed.
I guess my next question is what browser versions does react-viz support?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
docs enhancement
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@bhutanigautam @mcnuttandrew @lzhuor
You can’t perform that action at this time.