fix: scrollable workaround if opener tab is pinned

ArnaudLigny · Jan 17, 2024 · 7957127 · 7957127
1 parent 3741b1a
commit 7957127
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 5 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "open-new-tab-after-current-tab",
-  "version": "0.4.6",
+  "version": "0.4.7",
   "description": "Opens new tab after the active tab, instead of last position.",
   "homepage": "https://dev.ligny.org/Open-New-Tab-After-Current-Tab/",
   "author": "Arnaud Ligny",

diff --git a/src/background.js b/src/background.js
@@ -62,9 +62,14 @@ function moveIt(tab, event) {
       index: moveToIndex,
     }, movedTab => {
       // Workaround for the Chrome and Edge scrollable tabstrip issue
-      chrome.tabs.update(movedTab.openerTabId, {active: true}, () => {
-        console.log(tab.windowId + ': active moved tab');
-        chrome.tabs.update(movedTab.id, {active: true});
+      // only if opener tab is not pinned
+      chrome.tabs.get(movedTab.openerTabId, async tab => {
+        if (!tab.pinned) {
+          chrome.tabs.update(movedTab.openerTabId, {active: true}, () => {
+            console.log(tab.windowId + ': active moved tab');
+            chrome.tabs.update(movedTab.id, {active: true});
+          });
+        }
       });
     });
     if (currentGroup >= 0) {

diff --git a/src/manifest.json b/src/manifest.json
@@ -1,6 +1,6 @@
 {
   "name": "Open New Tab After Current Tab",
-  "version": "0.4.6",
+  "version": "0.4.7",
   "description": "__MSG_extDescription__",
   "homepage_url": "https://dev.ligny.org/Open-New-Tab-After-Current-Tab/",
   "author": "Arnaud Ligny",