Skip to content

Conversation

therahulbehera
Copy link
Contributor

@therahulbehera therahulbehera commented Oct 9, 2025

NavigationMenu was missing in the example, which was throwing an error while building in the Next.js app. It says NavigationMenuDemo, so I expected it to run directly when I pasted it in my code editor.

<NavigationMenu> was missing in the example, which was throwing an error while building in the Next.js app. It says NavigationMenuDemo, so I expected it to run directly when I pasted it in my code editor.
Copy link

vercel bot commented Oct 9, 2025

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

A member of the Team first needs to authorize it.

@therahulbehera therahulbehera changed the title Update navigation-menu.mdx missing <NavigationMenu> docs: Update navigation-menu.mdx missing <NavigationMenu> Oct 9, 2025
@shadcn
Copy link
Collaborator

shadcn commented Oct 15, 2025

Ah I think this is intentional. We want to show the item part only. Let's rename the component from NavigationMenuDemo to NavigationMenuItemDemo instead so it's clearer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants