fix(navigation-menu): Incorrect positioning and animations

[multiwebinc]

trafficstars

Fixes #172

Also adds animation on the NavigationMenuContent and NavigationMenuIndicator similar to https://www.radix-ui.com/docs/primitives/components/navigation-menu

[vercel[bot]]

@multiwebinc is attempting to deploy a commit to the shadcn-pro Team on Vercel.

A member of the Team first needs to authorize it.

[vercel[bot]]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
ui	❌ Failed (Inspect)			Apr 20, 2023 3:34pm

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
next-template	⬜️ Ignored (Inspect)			Apr 20, 2023 3:34pm

[shadcn]

@multiwebinc Here's the error:

./components/ui/navigation-menu.tsx
Error: Invalid Tailwind CSS classnames order  tailwindcss/classnames-order
Error: Classnames bg-popover, bg-white are conflicting!  tailwindcss/no-contradicting-classname

[multiwebinc]

@shadcn Hopefully that should fix it.

[reza-ebrahimi]

This is the same problem with issue #902

[shadcn]

@multiwebinc Does this still apply? Or did we fix it in the latest iterations?

[aFuzzyBear]

Hi folks, I was wondering if there was any movement on this PR? Im running into an issue #172 and hoping this would be the fix 🤞

Please do let me know, kinda blocked by this.

🖖💚🐻💚🖖

[multiwebinc]

@shadcn I haven't used this library for a while now. Maybe someone else could test it?

[qiaoandrew]

@shadcn I haven't used this library for a while now. Maybe someone else could test it?

I previously had some issues with the Navigation Menu's animations, but adding the changes by @multiwebinc seems to fix it. Thanks!

[chris-hud]

This worked for me, solved big issue with library.