celo-org / react-celo

Connect your react dApp to Celo
https://react-celo.vercel.app
MIT License
59 stars 43 forks source link

tailwindcss as peerDependency #41

Closed vibern0 closed 2 years ago

vibern0 commented 2 years ago

Hi,

we (@impactMarket) were adding use-contractkit to the web and realized that it increases de bundle significantly. It seems to be coming from tailwindcss. Since we can create a custom modal, would it also be possible to change tailwindcss to peerDependency, so those of us with a custom modal would not import a giant lib without being used?

Or do you have another suggestion?

dckesler commented 2 years ago

Hi! Yes this was a mistake for tailwind to be included as regular dependency. I've moved it over to devDependencies which should exclude it from the package. https://github.com/celo-org/use-contractkit/pull/49 should address this.