search

YIZHUANG / react-multi-carousel

A lightweight production-ready Carousel that rocks supports multiple items and server-side rendering with no dependency. Bundle size 2kb.
MIT License
1.25k stars 286 forks source link

fix: updated .npmignore and readme #352

Open falsepopsky opened 2 years ago

falsepopsky commented 2 years ago

Hello there

Edits

npmignore

I edited the npmignore file because, when you start a new project with this package, you get some files and folders that you actually don't use...

Preview on node_modules:

react

as you can see dev, storybook, etc.

ps: i removed node_modules for this reason:

Additionally, everything in node_modules is ignored, except for bundled dependencies. npm automatically handles this for you, so don't bother adding node_modules to .npmignore. ==> The .npmignore File

readme

then in the readme section Specific Props

added the property pauseOnHover that was missing and fixed the typo on partialVisible also, removed David Dependancy bag due to the unreliability of its service https://github.com/alanshaw/david/issues/171

falsepopsky commented 1 year ago

@YIZHUANG