From 36e62781e27c19fade2df4d6a63e9062862ce7b0 Mon Sep 17 00:00:00 2001
From: mauro-balades <mauro.balades@tutanota.com>
Date: Sat, 7 Sep 2024 23:20:55 +0200
Subject: [PATCH] refactor: Update tab navigation logic in ZenViewSplitter.mjs

---
 src/ZenViewSplitter.mjs | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/ZenViewSplitter.mjs b/src/ZenViewSplitter.mjs
index c102c4a..6d4b77f 100644
--- a/src/ZenViewSplitter.mjs
+++ b/src/ZenViewSplitter.mjs
@@ -591,9 +591,11 @@ var gZenViewSplitter = new class {
     }
     let nextTabIndex = tabs.indexOf(gBrowser.selectedTab) + 1;
     if (nextTabIndex >= tabs.length) {
-      nextTabIndex = 0;
+      // Find the first non-hidden tab
+      nextTabIndex = tabs.findIndex(tab => !tab.hidden);
     } else if (nextTabIndex < 0) {
-      nextTabIndex = tabs.length - 1;
+      // reverse find the first non-hidden tab
+      nextTabIndex = tabs.slice().reverse().findIndex(tab => !tab.hidden);
     }
     const selected_tabs = gBrowser.selectedTab.multiselected 
       ? gBrowser.selectedTabs : [gBrowser.selectedTab, tabs[nextTabIndex]];