Skip to content

Conversation

kevinwaxi
Copy link

  • Implement submenu component with shadcn-vue styling
  • Integrate submenu into existing navigation structure

- Implement submenu component with shadcn-vue styling - Integrate submenu into existing navigation structure
@tnylea
Copy link
Contributor

tnylea commented Mar 31, 2025

Cool, I like this. It's a good way to show an example of how to add children menu items. Can you also create a PR for this in the React Starter kit.

Thanks.

@tnylea tnylea added the Awaiting User Response Waiting for developers response label Mar 31, 2025
@kevinwaxi
Copy link
Author

Can you also create a PR for this in the React Starter kit.

Sadly i'm not that familiar with react but i can try replicating the code soon.

- Implement multi-level dropdown menus within breadcrumbs
- Update TypeScript interfaces for new data structure
@tnylea
Copy link
Contributor

tnylea commented Apr 15, 2025

Ok, this looks good. I'll work on implementing this in the React version. Thanks for those updates.

@taylorotwell, This will add a dropdown menu in the breadcrumbs if the user provides subItems key in their breadcrumbs array (screenshot below).

CleanShot 2025-04-15 at 06 35 26@2x

This will also add dropdown options to the sidebar Nav menu. When the developer uncomments the example from the AppSidebar.Vue

//     {
//     title: 'User Management',
//     href: '#',
//     icon: Shield,
//     children: [
//       {
//         title: 'Roles',
//         href: '#',
//       },
//       {
//         title: 'Permissions',
//         href: '#',
//       },
//       {
//         title: 'Users',
//         href: '#',
//       },
//     ],
//   },

It will show a dropdown menu item in the sidebar (screenshot below)

CleanShot 2025-04-15 at 06 26 07@2x

I've tested out and everything works great.

Thanks!

@tnylea tnylea added Approved Approved for merge and removed Awaiting User Response Waiting for developers response labels Apr 15, 2025
@taylorotwell
Copy link
Member

Not sure we really need this.

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

Labels

Approved Approved for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants