Small fixes for context menu

obsidianmd · Sep 22, 2024 · 1d67526 · 1d67526
1 parent 1ecf687
commit 1d67526
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 14 deletions.
diff --git a/src/background-firefox.ts b/src/background-firefox.ts
@@ -61,12 +61,6 @@ browser.runtime.onMessage.addListener((request, sender, sendResponse) => {
 });
 
 browser.runtime.onInstalled.addListener(() => {
-	browser.action.onClicked.addListener((tab) => {
-		if (tab.id) {
-			browser.tabs.sendMessage(tab.id, { action: "ping" });
-		}
-	});
-
 	function createContextMenu() {
 		browser.contextMenus.create({
 			id: "open-obsidian-clipper",
@@ -77,7 +71,7 @@ browser.runtime.onInstalled.addListener(() => {
 
 	browser.contextMenus.onClicked.addListener((info, tab) => {
 		if (info.menuItemId === "open-obsidian-clipper") {
-			browser.browserAction.openPopup();
+			browser.action.openPopup();
 		}
 	});
 

diff --git a/src/background-safari.ts b/src/background-safari.ts
@@ -61,12 +61,6 @@ browser.runtime.onMessage.addListener((request, sender, sendResponse) => {
 });
 
 browser.runtime.onInstalled.addListener(() => {
-	browser.action.onClicked.addListener((tab) => {
-		if (tab.id) {
-			browser.tabs.sendMessage(tab.id, { action: "ping" });
-		}
-	});
-
 	function createContextMenu() {
 		browser.contextMenus.create({
 			id: "open-obsidian-clipper",
@@ -77,7 +71,7 @@ browser.runtime.onInstalled.addListener(() => {
 
 	browser.contextMenus.onClicked.addListener((info, tab) => {
 		if (info.menuItemId === "open-obsidian-clipper") {
-			browser.browserAction.openPopup();
+			browser.action.openPopup();
 		}
 	});