draupnir4all-web

commit 4c54d5ebd11f283a4beee9ae91721f3f3e38eda6
parent 67d05c891d769795fa8cdbdca6eb0cef67a30fed
Author: MTRNord <mtrnord1@gmail.com>
Date:   Sat, 26 Apr 2025 22:05:23 +0200

Some more cleanup on the analytics page to remove the double header

Signed-off-by: MTRNord <mtrnord1@gmail.com>

Diffstat:
M
src/app/dashboard/analytics/page.tsx
 | 
28
++++++++++++----------------
M
src/app/dashboard/overview/page.tsx
 | 
3
+++
M
src/components/dashboard/tab-navigation.tsx
 | 
2
+-

3 files changed, 16 insertions(+), 17 deletions(-)
diff --git a/src/app/dashboard/analytics/page.tsx b/src/app/dashboard/analytics/page.tsx
@@ -93,22 +93,18 @@ export default function AnalyticsDashboard() {
         <>
             <TabNavigation selectedTeam={selectedTeam} currentTab="analytics" />
             <div className="space-y-4">
-                <div className="flex items-center justify-between">
-                    <h2 className="text-2xl font-bold">{selectedTeam.name} Analytics</h2>
-                    <div className="flex items-center gap-2">
-                        <Select value={timeRange} onValueChange={setTimeRange}>
-                            <SelectTrigger className="w-[120px] h-8 bg-gray-900 border-gray-800">
-                                <SelectValue placeholder="Time Range" />
-                            </SelectTrigger>
-                            <SelectContent className="bg-gray-900 border-gray-800">
-                                <SelectItem value="month">Past Month</SelectItem>
-                                <SelectItem value="quarter">Past Quarter</SelectItem>
-                                <SelectItem value="year">Past Year</SelectItem>
-                                <SelectItem value="all">All Time</SelectItem>
-                            </SelectContent>
-                        </Select>
-
-                    </div>
+                <div className="flex items-center  justify-end">
+                    <Select value={timeRange} onValueChange={setTimeRange}>
+                        <SelectTrigger className="w-[120px] h-8 bg-gray-900 border-gray-800">
+                            <SelectValue placeholder="Time Range" />
+                        </SelectTrigger>
+                        <SelectContent className="bg-gray-900 border-gray-800">
+                            <SelectItem value="month">Past Month</SelectItem>
+                            <SelectItem value="quarter">Past Quarter</SelectItem>
+                            <SelectItem value="year">Past Year</SelectItem>
+                            <SelectItem value="all">All Time</SelectItem>
+                        </SelectContent>
+                    </Select>
                 </div>
 
                 <div className="grid gap-4 md:grid-cols-2 lg:grid-cols-4">
diff --git a/src/app/dashboard/overview/page.tsx b/src/app/dashboard/overview/page.tsx
@@ -13,6 +13,9 @@ interface OverviewPageProps {
 }
 
 async function fetchData(teamIdParam?: string): Promise<OverviewPageProps> {
+    "use cache";
+    // TODO: Check if we really can cache the real data here
+
     const selectedTeam = mockTeams.find((t) => t.id === teamIdParam) || mockTeams[0];
     const reports = mockReports.filter((report) => report.teamId === selectedTeam.id);
     const policyLists = mockPolicyLists.filter((list) => list.teamId === selectedTeam.id);
diff --git a/src/components/dashboard/tab-navigation.tsx b/src/components/dashboard/tab-navigation.tsx
@@ -11,7 +11,7 @@ export default function TabNavigation({ selectedTeam, currentTab }: TabNavigatio
     return (
         <div className="flex items-center justify-between mb-8">
             <h1 className="text-3xl font-bold tracking-tight">{selectedTeam.name} Dashboard</h1>
-            <Tabs defaultValue={currentTab} className="hidden md:block">
+            <Tabs activationMode="manual" defaultValue={currentTab} className="hidden md:block">
                 <TabsList className="bg-gray-900">
                     <TabsTrigger value="overview" asChild>
                         <Link href={`/dashboard/overview?team=${selectedTeam.id}`}>Overview</Link>