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

Get rid of raf library #259

Open
1 of 3 tasks
darkowic opened this issue Sep 21, 2018 · 2 comments
Open
1 of 3 tasks

Get rid of raf library #259

darkowic opened this issue Sep 21, 2018 · 2 comments
Labels
6.1 good first issue

Comments

@darkowic
Copy link

I'm submitting a ...

  • bug report
  • feature request
  • support request

The requestAnimationFrame is now widely supported by browsers -https://caniuse.com/#search=requestanimationframe

The library is using polyfill for it - https://www.npmjs.com/package/raf . Removing the polyfill we could save ~30% of build size https://bundlephobia.com/[email protected]
@vcarl
Copy link
Contributor

vcarl commented Nov 13, 2018

This is a good idea! You're right, I feel fine not supporting older than IE10. I'll see about making this change soon.

@vcarl vcarl added the 6.1 label Nov 13, 2018
@SleepWalker
Copy link

moreover raf is required for recent React versions. So it should be actually already polyfilled by lib consumers.

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

No branches or pull requests
3 participants
@vcarl @SleepWalker @darkowic