From 2f9f42c9e26ebe7d8576d5975ad845a69874b316 Mon Sep 17 00:00:00 2001
From: Tom Butcher <thomasbutcher15@gmail.com>
Date: Thu, 17 Sep 2026 20:46:26 +0100
Subject: [PATCH] Refactor tab management in NavigationTabsContext

- Simplified the logic for adding new tabs by directly appending the next tab to the current tabs array, improving code readability and maintainability.
- Removed unnecessary calculations for determining the insertion index, streamlining the tab addition process.
---
 .../Dashboard/context/NavigationTabsContext.jsx      | 12 +-----------
 1 file changed, 1 insertion(+), 11 deletions(-)

diff --git a/src/components/Dashboard/context/NavigationTabsContext.jsx b/src/components/Dashboard/context/NavigationTabsContext.jsx
index 5b62d990..5779440b 100644
--- a/src/components/Dashboard/context/NavigationTabsContext.jsx
+++ b/src/components/Dashboard/context/NavigationTabsContext.jsx
@@ -112,17 +112,7 @@ export const NavigationTabsProvider = ({ children }) => {
       (tab) => tab.id === activeTabIdRef.current
     )
     const nextTab = cloneCurrentPageTab(currentTab, location)
-    setTabs((current) => {
-      const activeIndex = current.findIndex(
-        (tab) => tab.id === activeTabIdRef.current
-      )
-      const insertAt = activeIndex === -1 ? current.length : activeIndex + 1
-      return [
-        ...current.slice(0, insertAt),
-        nextTab,
-        ...current.slice(insertAt)
-      ]
-    })
+    setTabs((current) => [...current, nextTab])
     setActiveTabId(nextTab.id)
   }, [location])