diff --git a/docs/player_gm_ux_redesign_plan.md b/docs/player_gm_ux_redesign_plan.md index 0e2bcfe..7ee27e4 100644 --- a/docs/player_gm_ux_redesign_plan.md +++ b/docs/player_gm_ux_redesign_plan.md @@ -516,6 +516,10 @@ Acceptance criteria: ### Phase 2: Clean up critical table browsing +Status: + +- implemented in the web app on March 15, 2026 + Scope: - reframe tables page around reading and selecting outcomes diff --git a/src/RolemasterDb.App/Components/Pages/Tables.razor b/src/RolemasterDb.App/Components/Pages/Tables.razor index 2f41cab..715158b 100644 --- a/src/RolemasterDb.App/Components/Pages/Tables.razor +++ b/src/RolemasterDb.App/Components/Pages/Tables.razor @@ -9,31 +9,35 @@ Critical Tables
-
- - + @if (referenceData is null) { - + } - } - + else + { + @foreach (var table in referenceData.CriticalTables) + { + + } + } + +
+ +

Choose a table, then read from the roll band on the left across to the result you need.

@if (referenceData is null) { -

Loading reference data...

+

Loading table list...

} else if (!referenceData.CriticalTables.Any()) { @@ -53,9 +57,17 @@ } else if (tableDetail is { } detail) { + var readingHint = detail.Groups.Count > 0 + ? "Find the roll band on the left, then read across to the group and severity you need." + : "Find the roll band on the left, then read across to the severity you need."; +
-
-

@detail.DisplayName

+
+
+

@detail.DisplayName

+

@readingHint

+
+

Click any filled result to correct it.

@@ -168,7 +180,10 @@ @if (legendEntries.Count > 0) {
-

Affix legend

+
+

Reading help

+

These symbols show the effects attached to a result at a glance.

+
@foreach (var entry in legendEntries) { diff --git a/src/RolemasterDb.App/Components/Shared/CompactCriticalCell.razor b/src/RolemasterDb.App/Components/Shared/CompactCriticalCell.razor index fbc1b0c..b8fd06f 100644 --- a/src/RolemasterDb.App/Components/Shared/CompactCriticalCell.razor +++ b/src/RolemasterDb.App/Components/Shared/CompactCriticalCell.razor @@ -7,22 +7,27 @@

@Description

} - - - @if (Branches?.Count > 0) + @if ((Effects?.Count ?? 0) > 0 || (Branches?.Count ?? 0) > 0) { -
- @foreach (var branch in Branches) +