Skip to content

docs: update 2.keyboard-shortcuts.md to match implementation#1495

Open
morinokami wants to merge 1 commit intonpmx-dev:mainfrom
morinokami:update-keyboard-shortcuts
Open

docs: update 2.keyboard-shortcuts.md to match implementation#1495
morinokami wants to merge 1 commit intonpmx-dev:mainfrom
morinokami:update-keyboard-shortcuts

Conversation

@morinokami
Copy link
Contributor

  • Add ?, ,, c to global shortcuts section and move . to package page section
  • Remove unimplemented browse code section (file tree arrow keys, enter, escape)
  • Add package page section with ., d, c shortcuts
  • Move tip to global shortcuts section where it belongs

@vercel
Copy link

vercel bot commented Feb 14, 2026

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

Project Deployment Actions Updated (UTC)
docs.npmx.dev Ready Ready Preview, Comment Feb 14, 2026 7:39am
npmx.dev Ready Ready Preview, Comment Feb 14, 2026 7:39am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
npmx-lunaria Ignored Ignored Feb 14, 2026 7:39am

Request Review

@codecov
Copy link

codecov bot commented Feb 14, 2026

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!

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 14, 2026

📝 Walkthrough

Walkthrough

A documentation update for keyboard shortcuts that restructures and consolidates the reference guide. The global shortcuts table is expanded with new keys for specific actions, whilst navigation shortcuts are reorganised and renamed. The legacy "Browse code" section is replaced with a new "Package page" section containing simplified keyboard shortcuts. The changes affect 18 lines added and 18 removed with no modifications to public API signatures.

Possibly related PRs

Suggested reviewers

  • antfu
🚥 Pre-merge checks | ✅ 1 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Merge Conflict Detection ⚠️ Warning ⚠️ Unable to check for merge conflicts: Stream setup permanently failed: 13 INTERNAL: Received RST_STREAM with code 2 (Internal server error)
✅ Passed checks (1 passed)
Check name Status Explanation
Description check ✅ Passed The pull request description clearly outlines the changes made to the keyboard shortcuts documentation, matching the actual modifications in the changeset.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
⚔️ Resolve merge conflicts (beta)
  • Auto-commit resolved conflicts to branch update-keyboard-shortcuts
  • Post resolved changes as copyable diffs in a comment

No actionable comments were generated in the recent review. 🎉

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


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

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