Skip to content

Commit 178c37c

Browse files
committed
useKeyDownHandlers null safe
1 parent ad3fbbb commit 178c37c

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

client/modules/IDE/hooks/useKeyDownHandlers.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ export default function useKeyDownHandlers(keyHandlers) {
2020
const handlers = useRef(keyHandlers);
2121

2222
useEffect(() => {
23-
handlers.current = mapKeys(keyHandlers, (value, key) => key.toLowerCase());
23+
handlers.current = mapKeys(keyHandlers, (value, key) => key?.toLowerCase());
2424
}, [keyHandlers]);
2525

2626
/**
@@ -32,12 +32,12 @@ export default function useKeyDownHandlers(keyHandlers) {
3232
const isMac = navigator.userAgent.toLowerCase().indexOf('mac') !== -1;
3333
const isCtrl = isMac ? e.metaKey : e.ctrlKey;
3434
if (e.shiftKey && isCtrl) {
35-
handlers.current[`ctrl-shift-${e.key.toLowerCase()}`]?.(e);
35+
handlers.current[`ctrl-shift-${e.key?.toLowerCase()}`]?.(e);
3636
}
3737
if (isCtrl) {
38-
handlers.current[`ctrl-${e.key.toLowerCase()}`]?.(e);
38+
handlers.current[`ctrl-${e.key?.toLowerCase()}`]?.(e);
3939
}
40-
handlers.current[e.key.toLowerCase()]?.(e);
40+
handlers.current[e.key?.toLowerCase()]?.(e);
4141
}, []);
4242

4343
useEffect(() => {

0 commit comments

Comments
 (0)