fix: show loading spinner instead of empty state while sidebar tables load

Author: datlechin

TablePro/ViewModels/SidebarViewModel.swift

@@ -158,13 +158,11 @@ final class SidebarViewModel {
             NSLog("[SidebarVM] onAppear: tables not empty (%d), skipping", tables.count)
             return
         }
-        Task { @MainActor in
-            if DatabaseManager.shared.driver(for: connectionId) != nil {
-                NSLog("[SidebarVM] onAppear: driver found, loading tables")
-                loadTables()
-            } else {
-                NSLog("[SidebarVM] onAppear: driver is nil for %@", connectionId.uuidString)
-            }
+        if DatabaseManager.shared.driver(for: connectionId) != nil {
+            NSLog("[SidebarVM] onAppear: driver found, loading tables")
+            loadTables()
+        } else {
+            NSLog("[SidebarVM] onAppear: driver is nil for %@", connectionId.uuidString)
         }
     }
 

TablePro/Views/Sidebar/SidebarView.swift

@@ -149,7 +149,7 @@ struct SidebarView: View {
     private var tablesContent: some View {
         if let error = viewModel.errorMessage {
             errorState(message: error)
-        } else if tables.isEmpty && viewModel.isLoading {
+        } else if tables.isEmpty && hasActiveConnection {
             loadingState
         } else if tables.isEmpty {
             emptyState
@@ -158,6 +158,10 @@ struct SidebarView: View {
         }
     }
 
+    private var hasActiveConnection: Bool {
+        viewModel.isLoading || DatabaseManager.shared.driver(for: connectionId) != nil
+    }
+
     private var loadingState: some View {
         ProgressView()
             .frame(maxWidth: .infinity, maxHeight: .infinity)