From d6a187c01a461d6db432c1c9c2bd9591021fe9ae Mon Sep 17 00:00:00 2001
From: troogs <gitea@beging.de>
Date: Sat, 7 Feb 2026 18:30:03 +0100
Subject: [PATCH] Enhance routine run UI with selectable items, check icons,
 and weight select dropdown

---
 src/ASTRAIN.Client/Pages/Routines.razor    | 21 +++++++-----
 src/ASTRAIN.Client/Pages/Routines.razor.cs | 38 +++++++++++++++++++++-
 src/ASTRAIN.Client/wwwroot/css/app.css     |  9 +++++
 3 files changed, 59 insertions(+), 9 deletions(-)

diff --git a/src/ASTRAIN.Client/Pages/Routines.razor b/src/ASTRAIN.Client/Pages/Routines.razor
index f23f10d..62989dc 100644
--- a/src/ASTRAIN.Client/Pages/Routines.razor
+++ b/src/ASTRAIN.Client/Pages/Routines.razor
@@ -113,15 +113,20 @@
             <div class="list">
                 @foreach (var entry in RunEntries)
                 {
-                    <div class="list-item @(entry.Completed ? "done" : string.Empty)">
-                        <label class="checkbox-row">
-                            <input type="checkbox" checked="@entry.Completed" @onchange="() => ToggleRunCompleted(entry.ExerciseId)" />
-                            <span>@GetExerciseName(entry.ExerciseId)</span>
-                        </label>
-                        <div class="input-unit">
-                            <input class="input input-sm" type="number" step="0.5" @bind="entry.Weight" @bind:event="oninput" />
-                            <span class="unit">kg</span>
+                    <div class="list-item selectable @(entry.Completed ? "selected" : string.Empty)" @onclick="() => ToggleRunCompleted(entry.ExerciseId)">
+                        <div class="item-title">@GetExerciseName(entry.ExerciseId)</div>
+                        <div class="item-actions" @onclick:stopPropagation="true">
+                            <div class="input-unit">
+                                <select class="input input-sm" @bind="entry.Weight">
+                                    @foreach (var w in WeightOptions)
+                                    {
+                                        <option value="@w">@w</option>
+                                    }
+                                </select>
+                                <span class="unit">kg</span>
+                            </div>
                         </div>
+                        <span class="check-icon @(entry.Completed ? "visible" : string.Empty)" aria-hidden="true">✓</span>
                     </div>
                 }
             </div>
diff --git a/src/ASTRAIN.Client/Pages/Routines.razor.cs b/src/ASTRAIN.Client/Pages/Routines.razor.cs
index 4cd8432..fe91bb0 100644
--- a/src/ASTRAIN.Client/Pages/Routines.razor.cs
+++ b/src/ASTRAIN.Client/Pages/Routines.razor.cs
@@ -32,6 +32,42 @@ public partial class Routines
     private RoutineDto? ActiveRun { get; set; }
     private List<RoutineRunEntryDto> RunEntries { get; set; } = new();
 
+    /// <summary>
+    /// Available weight options for the routine run select input.
+    /// 10–40 kg in 2.5 kg steps, then 40–100 kg in 5 kg steps.
+    /// </summary>
+    private static readonly List<double> WeightOptions = BuildWeightOptions();
+
+    /// <summary>
+    /// Builds the list of available weight options for the routine run select input.
+    /// </summary>
+    /// <returns>A list of double values representing weight options.</returns>
+    private static List<double> BuildWeightOptions()
+    {
+        var options = new List<double>();
+        for (var w = 10.0; w <= 40.0; w += 2.5)
+        {
+            options.Add(w);
+        }
+
+        for (var w = 45.0; w <= 100.0; w += 5.0)
+        {
+            options.Add(w);
+        }
+
+        return options;
+    }
+
+    /// <summary>
+    /// Snaps a weight value to the nearest available option.
+    /// </summary>
+    /// <param name="weight">The weight value to snap to the nearest option.</param>
+    /// <returns>The nearest available weight option.</returns>
+    private static double SnapToNearest(double weight)
+    {
+        return WeightOptions.OrderBy(w => Math.Abs(w - weight)).First();
+    }
+
     /// <inheritdoc/>
     protected override async Task OnInitializedAsync()
     {
@@ -197,7 +233,7 @@ public partial class Routines
             .Select(e =>
             {
                 var last = lastRun.Entries.FirstOrDefault(x => x.ExerciseId == e.ExerciseId);
-                return new RoutineRunEntryDto(e.ExerciseId, last?.Weight ?? 0, false);
+                return new RoutineRunEntryDto(e.ExerciseId, SnapToNearest(last?.Weight ?? WeightOptions[0]), false);
             }).ToList();
     }
 
diff --git a/src/ASTRAIN.Client/wwwroot/css/app.css b/src/ASTRAIN.Client/wwwroot/css/app.css
index af2220f..f5331d1 100644
--- a/src/ASTRAIN.Client/wwwroot/css/app.css
+++ b/src/ASTRAIN.Client/wwwroot/css/app.css
@@ -91,6 +91,8 @@ p {
 
 .input-sm {
     max-width: 110px;
+    appearance: none;
+    -webkit-appearance: none;
 }
 
 .primary {
@@ -173,6 +175,13 @@ p {
     font-weight: 600;
 }
 
+.item-actions {
+    display: flex;
+    align-items: center;
+    gap: 0.5rem;
+    margin-left: auto;
+}
+
 .item-subtitle {
     color: #bdbdbd;
     font-size: 0.85rem;