Skip to content

feat: add keyboard shortcuts for common actions#1536

Open
Ashvin-KS wants to merge 2 commits into
aryandas2911:mainfrom
Ashvin-KS:feat/keyboard-shortcuts
Open

feat: add keyboard shortcuts for common actions#1536
Ashvin-KS wants to merge 2 commits into
aryandas2911:mainfrom
Ashvin-KS:feat/keyboard-shortcuts

Conversation

@Ashvin-KS

Copy link
Copy Markdown
Contributor

Summary

Adds keyboard shortcuts for common actions to improve productivity:

  • Ctrl+N: New task
  • Ctrl+K: Toggle search
  • Ctrl+D: Toggle dark mode
  • Esc: Close modal
  • ?: Show keyboard shortcuts help

Changes

  • Added useKeyboardShortcuts hook
  • Added KeyboardShortcutsHelp component
  • Added KeyboardShortcutsProvider wrapper
  • Integrated into App.jsx

Closes #1528

@aryandas2911

Copy link
Copy Markdown
Owner

lint pls

@Ashvin-KS Ashvin-KS force-pushed the feat/keyboard-shortcuts branch from 03f480c to 094d268 Compare June 19, 2026 06:55
@Ashvin-KS

Copy link
Copy Markdown
Contributor Author

@aryandas2911 Hi! This PR is approved and all checks are passing. Ready to merge whenever you get a chance. Thanks!

@Ashvin-KS

Copy link
Copy Markdown
Contributor Author

Ran in both frontend/ and backend/ - no errors in my changes. The 3 warnings are pre-existing in Profile.jsx (unrelated to keyboard shortcuts). PR is ready to merge.

- Ctrl+N: New task
- Ctrl+K: Toggle search
- Ctrl+D: Toggle dark mode
- Esc: Close modal
- ?: Show keyboard shortcuts help

Closes aryandas2911#1528
- Remove unused imports (useState, useEffect) from KeyboardShortcutsHelp
- Remove unused named export from KeyboardShortcutsProvider
- Remove unused useContext import
@Ashvin-KS Ashvin-KS force-pushed the feat/keyboard-shortcuts branch from 094d268 to 117dc94 Compare June 23, 2026 13:16
@Ashvin-KS

Copy link
Copy Markdown
Contributor Author

Not at all — this PR is complete and ready to merge. All checks are passing and the lint warnings you flagged were addressed (only pre-existing warnings unrelated to these changes). Feel free to merge whenever convenient!

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.

feat: Add keyboard shortcuts for common actions

2 participants