shadcn animations

[klagrida]

Hi,

Thanks for your library and the work you've put into it. I have a question about why you included ShadCN animations in the library instead of keeping it Tailwind CSS-specific.

Wouldn't it be better to separate these animations into a different file or library? This could help keep the core library more generic and flexible.

Thanks

[Wombosvideo]

I had this thought too when @nrjdalal suggested adding these animations. The reason I still went with it is the fact this allows shadcn or other library authors to focus on components instead of tinkering with animations and "polluting" their CSS. If they still need a custom animation, I'd be happy to add it to tw-animate-css.
Adding these extra animations to a separate file would mean users who are interested in these extra animations have to add a second @import statement to their CSS file. Not a big deal, but not really necessary either. And due to Tailwind CSS "tree-shaking" away all unused utilities, people who don't use these extra animations don't have to worry about extra bundle size.

To make it clear this is not just a 1:1 port of tailwindcss-animate, I will rename this project to tailwindcss-animate-x as soon as I finished writing the documentation. To keep backward compatibility, I will keep the tw-animate-css package up to date too, as an alias.

[Wombosvideo]

I don't know if it's still the right decision to rename this package, given the name is well-known now.

[nrjdalal]

@Khalilou88 try creating any project, and add @import "tw-animate-css", then run the build and check the dist css, search for accordion, collapsible or any other class that you haven'r referenced/used in your files, you won't find em.

Even if @Wombosvideo add 20-30 other classes, it won't matter.

Great statement @Wombosvideo, and due to Tailwind CSS "tree-shaking" away all unused utilities, people who don't use these extra animations don't have to worry about extra bundle size.

[nrjdalal]

also @Wombosvideo you should add that note in docs I guess, so peeps don't feel it's bloated or have bundlephobia

[klagrida]

@nrjdalal Thanks, will do.