Skip to content

feat: add context menu#799

Merged
pasqualevitiello merged 5 commits into
mainfrom
feat/add-context-menu
Jun 11, 2026
Merged

feat: add context menu#799
pasqualevitiello merged 5 commits into
mainfrom
feat/add-context-menu

Conversation

@pasqualevitiello

@pasqualevitiello pasqualevitiello commented Jun 11, 2026 •
edited by devin-ai-integration Bot
Loading

Copy link
Copy Markdown
Contributor

@vercel

vercel Bot commented Jun 11, 2026 •
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
coss-com Ready Ready Preview, Comment Jun 11, 2026 3:56pm
coss-com-ui Ready Ready Preview, Comment Jun 11, 2026 3:56pm
coss-examples-calcom Ready Ready Preview, Comment Jun 11, 2026 3:56pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
coss-com-origin Skipped Skipped Jun 11, 2026 3:56pm

Request Review

devin-ai-integration[bot]
devin-ai-integration Bot reviewed Jun 11, 2026
View reviewed changes

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 1 potential issue.

Open in Devin Review

Comment thread apps/ui/registry/default/ui/menu.tsx

@devin-ai-integration devin-ai-integration Bot Jun 11, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Missing DropdownMenuLinkItem alias export for MenuLinkItem

Every Menu* component has a corresponding DropdownMenu* alias export (e.g., MenuItem as DropdownMenuItem, MenuCheckboxItem as DropdownMenuCheckboxItem), but the newly added MenuLinkItem is missing its DropdownMenuLinkItem alias in the re-export block at apps/ui/registry/default/ui/menu.tsx:334-352 and packages/ui/src/components/menu.tsx:334-352. Users who rely on the DropdownMenu* import pattern (e.g. shadcn/Radix migration) won't be able to import DropdownMenuLinkItem. The docs at apps/ui/content/docs/components/menu.mdx:172 also don't mention the alias, unlike every other component section on the same page.

(Refers to lines 334-352)

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

@vercel vercel Bot temporarily deployed to Preview – coss-com-origin June 11, 2026 15:54 Inactive
@pasqualevitiello pasqualevitiello merged commit 5ed9f80 into main Jun 11, 2026
8 checks passed
@pasqualevitiello pasqualevitiello deleted the feat/add-context-menu branch June 11, 2026 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@devin-ai-integration devin-ai-integration[bot] devin-ai-integration[bot] left review comments

@PeerRich PeerRich PeerRich approved these changes

Assignees

No one assigned

Labels

area:apps/ui docs pkg:ui

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pasqualevitiello @PeerRich