Require only necessary parts of CoreJS

missive / emoji-mart

🏪 One component to pick them all

https://missiveapp.com/open/emoji-mart

MIT License

8.59k stars 826 forks source link

Require only necessary parts of CoreJS #112

Closed ai closed 7 years ago

ai commented 7 years ago

Right now EmojiMart requires whole CoreJS. And it is the huge dependency, about 15%.

Maybe we can require only some parts of CoreJS? What example ES6 methods do we use in EmojiMart?

EtienneLem commented 7 years ago

I’m not even sure which ones are needed nowadays. Original issue states for..of (https://github.com/missive/emoji-mart/pull/92). I’m wondering whether that is the only concern or there’s more to it. If it is, we could easily use good old for loops instead and drop babel-runtime.

ai commented 7 years ago

Hm. technically, we can rewrite for-of to old for (var i = 0…) to remove CoreJS completely.

bniwredyc commented 7 years ago

Hi, guys! I've done it: https://github.com/missive/emoji-mart/pull/113