From e67bb55487211ed622b032fee4445bfda2c93a9e Mon Sep 17 00:00:00 2001
From: Pat Hartl <pat@pathar.tl>
Date: Tue, 11 Mar 2025 17:27:57 -0500
Subject: [PATCH] Display different message for empty library vs no filtered
 results

Fixes #193
---
 .../UI/Components/LibraryItemFilter.razor     |  4 +-
 .../UI/Components/SavesDialog.razor           |  3 +-
 .../Depot/Components/DepotGameDetails.razor   |  2 +-
 .../UI/Library/Components/LibraryList.razor   | 46 +++++++++++--------
 4 files changed, 33 insertions(+), 22 deletions(-)

diff --git a/LANCommander.Launcher/UI/Components/LibraryItemFilter.razor b/LANCommander.Launcher/UI/Components/LibraryItemFilter.razor
index 0dc430d5..b184a5fc 100644
--- a/LANCommander.Launcher/UI/Components/LibraryItemFilter.razor
+++ b/LANCommander.Launcher/UI/Components/LibraryItemFilter.razor
@@ -91,12 +91,12 @@
 
     }
 
-    async Task ResetFilter()
+    public async Task ResetFilter()
     {
         await LibraryService.Filter.ResetFilter();
     }
 
-    async Task ApplyFilter()
+    public async Task ApplyFilter()
     {
         await LibraryService.Filter.UpdateFilter();
         await Popover.Close();
diff --git a/LANCommander.Launcher/UI/Components/SavesDialog.razor b/LANCommander.Launcher/UI/Components/SavesDialog.razor
index c99c9fc5..3cf668b4 100644
--- a/LANCommander.Launcher/UI/Components/SavesDialog.razor
+++ b/LANCommander.Launcher/UI/Components/SavesDialog.razor
@@ -89,6 +89,7 @@
 
     async Task LoadData()
     {
-        Saves = (await SaveService.Get(Options.Id)).OrderByDescending(s => s.CreatedOn);
+        Saves = await SaveService.Get(Options.Id);
+        Saves = Saves.OrderByDescending(s => s.CreatedOn);
     }
 }
diff --git a/LANCommander.Launcher/UI/Depot/Components/DepotGameDetails.razor b/LANCommander.Launcher/UI/Depot/Components/DepotGameDetails.razor
index de129eaf..9b7d3e04 100644
--- a/LANCommander.Launcher/UI/Depot/Components/DepotGameDetails.razor
+++ b/LANCommander.Launcher/UI/Depot/Components/DepotGameDetails.razor
@@ -14,7 +14,7 @@
 <Drawer Placement="DrawerPlacement.Bottom" Class="depot-game-details" @bind-Visible="@Visible" OnClose="Hide">
     @if (SelectedGame != null)
     {
-        <div class="game-details no-scrollbar">
+        <div @key="SelectedGame.Id" class="game-details no-scrollbar">
             <Button Class="game-details-close-btn" Type="@ButtonType.Text" Icon="@IconType.Outline.Close" OnClick="Hide" />
 
             @if (SelectedGame != null)
diff --git a/LANCommander.Launcher/UI/Library/Components/LibraryList.razor b/LANCommander.Launcher/UI/Library/Components/LibraryList.razor
index 3a74e2ba..fdfa1ed3 100644
--- a/LANCommander.Launcher/UI/Library/Components/LibraryList.razor
+++ b/LANCommander.Launcher/UI/Library/Components/LibraryList.razor
@@ -7,7 +7,7 @@
 @inject NavigationManager NavigationManager
 
 <Flex Vertical Class="library-list">
-    @if (LibraryItems == null || !LibraryItems.Any())
+    @if (LibraryService.Items == null || !LibraryService.Items.Any())
     {
         <Empty Simple Description="false">
             <Button Type="ButtonType.Primary" OnClick="@(() => NavigationManager.NavigateTo("/Depot"))">Browse Depot</Button>
@@ -15,29 +15,38 @@
     }
     else
     {
-        <Collapse>
-            @if (Groups.Count > 1)
-            {
-                foreach (var group in Groups.OrderByTitle(g => g))
+        if (LibraryItems == null || !LibraryItems.Any())
+        {
+            <Empty Simple Description="@("No Results Found")">
+                <Button Type="ButtonType.Primary" OnClick="Filter.ResetFilter">Reset Filter</Button>
+            </Empty>
+        }
+        else
+        {
+            <Collapse>
+                @if (Groups.Count > 1)
                 {
-                    <Panel Header="@group" Key="@group">
-                        <AntList DataSource="@(LibraryItems.Where(i => i.Groups.Contains(group) || (group == "Uncategorized" && !i.Groups.Any())))" TItem="Models.ListItem" Class="ant-list-clickable" Size="ListSize.Small">
+                    foreach (var group in Groups.OrderByTitle(g => g))
+                    {
+                        <Panel Header="@group" Key="@group">
+                            <AntList DataSource="@(LibraryItems.Where(i => i.Groups.Contains(group) || (group == "Uncategorized" && !i.Groups.Any())))" TItem="Models.ListItem" Class="ant-list-clickable" Size="ListSize.Small">
+                                <LibraryListItem Model="@context" OnClick="() => SelectItem(context)" @bind-SelectedItem="SelectedItem"/>
+                            </AntList>
+                        </Panel>
+                    }
+                }
+                else
+                {
+                    <Panel Class="no-header">
+                        <AntList DataSource="LibraryItems" TItem="Models.ListItem" Class="ant-list-clickable" Size="ListSize.Small">
                             <LibraryListItem Model="@context" OnClick="() => SelectItem(context)" @bind-SelectedItem="SelectedItem"/>
                         </AntList>
                     </Panel>
                 }
-            }
-            else
-            {
-                <Panel Class="no-header">
-                    <AntList DataSource="LibraryItems" TItem="Models.ListItem" Class="ant-list-clickable" Size="ListSize.Small">
-                        <LibraryListItem Model="@context" OnClick="() => SelectItem(context)" @bind-SelectedItem="SelectedItem"/>
-                    </AntList>
-                </Panel>
-            }
-        </Collapse>
+            </Collapse>
+        }
 
-        <LibraryItemFilter/>
+        <LibraryItemFilter @ref="Filter" />
     }
 </Flex>
 
@@ -47,6 +56,7 @@
 
     List<string> Groups = new();
     IEnumerable<Models.ListItem> LibraryItems;
+    LibraryItemFilter Filter;
 
     protected override async Task OnInitializedAsync()
     {