From 67237982c89f5904ca966d629338f4c1636601d5 Mon Sep 17 00:00:00 2001
From: "santasri.pachhal" <santasri.pachhal@sentientgeeks.com>
Date: Wed, 24 Jun 2026 00:36:27 +0530
Subject: [PATCH] feat(results): add video results skeleton and refresh
 detection summary UI

---
 .../components/VideoResultsSkeleton.tsx       | 105 ++++++++++++++++++
 src/app/(modules)/results/[videoId]/page.tsx  |   9 +-
 src/components/video/annotatedVideoPlayer.tsx |   2 +-
 src/components/video/currentDetectionBar.tsx  |  82 +++++++++-----
 4 files changed, 165 insertions(+), 33 deletions(-)
 create mode 100644 src/app/(modules)/results/[videoId]/components/VideoResultsSkeleton.tsx

diff --git a/src/app/(modules)/results/[videoId]/components/VideoResultsSkeleton.tsx b/src/app/(modules)/results/[videoId]/components/VideoResultsSkeleton.tsx
new file mode 100644
index 0000000..47e2de8
--- /dev/null
+++ b/src/app/(modules)/results/[videoId]/components/VideoResultsSkeleton.tsx
@@ -0,0 +1,105 @@
+'use client';
+
+import { Skeleton } from '@/components/ui/skeleton';
+
+function StatSkeleton() {
+  return (
+    <div className="flex flex-col items-center rounded-lg border border-muted bg-muted/30 p-4 text-center">
+      <Skeleton className="mb-2 size-9 rounded-md" />
+      <Skeleton className="h-6 w-12" />
+      <Skeleton className="mt-2 h-3 w-20" />
+    </div>
+  );
+}
+
+function DetectionLogSkeleton() {
+  return (
+    <div className="rounded-md border bg-card p-4">
+      <div className="mb-3 flex items-start justify-between gap-4">
+        <div className="space-y-2">
+          <Skeleton className="h-4 w-36" />
+          <Skeleton className="h-3 w-20" />
+        </div>
+        <div className="space-y-2">
+          <Skeleton className="h-3 w-12" />
+          <Skeleton className="h-3 w-10" />
+        </div>
+      </div>
+      <div className="space-y-2">
+        <Skeleton className="h-3 w-32" />
+        <Skeleton className="h-3 w-44" />
+      </div>
+    </div>
+  );
+}
+
+export function VideoResultsSkeleton() {
+  return (
+    <div className="space-y-8">
+      <div className="flex flex-col gap-4 sm:flex-row sm:items-center sm:justify-between">
+        <div className="flex min-w-0 items-start gap-3">
+          <Skeleton className="size-10 rounded-md" />
+          <div className="min-w-0 space-y-2">
+            <Skeleton className="h-7 w-60 max-w-full" />
+            <Skeleton className="h-3 w-80 max-w-full" />
+          </div>
+        </div>
+        <Skeleton className="h-8 w-20" />
+      </div>
+
+      <section className="overflow-hidden rounded-xl border bg-card">
+        <div className="border-b p-6 pb-4">
+          <div className="flex items-center gap-3">
+            <Skeleton className="size-10 rounded" />
+            <div className="space-y-2">
+              <Skeleton className="h-5 w-44" />
+              <Skeleton className="h-3 w-64" />
+            </div>
+          </div>
+        </div>
+
+        <div className="p-6">
+          <div className="grid grid-cols-1 gap-6 lg:grid-cols-3">
+            <div className="space-y-6 lg:col-span-2">
+              <Skeleton className="aspect-video w-full rounded-lg border" />
+
+              <section className="overflow-hidden rounded-lg border bg-card/80">
+                <div className="flex flex-col gap-3 border-b px-4 py-3 sm:flex-row sm:items-center sm:justify-between">
+                  <div className="space-y-2">
+                    <Skeleton className="h-4 w-48" />
+                    <Skeleton className="h-3 w-72 max-w-full" />
+                  </div>
+                  <Skeleton className="h-8 w-48" />
+                </div>
+                <div className="grid grid-cols-1 gap-2 p-3 sm:grid-cols-3">
+                  {Array.from({ length: 3 }).map((_, index) => (
+                    <Skeleton key={index} className="h-16 w-full rounded-md" />
+                  ))}
+                </div>
+              </section>
+            </div>
+
+            <div className="space-y-4">
+              <div className="flex items-center justify-between">
+                <Skeleton className="h-5 w-36" />
+              </div>
+              <div className="h-107.5 rounded-lg border bg-muted/20 p-4">
+                <div className="space-y-3">
+                  {Array.from({ length: 5 }).map((_, index) => (
+                    <DetectionLogSkeleton key={index} />
+                  ))}
+                </div>
+              </div>
+            </div>
+          </div>
+
+          <div className="mt-6 grid grid-cols-2 gap-3 md:grid-cols-5">
+            {Array.from({ length: 5 }).map((_, index) => (
+              <StatSkeleton key={index} />
+            ))}
+          </div>
+        </div>
+      </section>
+    </div>
+  );
+}
diff --git a/src/app/(modules)/results/[videoId]/page.tsx b/src/app/(modules)/results/[videoId]/page.tsx
index 2de7fda..55b77b9 100644
--- a/src/app/(modules)/results/[videoId]/page.tsx
+++ b/src/app/(modules)/results/[videoId]/page.tsx
@@ -2,13 +2,14 @@
 
 import { useMemo } from 'react';
 import { useParams, useRouter } from 'next/navigation';
-import { ArrowLeft, Loader2, TrendingUp } from 'lucide-react';
+import { ArrowLeft, TrendingUp } from 'lucide-react';
 import VideoPlayerSection from '@/components/videoPlayerSection';
 import { PageHeader } from '@/components/page-header';
 import { Button } from '@/components/ui/button';
 import { CompletedVideoResult, DetectionType } from '@/types';
 import { getDetectionModeConfig } from '@/constants/detectionModeConfig';
 import { useVideoResultsQuery } from '../hooks/useVideoResults';
+import { VideoResultsSkeleton } from './components/VideoResultsSkeleton';
 
 const inferDetectionType = (data: CompletedVideoResult): DetectionType => {
   if (data.detection_mode) return data.detection_mode as DetectionType;
@@ -50,11 +51,7 @@ export default function VideoResultsPage() {
   };
 
   if (isLoading) {
-    return (
-      <div className="min-h-[60vh] flex items-center justify-center">
-        <Loader2 className="h-8 w-8 animate-spin text-primary" />
-      </div>
-    );
+    return <VideoResultsSkeleton />;
   }
 
   if (error) {
diff --git a/src/components/video/annotatedVideoPlayer.tsx b/src/components/video/annotatedVideoPlayer.tsx
index b3066d8..027d04f 100644
--- a/src/components/video/annotatedVideoPlayer.tsx
+++ b/src/components/video/annotatedVideoPlayer.tsx
@@ -53,7 +53,7 @@ export default function AnnotatedVideoPlayer({
           ) : (
             <>
               <Loader2 className="h-7 w-7 animate-spin text-primary" />
-              <p className="text-xs font-medium uppercase tracking-widest">
+              <p className="text-xs">
                 {isLoading ? 'Loading annotated video...' : 'Preparing video...'}
               </p>
             </>
diff --git a/src/components/video/currentDetectionBar.tsx b/src/components/video/currentDetectionBar.tsx
index 371fb83..9f81cc5 100644
--- a/src/components/video/currentDetectionBar.tsx
+++ b/src/components/video/currentDetectionBar.tsx
@@ -1,7 +1,8 @@
 'use client';
 
-import { MapPin } from 'lucide-react';
-import { CompletedVideoResult, DetectionResultLog } from '@/types';
+import { Activity, AlertTriangle, MapPin, SignpostBig } from 'lucide-react';
+
+import type { CompletedVideoResult, DetectionResultLog } from '@/types';
 
 interface CurrentDetectionBarProps {
   activeLog?: DetectionResultLog;
@@ -17,17 +18,23 @@ export default function CurrentDetectionBar({
     {
       label: 'Potholes',
       value: currentCounts.unique_potholes || 0,
+      icon: AlertTriangle,
       className: 'text-orange-500',
+      iconClassName: 'bg-orange-500/10 text-orange-500',
     },
     {
       label: 'Signboards',
       value: currentCounts.unique_signboards || 0,
+      icon: SignpostBig,
       className: 'text-blue-500',
+      iconClassName: 'bg-blue-500/10 text-blue-500',
     },
     {
       label: 'Total',
       value: currentCounts.total_detections || 0,
+      icon: Activity,
       className: 'text-green-500',
+      iconClassName: 'bg-green-500/10 text-green-500',
     },
   ];
   const latitude = activeLog?.latitude;
@@ -38,33 +45,56 @@ export default function CurrentDetectionBar({
       : undefined;
 
   return (
-    <div className="rounded-lg border bg-card px-4 py-3">
-      {coordinates && (
-        <div className="mb-3 flex justify-end">
-          <div className="flex max-w-full items-center gap-2 rounded-md border bg-muted/30 px-3 py-1.5">
-            <MapPin className="size-3.5 shrink-0 text-muted-foreground" />
-            <span className="truncate font-mono text-xs font-semibold text-foreground">
+    <section className="overflow-hidden rounded-lg border bg-card/80">
+      <div className="flex flex-col gap-3 border-b px-4 py-3 sm:flex-row sm:items-center sm:justify-between">
+        <div>
+          <p className="text-sm font-semibold text-foreground">
+            Current Detection Summary
+          </p>
+          <p className="text-xs text-muted-foreground">
+            Counts update as the playback timeline moves.
+          </p>
+        </div>
+
+        {coordinates ? (
+          <div className="flex max-w-full items-center gap-2 rounded-md bg-muted/40 px-3 py-1.5 text-xs text-muted-foreground">
+            <MapPin className="size-3.5 shrink-0" />
+            <span className="shrink-0 font-medium">Location</span>
+            <span className="truncate font-mono font-semibold text-foreground">
               {coordinates}
             </span>
           </div>
-        </div>
-      )}
-
-      <div className="grid grid-cols-1 gap-3 sm:grid-cols-3">
-        {countItems.map((item) => (
-          <div
-            key={item.label}
-            className="flex items-center justify-between gap-3 rounded-lg border bg-muted/20 px-4 py-2"
-          >
-            <p className="text-xs font-bold uppercase tracking-wide text-muted-foreground">
-              {item.label}
-            </p>
-            <p className={`text-2xl font-bold leading-none ${item.className}`}>
-              {item.value}
-            </p>
-          </div>
-        ))}
+        ) : null}
       </div>
-    </div>
+
+      <div className="grid grid-cols-1 gap-2 p-3 sm:grid-cols-3">
+        {countItems.map((item) => {
+          const Icon = item.icon;
+
+          return (
+            <div
+              key={item.label}
+              className="flex items-center justify-between gap-3 rounded-md bg-muted/25 px-3 py-2.5 ring-1 ring-border/60"
+            >
+              <div className="flex min-w-0 items-center gap-3">
+                <span
+                  className={`flex size-8 shrink-0 items-center justify-center rounded-md ${item.iconClassName}`}
+                >
+                  <Icon className="size-4" />
+                </span>
+                <p className="truncate text-xs font-semibold uppercase tracking-wide text-muted-foreground">
+                  {item.label}
+                </p>
+              </div>
+              <p
+                className={`text-2xl font-bold leading-none ${item.className}`}
+              >
+                {item.value}
+              </p>
+            </div>
+          );
+        })}
+      </div>
+    </section>
   );
 }