matty/ncd-fe
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

nicolium: allow hotkeys to continue propagating

Browse Source 
Signed-off-by: nicole mikołajczyk <git@mkljczk.pl>
This commit is contained in:
nicole mikołajczyk
2026-03-05 10:49:41 +01:00
parent d4a92abf9d
commit 395553bc6b
1 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
packages/nicolium/src/features/ui/components/hotkeys.tsx
View File

@ -200,7 +200,7 @@ function useHotkeys<T extends HTMLElement>(handlers: HandlerMap) {
const matchCandidates: {
// A candidate will be have an undefined handler if it's matched,
// but handled in a parent component rather than this one.
handler: ((event: KeyboardEvent) => void) | undefined;
handler: ((event: KeyboardEvent) => void | boolean) | undefined;
priority: number;
}[] = [];
@ -220,9 +220,11 @@ function useHotkeys<T extends HTMLElement>(handlers: HandlerMap) {
const bestMatchingHandler = matchCandidates.at(0)?.handler;
if (bestMatchingHandler) {
bestMatchingHandler(event);
event.stopPropagation();
event.preventDefault();
const result = bestMatchingHandler(event);
if (result !== false) {
event.stopPropagation();
event.preventDefault();
}
}
// Add last keypress to buffer