Skip to content

fix: allow native text-editing shortcuts in focused inputs#203

Open
knabe wants to merge 1 commit intoavihaymenahem:mainfrom
knabe:fix/native-text-shortcuts-in-inputs
Open

fix: allow native text-editing shortcuts in focused inputs#203
knabe wants to merge 1 commit intoavihaymenahem:mainfrom
knabe:fix/native-text-shortcuts-in-inputs

Conversation

@knabe
Copy link
Copy Markdown
Contributor

@knabe knabe commented Mar 1, 2026

Summary

Ctrl/Cmd+A (select all) was intercepted by the app's keyboard shortcut handler even when an input/textarea was focused, preventing native text selection in the search box and other inputs.

Now bypasses app shortcut handling for Ctrl/Cmd + A/C/X/V/Z when an input element is focused, letting the browser handle them natively.

Changes

  • Added input focus check before Ctrl/Cmd shortcut processing in useKeyboardShortcuts.ts

Type of Change

  • Bug fix
  • New feature
  • Enhancement (improving existing feature)
  • Refactor (no behavior change)
  • Documentation
  • CI/Build

Testing

  • Existing tests pass (npm run test)
  • [] New tests added (if applicable)
  • Manually tested

Ctrl/Cmd+A (select all) was intercepted by the app's keyboard
shortcut handler even when an input/textarea was focused, preventing
native text selection in the search box and other inputs.

Now bypasses app shortcut handling for Ctrl/Cmd + A/C/X/V/Z when
an input element is focused, letting the browser handle them natively.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@knabe knabe requested a review from avihaymenahem as a code owner March 1, 2026 06:03
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