From 2f435f835d81e6df33610fbd4090e6bcf4289c89 Mon Sep 17 00:00:00 2001
From: Bread Genie <msuhailbh07@gmail.com>
Date: Tue, 5 Nov 2024 15:43:47 +0530
Subject: [PATCH] refactor(SiteInsights): deduplicate code and minor style
 changes

---
 .../src2/components/site/SiteInsights.vue     | 36 ++++++++-----------
 1 file changed, 15 insertions(+), 21 deletions(-)

diff --git a/dashboard/src2/components/site/SiteInsights.vue b/dashboard/src2/components/site/SiteInsights.vue
index 802e2eafdf..606edc4bf0 100644
--- a/dashboard/src2/components/site/SiteInsights.vue
+++ b/dashboard/src2/components/site/SiteInsights.vue
@@ -5,35 +5,29 @@
 			'flex-col': $isMobile
 		}"
 	>
-		<div v-if="$isMobile" class="ml-5 mt-5 w-60 divide-y rounded-sm border">
+		<div
+			:class="{
+				'ml-5 mt-5 w-60 divide-y rounded-sm border': $isMobile,
+				'w-60': !$isMobile
+			}"
+		>
 			<template v-for="tab in tabs">
 				<router-link
 					:to="{ name: tab.value }"
-					v
 					class="flex cursor-pointer text-base text-gray-600 hover:bg-gray-100"
 					:class="{
 						' bg-gray-50 text-gray-800': isActiveTab(tab),
-						'text-gray-600': !isActiveTab(tab)
+						'text-gray-600': !isActiveTab(tab),
+						'border-b': !$isMobile
 					}"
 				>
-					<div class="px-4 py-2">
-						{{ tab.label }}
-					</div>
-				</router-link>
-			</template>
-		</div>
-		<div v-if="!$isMobile" class="w-60">
-			<template v-for="tab in tabs">
-				<router-link
-					:to="{ name: tab.value }"
-					v
-					class="flex cursor-pointer border-b text-base text-gray-600 hover:bg-gray-100"
-					:class="{
-						' bg-gray-50 text-gray-800': isActiveTab(tab),
-						'text-gray-600': !isActiveTab(tab)
-					}"
-				>
-					<div class="px-4 py-2">
+					<div
+						class="px-4"
+						:class="{
+							'py-2': $isMobile,
+							'py-2.5': !$isMobile
+						}"
+					>
 						{{ tab.label }}
 					</div>
 				</router-link>