From 49365ba216a8e9fc5af59080f6034d40c512dbe4 Mon Sep 17 00:00:00 2001
From: Bertrand THOMAS <bertrand@devpro.fr>
Date: Tue, 27 Oct 2026 14:58:28 +0100
Subject: [PATCH 1/5] Dump version 2.4.3

---
 Directory.Build.props | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Directory.Build.props b/Directory.Build.props
index 806acfc8..ba483fa5 100644
--- a/Directory.Build.props
+++ b/Directory.Build.props
@@ -5,7 +5,7 @@
 
   <PropertyGroup>
     <!-- Tip: Edit this value to change the current MAJOR.MINOR.PATCH version -->
-    <VersionPrefix>2.4.2</VersionPrefix>
+    <VersionPrefix>2.4.3</VersionPrefix>
   </PropertyGroup>
 
   <Choose>

From e595bf6225b4bd53fdee250dd40dbec4e552d191 Mon Sep 17 00:00:00 2001
From: Bertrand THOMAS <bertrand@devpro.fr>
Date: Tue, 27 Oct 2026 15:12:09 +0100
Subject: [PATCH 2/5] Show the app version on the admin page and widen the
 layout on large screens

The content column, list rows, grids and covers grow above 1400px, 1600px and 1920px.
The screenshot harness gains a 2560x1440 pass.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 docs/blazor-ui.md                             |  5 ++
 docs/testing.md                               |  2 +-
 .../ReferenceDataAdminPage.razor              |  1 +
 src/BlazorApp/Components/Shared/AppVersion.cs | 28 +++++++++++
 src/BlazorApp/wwwroot/app.css                 | 46 ++++++++++++++++++-
 .../Smoke/MobileScreenshotTest.cs             | 17 +++++++
 .../Components/Shared/AppVersionTest.cs       | 31 +++++++++++++
 7 files changed, 128 insertions(+), 2 deletions(-)
 create mode 100644 src/BlazorApp/Components/Shared/AppVersion.cs
 create mode 100644 test/BlazorApp.UnitTests/Components/Shared/AppVersionTest.cs

diff --git a/docs/blazor-ui.md b/docs/blazor-ui.md
index 27f9512a..c2eb3e11 100644
--- a/docs/blazor-ui.md
+++ b/docs/blazor-ui.md
@@ -100,6 +100,11 @@ Sonar reports every `!important` (`css:S9423`), and an `!important` theme also s
 
 `wwwroot/Keeptrack.BlazorApp.lib.module.js` is loaded automatically by name (never a manual `<script>` tag), and is where `blazor.addEventListener('enhancedload', ...)` re-applies client-side DOM state.
 
+### Responsive widths
+
+Phone below 768px, laptop up to 1400px, and the large-screen block last in `app.css` above it, so its overrides win the cascade.
+At 1400px and up the content column and the detail measures widen, at 1600px list rows go two per line instead of stretching, and at 1920px the root font size grows with the px measures (covers, grid columns, thumbnails) widened to match, since rem does not reach them.
+
 ### Icons
 
 **Icons are Unicode with text presentation** (`◈ ✓ ✕ ★ ▶ ↻ ⌂ ⚙ ♪ ◼ ▭ ▬ ◆`), never a codepoint with `Emoji_Presentation=Yes` (`⭐`, `👁`), never with U+FE0F.
diff --git a/docs/testing.md b/docs/testing.md
index e98fad20..b3abdae6 100644
--- a/docs/testing.md
+++ b/docs/testing.md
@@ -50,7 +50,7 @@ Gherkin was rejected as twice the code for the same coverage.
 - **`ExploreSmokeTest` seeds `explore_catalogue` directly** (`End2EndFixture.SeedExploreCatalogueAsync`, self-hosted only), since the e2e host never runs the refresh.
   It seeds every ranking `ExploreRankings.Rankings` declares for a domain (the one read follows a stored admin setting), uses real provider ids only for the two `Add` cases (TMDB 278, IGDB 72),
   and its `[Theory]`s cover movies and video games, which differ in tab gating, ranking and id space.
-- `MobileScreenshotTest` is an assertion-free visual harness behind `E2E_MOBILE_CHECK=true`, capturing every page at a phone viewport for review.
+- `MobileScreenshotTest` is an assertion-free visual harness behind `E2E_MOBILE_CHECK=true`, capturing every page at a phone viewport, then a sample at 1280px and at 2560px, for review.
 
 **`PageBase.WaitForReadyAsync` reloads once and re-asserts** (`ExpectWithReloadAsync`, also behind `ListPage.ExpectRowThumbnailAsync`), for two cases where waiting never helps:
 
diff --git a/src/BlazorApp/Components/ReferenceDataAdmin/ReferenceDataAdminPage.razor b/src/BlazorApp/Components/ReferenceDataAdmin/ReferenceDataAdminPage.razor
index da6098ca..91390724 100644
--- a/src/BlazorApp/Components/ReferenceDataAdmin/ReferenceDataAdminPage.razor
+++ b/src/BlazorApp/Components/ReferenceDataAdmin/ReferenceDataAdminPage.razor
@@ -3,6 +3,7 @@
 
 <div class="kt-page-header">
     <h1>Reference data</h1>
+    <span class="kt-app-version" title="Running build">v@(AppVersion.Current)</span>
 </div>
 
 @* Linking unresolved titles is the frequent admin task, so it comes first, and the occasional sync, export and import sit below it. *@
diff --git a/src/BlazorApp/Components/Shared/AppVersion.cs b/src/BlazorApp/Components/Shared/AppVersion.cs
new file mode 100644
index 00000000..6874066a
--- /dev/null
+++ b/src/BlazorApp/Components/Shared/AppVersion.cs
@@ -0,0 +1,28 @@
+using System.Reflection;
+
+namespace Keeptrack.BlazorApp.Components.Shared;
+
+/// <summary>
+/// The running build's version, from the informational version that <c>Directory.Build.props</c> and the SDK's source revision produce.
+/// </summary>
+public static class AppVersion
+{
+    private const int ShortHashLength = 7;
+
+    public static string Current { get; } =
+        Format(typeof(AppVersion).Assembly.GetCustomAttribute<AssemblyInformationalVersionAttribute>()?.InformationalVersion);
+
+    public static string Format(string? informationalVersion)
+    {
+        if (string.IsNullOrEmpty(informationalVersion))
+        {
+            return "unknown";
+        }
+
+        // The SDK appends the full 40-character commit hash, which is unreadable in a header and as identifying at git's usual 7.
+        var plus = informationalVersion.IndexOf('+');
+        return plus >= 0 && informationalVersion.Length - plus - 1 > ShortHashLength
+            ? informationalVersion[..(plus + 1 + ShortHashLength)]
+            : informationalVersion;
+    }
+}
diff --git a/src/BlazorApp/wwwroot/app.css b/src/BlazorApp/wwwroot/app.css
index 9fc54d54..a35238ce 100644
--- a/src/BlazorApp/wwwroot/app.css
+++ b/src/BlazorApp/wwwroot/app.css
@@ -392,7 +392,8 @@ div.content,
 /* Inventory rows in list view only, added by InventoryList, never to the grid.
    A .kt-item-row clusters its content left and pushes its delete button to the far edge, so across the full 1100px column the title and its delete icon sit ~900px apart.
    Capping the measure keeps a row readable as one object.
-   The grid, the admin queues and the dense car and health journals take the full width. */
+   The grid, the admin queues and the dense car and health journals take the full width.
+   On a large screen the cap lifts and the rows go two per line instead (see Large screens). */
 .kt-table-wrap-list { max-width: 920px; margin-inline: auto; }
 
 .kt-table-header {
@@ -699,6 +700,8 @@ a.kt-item-row { color: inherit; text-decoration: none; }
 }
 .kt-page-header h1 { font-size: 2rem; color: var(--kt-text); margin: 0; line-height: 1; }
 .kt-page-header h1 span { color: var(--kt-accent); }
+/* The running build on the admin page, so a deployment can be checked from the browser. */
+.kt-app-version { color: var(--kt-text-muted); font-size: 0.8rem; font-variant-numeric: tabular-nums; }
 
 /* ── Quick Add ─────────────────────────────────────────────────────── */
 .kt-quickadd-back {
@@ -1872,3 +1875,44 @@ a.kt-item-row { color: inherit; text-decoration: none; }
   background: var(--kt-surface);
 }
 .kt-dropzone-icon { font-size: 2.25rem; display: block; margin-bottom: 1rem; color: var(--kt-text-subtle); }
+
+/* ── Large screens ─────────────────────────────────────────────────── */
+/* Last in the file so these overrides win the cascade over every base rule they widen.
+   The base measures suit a laptop, and on a large monitor they leave the content as a narrow strip between two empty margins.
+   Text grows with the root font size, while px measures (covers, grid columns) are widened explicitly since rem does not reach them. */
+@media (min-width: 1400px) {
+  article.content, div.content, .content { max-width: 1440px; padding: 2.5rem 3.5rem; }
+  .kt-search-input-wrap { max-width: 480px; }
+  .kt-detail-hero { max-width: 1100px; }
+  .kt-form-card > .row { max-width: 1100px; }
+  .kt-game-page { max-width: 1100px; }
+}
+
+/* Rows go two per line rather than stretching, which keeps each row's title and delete button close together without leaving the sides empty. */
+@media (min-width: 1600px) {
+  .kt-table-wrap-list { max-width: none; }
+  .kt-item-list { display: grid; grid-template-columns: repeat(2, minmax(0, 1fr)); }
+  .kt-item-list > .kt-item-row:nth-child(odd) { border-right: 1px solid var(--kt-border); }
+}
+
+@media (min-width: 1920px) {
+  html, body { font-size: 16px; }
+  article.content, div.content, .content { max-width: 1800px; padding: 3rem 4rem; }
+  .sidebar { width: 250px; }
+  .kt-search-input-wrap { max-width: 560px; }
+  .kt-sort-select { width: 96px; }
+  .kt-item-thumb { width: 52px; height: 78px; }
+  .kt-item-thumb.square { width: 64px; height: 64px; }
+  .kt-item-thumb.wide { width: 112px; height: 63px; }
+  .kt-item-grid { grid-template-columns: repeat(auto-fill, minmax(175px, 1fr)); gap: 1.5rem 1.25rem; }
+  .kt-item-grid.square { grid-template-columns: repeat(auto-fill, minmax(200px, 1fr)); }
+  .kt-item-grid.wide { grid-template-columns: repeat(auto-fill, minmax(320px, 1fr)); }
+  .kt-detail-hero { max-width: 1300px; --kt-hero-cover-width: 280px; }
+  .kt-detail-hero.square { --kt-hero-cover-width: 320px; }
+  .kt-form-card > .row { max-width: 1300px; }
+  .kt-game-page { max-width: 1300px; }
+}
+
+@media (min-width: 2400px) {
+  html, body { font-size: 17px; }
+}
diff --git a/test/BlazorApp.PlaywrightTests/Smoke/MobileScreenshotTest.cs b/test/BlazorApp.PlaywrightTests/Smoke/MobileScreenshotTest.cs
index db4d8307..37e44921 100644
--- a/test/BlazorApp.PlaywrightTests/Smoke/MobileScreenshotTest.cs
+++ b/test/BlazorApp.PlaywrightTests/Smoke/MobileScreenshotTest.cs
@@ -184,6 +184,23 @@ public async Task CaptureAllPagesAtPhoneViewport()
             await SetListViewPreferenceAsync(null);
             await CaptureAsync("/books", "books-list-desktop");
 
+            // A large monitor, where a layout tuned for a laptop leaves the content as a narrow strip between two empty margins.
+            await Page.SetViewportSizeAsync(2560, 1440);
+            await CaptureAsync("/", "home-large");
+            await CaptureAsync("/books", "books-list-large");
+            await CaptureAsync("/tv-shows", "tv-shows-list-large");
+            await SetListViewPreferenceAsync("grid");
+            await CaptureAsync("/books", "books-grid-large");
+            await CaptureAsync("/video-games", "video-games-grid-large");
+            await SetListViewPreferenceAsync(null);
+            await CaptureAsync("/watch-next", "watch-next-large");
+            await CaptureAsync("/admin/reference-data", "admin-reference-data-large");
+            await CaptureDetailByTitleAsync("/video-games", "Half-Life 2", "video-game-detail-large");
+            await CaptureDetailByTitleAsync("/albums", "Nevermind", "album-detail-large");
+            await CaptureFirstDetailAsync("/movies", "movie-detail-large");
+            await CaptureFirstDetailAsync("/cars", "car-detail-large");
+            await Page.SetViewportSizeAsync(1280, 900);
+
             // A dark-theme sample of the densest pages.
             await Page.EmulateMediaAsync(new PageEmulateMediaOptions { ColorScheme = ColorScheme.Dark });
             await CaptureAsync("/movies", "movies-dark");
diff --git a/test/BlazorApp.UnitTests/Components/Shared/AppVersionTest.cs b/test/BlazorApp.UnitTests/Components/Shared/AppVersionTest.cs
new file mode 100644
index 00000000..e0f01f3f
--- /dev/null
+++ b/test/BlazorApp.UnitTests/Components/Shared/AppVersionTest.cs
@@ -0,0 +1,31 @@
+using AwesomeAssertions;
+using Keeptrack.BlazorApp.Components.Shared;
+using Xunit;
+
+namespace Keeptrack.BlazorApp.UnitTests.Components.Shared;
+
+[Trait("Category", "UnitTests")]
+public class AppVersionTest
+{
+    [Fact]
+    public void Format_ShortensTheCommitHash()
+    {
+        AppVersion.Format("2.4.2-alpha-000123+0dbff56febdf601e3e03e6ee5868bb1bc4c8dc71").Should().Be("2.4.2-alpha-000123+0dbff56");
+    }
+
+    [Theory]
+    [InlineData("2.4.2", "2.4.2")]
+    [InlineData("2.4.2+abc", "2.4.2+abc")]
+    [InlineData(null, "unknown")]
+    [InlineData("", "unknown")]
+    public void Format_KeepsAVersionWithoutALongHash(string? informationalVersion, string expected)
+    {
+        AppVersion.Format(informationalVersion).Should().Be(expected);
+    }
+
+    [Fact]
+    public void Current_IsTheBlazorAppAssemblyVersion()
+    {
+        AppVersion.Current.Should().StartWith(typeof(AppVersion).Assembly.GetName().Version!.ToString(3));
+    }
+}

From 5a9f8cb8b8504111b0da1db38eb55c5c8994e66b Mon Sep 17 00:00:00 2001
From: Bertrand THOMAS <bertrand@devpro.fr>
Date: Tue, 27 Oct 2026 15:12:18 +0100
Subject: [PATCH 3/5] Update NuGet packages October 27, 2026

---
 Directory.Packages.props | 80 ++++++++++++++++++++--------------------
 1 file changed, 40 insertions(+), 40 deletions(-)

diff --git a/Directory.Packages.props b/Directory.Packages.props
index f3d155b2..3575f1d1 100644
--- a/Directory.Packages.props
+++ b/Directory.Packages.props
@@ -1,40 +1,40 @@
-<Project>
-  <PropertyGroup>
-    <ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally>
-    <CentralPackageTransitivePinningEnabled>true</CentralPackageTransitivePinningEnabled>
-  </PropertyGroup>
-  <ItemGroup>
-    <PackageVersion Include="AwesomeAssertions" Version="9.6.0" />
-    <PackageVersion Include="BCrypt.Net-Next" Version="4.2.0" />
-    <PackageVersion Include="Bogus" Version="35.6.5" />
-    <PackageVersion Include="ClosedXML" Version="0.105.1" />
-    <PackageVersion Include="CsvHelper" Version="33.1.0" />
-    <PackageVersion Include="FirebaseAdmin" Version="3.7.0" />
-    <PackageVersion Include="Microsoft.AspNetCore.Authentication.Google" Version="10.0.13" />
-    <PackageVersion Include="Microsoft.AspNetCore.Authentication.JwtBearer" Version="10.0.13" />
-    <PackageVersion Include="Microsoft.AspNetCore.Authentication.OpenIdConnect" Version="10.0.13" />
-    <PackageVersion Include="Microsoft.AspNetCore.Mvc.Testing" Version="10.0.13" />
-    <PackageVersion Include="Microsoft.AspNetCore.OpenApi" Version="10.0.13" />
-    <PackageVersion Include="Microsoft.Bcl.AsyncInterfaces" Version="10.0.13" />
-    <PackageVersion Include="Microsoft.Extensions.Http.Resilience" Version="10.11.0" />
-    <PackageVersion Include="Microsoft.Extensions.Logging.Abstractions" Version="10.0.13" />
-    <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="18.11.0" />
-    <PackageVersion Include="Microsoft.OpenApi" Version="2.13.0" />
-    <PackageVersion Include="Microsoft.Playwright" Version="1.63.0" />
-    <PackageVersion Include="Microsoft.Playwright.Xunit.v3" Version="1.63.0" />
-    <PackageVersion Include="Microsoft.Testing.Extensions.CodeCoverage" Version="18.12.0" />
-    <PackageVersion Include="Microsoft.Testing.Extensions.TrxReport" Version="2.5.1" />
-    <PackageVersion Include="MongoDB.Bson" Version="3.12.0" />
-    <PackageVersion Include="MongoDB.Driver" Version="3.12.0" />
-    <PackageVersion Include="Moq" Version="4.21.0" />
-    <PackageVersion Include="Riok.Mapperly" Version="4.3.1" />
-    <PackageVersion Include="Scalar.AspNetCore" Version="2.18.0" />
-    <PackageVersion Include="SharpCompress" Version="0.50.6" />
-    <PackageVersion Include="System.CodeDom" Version="10.0.13" />
-    <PackageVersion Include="System.Management" Version="10.0.13" />
-    <PackageVersion Include="SystemTextJson.JsonDiffPatch" Version="2.0.0" />
-    <PackageVersion Include="Withywoods.AspNetCore.Mvc.Testing" Version="2.0.5" />
-    <PackageVersion Include="Withywoods.Configuration" Version="2.0.5" />
-    <PackageVersion Include="xunit.v3.mtp-v2" Version="4.0.2" />
-  </ItemGroup>
-</Project>
+<Project>
+  <PropertyGroup>
+    <ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally>
+    <CentralPackageTransitivePinningEnabled>true</CentralPackageTransitivePinningEnabled>
+  </PropertyGroup>
+  <ItemGroup>
+    <PackageVersion Include="AwesomeAssertions" Version="9.6.0" />
+    <PackageVersion Include="BCrypt.Net-Next" Version="4.2.0" />
+    <PackageVersion Include="Bogus" Version="35.6.5" />
+    <PackageVersion Include="ClosedXML" Version="0.105.1" />
+    <PackageVersion Include="CsvHelper" Version="33.1.0" />
+    <PackageVersion Include="FirebaseAdmin" Version="3.7.0" />
+    <PackageVersion Include="Microsoft.AspNetCore.Authentication.Google" Version="10.0.14" />
+    <PackageVersion Include="Microsoft.AspNetCore.Authentication.JwtBearer" Version="10.0.14" />
+    <PackageVersion Include="Microsoft.AspNetCore.Authentication.OpenIdConnect" Version="10.0.14" />
+    <PackageVersion Include="Microsoft.AspNetCore.Mvc.Testing" Version="10.0.14" />
+    <PackageVersion Include="Microsoft.AspNetCore.OpenApi" Version="10.0.14" />
+    <PackageVersion Include="Microsoft.Bcl.AsyncInterfaces" Version="10.0.14" />
+    <PackageVersion Include="Microsoft.Extensions.Http.Resilience" Version="10.11.0" />
+    <PackageVersion Include="Microsoft.Extensions.Logging.Abstractions" Version="10.0.14" />
+    <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="18.11.0" />
+    <PackageVersion Include="Microsoft.OpenApi" Version="2.13.0" />
+    <PackageVersion Include="Microsoft.Playwright" Version="1.64.1" />
+    <PackageVersion Include="Microsoft.Playwright.Xunit.v3" Version="1.64.1" />
+    <PackageVersion Include="Microsoft.Testing.Extensions.CodeCoverage" Version="18.12.0" />
+    <PackageVersion Include="Microsoft.Testing.Extensions.TrxReport" Version="2.5.1" />
+    <PackageVersion Include="MongoDB.Bson" Version="3.13.0" />
+    <PackageVersion Include="MongoDB.Driver" Version="3.13.0" />
+    <PackageVersion Include="Moq" Version="4.21.0" />
+    <PackageVersion Include="Riok.Mapperly" Version="4.3.1" />
+    <PackageVersion Include="Scalar.AspNetCore" Version="2.18.3" />
+    <PackageVersion Include="SharpCompress" Version="0.50.6" />
+    <PackageVersion Include="System.CodeDom" Version="10.0.14" />
+    <PackageVersion Include="System.Management" Version="10.0.14" />
+    <PackageVersion Include="SystemTextJson.JsonDiffPatch" Version="2.0.0" />
+    <PackageVersion Include="Withywoods.AspNetCore.Mvc.Testing" Version="2.0.5" />
+    <PackageVersion Include="Withywoods.Configuration" Version="2.0.5" />
+    <PackageVersion Include="xunit.v3.mtp-v2" Version="4.0.2" />
+  </ItemGroup>
+</Project>

From 566fb933b144033e7a5a34eb0db9483b2db5c88e Mon Sep 17 00:00:00 2001
From: Bertrand THOMAS <bertrand@devpro.fr>
Date: Tue, 27 Oct 2026 15:28:36 +0100
Subject: [PATCH 4/5] Delete a media item from its detail page and draw a
 projector for movies

The admin unlink takes a broken-chain icon, so the trash means deleting the item.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 docs/blazor-ui.md                             |  9 +++-
 .../Inventory/Pages/AlbumDetail.razor         |  1 +
 .../Inventory/Pages/BookDetail.razor          |  1 +
 .../Inventory/Pages/MovieDetail.razor         |  1 +
 .../Inventory/Pages/TvShowDetail.razor        |  1 +
 .../Inventory/Pages/VideoGameDetail.razor     |  1 +
 .../Inventory/Shared/DeleteItemControl.razor  | 47 +++++++++++++++++++
 .../Shared/ReferenceMatchControls.razor       |  2 +-
 src/BlazorApp/Components/Shared/Icon.razor    | 13 ++++-
 .../Pages/ReferenceableDetailPageBase.cs      | 11 +++++
 .../Smoke/MovieSmokeTest.cs                   | 29 +++++++++++-
 11 files changed, 111 insertions(+), 5 deletions(-)
 create mode 100644 src/BlazorApp/Components/Inventory/Shared/DeleteItemControl.razor

diff --git a/docs/blazor-ui.md b/docs/blazor-ui.md
index c2eb3e11..6e6c3dea 100644
--- a/docs/blazor-ui.md
+++ b/docs/blazor-ui.md
@@ -79,6 +79,10 @@ A missing item is a 404 at every layer:
 - **Enter takes the highlight, Tab only one reached with the arrow keys**, since completing on Tab turns "Dr Kim" into "Dr Kimura".
   Escape drops the highlight, and there is no `preventDefault` on keydown.
 
+**`DeleteItemControl`** deletes a media item from its detail page, with the same confirmation as a list row.
+It replaces the history entry with the list, so browser back skips the deleted item.
+The admin unlink beside it uses the broken-chain icon, since the trash means deleting the item.
+
 ## Razor gotchas
 
 - **A `string`-typed component `[Parameter]` needs the `@` prefix.**
@@ -103,7 +107,10 @@ Sonar reports every `!important` (`css:S9423`), and an `!important` theme also s
 ### Responsive widths
 
 Phone below 768px, laptop up to 1400px, and the large-screen block last in `app.css` above it, so its overrides win the cascade.
-At 1400px and up the content column and the detail measures widen, at 1600px list rows go two per line instead of stretching, and at 1920px the root font size grows with the px measures (covers, grid columns, thumbnails) widened to match, since rem does not reach them.
+From 1400px, the content column and the detail measures widen.
+From 1600px, list rows go two per line instead of stretching.
+From 1920px, the root font size grows.
+The px measures (covers, grid columns, thumbnails) are widened alongside it, since rem does not reach them.
 
 ### Icons
 
diff --git a/src/BlazorApp/Components/Inventory/Pages/AlbumDetail.razor b/src/BlazorApp/Components/Inventory/Pages/AlbumDetail.razor
index 4fd86a5f..2d6147a5 100644
--- a/src/BlazorApp/Components/Inventory/Pages/AlbumDetail.razor
+++ b/src/BlazorApp/Components/Inventory/Pages/AlbumDetail.razor
@@ -31,6 +31,7 @@ else
             <ReferenceMatchControls TDto="AlbumDto" Api="Api" ItemId="@(Album.Id!)" ReferenceId="@Album.ReferenceId" ReferenceTitle="@Reference?.Title"
                                     ReferenceDataDescription="cover, tracklist, artist"
                                     OnChecked="OnReferenceCheckedAsync" OnUnlinked="LoadAsync"/>
+            <DeleteItemControl TDto="AlbumDto" Api="Api" ItemId="@(Album.Id!)" ItemName="album" ListRoute="/albums"/>
         </div>
         <div class="d-flex gap-2">
             <button class="btn btn-sm kt-toggle-btn @(Album.IsFavorite ? "active" : "")" @onclick="ToggleFavoriteAsync">Favorites</button>
diff --git a/src/BlazorApp/Components/Inventory/Pages/BookDetail.razor b/src/BlazorApp/Components/Inventory/Pages/BookDetail.razor
index dc372548..ba0e5e3d 100644
--- a/src/BlazorApp/Components/Inventory/Pages/BookDetail.razor
+++ b/src/BlazorApp/Components/Inventory/Pages/BookDetail.razor
@@ -31,6 +31,7 @@ else
             <ReferenceMatchControls TDto="BookDto" Api="Api" ItemId="@(Book.Id!)" ReferenceId="@Book.ReferenceId" ReferenceTitle="@Reference?.Title"
                                     ReferenceDataDescription="cover, synopsis, genre/language/ISBN autofill"
                                     OnChecked="OnReferenceCheckedAsync" OnUnlinked="LoadAsync"/>
+            <DeleteItemControl TDto="BookDto" Api="Api" ItemId="@(Book.Id!)" ItemName="book" ListRoute="/books"/>
         </div>
         @* "Read" is not here but on the card below as a .kt-corner-flag (see MovieDetail). *@
         <div class="d-flex gap-2 flex-wrap">
diff --git a/src/BlazorApp/Components/Inventory/Pages/MovieDetail.razor b/src/BlazorApp/Components/Inventory/Pages/MovieDetail.razor
index f30a9300..49231cf9 100644
--- a/src/BlazorApp/Components/Inventory/Pages/MovieDetail.razor
+++ b/src/BlazorApp/Components/Inventory/Pages/MovieDetail.razor
@@ -31,6 +31,7 @@ else
             <ReferenceMatchControls TDto="MovieDto" Api="Api" ItemId="@(Movie.Id!)" ReferenceId="@Movie.ReferenceId" ReferenceTitle="@Reference?.Title"
                                     ReferenceDataDescription="cover, synopsis, cast/genres"
                                     OnChecked="OnReferenceCheckedAsync" OnUnlinked="LoadAsync"/>
+            <DeleteItemControl TDto="MovieDto" Api="Api" ItemId="@(Movie.Id!)" ItemName="movie" ListRoute="/movies"/>
         </div>
         @* "Watched" is not here but on the card below as a .kt-corner-flag, since it outranks these three flags,
            and its success green says "finished" where their accent blue says "flagged". *@
diff --git a/src/BlazorApp/Components/Inventory/Pages/TvShowDetail.razor b/src/BlazorApp/Components/Inventory/Pages/TvShowDetail.razor
index fa2c25bb..65080e5c 100644
--- a/src/BlazorApp/Components/Inventory/Pages/TvShowDetail.razor
+++ b/src/BlazorApp/Components/Inventory/Pages/TvShowDetail.razor
@@ -31,6 +31,7 @@ else
             <ReferenceMatchControls TDto="TvShowDto" Api="Api" ItemId="@(_show.Id!)" ReferenceId="@_show.ReferenceId" ReferenceTitle="@_reference?.Title"
                                     ReferenceDataDescription="cover, synopsis, cast/genres, episode guide"
                                     OnChecked="OnReferenceCheckedAsync" OnUnlinked="LoadAsync"/>
+            <DeleteItemControl TDto="TvShowDto" Api="Api" ItemId="@(_show.Id!)" ItemName="TV show" ListRoute="/tv-shows"/>
         </div>
         <div class="d-flex gap-2">
             <button class="btn btn-sm kt-toggle-btn @(_show.IsFavorite ? "active" : "")" @onclick="ToggleFavoriteAsync">Favorites</button>
diff --git a/src/BlazorApp/Components/Inventory/Pages/VideoGameDetail.razor b/src/BlazorApp/Components/Inventory/Pages/VideoGameDetail.razor
index 271932d3..10a6b172 100644
--- a/src/BlazorApp/Components/Inventory/Pages/VideoGameDetail.razor
+++ b/src/BlazorApp/Components/Inventory/Pages/VideoGameDetail.razor
@@ -34,6 +34,7 @@ else
             <ReferenceMatchControls TDto="VideoGameDto" Api="Api" ItemId="@(Game.Id!)" ReferenceId="@Game.ReferenceId" ReferenceTitle="@Reference?.Title"
                                     ReferenceDataDescription="cover, synopsis, genres/platforms"
                                     OnChecked="OnReferenceCheckedAsync" OnUnlinked="LoadAsync"/>
+            <DeleteItemControl TDto="VideoGameDto" Api="Api" ItemId="@(Game.Id!)" ItemName="video game" ListRoute="/video-games"/>
         </div>
         <div class="d-flex gap-2">
             <button class="btn btn-sm kt-toggle-btn @(Game.IsWishlisted ? "active" : "")" @onclick="ToggleWishlistedAsync"><Icon Name="wishlist" Size="13"/> Wishlist</button>
diff --git a/src/BlazorApp/Components/Inventory/Shared/DeleteItemControl.razor b/src/BlazorApp/Components/Inventory/Shared/DeleteItemControl.razor
new file mode 100644
index 00000000..eb86c651
--- /dev/null
+++ b/src/BlazorApp/Components/Inventory/Shared/DeleteItemControl.razor
@@ -0,0 +1,47 @@
+@typeparam TDto where TDto : Keeptrack.Common.System.IHasId
+@inject NavigationManager Navigation
+@* A detail page's own delete, with the same confirmation as a list row's. *@
+
+@if (_error is not null)
+{
+    <span class="kt-inline-toast danger">@_error</span>
+}
+<button class="kt-icon-btn" title="Delete this @ItemName" aria-label="Delete" @onclick="() => _pending = true">
+    <Icon Name="trash" Size="15"/>
+</button>
+<ConfirmModal Show="_pending"
+              Title="Delete this item?"
+              Message="@($"This will permanently delete this {ItemName}. This can't be undone.")"
+              OnConfirm="DeleteAsync" OnCancel="() => _pending = false"/>
+
+@code {
+    [Parameter, EditorRequired] public required InventoryApiClientBase<TDto> Api { get; set; }
+
+    [Parameter, EditorRequired] public required string ItemId { get; set; }
+
+    /// <summary>Lower case, as it reads in the confirmation, for example "movie".</summary>
+    [Parameter, EditorRequired] public required string ItemName { get; set; }
+
+    [Parameter, EditorRequired] public required string ListRoute { get; set; }
+
+    private bool _pending;
+    private string? _error;
+
+    private async Task DeleteAsync()
+    {
+        _pending = false;
+        _error = null;
+        try
+        {
+            await Api.DeleteAsync(ItemId);
+        }
+        catch (Exception ex)
+        {
+            _error = $"Error: {ex.Message}";
+            return;
+        }
+
+        // Replaces the history entry, so browser back skips the page of an item that no longer exists.
+        Navigation.NavigateTo(ListRoute, replace: true);
+    }
+}
diff --git a/src/BlazorApp/Components/Inventory/Shared/ReferenceMatchControls.razor b/src/BlazorApp/Components/Inventory/Shared/ReferenceMatchControls.razor
index 84275e6e..7e83cfce 100644
--- a/src/BlazorApp/Components/Inventory/Shared/ReferenceMatchControls.razor
+++ b/src/BlazorApp/Components/Inventory/Shared/ReferenceMatchControls.razor
@@ -19,7 +19,7 @@
         <Authorized>
             <button class="kt-icon-btn" title="Unlink and remove this reference" aria-label="Unlink reference"
                     @onclick="() => _pendingUnlink = true">
-                <Icon Name="trash" Size="15"/>
+                <Icon Name="unlink" Size="15"/>
             </button>
         </Authorized>
     </AuthorizeView>
diff --git a/src/BlazorApp/Components/Shared/Icon.razor b/src/BlazorApp/Components/Shared/Icon.razor
index 443f8aba..5698537f 100644
--- a/src/BlazorApp/Components/Shared/Icon.razor
+++ b/src/BlazorApp/Components/Shared/Icon.razor
@@ -38,8 +38,10 @@
             break;
 
         case "movies":
-            <rect x="3" y="4.5" width="18" height="15" rx="2"/>
-            <path d="M3 9.5h18M3 14.5h18M8 4.5v15M16 4.5v15"/>
+            <circle cx="6.5" cy="6.5" r="3"/>
+            <circle cx="13.5" cy="6.5" r="3"/>
+            <rect x="3" y="10.5" width="13" height="8.5" rx="1.5"/>
+            <path d="M16 13.4 21 11v7.5l-5-2.4"/>
             break;
 
         case "tv-shows":
@@ -139,6 +141,13 @@
             <path d="M10.2 10.5v7M13.8 10.5v7"/>
             break;
 
+        @* A broken chain, distinct from "trash": it removes the link to the reference, never the item itself. *@
+        case "unlink":
+            <path d="M13 6.4 14.4 5a3.8 3.8 0 0 1 5.4 5.4l-1.4 1.4"/>
+            <path d="M11 17.6 9.6 19a3.8 3.8 0 0 1-5.4-5.4l1.4-1.4"/>
+            <path d="M8 3v2.6M3 8h2.6M16 21v-2.6M21 16h-2.6"/>
+            break;
+
         case "external-link":
             <path d="M18 13v6a2 2 0 0 1-2 2H5a2 2 0 0 1-2-2V8a2 2 0 0 1 2-2h6"/>
             <path d="M15 3h6v6"/>
diff --git a/test/BlazorApp.PlaywrightTests/Pages/ReferenceableDetailPageBase.cs b/test/BlazorApp.PlaywrightTests/Pages/ReferenceableDetailPageBase.cs
index a3c7fb54..f62ac070 100644
--- a/test/BlazorApp.PlaywrightTests/Pages/ReferenceableDetailPageBase.cs
+++ b/test/BlazorApp.PlaywrightTests/Pages/ReferenceableDetailPageBase.cs
@@ -20,6 +20,17 @@ public abstract class ReferenceableDetailPageBase(IPage page) : DetailPageBase(p
 
     private ILocator ReferenceToast => Page.Locator(".kt-inline-toast");
 
+    /// <summary>
+    /// Deletes the item from its own page, the confirm click scoped to the modal, whose button shares the "Delete" name.
+    /// Opening the confirmation is the first click after load, so it goes through <see cref="PageBase.ClickUntilAsync(ILocator, ILocator, int)"/>.
+    /// </summary>
+    public async Task DeleteItemAsync()
+    {
+        var confirm = Page.Locator(".kt-modal").GetByRole(AriaRole.Button, new LocatorGetByRoleOptions { Name = "Delete", Exact = true });
+        await ClickUntilAsync(Page.Locator(".kt-page-header").GetByRole(AriaRole.Button, new LocatorGetByRoleOptions { Name = "Delete", Exact = true }), confirm);
+        await confirm.ClickAsync();
+    }
+
     /// <summary>
     /// The non-admin "check for reference match" icon button.
     /// </summary>
diff --git a/test/BlazorApp.PlaywrightTests/Smoke/MovieSmokeTest.cs b/test/BlazorApp.PlaywrightTests/Smoke/MovieSmokeTest.cs
index 95018bfb..49ccc797 100644
--- a/test/BlazorApp.PlaywrightTests/Smoke/MovieSmokeTest.cs
+++ b/test/BlazorApp.PlaywrightTests/Smoke/MovieSmokeTest.cs
@@ -1,6 +1,9 @@
+using System.Net;
+using System.Text.RegularExpressions;
 using System.Threading.Tasks;
 using Keeptrack.BlazorApp.PlaywrightTests.Hosting;
 using Keeptrack.BlazorApp.PlaywrightTests.Pages;
+using Keeptrack.WebApi.Contracts.Dto;
 using Microsoft.Playwright;
 using Xunit;
 
@@ -11,7 +14,7 @@ namespace Keeptrack.BlazorApp.PlaywrightTests.Smoke;
 /// </summary>
 [Trait("Category", "E2eTests")]
 [Trait("Mode", "Mutating")]
-public class MovieSmokeTest(End2EndFixture fixture) : SmokeTestBase(fixture)
+public partial class MovieSmokeTest(End2EndFixture fixture) : SmokeTestBase(fixture)
 {
     private const string Title = "The Terminator";
     private const string Year = "1984";
@@ -37,4 +40,28 @@ public async Task AddLinkAndDelete_MovieThroughTheList()
 
         await Assertions.Expect(detail.CoverImage).ToBeVisibleAsync();
     }
+
+    [Fact]
+    public async Task Delete_FromTheDetailPage_ReturnsToTheListWithoutTheItem()
+    {
+        SkipIfReadOnly();
+        // a title no provider holds, so no background link lands while the page is open
+        const string title = "Zzq Detail Delete Movie";
+        var movie = await CreateItemAsync("api/movies", new MovieDto { Title = title });
+
+        await Page.GotoAsync($"/movies/{movie.Id}");
+        var detail = new MovieDetailPage(Page);
+        await detail.WaitForReadyAsync();
+        await detail.DeleteItemAsync();
+
+        var list = new ListPage(Page, "/movies", "Movies");
+        await list.WaitForReadyAsync();
+        await Assertions.Expect(Page).ToHaveURLAsync(MoviesListRegex());
+        await Assertions.Expect(list.Row(title)).ToHaveCountAsync(0);
+        var response = await Fixture.ApiHttpClient.GetAsync($"api/movies/{movie.Id}", TestContext.Current.CancellationToken);
+        Assert.Equal(HttpStatusCode.NotFound, response.StatusCode);
+    }
+
+    [GeneratedRegex("/movies$")]
+    private static partial Regex MoviesListRegex();
 }

From 42860a4e674aa9682265b9ed99d2d263fd52a6e9 Mon Sep 17 00:00:00 2001
From: Bertrand THOMAS <bertrand@devpro.fr>
Date: Tue, 27 Oct 2026 15:35:33 +0100
Subject: [PATCH 5/5] Make sure read permission by default in GitHub pipelines

---
 .github/workflows/ci.yaml  | 1 -
 .github/workflows/pkg.yaml | 3 +++
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
index 6dc1a40c..cdec53ef 100644
--- a/.github/workflows/ci.yaml
+++ b/.github/workflows/ci.yaml
@@ -28,7 +28,6 @@ concurrency:
 
 permissions:
   contents: read
-  pull-requests: write
 
 jobs:
   markup-lint:
diff --git a/.github/workflows/pkg.yaml b/.github/workflows/pkg.yaml
index 558fa3ee..d7e1f52d 100644
--- a/.github/workflows/pkg.yaml
+++ b/.github/workflows/pkg.yaml
@@ -7,6 +7,9 @@ on:
     branches: [main]
   workflow_dispatch: {}
 
+permissions:
+  contents: read
+
 jobs:
   git-check:
     name: Git