daonanchao/vibe-kanban
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix multi-shortcut issue (#856)

Browse Source
This commit is contained in:
Louis Knight-Webb
2025-09-25 19:53:32 +01:00
committed by GitHub
parent 1c23d4fd11
commit 4f343fdb8f
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
frontend/src/keyboard/useSemanticKey.ts
View File

@@ -1,3 +1,4 @@
import { useMemo } from 'react';
import { import {
useKeyboardShortcut, useKeyboardShortcut,
type KeyboardShortcutOptions, type KeyboardShortcutOptions,
@@ -33,8 +34,13 @@ export function createSemanticHook<A extends Action>(action: A) {
// Use 'when' as alias for 'enabled' if provided // Use 'when' as alias for 'enabled' if provided
const isEnabled = when !== undefined ? when : enabled; const isEnabled = when !== undefined ? when : enabled;
const keys = getKeysFor(action, scope); // Memoize to get stable array references and prevent unnecessary re-registrations
const binding = getBindingFor(action, scope); const keys = useMemo(() => getKeysFor(action, scope), [action, scope]);
const binding = useMemo(
() => getBindingFor(action, scope),
[action, scope]
);
const keyboardShortcutOptions: KeyboardShortcutOptions = {}; const keyboardShortcutOptions: KeyboardShortcutOptions = {};
if (enableOnContentEditable !== undefined) if (enableOnContentEditable !== undefined)