From c7d2cc47e11e732527abf3b931d611dda8545bae Mon Sep 17 00:00:00 2001 From: dtoro Date: Tue, 7 Apr 2026 22:56:09 +0200 Subject: [PATCH] chore: drop E shortcut for preview, keep only Space E was a Lightroom holdover and overlapped with the natural rating / flag culling shortcuts. Space is the only binding now (double-click on a thumbnail still works). Hints pill updated. Co-Authored-By: Claude Opus 4.6 (1M context) --- frontend/src/components/KeyboardHints.tsx | 4 ++-- frontend/src/hooks/useKeyboardShortcuts.ts | 5 ++--- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/frontend/src/components/KeyboardHints.tsx b/frontend/src/components/KeyboardHints.tsx index 5b19cd3..d182398 100644 --- a/frontend/src/components/KeyboardHints.tsx +++ b/frontend/src/components/KeyboardHints.tsx @@ -13,14 +13,14 @@ export function KeyboardHints() { { key: '1-5', action: 'Rate' }, { key: 'P', action: 'Pick' }, { key: 'X', action: 'Discard' }, - { key: 'E / Space', action: 'Preview' }, + { key: 'Space', action: 'Preview' }, { key: 'Esc', action: 'Deselect' }, ] : [ { key: '↑↓←→', action: 'Navigate' }, { key: 'Click', action: 'Select' }, { key: 'Shift+Click', action: 'Range' }, - { key: 'E / Space', action: 'Preview' }, + { key: 'Space', action: 'Preview' }, { key: '\\', action: 'Filters' }, { key: '/', action: 'Search' }, ] diff --git a/frontend/src/hooks/useKeyboardShortcuts.ts b/frontend/src/hooks/useKeyboardShortcuts.ts index 4b388b2..a043186 100644 --- a/frontend/src/hooks/useKeyboardShortcuts.ts +++ b/frontend/src/hooks/useKeyboardShortcuts.ts @@ -75,8 +75,8 @@ export function useKeyboardShortcuts(props: KeyboardShortcutsProps) { useHotkeys('/', focusSearch, HK_OPTS) useHotkeys('mod+f', focusSearch, HK_OPTS) - // E and Space both toggle the preview view (open from grid, close from - // preview). Double-click on a thumbnail does the same. + // Space toggles the preview view (open from grid, close from preview). + // Double-click on a thumbnail does the same. const openPreviewFromGrid = () => { const id = usePhotoStore.getState().activePhotoId ?? getFirstPhotoId?.() ?? null if (id) openPreview(id) @@ -87,7 +87,6 @@ export function useKeyboardShortcuts(props: KeyboardShortcutsProps) { else openPreviewFromGrid() } - useHotkeys('e', togglePreview, HK_OPTS, [isPreview, getFirstPhotoId]) useHotkeys('space', togglePreview, HK_OPTS, [isPreview, getFirstPhotoId]) // ── Culling shortcuts (work in both grid and preview) ────────────────────