From a2eb5bfcd770544e2a6ef2d75423c6f6997fc745 Mon Sep 17 00:00:00 2001
From: Matt <mattskent@gmail.com>
Date: Sat, 4 Jan 2025 00:23:42 -0500
Subject: [PATCH] Add missing skeletons

---
 .../dashboard/MiniRealtimeAnalytics.jsx       | 175 +++++++++---------
 .../components/dashboard/MiniSalesChart.jsx   |   4 +-
 2 files changed, 92 insertions(+), 87 deletions(-)

diff --git a/dashboard/src/components/dashboard/MiniRealtimeAnalytics.jsx b/dashboard/src/components/dashboard/MiniRealtimeAnalytics.jsx
index 250a514..f15f446 100644
--- a/dashboard/src/components/dashboard/MiniRealtimeAnalytics.jsx
+++ b/dashboard/src/components/dashboard/MiniRealtimeAnalytics.jsx
@@ -17,6 +17,35 @@ import {
   SkeletonBarChart,
   processBasicData,
 } from "./RealtimeAnalytics";
+import { Skeleton } from "@/components/ui/skeleton";
+
+const SkeletonCard = ({ colorScheme = "sky" }) => (
+  <Card className={`w-full h-[150px] bg-gradient-to-br from-sky-900 to-sky-800 backdrop-blur-sm`}>
+    <CardHeader className="flex flex-row items-center justify-between space-y-0 p-4 pb-2">
+      <CardTitle>
+        <div className="space-y-2">
+          <Skeleton className={`h-4 w-24 bg-${colorScheme}-300/20`} />
+        </div>
+      </CardTitle>
+      <div className="relative p-2">
+        <div className={`absolute inset-0 rounded-full bg-${colorScheme}-300/20`} />
+        <div className="h-5 w-5 relative rounded-full bg-${colorScheme}-300/20" />
+      </div>
+    </CardHeader>
+    <CardContent className="p-4 pt-0">
+      <div className="space-y-4">
+        <div className="space-y-2">
+          <Skeleton className={`h-8 w-32 bg-${colorScheme}-300/20`} />
+          <div className="flex justify-between items-center">
+            <div className="space-y-1">
+              <Skeleton className={`h-4 w-24 bg-${colorScheme}-300/20`} />
+            </div>
+          </div>
+        </div>
+      </div>
+    </CardContent>
+  </Card>
+);
 
 const MiniRealtimeAnalytics = () => {
   const [basicData, setBasicData] = useState({
@@ -76,95 +105,68 @@ const MiniRealtimeAnalytics = () => {
     };
   }, [isPaused]);
 
-  if (loading && !basicData) {
-    return (
-      <div>
-        <div className="grid grid-cols-2 gap-2 mt-1 mb-2">
-          <Card className="h-[150px] bg-gradient-to-br from-sky-900 to-sky-800 backdrop-blur-sm">
-            <CardHeader className="flex flex-row items-center justify-between space-y-0 p-4 pb-2">
-              <CardTitle className="text-sky-100 font-bold text-md">
-                <Skeleton className="h-4 w-24 bg-sky-700" />
-              </CardTitle>
-              <div className="relative p-2">
-                <div className="absolute inset-0 rounded-full bg-sky-300" />
-                <Skeleton className="h-5 w-5 bg-sky-700 relative rounded-full" />
-              </div>
-            </CardHeader>
-            <CardContent className="p-4 pt-0">
-              <div className="space-y-2">
-                <Skeleton className="h-8 w-20 bg-sky-700" />
-                <Skeleton className="h-4 w-32 bg-sky-700" />
-              </div>
-            </CardContent>
-          </Card>
-          <Card className="h-[150px] bg-gradient-to-br from-sky-900 to-sky-800 backdrop-blur-sm">
-            <CardHeader className="flex flex-row items-center justify-between space-y-0 p-4 pb-2">
-              <CardTitle className="text-sky-100 font-bold text-md">
-                <Skeleton className="h-4 w-24 bg-sky-700" />
-              </CardTitle>
-              <div className="relative p-2">
-                <div className="absolute inset-0 rounded-full bg-sky-300" />
-                <Skeleton className="h-5 w-5 bg-sky-700 relative rounded-full" />
-              </div>
-            </CardHeader>
-            <CardContent className="p-4 pt-0">
-              <div className="space-y-2">
-                <Skeleton className="h-8 w-20 bg-sky-700" />
-                <Skeleton className="h-4 w-32 bg-sky-700" />
+  const renderContent = () => {
+    if (error) {
+      return (
+        <Alert variant="destructive" className="mb-4">
+          <AlertTriangle className="h-4 w-4" />
+          <AlertDescription>{error}</AlertDescription>
+        </Alert>
+      );
+    }
+
+    if (loading) {
+      return (
+        <div>
+          <div className="grid grid-cols-2 gap-2 mt-1 mb-2">
+            <SkeletonCard colorScheme="sky" />
+            <SkeletonCard colorScheme="sky" />
+          </div>
+
+          <Card className="bg-gradient-to-br from-sky-900 to-sky-800 backdrop-blur-sm">
+            <CardContent className="p-4">
+              <div className="h-[216px]">
+                <div className="h-full w-full relative">
+                  {/* Grid lines */}
+                  {[...Array(5)].map((_, i) => (
+                    <div
+                      key={i}
+                      className="absolute w-full h-px bg-sky-300/20"
+                      style={{ top: `${(i + 1) * 20}%` }}
+                    />
+                  ))}
+                  {/* Y-axis labels */}
+                  <div className="absolute left-0 top-0 bottom-0 w-8 flex flex-col justify-between py-4">
+                    {[...Array(5)].map((_, i) => (
+                      <Skeleton key={i} className="h-3 w-6 bg-sky-300/20 rounded-sm" />
+                    ))}
+                  </div>
+                  {/* X-axis labels */}
+                  <div className="absolute left-8 right-4 bottom-0 flex justify-between">
+                    {[...Array(6)].map((_, i) => (
+                      <Skeleton key={i} className="h-3 w-8 bg-sky-300/20 rounded-sm" />
+                    ))}
+                  </div>
+                  {/* Bars */}
+                  <div className="absolute inset-x-8 bottom-6 top-4 flex items-end justify-between gap-1">
+                    {[...Array(24)].map((_, i) => (
+                      <div
+                        key={i}
+                        className="w-2 bg-sky-300/20 rounded-sm"
+                        style={{ height: `${Math.random() * 80 + 10}%` }}
+                      />
+                    ))}
+                  </div>
+                </div>
               </div>
             </CardContent>
           </Card>
         </div>
+      );
+    }
 
-        <Card className="bg-gradient-to-br from-sky-900 to-sky-800 backdrop-blur-sm">
-          <CardContent className="p-4">
-            <div className="h-[230px] relative">
-              {/* Grid lines */}
-              {[...Array(5)].map((_, i) => (
-                <div
-                  key={i}
-                  className="absolute w-full h-px bg-sky-700"
-                  style={{ top: `${(i + 1) * 20}%` }}
-                />
-              ))}
-              {/* Y-axis labels */}
-              <div className="absolute left-0 top-0 bottom-0 w-8 flex flex-col justify-between py-4">
-                {[...Array(5)].map((_, i) => (
-                  <Skeleton key={i} className="h-3 w-6 bg-sky-700 rounded-sm" />
-                ))}
-              </div>
-              {/* X-axis labels */}
-              <div className="absolute left-8 right-4 bottom-0 flex justify-between">
-                {[...Array(6)].map((_, i) => (
-                  <Skeleton key={i} className="h-3 w-8 bg-sky-700 rounded-sm" />
-                ))}
-              </div>
-              {/* Bars */}
-              <div className="absolute inset-x-8 bottom-6 top-4 flex items-end justify-between gap-1">
-                {[...Array(24)].map((_, i) => (
-                  <div
-                    key={i}
-                    className="w-2 bg-sky-700 rounded-sm"
-                    style={{ height: `${Math.random() * 80 + 10}%` }}
-                  />
-                ))}
-              </div>
-            </div>
-          </CardContent>
-        </Card>
-      </div>
-    );
-  }
-
-  return (
-   <div>
-        {error && (
-          <Alert variant="destructive" className="mb-4">
-            <AlertTriangle className="h-4 w-4" />
-            <AlertDescription>{error}</AlertDescription>
-          </Alert>
-        )}
-
+    return (
+      <div>
         <div className="grid grid-cols-2 gap-2 mt-1 mb-2">
           {summaryCard(
             "Last 30 Minutes",
@@ -245,7 +247,10 @@ const MiniRealtimeAnalytics = () => {
           </CardContent>
         </Card>
       </div>
-  );
+    );
+  };
+
+  return renderContent();
 };
 
 export default MiniRealtimeAnalytics; 
\ No newline at end of file
diff --git a/dashboard/src/components/dashboard/MiniSalesChart.jsx b/dashboard/src/components/dashboard/MiniSalesChart.jsx
index 79d9b9f..d1f44d1 100644
--- a/dashboard/src/components/dashboard/MiniSalesChart.jsx
+++ b/dashboard/src/components/dashboard/MiniSalesChart.jsx
@@ -309,7 +309,7 @@ const MiniSalesChart = ({ className = "" }) => {
               trendValue={`${Math.abs(Math.round(summaryStats.growth.revenue))}%`}
               colorClass="text-emerald-300"
               titleClass="text-emerald-300 font-bold text-md"
-              descriptionClass="text-emerald-300 text-md font-semibold"
+              descriptionClass="text-emerald-300 text-md font-semibold pb-1"
               icon={PiggyBank}
               iconColor="text-emerald-900"
               iconBackground="bg-emerald-300"
@@ -324,7 +324,7 @@ const MiniSalesChart = ({ className = "" }) => {
               trendValue={`${Math.abs(Math.round(summaryStats.growth.orders))}%`}
               colorClass="text-blue-300"
               titleClass="text-blue-300 font-bold text-md"
-              descriptionClass="text-blue-300 text-md font-semibold"
+              descriptionClass="text-blue-300 text-md font-semibold pb-1"
               icon={Truck}
               iconColor="text-blue-900"
               iconBackground="bg-blue-300"