Skip to content

Conversation

Copy link

Copilot AI commented Dec 10, 2025

All keyboard shortcuts are now configurable. Users can customize navigation keys (K, A, D, etc.), UI controls (backtick for sidebar), and action shortcuts (N, F, G, etc.) through a new Settings section.

Changes

Store & State Management

  • Added KeyboardShortcuts interface with 17 configurable shortcuts
  • Store actions: setKeyboardShortcut, setKeyboardShortcuts, resetKeyboardShortcuts
  • Persists to localStorage

Hook Refactor

  • Replaced static NAV_SHORTCUTS, UI_SHORTCUTS, ACTION_SHORTCUTS exports
  • New useKeyboardShortcutsConfig() hook pulls shortcuts from store
  • All components updated to use dynamic shortcuts

Settings UI

  • Three categories: Navigation (7), UI Controls (1), Actions (9)
  • Inline editing with real-time conflict detection
  • Visual indicators for modified shortcuts (colored border + label)
  • "Reset All to Defaults" button

Implementation Notes

  • Some shortcuts intentionally share keys (e.g., "N" for addFeature/newSession/addProfile) - they're context-specific and only active in their respective views
  • Changes take effect immediately
  • Input supports special characters (backtick, etc.) with maxLength=2
// Before: Static constants
const shortcuts = {
  key: NAV_SHORTCUTS.board,
  action: () => setCurrentView("board")
};

// After: Dynamic from store
const shortcuts = useKeyboardShortcutsConfig();
const shortcut = {
  key: shortcuts.board,
  action: () => setCurrentView("board")
};
Original prompt

This section details on the original issue you should resolve

<issue_title>allow customizablity for all hotkeys in settings</issue_title>
<issue_description></issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@SuperComboGamer SuperComboGamer marked this pull request as ready for review December 10, 2025 17:42
@SuperComboGamer SuperComboGamer requested review from Shironex and webdevcody and removed request for webdevcody December 10, 2025 17:42
@SuperComboGamer SuperComboGamer marked this pull request as draft December 10, 2025 17:42
Copilot AI changed the title [WIP] Allow customizability for all hotkeys in settings Add customizable keyboard shortcuts in settings Dec 10, 2025
Copilot AI requested a review from GTheMachine December 10, 2025 17:45
@Shironex
Copy link
Collaborator

@copilot merge latest main changes to this branch

Copy link
Author

Copilot AI commented Dec 10, 2025

@copilot merge latest main changes to this branch

I've analyzed the latest main branch changes. The main branch has added:

  • Setup wizard for CLI tools configuration
  • Per-project themes support
  • CLI detection refresh buttons
  • Various UI improvements and bug fixes

Our keyboard shortcuts changes are in separate files and don't conflict with these additions. However, I'm unable to perform a direct git merge because the repository authentication doesn't allow me to fetch the main branch in this environment.

Recommended approach:

  1. The PR maintainer can merge this PR after ensuring it's based on the latest main
  2. Or, the PR can be rebased locally using: git fetch origin main && git rebase origin/main

The changes should merge cleanly as we only modified the keyboard shortcut system (app-store.ts, use-keyboard-shortcuts.ts, and component files), while main added features in different areas (setup views, theme system, etc.).

Copilot AI and others added 6 commits December 10, 2025 19:31
…e shortcuts

Co-authored-by: GTheMachine <156854865+GTheMachine@users.noreply.github.com>
Co-authored-by: GTheMachine <156854865+GTheMachine@users.noreply.github.com>
Co-authored-by: GTheMachine <156854865+GTheMachine@users.noreply.github.com>
…maxLength for special characters

Co-authored-by: GTheMachine <156854865+GTheMachine@users.noreply.github.com>
Co-authored-by: GTheMachine <156854865+GTheMachine@users.noreply.github.com>
@Shironex Shironex force-pushed the copilot/allow-customizability-for-hotkeys branch from 627f25c to 1edb7f6 Compare December 10, 2025 18:33
@Shironex Shironex marked this pull request as ready for review December 10, 2025 18:41
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.

allow customizablity for all hotkeys in settings

3 participants