From 485276e46caf51b6671a13cdd55f95342f12864b Mon Sep 17 00:00:00 2001 From: MohamedBassem Date: Sun, 18 Aug 2024 23:07:02 +0300 Subject: fix(extension): Upgrade crxjs to 2.0 to add support for firefox --- apps/browser-extension/src/background/background.ts | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'apps/browser-extension/src') diff --git a/apps/browser-extension/src/background/background.ts b/apps/browser-extension/src/background/background.ts index cab58aa9..7622c07e 100644 --- a/apps/browser-extension/src/background/background.ts +++ b/apps/browser-extension/src/background/background.ts @@ -69,9 +69,11 @@ async function handleContextMenuClick(info: chrome.contextMenus.OnClickData) { }; } if (newBookmark) { - await chrome.storage.session.set({ + chrome.storage.session.set({ [NEW_BOOKMARK_REQUEST_KEY_NAME]: newBookmark, }); + // NOTE: Firefox only allows opening context menus if it's triggered by a user action. + // awaiting on any promise before calling this function will lose the "user action" context. await chrome.action.openPopup(); } } -- cgit v1.2.3-70-g09d2