From f2dcff7b71a0f375e4fe4d2dfcfa8e7cbff4f291 Mon Sep 17 00:00:00 2001
From: Timothy Cardoza <tcardoza@cardoza.construction>
Date: Wed, 18 Mar 2026 18:51:26 -0500
Subject: [PATCH] Load in improvements

---
 src/AppContainer.tsx              | 11 ++++-
 src/ItemExtractor.tsx             |  2 +-
 src/components/CloudLoadModal.tsx | 68 +++++++++++++++++--------------
 3 files changed, 48 insertions(+), 33 deletions(-)

diff --git a/src/AppContainer.tsx b/src/AppContainer.tsx
index 9c0b4ea..a4bd07c 100644
--- a/src/AppContainer.tsx
+++ b/src/AppContainer.tsx
@@ -2,6 +2,7 @@ import type { Component } from 'solid-js';
 import { A, useLocation, useNavigate } from '@solidjs/router';
 import { Show } from 'solid-js';
 import { authStore } from './store/authStore';
+import { appStore } from './store/appStore';
 
 interface AppContainerProps {
     children?: any;
@@ -25,7 +26,15 @@ const AppContainer: Component<AppContainerProps> = (props) => {
                     <div class="flex justify-between h-16">
                         <div class="flex items-center gap-8">
                             <div class="flex-shrink-0 flex items-center">
-                                <span class="font-bold text-xl text-gray-900 tracking-tight">EstiMaker</span>
+                                <button 
+                                    onClick={() => {
+                                        appStore.resetEstimate();
+                                        navigate('/');
+                                    }}
+                                    class="font-bold text-xl text-gray-900 tracking-tight hover:text-blue-600 transition-colors"
+                                >
+                                    EstiMaker
+                                </button>
                             </div>
                             <nav class="flex space-x-4">
                                 <A 
diff --git a/src/ItemExtractor.tsx b/src/ItemExtractor.tsx
index da4a797..458b942 100644
--- a/src/ItemExtractor.tsx
+++ b/src/ItemExtractor.tsx
@@ -64,7 +64,7 @@ const ItemExtractor: Component<ItemExtractorProps> = () => {
                     <Show when={mode() === 'initial'}>
                         <div class="flex flex-col items-center justify-center space-y-8 py-12">
                             <div class="text-center mb-4">
-                                <h1 class="text-4xl font-black text-gray-900 mb-3 tracking-tight lowercase">EstiMaker</h1>
+                                <h1 class="text-4xl font-black text-gray-900 mb-3 tracking-tight">EstiMaker</h1>
                                 <p class="text-gray-500 text-lg">Start a new project or continue where you left off.</p>
                             </div>
 
diff --git a/src/components/CloudLoadModal.tsx b/src/components/CloudLoadModal.tsx
index 52d665d..1a175d5 100644
--- a/src/components/CloudLoadModal.tsx
+++ b/src/components/CloudLoadModal.tsx
@@ -1,7 +1,7 @@
 import type { Component } from 'solid-js';
 import { createSignal, Show, For, createEffect } from 'solid-js';
 import { Portal } from 'solid-js/web';
-import { X, User } from 'lucide-solid';
+import { X } from 'lucide-solid';
 import { pb, COLLECTIONS } from '../utils/db';
 import { appStore } from '../store/appStore';
 import { normalizeScopes } from '../utils/scope-utils';
@@ -14,6 +14,7 @@ interface CloudLoadModalProps {
 const CloudLoadModal: Component<CloudLoadModalProps> = (props) => {
     const [cloudEstimates, setCloudEstimates] = createSignal<any[]>([]);
     const [isCloudLoading, setIsCloudLoading] = createSignal(false);
+    const [loadingRecordId, setLoadingRecordId] = createSignal<string | null>(null);
 
     createEffect(() => {
         if (props.show) {
@@ -24,8 +25,10 @@ const CloudLoadModal: Component<CloudLoadModalProps> = (props) => {
     const loadCloudEstimatesList = async () => {
         setIsCloudLoading(true);
         try {
+            // Optimized: Only fetch metadata fields to keep the list response small and fast
             const records = await pb.collection(COLLECTIONS.ESTIMATES).getFullList({
-                sort: '-created'
+                sort: '-created',
+                fields: 'id,name,created,updated'
             });
             setCloudEstimates(records);
         } catch (err) {
@@ -36,28 +39,36 @@ const CloudLoadModal: Component<CloudLoadModalProps> = (props) => {
         }
     };
 
-    const loadFromCloud = (record: any) => {
-        const data = record.items; // items holds the full JSON blob 
-        if (!data) return;
+    const loadFromCloud = async (recordMetadata: any) => {
+        setLoadingRecordId(recordMetadata.id);
+        try {
+            // Fetch the full record with all fields (items, history) only when requested
+            const record = await pb.collection(COLLECTIONS.ESTIMATES).getOne(recordMetadata.id);
+            const data = record.items; 
+            if (!data) return;
 
-        if (data.scopes) appStore.setScopes(normalizeScopes(data.scopes));
-        if (data.subScopes) appStore.setSubScopes(data.subScopes);
-        if (data.items) appStore.setEstimateItems(data.items);
-        if (data.clientInfo) appStore.setClientInfo(data.clientInfo);
-        if (data.jobInfo) appStore.setJobInfo(data.jobInfo);
-        if (data.generalEstimateNotes !== undefined) appStore.setGeneralEstimateNotes(data.generalEstimateNotes);
-        if (data.generalPreNotes !== undefined) appStore.setGeneralPreNotes(data.generalPreNotes);
-        
-        appStore.setEstimateName(record.name);
-        appStore.setEstimateId(record.id);
-        appStore.setHistory(record.history || []);
-        appStore.setLatestSnapshot(record.items);
-        appStore.setIsLatestVersion(true);
-        
-        // Also clear out the extracted CSV items so we don't accidentally switch back to the upload screen or cause conflicts
-        appStore.setItems([]);
-        
-        props.onClose();
+            if (data.scopes) appStore.setScopes(normalizeScopes(data.scopes));
+            if (data.subScopes) appStore.setSubScopes(data.subScopes);
+            if (data.items) appStore.setEstimateItems(data.items);
+            if (data.clientInfo) appStore.setClientInfo(data.clientInfo);
+            if (data.jobInfo) appStore.setJobInfo(data.jobInfo);
+            if (data.generalEstimateNotes !== undefined) appStore.setGeneralEstimateNotes(data.generalEstimateNotes);
+            if (data.generalPreNotes !== undefined) appStore.setGeneralPreNotes(data.generalPreNotes);
+            
+            appStore.setEstimateName(record.name);
+            appStore.setEstimateId(record.id);
+            appStore.setHistory(record.history || []);
+            appStore.setLatestSnapshot(record.items);
+            appStore.setIsLatestVersion(true);
+            
+            appStore.setItems([]);
+            props.onClose();
+        } catch (err) {
+            console.error(err);
+            alert('Failed to load full estimate data.');
+        } finally {
+            setLoadingRecordId(null);
+        }
     };
 
     return (
@@ -85,18 +96,13 @@ const CloudLoadModal: Component<CloudLoadModalProps> = (props) => {
                                                     <p class="text-[10px] text-muted-foreground font-medium uppercase tracking-wider">
                                                         {new Date(record.created).toLocaleDateString()}
                                                     </p>
-                                                    <Show when={record.items?.savedBy}>
-                                                        <div class="flex items-center gap-1.5 mt-1.5 grayscale opacity-70">
-                                                            <User class="w-3 h-3 text-muted-foreground" />
-                                                            <span class="text-[10px] font-bold text-muted-foreground uppercase tracking-tight">{record.items.savedBy}</span>
-                                                        </div>
-                                                    </Show>
                                                 </div>
                                                 <button
+                                                    disabled={loadingRecordId() !== null}
                                                     onClick={() => loadFromCloud(record)}
-                                                    class="px-4 py-2 bg-background border border-border rounded-xl text-xs font-bold shadow-sm hover:text-primary hover:border-primary/50 transition-all opacity-0 group-hover:opacity-100"
+                                                    class="px-4 py-2 bg-background border border-border rounded-xl text-xs font-bold shadow-sm hover:text-primary hover:border-primary/50 transition-all opacity-0 group-hover:opacity-100 disabled:opacity-50"
                                                 >
-                                                    Load
+                                                    {loadingRecordId() === record.id ? 'Fetching...' : 'Load'}
                                                 </button>
                                             </div>
                                         )}