From 83e8f8ff23efca34764532fdddd05bf3f281bf99 Mon Sep 17 00:00:00 2001
From: Marco Minerva <marco.minerva@gmail.com>
Date: Fri, 14 Feb 2025 17:38:03 +0100
Subject: [PATCH] Update document management UI and functionality

- Changed the "Documents" icon in `MainLayout.razor`.
- Added a `ConfirmDialog` component in `Documents.razor` for user confirmations.
- Adjusted the capitalization of the creation date column header.
- Restructured the delete button layout for better responsiveness and changed its icon.
- Updated lifecycle method from `OnInitializedAsync` to `OnAfterRenderAsync` for improved document loading.
- Introduced a new asynchronous method for obtaining local date strings.
- Updated toast messages to use the new date formatting method.
- Added a new method for converting UTC dates to local time.
- Introduced a `SelectableDocument` record class to enhance document management.
---
 .../Components/Layout/MainLayout.razor        |  2 +-
 .../Components/Pages/Documents.razor          | 51 +++++++++++++++----
 2 files changed, 42 insertions(+), 11 deletions(-)

diff --git a/SqlDatabaseVectorSearch/Components/Layout/MainLayout.razor b/SqlDatabaseVectorSearch/Components/Layout/MainLayout.razor
index 54a9c59..11dbb73 100644
--- a/SqlDatabaseVectorSearch/Components/Layout/MainLayout.razor
+++ b/SqlDatabaseVectorSearch/Components/Layout/MainLayout.razor
@@ -42,7 +42,7 @@
             new() { Id = "1", Href = "/", IconName = IconName.HouseDoorFill, Text = "Home", Match = NavLinkMatch.All},
             new() { Id = "2", Href = "/counter", IconName = IconName.PlusSquareFill, Text = "Counter"},
             new() { Id = "3", Href = "/weather", IconName = IconName.Table, Text = "Weather"},
-            new() { Id = "4", Href= "/documents", IconName = IconName.FileTypeTxt, Text = "Documents" }
+            new() { Id = "4", Href= "/documents", IconName = IconName.FileText, Text = "Documents" }
         ];
 
         return navItems;
diff --git a/SqlDatabaseVectorSearch/Components/Pages/Documents.razor b/SqlDatabaseVectorSearch/Components/Pages/Documents.razor
index 2751635..19889a3 100644
--- a/SqlDatabaseVectorSearch/Components/Pages/Documents.razor
+++ b/SqlDatabaseVectorSearch/Components/Pages/Documents.razor
@@ -6,6 +6,8 @@
 @inject IJSRuntime JSRuntime
 
 <Toasts class="p-3" Messages="messages" Placement="ToastsPlacement.TopRight" />
+<ConfirmDialog @ref="dialog" />
+
 <PageTitle>Documents</PageTitle>
 
 <h2 class="mb-4">Upload new document</h2>
@@ -40,7 +42,7 @@
                 <th></th>
                 <th>Name</th>
                 <th>Number of chunks</th>
-                <th>Creation Date</th>
+                <th>Creation date</th>
             </tr>
         </thead>
         <tbody>
@@ -58,12 +60,19 @@
         </tbody>
     </table>
 
-    <Button @ref="deleteButton" Color="ButtonColor.Danger" Disabled="@(!documents.Any(d => d.IsSelected))" @onclick="DeleteSelectedDocuments">
-        <Icon Name="IconName.X" /><span class="d-none d-lg-inline ps-3">Delete</span>
-    </Button>
+    <div class="row">
+        <div class="col-md-2 col-sm-3 col-2">
+            <div class="d-grid gap-2">
+                <Button @ref="deleteButton" Color="ButtonColor.Danger" Disabled="@(!documents.Any(d => d.IsSelected))" @onclick="DeleteSelectedDocuments">
+                    <Icon Name="IconName.Trash" /><span class="d-none d-lg-inline ps-3">Delete</span>
+                </Button>
+            </div>
+        </div>
+    </div>
 }
 
 @code {
+    private ConfirmDialog dialog = default!;
     private Button uploadButton = default!;
     private Button deleteButton = default!;
 
@@ -73,8 +82,13 @@
     [SupplyParameterFromForm]
     public IBrowserFile? File { get; set; }
 
-    protected override async Task OnInitializedAsync()
+    protected override async Task OnAfterRenderAsync(bool firstRender)
     {
+        if (!firstRender)
+        {
+            return;
+        }
+
         await LoadDocumentsAsync();
     }
 
@@ -87,9 +101,11 @@
         {
             documents.Add(new SelectableDocument(dbDocument.Id, dbDocument.Name, dbDocument.CreationDate, dbDocument.ChunkCount)
                 {
-                    LocalCreationDateString = await JSRuntime.InvokeAsync<string>("getLocalTime", dbDocument.CreationDate)
+                    LocalCreationDateString = await GetLocalDateTimeStringAsync(dbDocument.CreationDate)
                 });
         }
+
+        StateHasChanged();
     }
 
     private void HandleFileSelected(InputFileChangeEventArgs e)
@@ -114,7 +130,7 @@
             await vectorSearchService.ImportAsync(stream, File.Name, MimeUtility.GetMimeMapping(File.Name), null);
 
 
-            CreateToastMessage(ToastType.Success, "Upload document", $"The document {File.Name} has been successfully uploaded and indexed.", DateTime.Now.ToString("g"));
+            CreateToastMessage(ToastType.Success, "Upload document", $"The document {File.Name} has been successfully uploaded and indexed.", await GetLocalDateTimeStringAsync(DateTimeOffset.UtcNow));
 
             await LoadDocumentsAsync();
         }
@@ -128,9 +144,19 @@
     {
         var selectedDocuments = documents?.Where(d => d.IsSelected) ?? [];
 
+        var confirmation = await dialog.ShowAsync(
+            title: "Delete the selected document?",
+            message1: "This will delete the documents and all the corresponding embeddings. The operation cannot be undone.",
+            message2: "Do you want to proceed?");
+
+        if (!confirmation)
+        {
+            return;
+        }
+
         try
         {
-            uploadButton.ShowLoading();
+            deleteButton.ShowLoading();
 
             foreach (var document in selectedDocuments)
             {
@@ -138,11 +164,11 @@
             }
 
             await LoadDocumentsAsync();
-            CreateToastMessage(ToastType.Info, "Delete documents", "The selected documents have been successfully deleted.", DateTime.Now.ToString("g"));
+            CreateToastMessage(ToastType.Info, "Delete documents", "The selected documents have been successfully deleted.", await GetLocalDateTimeStringAsync(DateTimeOffset.UtcNow));
         }
         finally
         {
-            uploadButton.HideLoading();
+            deleteButton.HideLoading();
         }
     }
 
@@ -160,6 +186,11 @@
         messages.Add(toastMessage);
     }
 
+    private async Task<string> GetLocalDateTimeStringAsync(DateTimeOffset dateTime)
+    {
+        return await JSRuntime.InvokeAsync<string>("getLocalTime", dateTime);
+    }
+
     private record class SelectableDocument(Guid Id, string Name, DateTimeOffset CreationDate, int ChunkCount) : Document(Id, Name, CreationDate, ChunkCount)
     {
         public bool IsSelected { get; set; }