Skip to content

feat(ui): add visible keyboard shortcut hints to the command palette#2911

Open
BittuBarnwal7479 wants to merge 1 commit into
npmx-dev:mainfrom
BittuBarnwal7479:feat/add-command-palette-keyboard-hints
Open

feat(ui): add visible keyboard shortcut hints to the command palette#2911
BittuBarnwal7479 wants to merge 1 commit into
npmx-dev:mainfrom
BittuBarnwal7479:feat/add-command-palette-keyboard-hints

Conversation

@BittuBarnwal7479

Copy link
Copy Markdown
Contributor

🔗 Linked issue

#2910

🧭 Context

image

📚 Description

The command palette supports keyboard navigation, but the available shortcuts are not visible to users.

Currently users can navigate with:

↑ / ↓ to move between results
Enter to select
Esc to close
However, these shortcuts are not displayed anywhere in the command palette UI.

@vercel

vercel Bot commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Jun 14, 2026 5:43pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Jun 14, 2026 5:43pm
npmx-lunaria Ignored Ignored Jun 14, 2026 5:43pm

Request Review

@coderabbitai

coderabbitai Bot commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Adds a keyboard shortcut hint footer to the command palette component, displaying Arrow Up/Down, Enter, and Esc hint labels via kbd elements. Corresponding navigate, select, and close keys are added to the English locale file and i18n schema. The header's bottom border is also removed. A new test asserts the footer renders correctly.

Changes

Command Palette Keyboard Shortcut Footer

Layer / File(s) Summary
i18n contract: footer translation keys and schema
i18n/schema.json, i18n/locales/en.json
command_palette.footer object with navigate, select, and close string properties is added to both the JSON schema and the English locale file.
CommandPalette footer UI and header border removal
app/components/CommandPalette.client.vue
Header container div loses the border-b border-border/70 classes, and a new footer block is inserted before the results list, rendering kbd shortcut hints bound to the new i18n keys.
Footer rendering test
test/nuxt/components/CommandPalette.spec.ts
New test case mounts the palette and asserts the footer DOM element contains the expected shortcut hint copy for navigate, select, and close.

Possibly related issues

Suggested reviewers

  • knowler
  • graphieros
  • ghostdevv
🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description check ✅ Passed The description is directly related to the changeset, providing context about the problem (keyboard shortcuts not being visible) and the solution (displaying them in the UI).
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The pull request title 'feat(ui): add visible keyboard shortcut hints to the command palette' accurately and concisely summarizes the main change: adding visible keyboard shortcut hints (↑/↓, Enter, Esc) to the command palette footer.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions

Copy link
Copy Markdown

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
i18n/locales/en.json Source changed, localizations will be marked as outdated.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@codecov

codecov Bot commented Jun 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@BittuBarnwal7479 BittuBarnwal7479 changed the title feat(search): adds visible keyboard shortcut hints to the command palette feat(ui): add visible keyboard shortcut hints to the command palette Jun 14, 2026
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