Merge pull request #296 from LiamMorrow/fix-stats-filtering

LiamMorrow · Oct 15, 2024 · d78f44c · d78f44c
2 parents 3d65160 + d313e1d
commit d78f44c
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/LiftLog.Ui/Pages/StatsPage.razor b/LiftLog.Ui/Pages/StatsPage.razor
@@ -77,7 +77,7 @@ else if (!StatsState.Value.IsLoading && StatsState.Value.OverallView is not null
         <Card class=@CardClass>
             <StatGraphCardContent Title="Sessions" Statistics="overallStats.SessionStats" RenderDelay="TimeSpan.FromMilliseconds(200)"/>
         </Card>
-        <CardList Items="@(UnpinnedExercisesToShow.IndexedTuples())">
+        <CardList KeySelector="context=>context.Item.ExerciseName" Items="@(UnpinnedExercisesToShow.IndexedTuples())" >
             <StatGraphCardContent Title="@context.Item.ExerciseName" Statistics="[context.Item.Statistics, context.Item.OneRepMaxStatistics]" RenderDelay="TimeSpan.FromMilliseconds(200 + context.Index * 200)"/>
         </CardList>
         @if(SearchTerm != "" && !UnpinnedExercisesToShow.Any())

diff --git a/LiftLog.Ui/Shared/Presentation/CardList.razor b/LiftLog.Ui/Shared/Presentation/CardList.razor
@@ -1,9 +1,13 @@
 @typeparam TItem
 
 <div @attributes="AdditionalAttributes" class=" @(AdditionalAttributes?.GetValueOrDefault("class")) flex flex-col gap-2 p-2 cardlist">
+    @{
+        var index = 0;
+    }
     @foreach (var item in Items)
     {
         <Card
+            @key="KeySelector?.Invoke(item) ?? index++"
             class="@CardClass"
             Type="CardType"
             OnLongPress=@(OnLongPress.HasDelegate ? ()=>OnLongPress.InvokeAsync(item) : default!)
@@ -30,6 +34,8 @@
 
     [Parameter] public Card.CardType CardType { get; set; } = Card.CardType.Outlined;
 
+    [Parameter] public Func<TItem, object>? KeySelector { get; set; }
+
     [Parameter(CaptureUnmatchedValues = true)]
     public Dictionary<string, object>? AdditionalAttributes { get; set; }
 }