Skip to content

feat: publish dropdown-menu - #23

Merged
zyx1121 merged 1 commit into
mainfrom
feat/dropdown-menu
Oct 7, 2026
Merged

zyx1121 merged 1 commit into
mainfrom
feat/dropdown-menu

Conversation

@zyx1121

@zyx1121 zyx1121 commented Oct 7, 2026

Copy link
Copy Markdown
Member

Summary

Portal uses DropdownMenu 11 times: 49 items, 5 labels, 10 separators, 1 submenu; no item variants, checkbox or radio items.

  • dropdown-menu: frosted surface (p-1, rounded-menu, min-w-48), 40px rows with rounded-control, muted labels, separators that reach the menu edge, a submenu whose first item lines up with its trigger (alignOffset = padding + border).
  • Dropped: checkbox / radio items, shortcuts, inset, destructive items, the portal wrapper.
  • Site: an icon-button menu next to the buttons.

Test plan

  • Sandbox: all checks, compose healthy
  • Chromium: menu 192px / 20px radius; submenu's first item at the trigger's y (697 = 697), flips left at the viewport edge; no console errors
  • CI green

🤖 Generated with Claude Code

Items, group labels, separators and one submenu level, which is what
portal uses (49 items, 5 labels, 10 separators, 1 submenu). Frosted
surface with p-1 and rounded-menu, 40px rows with rounded-control; the
submenu lines its first item up with its trigger. No checkbox, radio,
shortcut, inset or destructive items (0 uses).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@zyx1121
zyx1121 merged commit c830b65 into main Oct 7, 2026
2 checks passed
@zyx1121
zyx1121 deleted the feat/dropdown-menu branch October 7, 2026 06:54
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.

1 participant