From ccb9f2d9854e3a6537f58b13b0ea2f898717e775 Mon Sep 17 00:00:00 2001 From: mfoltz Date: Sat, 9 May 2026 14:33:31 -0500 Subject: [PATCH 1/4] Move NPC summary into the quick facts rail --- src/components/db/DbDetailView.test.tsx | 3 ++ src/components/db/DbDetailView.tsx | 41 ++++++++++++++++--------- 2 files changed, 29 insertions(+), 15 deletions(-) diff --git a/src/components/db/DbDetailView.test.tsx b/src/components/db/DbDetailView.test.tsx index 79cf9335c8..f5137a9a16 100644 --- a/src/components/db/DbDetailView.test.tsx +++ b/src/components/db/DbDetailView.test.tsx @@ -133,6 +133,9 @@ test("structured NPC detail keeps summary cues while consolidating duplicate rel assert.match(html, /⚑/); assert.match(html, /◇/); assert.match(html, /✧/); + assert.match(html, /Quick Facts/); + assert.match(html, /Quick Facts<\/div>
NPC Archive<\/div>/); + assert.match(html, /
{detailIcon && !showSummaryRail && section !== "items" ? ( {showSummaryRail ? ( - ) : null} From b0a381bd14767e6db57890bab0e19ecb1423a041 Mon Sep 17 00:00:00 2001 From: mfoltz Date: Sat, 9 May 2026 17:45:33 -0500 Subject: [PATCH 4/4] Keep overflow fact rows visible on mobile and desktop --- src/components/db/DbDetailView.test.tsx | 4 ++-- src/components/db/DbDetailView.tsx | 9 ++++++++- 2 files changed, 10 insertions(+), 3 deletions(-) diff --git a/src/components/db/DbDetailView.test.tsx b/src/components/db/DbDetailView.test.tsx index bcf0472540..a2f130f372 100644 --- a/src/components/db/DbDetailView.test.tsx +++ b/src/components/db/DbDetailView.test.tsx @@ -97,10 +97,10 @@ test("structured recipe detail keeps summary cues while consolidating duplicate assert.equal(countMatches(html, /href="#relation-repair-costs"/g), 0); }); -test("structured recipe detail keeps non-summary fact rows in the quick facts rail", () => { +test("structured recipe detail keeps non-summary fact rows visible on mobile and desktop", () => { const html = renderDetail({ ...recipeDetailFixture, alwaysUnlocked: true, hideInStation: true, ignoreServerSettings: true }, "recipes"); - assert.match(html, /Ignores Server Settings/); + assert.equal(countMatches(html, /Ignores Server Settings/g), 2); }); test("structured item detail keeps localized copy while consolidating duplicate relation sections", () => { diff --git a/src/components/db/DbDetailView.tsx b/src/components/db/DbDetailView.tsx index e790435dbc..d5017b75e3 100644 --- a/src/components/db/DbDetailView.tsx +++ b/src/components/db/DbDetailView.tsx @@ -1641,7 +1641,14 @@ function renderHero(section: DbSection, detail: DbEntityDetail, factRows: DbDisp

) : null} - {placeStructuredSummaryInRail ?
{structuredSummary}
: structuredSummary} + {placeStructuredSummaryInRail ? ( +
+ {structuredSummary} + {summaryFactRows.length > 0 ?
{renderSummaryRows(summaryFactRows)}
: null} +
+ ) : ( + structuredSummary + )} {detailIcon && !showSummaryRail && section !== "items" ? (