From 4c6f5658550d333b06d1556a3df4d6104ee3f9de Mon Sep 17 00:00:00 2001
From: rcourtman <courtmanr@gmail.com>
Date: Sun, 9 Nov 2025 22:35:32 +0000
Subject: [PATCH] fix: sparkline canvas wrapper display mode for flex layout

Change sparkline wrapper from inline-block to block w-full to properly
fill flex parent container. Inline-block was preventing the canvas from
calculating the correct width when width={0} (auto-size mode).
---
 frontend-modern/src/components/shared/Sparkline.tsx |  4 ++--
 frontend-modern/src/stores/metricsHistory.ts        | 11 ++++++++++-
 2 files changed, 12 insertions(+), 3 deletions(-)

diff --git a/frontend-modern/src/components/shared/Sparkline.tsx b/frontend-modern/src/components/shared/Sparkline.tsx
index a4531af..3dd7b1d 100644
--- a/frontend-modern/src/components/shared/Sparkline.tsx
+++ b/frontend-modern/src/components/shared/Sparkline.tsx
@@ -255,10 +255,10 @@ export const Sparkline: Component<SparklineProps> = (props) => {
   };
 
   return (
-    <div class="relative inline-block">
+    <div class="relative block w-full">
       <canvas
         ref={canvasRef}
-        class="inline-block cursor-crosshair"
+        class="block cursor-crosshair"
         style={{
           width: `${width()}px`,
           height: `${height()}px`,
diff --git a/frontend-modern/src/stores/metricsHistory.ts b/frontend-modern/src/stores/metricsHistory.ts
index 95afefc..60f17ce 100644
--- a/frontend-modern/src/stores/metricsHistory.ts
+++ b/frontend-modern/src/stores/metricsHistory.ts
@@ -112,10 +112,19 @@ function saveToLocalStorage(): void {
 function loadFromLocalStorage(): void {
   try {
     const stored = localStorage.getItem(STORAGE_KEY);
-    if (!stored) return;
+    if (!stored) {
+      logger.debug('[MetricsHistory] No stored data found');
+      return;
+    }
 
     const payload = JSON.parse(stored);
 
+    if (!payload || typeof payload !== 'object') {
+      logger.warn('[MetricsHistory] Invalid payload format, clearing');
+      localStorage.removeItem(STORAGE_KEY);
+      return;
+    }
+
     // Check version and age
     if (payload.version !== STORAGE_VERSION) {
       logger.warn('[MetricsHistory] Storage version mismatch, clearing');