chore: simplify anchor gathering

igorlogius · Mar 28, 2024 · ad31ad0 · ad31ad0
1 parent c970666
commit ad31ad0
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 43 deletions.
diff --git a/background.js b/background.js
@@ -90,7 +90,16 @@ browser.menus.create({
       // link
       createTempContainerTab(clickdata.linkUrl, openAsActive);
     } else if (clickdata.selectionText) {
-      const links = await browser.tabs.sendMessage(tab.id, "getLinks");
+      const ret = await browser.tabs.executeScript({
+        code: `
+          selection = getSelection();
+             [...document.links]
+          .filter((anchor) => selection.containsNode(anchor, true))
+        .map((link) => link.href );
+          `,
+      });
+
+      const links = ret[0];
 
       for (const link of links) {
         createTempContainerTab(link, false);

diff --git a/content.js b/content.js
diff --git a/manifest.json b/manifest.json
@@ -17,14 +17,6 @@
       "description": "Open current tab url in multiple temp container"
     }
   },
-  "content_scripts": [
-    {
-      "matches": ["<all_urls>"],
-      "all_frames": true,
-      "js": ["content.js"],
-      "run_at": "document_start"
-    }
-  ],
   "description": "Open the page,tab,link or bookmark in a new temporary container tab",
   "icons": {
     "96": "icon.png"
@@ -35,6 +27,7 @@
     "page": "options.html"
   },
   "permissions": [
+    "activeTab",
     "webNavigation",
     "storage",
     "contextualIdentities",
@@ -43,5 +36,5 @@
     "tabs"
   ],
   "optional_permissions": ["bookmarks", "history"],
-  "version": "1.1.24"
+  "version": "1.1.25"
 }