From 71bc9e841878b7f909fb0b458f818a32bd7edddb Mon Sep 17 00:00:00 2001 From: Rikinshah787 Date: Tue, 25 Aug 2026 16:07:37 -0700 Subject: [PATCH] fix(browser-extension): make the save-memory keyboard shortcut fire The keydown handler checked event.shiftKey && event.key === "m", but with Shift held the key value is "M", so the condition could never be true and Ctrl/Cmd+Shift+M did nothing on any page. Compare the key case-insensitively instead. --- apps/browser-extension/entrypoints/content/shared.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/apps/browser-extension/entrypoints/content/shared.ts b/apps/browser-extension/entrypoints/content/shared.ts index 126479089..8c3da811e 100644 --- a/apps/browser-extension/entrypoints/content/shared.ts +++ b/apps/browser-extension/entrypoints/content/shared.ts @@ -97,7 +97,8 @@ export function setupGlobalKeyboardShortcut() { if ( (event.ctrlKey || event.metaKey) && event.shiftKey && - event.key === "m" + // with Shift held, event.key is "M", never "m" + event.key.toLowerCase() === "m" ) { event.preventDefault() await saveMemory("keyboard_shortcut")