feat: Enhance tab functionality by adding defaultUrl to tab properties and refining terminal tab visibility logic

Author: AbhishekA1509

src/components/ResourceBrowser/ResourceList/ResourceList.tsx

@@ -657,7 +657,11 @@ const ResourceList = () => {
                         const currentTab = tabs[index]
                         // We will render the fixed tab if it is selected, alive or it should remain mounted
                         // Not using filter, as the index is directly coupled with tabs indexes
-                        if (currentTab.isSelected || currentTab.shouldRemainMounted || currentTab.isAlive) {
+                        if (
+                            currentTab.isSelected ||
+                            currentTab.shouldRemainMounted ||
+                            (currentTab.isAlive && currentTab.id === ResourceBrowserTabsId.terminal)
+                        ) {
                             /* NOTE: need to retain terminal layout. Thus hiding it through visibility */
                             const hideClassName =
                                 tabs[index].name === AppDetailsTabs.terminal

src/components/common/DynamicTabs/types.ts

@@ -94,7 +94,9 @@ export interface PopulateTabDataPropsType
             | 'hideName'
             | 'id'
         >,
-        Required<Pick<DynamicTabType, 'shouldRemainMounted' | 'title' | 'showNameOnSelect' | 'isAlpha'>> {}
+        Required<
+            Pick<DynamicTabType, 'shouldRemainMounted' | 'title' | 'showNameOnSelect' | 'isAlpha' | 'defaultUrl'>
+        > {}
 
 export interface AddTabParamsType
     extends Pick<PopulateTabDataPropsType, 'name' | 'url' | 'tippyConfig'>,

src/components/common/DynamicTabs/useTabs.ts

@@ -73,6 +73,7 @@ export function useTabs(persistenceKey: string, fallbackTabIndex = FALLBACK_TAB)
         lastActiveTabId,
         shouldRemainMounted,
         isAlpha,
+        defaultUrl,
     }: PopulateTabDataPropsType): DynamicTabType => ({
         id,
         name,
@@ -90,6 +91,7 @@ export function useTabs(persistenceKey: string, fallbackTabIndex = FALLBACK_TAB)
         lastActiveTabId,
         shouldRemainMounted,
         isAlpha: isAlpha || false,
+        defaultUrl: defaultUrl || null,
     })
 
     const getTabDataFromLocalStorage = () => localStorage.getItem(TAB_DATA_LOCAL_STORAGE_KEY)
@@ -162,6 +164,7 @@ export function useTabs(persistenceKey: string, fallbackTabIndex = FALLBACK_TAB)
             lastActiveTabId: null,
             shouldRemainMounted: _initTab.shouldRemainMounted,
             isAlpha: _initTab.isAlpha,
+            defaultUrl: _initTab.defaultUrl,
         })
     }
 
@@ -335,6 +338,7 @@ export function useTabs(persistenceKey: string, fallbackTabIndex = FALLBACK_TAB)
                             lastActiveTabId: getLastActiveTabIdFromTabs(prevTabs, _id),
                             shouldRemainMounted: false,
                             isAlpha: false,
+                            defaultUrl: null,
                         }),
                     )
                 }
@@ -373,6 +377,7 @@ export function useTabs(persistenceKey: string, fallbackTabIndex = FALLBACK_TAB)
                                       ...tab,
                                       isSelected: false,
                                       isAlive: false,
+                                      url: tab.defaultUrl || tab.url,
                                   }
                               }