feat: remove prop for align active border in tab group

arunjaindev · arunjaindev · commit 7c25cbb75697 · 2025-04-28T14:11:24.000+05:30
diff --git a/src/Pages/GlobalConfigurations/Authorization/Shared/components/AppPermissions/AppPermissions.component.tsx b/src/Pages/GlobalConfigurations/Authorization/Shared/components/AppPermissions/AppPermissions.component.tsx
@@ -996,7 +996,6 @@ const AppPermissions = () => {
                               }
                             : [],
                     )}
-                    alignActiveBorderWithContainer
                 />
             </div>
             <div>
diff --git a/src/components/ApplicationGroup/AppGroupDetailsRoute.tsx b/src/components/ApplicationGroup/AppGroupDetailsRoute.tsx
@@ -674,7 +674,7 @@ export const EnvHeader = ({
             },
         ]
 
-        return <TabGroup tabs={tabs} hideTopPadding alignActiveBorderWithContainer />
+        return <TabGroup tabs={tabs} hideTopPadding />
     }
 
     const renderBreadcrumbs = () => {
diff --git a/src/components/ApplicationGroup/CreateAppGroup.tsx b/src/components/ApplicationGroup/CreateAppGroup.tsx
@@ -208,7 +208,7 @@ export default function CreateAppGroup({
         }
 
         return (
-            <div className='flexbox-col flex-grow-1'>
+            <div className="flexbox-col flex-grow-1">
                 <SearchBar
                     inputProps={{
                         placeholder: `Search ${filterParentTypeMsg}'s`,
@@ -218,7 +218,7 @@ export default function CreateAppGroup({
                     handleEnter={handleAppSearchEnterChange}
                     dataTestId="create-app-group"
                 />
-                <div className='flexbox-col flex-grow-1'>
+                <div className="flexbox-col flex-grow-1">
                     {filteredAuthList.length <= 0 && filteredUnAuthList.length <= 0
                         ? renderEmptyState()
                         : filteredAuthList.map((app) => {
@@ -271,7 +271,7 @@ export default function CreateAppGroup({
         }
 
         return (
-            <div className='flexbox-col flex-grow-1'>
+            <div className="flexbox-col flex-grow-1">
                 <SearchBar
                     inputProps={{
                         placeholder: `Search ${filterParentTypeMsg}'s`,
@@ -280,7 +280,7 @@ export default function CreateAppGroup({
                     initialSearchText={allAppSearchText}
                     handleEnter={handleAllAppSearchEnterChange}
                 />
-                <div className='flexbox-col flex-grow-1'>
+                <div className="flexbox-col flex-grow-1">
                     {filteredAllApps.length <= 0
                         ? renderEmptyState()
                         : filteredAllApps.map((app) => (
@@ -377,7 +377,7 @@ export default function CreateAppGroup({
                         }
                     />
                 </div>
-                <div className='flexbox-col flex-grow-1'>
+                <div className="flexbox-col flex-grow-1">
                     <div className="dc__border-bottom mb-8">
                         <TabGroup
                             tabs={[
@@ -395,7 +395,6 @@ export default function CreateAppGroup({
                                 ),
                             ]}
                             hideTopPadding
-                            alignActiveBorderWithContainer
                         />
                     </div>
                     {selectedTab === CreateGroupTabs.SELECTED_APPS || selectedTab === CreateGroupTabs.SELECTED_ENV
diff --git a/src/components/CIPipelineN/CIPipeline.tsx b/src/components/CIPipelineN/CIPipeline.tsx
@@ -822,7 +822,6 @@ export default function CIPipeline({
                                       ]
                             }
                             hideTopPadding
-                            alignActiveBorderWithContainer
                         />
                     </div>
                 )}
diff --git a/src/components/ClusterNodes/NodeDetails.tsx b/src/components/ClusterNodes/NodeDetails.tsx
@@ -238,7 +238,7 @@ const NodeDetails = ({ addTab, lowercaseKindToResourceGroupMap, updateTabUrl }:
 
         return (
             <div className="pl-20 dc__border-bottom flex dc__gap-16">
-                <TabGroup tabs={tabs} alignActiveBorderWithContainer size={ComponentSizeType.medium} />
+                <TabGroup tabs={tabs} size={ComponentSizeType.medium} />
                 {nodeControls()}
             </div>
         )
@@ -395,7 +395,7 @@ const NodeDetails = ({ addTab, lowercaseKindToResourceGroupMap, updateTabUrl }:
             <div>
                 <div className="dc__border-bottom dc__position-sticky dc__top-0 dc__zi-1 bg__primary">
                     <div className="en-2 bw-1 dc__top-radius-4 bg__primary dc__no-bottom-border px-20">
-                        <TabGroup tabs={tabs} alignActiveBorderWithContainer />
+                        <TabGroup tabs={tabs} />
                     </div>
                 </div>
                 <div className="en-2 bw-1 br-4 dc__no-top-radius dc__no-top-border bg__primary mb-20">
diff --git a/src/components/Jobs/JobDetails/JobDetails.tsx b/src/components/Jobs/JobDetails/JobDetails.tsx
@@ -211,7 +211,7 @@ const JobHeader = ({ jobName }: { jobName: string }) => {
             },
         ]
 
-        return <TabGroup tabs={tabs} hideTopPadding alignActiveBorderWithContainer />
+        return <TabGroup tabs={tabs} hideTopPadding />
     }
 
     const renderBreadcrumbs = () => {
diff --git a/src/components/app/details/AppHeader.tsx b/src/components/app/details/AppHeader.tsx
@@ -240,7 +240,7 @@ export const AppHeader = ({
             },
         ]
 
-        return <TabGroup tabs={tabs} hideTopPadding alignActiveBorderWithContainer />
+        return <TabGroup tabs={tabs} hideTopPadding />
     }
 
     const renderBreadcrumbs = () => (
diff --git a/src/components/app/details/cIDetails/CIDetails.tsx b/src/components/app/details/cIDetails/CIDetails.tsx
@@ -495,7 +495,6 @@ export const Details = ({
                                       ]
                                     : []),
                             ]}
-                            alignActiveBorderWithContainer
                         />
                     </div>
                 </>
diff --git a/src/components/app/list-new/AppList.tsx b/src/components/app/list-new/AppList.tsx
@@ -398,7 +398,7 @@ const AppList = ({ isArgoInstalled }: AppListPropType) => {
 
         return (
             <div className="app-tabs-wrapper px-20">
-                <TabGroup tabs={tabs} rightComponent={rightComponent} alignActiveBorderWithContainer />
+                <TabGroup tabs={tabs} rightComponent={rightComponent} />
             </div>
         )
     }
diff --git a/src/components/cdPipeline/CDPipeline.tsx b/src/components/cdPipeline/CDPipeline.tsx
@@ -1307,7 +1307,6 @@ export default function CDPipeline({
                                 getNavLink(`post-build`, BuildStageVariable.PostBuild),
                             ]}
                             hideTopPadding
-                            alignActiveBorderWithContainer
                         />
                     </div>
                 )}
@@ -1439,7 +1438,6 @@ export default function CDPipeline({
                                           },
                                       }]),
                             ]}
-                            alignActiveBorderWithContainer
                         />
                     </div>
                 )}
diff --git a/src/components/ciPipeline/Webhook/WebhookDetailsModal.tsx b/src/components/ciPipeline/Webhook/WebhookDetailsModal.tsx
@@ -300,7 +300,6 @@ export const WebhookDetailsModal = ({ close, isTemplateView }: WebhookDetailType
                         },
                     }))}
                     hideTopPadding
-                    alignActiveBorderWithContainer={!isChildTab}
                     size={isChildTab ? ComponentSizeType.medium : ComponentSizeType.large}
                 />
             </div>
diff --git a/src/components/notifications/Notifications.tsx b/src/components/notifications/Notifications.tsx
@@ -60,7 +60,6 @@ export default class Notifications extends Component<NotificationsProps, {}> {
                                 },
                             },
                         ]}
-                        alignActiveBorderWithContainer
                     />
                 </div>
                 <ErrorBoundary>
diff --git a/src/components/security/Security.tsx b/src/components/security/Security.tsx
@@ -62,7 +62,6 @@ export class Security extends Component<SecurityProps> {
                     },
                 ]}
                 hideTopPadding
-                alignActiveBorderWithContainer
             />
         )
     }
diff --git a/src/components/v2/appDetails/k8Resource/nodeDetail/EphemeralContainerDrawer.tsx b/src/components/v2/appDetails/k8Resource/nodeDetail/EphemeralContainerDrawer.tsx
@@ -342,7 +342,6 @@ const EphemeralContainerDrawer = ({
                         },
                     ]}
                     hideTopPadding
-                    alignActiveBorderWithContainer
                 />
             </div>
         )
diff --git a/src/components/v2/appDetails/k8Resource/nodeDetail/NodeDetail.component.tsx b/src/components/v2/appDetails/k8Resource/nodeDetail/NodeDetail.component.tsx
@@ -492,7 +492,7 @@ const NodeDetailComponent = ({
         <>
             <div className="w-100 pr-20 pl-20 bg__primary flex border__secondary--bottom dc__content-space h-32">
                 <div className="flex left">
-                    <TabGroup tabs={TAB_GROUP_CONFIG} size={ComponentSizeType.medium} alignActiveBorderWithContainer />
+                    <TabGroup tabs={TAB_GROUP_CONFIG} size={ComponentSizeType.medium} />
                     {selectedTabName === NodeDetailTab.TERMINAL && (
                         <>
                             <div className="ml-12 mr-5 tab-cell-border" />
diff --git a/src/components/v2/appDetails/k8Resource/nodeType/Node.component.tsx b/src/components/v2/appDetails/k8Resource/nodeType/Node.component.tsx
@@ -543,7 +543,6 @@ const NodeComponent = ({
                                         },
                                     ]}
                                     size={ComponentSizeType.xl}
-                                    alignActiveBorderWithContainer
                                 />
                             </div>
                         )}
diff --git a/src/components/v2/appDetails/k8Resource/nodeType/PodHeader.component.tsx b/src/components/v2/appDetails/k8Resource/nodeType/PodHeader.component.tsx
@@ -93,7 +93,7 @@ const PodHeaderComponent = ({ callBack }: { callBack: (isNewPod: boolean) => voi
 
     return (
         <div className="dc__border-bottom px-16">
-            <TabGroup tabs={tabs} size={ComponentSizeType.xl} alignActiveBorderWithContainer />
+            <TabGroup tabs={tabs} size={ComponentSizeType.xl} />
         </div>
     )
 }
diff --git a/src/components/v2/appDetails/sourceInfo/scaleWorkloads/ScaleWorkloadsModal.component.tsx b/src/components/v2/appDetails/sourceInfo/scaleWorkloads/ScaleWorkloadsModal.component.tsx
@@ -228,7 +228,6 @@ export default function ScaleWorkloadsModal({ appId, onClose, history }: ScaleWo
                             'data-testid': `scale-workloads-tab-${index}`,
                         },
                     }))}
-                    alignActiveBorderWithContainer
                 />
             </div>
         )
diff --git a/src/components/v2/chartDeploymentHistory/ChartDeploymentHistory.component.tsx b/src/components/v2/chartDeploymentHistory/ChartDeploymentHistory.component.tsx
@@ -403,7 +403,6 @@ const ChartDeploymentHistory = ({
                             'data-testid': `nav-bar-option-${index}`,
                         },
                     }))}
-                    alignActiveBorderWithContainer
                 />
             </div>
         )
diff --git a/src/components/v2/devtronStackManager/AboutDevtronView.tsx b/src/components/v2/devtronStackManager/AboutDevtronView.tsx
@@ -68,7 +68,6 @@ const AboutDevtronView = ({
                             onClick: () => handleTabChange(index),
                         },
                     }))}
-                    alignActiveBorderWithContainer
                 />
             </div>
         )
diff --git a/src/components/v2/headers/AppHeader.component.tsx b/src/components/v2/headers/AppHeader.component.tsx
@@ -117,7 +117,6 @@ const AppHeaderComponent = () => {
                     },
                 ]}
                 hideTopPadding
-                alignActiveBorderWithContainer
             />
         )
     }
diff --git a/src/components/v2/headers/ChartHeader.component.tsx b/src/components/v2/headers/ChartHeader.component.tsx
@@ -114,7 +114,7 @@ const ChartHeaderComponent = ({ errorResponseCode }: ChartHeaderComponentType) =
             )
         }
 
-        return <TabGroup tabs={tabs} hideTopPadding alignActiveBorderWithContainer />
+        return <TabGroup tabs={tabs} hideTopPadding />
     }
 
     return (
diff --git a/src/components/v2/headers/EAHeader.component.tsx b/src/components/v2/headers/EAHeader.component.tsx
@@ -90,7 +90,7 @@ const EAHeaderComponent = ({ title, redirectURL, showAppDetailsOnly = false }: E
             )
         }
 
-        return <TabGroup tabs={tabs} hideTopPadding alignActiveBorderWithContainer />
+        return <TabGroup tabs={tabs} hideTopPadding />
     }
 
     return (

Original file line number	Diff line number	Diff line change
`@@ -996,7 +996,6 @@ const AppPermissions = () => {`
`996`	`996`	`}`
`997`	`997`	`: [],`
`998`	`998`	`)}`
`999`		`- alignActiveBorderWithContainer`
`1000`	`999`	`/>`
`1001`	`1000`	`</div>`
`1002`	`1001`	`<div>`
Original file line number	Diff line number	Diff line change
`@@ -674,7 +674,7 @@ export const EnvHeader = ({`
`674`	`674`	`},`
`675`	`675`	`]`
`676`	`676`
`677`		`- return <TabGroup tabs={tabs} hideTopPadding alignActiveBorderWithContainer />`
	`677`	`+ return <TabGroup tabs={tabs} hideTopPadding />`
`678`	`678`	`}`
`679`	`679`
`680`	`680`	`const renderBreadcrumbs = () => {`
Original file line number	Diff line number	Diff line change
`@@ -822,7 +822,6 @@ export default function CIPipeline({`
`822`	`822`	`]`
`823`	`823`	`}`
`824`	`824`	`hideTopPadding`
`825`		`- alignActiveBorderWithContainer`
`826`	`825`	`/>`
`827`	`826`	`</div>`
`828`	`827`	`)}`
Original file line number	Diff line number	Diff line change
`@@ -211,7 +211,7 @@ const JobHeader = ({ jobName }: { jobName: string }) => {`
`211`	`211`	`},`
`212`	`212`	`]`
`213`	`213`
`214`		`- return <TabGroup tabs={tabs} hideTopPadding alignActiveBorderWithContainer />`
	`214`	`+ return <TabGroup tabs={tabs} hideTopPadding />`
`215`	`215`	`}`
`216`	`216`
`217`	`217`	`const renderBreadcrumbs = () => {`
Original file line number	Diff line number	Diff line change
`@@ -240,7 +240,7 @@ export const AppHeader = ({`
`240`	`240`	`},`
`241`	`241`	`]`
`242`	`242`
`243`		`- return <TabGroup tabs={tabs} hideTopPadding alignActiveBorderWithContainer />`
	`243`	`+ return <TabGroup tabs={tabs} hideTopPadding />`
`244`	`244`	`}`
`245`	`245`
`246`	`246`	`const renderBreadcrumbs = () => (`
Original file line number	Diff line number	Diff line change
`@@ -495,7 +495,6 @@ export const Details = ({`
`495`	`495`	`]`
`496`	`496`	`: []),`
`497`	`497`	`]}`
`498`		`- alignActiveBorderWithContainer`
`499`	`498`	`/>`
`500`	`499`	`</div>`
`501`	`500`	`</>`
Original file line number	Diff line number	Diff line change
`@@ -398,7 +398,7 @@ const AppList = ({ isArgoInstalled }: AppListPropType) => {`
`398`	`398`
`399`	`399`	`return (`
`400`	`400`	`<div className="app-tabs-wrapper px-20">`
`401`		`- <TabGroup tabs={tabs} rightComponent={rightComponent} alignActiveBorderWithContainer />`
	`401`	`+ <TabGroup tabs={tabs} rightComponent={rightComponent} />`
`402`	`402`	`</div>`
`403`	`403`	`)`
`404`	`404`	`}`