react-sticky icon indicating copy to clipboard operation
react-sticky copied to clipboard

Get rid of `raf` library

Open darkowic opened this issue 7 years ago • 2 comments

I'm submitting a ...

  • [ ] bug report
  • [x] 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]

darkowic avatar Sep 21 '18 07:09 darkowic

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 avatar Nov 13 '18 20:11 vcarl

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

SleepWalker avatar Jan 11 '19 17:01 SleepWalker