Skip to content

Ellipsis only for menus; verbs for non-menu steps - #400

Merged
TheGreatAxios merged 1 commit into
mainfrom
cl-6785-ellipsis-control-opens-a-surface-that-is-not-a-menu
Aug 24, 2026
Merged

TheGreatAxios merged 1 commit into
mainfrom
cl-6785-ellipsis-control-opens-a-surface-that-is-not-a-menu

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Summary

  • Rename skill-detail Save… → Review (opens a confirmation surface, not a menu).
  • Rename shell context-menu Search… → Search (opens command palette, not a submenu).
  • Update skill-detail tests to match the new label.

Closes CL-6785

Test plan

  • bun test ./test/skill-detail-page.test.tsx in apps/web (21 pass)

@TheGreatAxios
TheGreatAxios merged commit 89af5ae into main Aug 24, 2026
5 checks passed
@TheGreatAxios
TheGreatAxios deleted the cl-6785-ellipsis-control-opens-a-surface-that-is-not-a-menu branch August 24, 2026 16:44
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