From 335fe9821b20dc01177519c6437e438a583ec751 Mon Sep 17 00:00:00 2001
From: Copilot <198982749+Copilot@users.noreply.github.com>
Date: Sat, 4 Oct 2025 14:37:10 +0200
Subject: [PATCH] Improve loading speed with parallel transport calls and
 non-blocking UI (#52)

* Initial plan

* Implement parallel transport loading and non-blocking UI improvements

Co-authored-by: clayschaad <11883505+clayschaad@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: clayschaad <11883505+clayschaad@users.noreply.github.com>
---
 Client/Pages/Index.razor    | 16 +++++++++++----
 Client/Pages/Index.razor.cs | 41 ++++++++++++++++++++++++++++++-------
 2 files changed, 46 insertions(+), 11 deletions(-)

diff --git a/Client/Pages/Index.razor b/Client/Pages/Index.razor
index 503a72e..751dccb 100644
--- a/Client/Pages/Index.razor
+++ b/Client/Pages/Index.razor
@@ -111,8 +111,8 @@ else
                             <div class="tooltip-wrapper">
                                 <button class="shift-button @(SelectedSchedule.ContainsKey(day) && SelectedSchedule[day] == shift.Name ? "selected" : "")"
                                         @onclick="() => SelectShift(day, shift.Name)"
-                                        disabled="@_isLoadingTransport">
-                                    @if (_isLoadingTransport && SelectedSchedule.ContainsKey(day) && SelectedSchedule[day] == shift.Name)
+                                        disabled="@IsLoadingTransportForDay(day)">
+                                    @if (IsLoadingTransportForDay(day) && SelectedSchedule.ContainsKey(day) && SelectedSchedule[day] == shift.Name)
                                     {
                                         <span class="loading-spinner">⏳</span>
                                     }
@@ -169,8 +169,8 @@ else
                         <div class="tooltip-wrapper">
                             <button class="shift-button @(SelectedSchedule.ContainsKey(day) && SelectedSchedule[day] == shift.Name ? "selected" : "")"
                                     @onclick="() => SelectShift(day, shift.Name)"
-                                    disabled="@_isLoadingTransport">
-                                @if (_isLoadingTransport && SelectedSchedule.ContainsKey(day) && SelectedSchedule[day] == shift.Name)
+                                    disabled="@IsLoadingTransportForDay(day)">
+                                @if (IsLoadingTransportForDay(day) && SelectedSchedule.ContainsKey(day) && SelectedSchedule[day] == shift.Name)
                                 {
                                     <span class="loading-spinner">⏳</span>
                                 }
@@ -217,3 +217,11 @@ else
 <ConfigurationDialog IsVisible="_showConfigDialog" 
                      OnClose="HideConfiguration" 
                      OnConfigurationChanged="OnConfigurationChanged" />
+
+@code {
+    // Helper method to check if a specific day is loading transport data
+    private bool IsLoadingTransportForDay(DateTime day)
+    {
+        return _isLoadingTransportPerDay.GetValueOrDefault(day, false);
+    }
+}
diff --git a/Client/Pages/Index.razor.cs b/Client/Pages/Index.razor.cs
index a6e4eef..9490291 100644
--- a/Client/Pages/Index.razor.cs
+++ b/Client/Pages/Index.razor.cs
@@ -14,9 +14,9 @@ namespace ShiftScheduler.Client.Pages
         private List<Shift> Shifts { get; set; } = new();
         private Dictionary<DateTime, string> SelectedSchedule { get; set; } = new();
         private Dictionary<DateTime, ShiftWithTransport> SelectedShiftsWithTransport { get; set; } = new();
+        private Dictionary<DateTime, bool> _isLoadingTransportPerDay { get; set; } = new();
 
         private bool _isCurrentMonth = true;
-        private bool _isLoadingTransport = false;
         private bool _isLoadingInitial = false;
         private bool _showConfigDialog = false;
         private bool _isSyncing = false;
@@ -31,12 +31,19 @@ namespace ShiftScheduler.Client.Pages
         {
             _isLoadingInitial = true;
             
+            // Load shifts first to show the calendar immediately
             Shifts = await HttpClient.GetFromJsonAsync<List<Shift>>("api/shift/shifts") ?? new();
-
-            await LoadScheduleFromStorage();
             
+            // Show calendar immediately, don't wait for transport data
             _isLoadingInitial = false;
             StateHasChanged();
+            
+            // Load schedule and transport data in the background
+            _ = Task.Run(async () =>
+            {
+                await LoadScheduleFromStorage();
+                await InvokeAsync(StateHasChanged);
+            });
         }
 
         private async Task SelectCurrentMonth()
@@ -57,8 +64,8 @@ namespace ShiftScheduler.Client.Pages
         {
             SelectedSchedule[day] = shiftName;
             
-            // Get transport information for this shift and date
-            _isLoadingTransport = true;
+            // Get transport information for this shift and date - per day loading
+            _isLoadingTransportPerDay[day] = true;
             StateHasChanged();
             
             try
@@ -93,7 +100,7 @@ namespace ShiftScheduler.Client.Pages
             }
             finally
             {
-                _isLoadingTransport = false;
+                _isLoadingTransportPerDay[day] = false;
                 StateHasChanged();
             }
             
@@ -312,7 +319,18 @@ namespace ShiftScheduler.Client.Pages
         {
             SelectedShiftsWithTransport.Clear();
             
+            // Clear existing per-day loading states
+            _isLoadingTransportPerDay.Clear();
+            
+            // Mark all days as loading
             foreach (var kvp in SelectedSchedule)
+            {
+                _isLoadingTransportPerDay[kvp.Key] = true;
+            }
+            StateHasChanged();
+
+            // Create parallel tasks for all transport data requests
+            var transportTasks = SelectedSchedule.Select(async kvp =>
             {
                 var day = kvp.Key;
                 var shiftName = kvp.Value;
@@ -347,7 +365,16 @@ namespace ShiftScheduler.Client.Pages
                         };
                     }
                 }
-            }
+                finally
+                {
+                    // Mark this day as no longer loading
+                    _isLoadingTransportPerDay[day] = false;
+                    await InvokeAsync(StateHasChanged);
+                }
+            }).ToArray();
+
+            // Wait for all transport data to load
+            await Task.WhenAll(transportTasks);
         }
 
         private async Task ResetSchedule()