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

Published 20 hours ago • verified publisher icon andyyou

lib size is enormous 1.07Mb

Open Stas-Buzunko opened this issue 6 years ago • 6 comments

I'm sorry if it's me doing something wrong (but i doubt it) But bundle size is huge due to react-coverflow size. screenshot 2018-12-16 at 14 39 59 In my opinion, it's not ok for carousel library to take 47% of bundle size

Stas-Buzunko avatar Dec 16 '18 12:12 Stas-Buzunko

Yikes, that is entirely too massive @Stas-Buzunko, I'll take a look at ways to reduce it's size ASAP. Thanks for bringing it to my attention

asalem1 avatar Dec 20 '18 15:12 asalem1

Hi @Asalem1 , any updates on this one.

wzard avatar Jan 04 '19 16:01 wzard

@wzard sorry, none so far. Just had a baby girl and haven't 19 days ago so i'm a little behind. I'll take a look at it soon

asalem1 avatar Jan 11 '19 15:01 asalem1

@Asalem1 Oh. Congratulations Asalem1. :) Take your time. Your baby needs you more than random internet strangers.

wzard avatar Jan 12 '19 12:01 wzard

Thanks for the patience and for the well wishes. Just cleaned up webpack a bit, it's still higher than I'd like it to be so I'll keep this open but know that this is a serious concern that I'm currently address (along with the other issues). Thanks again for the support and patience

asalem1 avatar Jan 26 '19 20:01 asalem1

Alright, final update on this ticket. The published size has been reduced a bit, but it seems like the main culprit in the large dependency size is the radium dependency. My intent is to remove the dependency and see if I can replicate its functionality without losing performance. I'm going to leave this issue open until then.

asalem1 avatar Mar 09 '19 16:03 asalem1