Skip to content

fix: disable some keyboard shortcuts when it's disabled in settings#1780

Draft
alex-key wants to merge 2 commits intonpmx-dev:mainfrom
alex-key:fix/some-shortkeys-work-when-disabled-in-settings
Draft

fix: disable some keyboard shortcuts when it's disabled in settings#1780
alex-key wants to merge 2 commits intonpmx-dev:mainfrom
alex-key:fix/some-shortkeys-work-when-disabled-in-settings

Conversation

@alex-key
Copy link
Contributor

@alex-key alex-key commented Mar 1, 2026

🔗 Linked issue

Partially resolves #1779

🧭 Context

In #1537 we've added a mechanism which allows user to disable all custom keyboard shortcuts in Settings. So it does not conflict with browser, os or other tools user might use to browse

📚 Description

Some of widgets in the app still use custom keyboard shortcuts. This PR adds logic to disable custom behavior for the following areas:

  • Search
  • Compare page package search
  • Playgorund dropdown

@vercel
Copy link

vercel bot commented Mar 1, 2026

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

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Mar 1, 2026 1:53pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Mar 1, 2026 1:53pm
npmx-lunaria Ignored Ignored Mar 1, 2026 1:53pm

Request Review

@codecov
Copy link

codecov bot commented Mar 1, 2026

Codecov Report

❌ Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
app/components/Package/Playgrounds.vue 0.00% 1 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

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.

Disable more keyboard shortcuts if off in Settings

1 participant