Skip to content

Commit 6844825

Browse files
ToolboxAidToolboxAid
committed
Polish System Health Theme V2 UI
1 parent 5cf6a87 commit 6844825

11 files changed

Lines changed: 379 additions & 465 deletions

‎assets/theme-v2/css/status.css‎

Lines changed: 52 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -289,3 +289,55 @@ body.tool-focus-mode .tool-center-panel {
289289
max-width: 30vw
290290
}
291291
}
292+
293+
[data-admin-system-health] [data-health-status] {
294+
font-weight: var(--font-weight-heavy);
295+
letter-spacing: 0;
296+
text-transform: uppercase
297+
}
298+
299+
[data-admin-system-health] td[data-health-status] {
300+
width: 1%;
301+
min-width: 7rem;
302+
text-align: center;
303+
white-space: nowrap
304+
}
305+
306+
[data-admin-system-health] p[data-health-status] {
307+
display: inline-flex;
308+
align-items: center;
309+
justify-content: center;
310+
width: fit-content;
311+
max-width: 100%;
312+
min-height: var(--space-28);
313+
padding: var(--space-4) var(--space-10);
314+
border: var(--border-standard);
315+
border-radius: var(--radius-pill);
316+
overflow-wrap: anywhere
317+
}
318+
319+
[data-admin-system-health] [data-health-status="PASS"] {
320+
border-color: color-mix(in srgb, var(--green) 58%, var(--line));
321+
background: color-mix(in srgb, var(--green) 16%, transparent);
322+
color: var(--green)
323+
}
324+
325+
[data-admin-system-health] [data-health-status="WARN"],
326+
[data-admin-system-health] [data-health-status="PENDING"] {
327+
border-color: color-mix(in srgb, var(--gold) 54%, var(--line));
328+
background: color-mix(in srgb, var(--gold) 13%, transparent);
329+
color: var(--gold)
330+
}
331+
332+
[data-admin-system-health] [data-health-status="FAIL"] {
333+
border-color: color-mix(in srgb, var(--red) 58%, var(--line));
334+
background: color-mix(in srgb, var(--red) 15%, transparent);
335+
color: var(--deep-red)
336+
}
337+
338+
[data-admin-system-health] [data-health-status="INFO"],
339+
[data-admin-system-health] [data-health-status="SKIP"] {
340+
border-color: color-mix(in srgb, var(--cyan) 52%, var(--line));
341+
background: color-mix(in srgb, var(--cyan) 12%, transparent);
342+
color: var(--cyan)
343+
}

‎assets/theme-v2/css/tables.css‎

Lines changed: 37 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -178,3 +178,40 @@ td {
178178
max-width: 100%;
179179
width: 100%
180180
}
181+
182+
[data-admin-system-health] .table-wrapper {
183+
border: var(--border-standard);
184+
border-radius: var(--radius-lg);
185+
background: var(--panel-overlay);
186+
box-shadow: var(--shadow-sm)
187+
}
188+
189+
[data-admin-system-health] .data-table {
190+
min-width: 720px;
191+
background: transparent
192+
}
193+
194+
[data-admin-system-health] .data-table caption {
195+
padding: var(--space-12) var(--space-14);
196+
border-bottom: var(--border-standard);
197+
background: var(--panel-soft);
198+
color: var(--gold)
199+
}
200+
201+
[data-admin-system-health] .data-table th,
202+
[data-admin-system-health] .data-table td {
203+
padding: var(--space-10) var(--space-12);
204+
vertical-align: top
205+
}
206+
207+
[data-admin-system-health] .data-table th {
208+
color: var(--cyan)
209+
}
210+
211+
[data-admin-system-health] .data-table tbody tr:nth-child(even) {
212+
background: var(--panel-soft-subtle)
213+
}
214+
215+
[data-admin-system-health] .data-table td {
216+
overflow-wrap: anywhere
217+
}

‎docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish.md‎

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
# PR_26177_CHARLIE_035-system-health-ui-polish
2+
3+
Team: Charlie
4+
Branch: pr/26177-CHARLIE-035-system-health-ui-polish
5+
Base: pr/26177-CHARLIE-034-startup-runtime-report-cleanup
6+
Lifecycle: Build / Validation
7+
Repair: Rebased onto repaired PR_26177_CHARLIE_034 branch on 2026-06-25.
8+
9+
## Scope
10+
- Added Theme V2 styling for System Health table wrappers, captions, row rhythm, and dense table spacing.
11+
- Added Theme V2 status treatment for System Health status indicators.
12+
- Kept the change visual-only with no new API, runtime, UI behavior, or page-local CSS.
13+
14+
## Changed Files
15+
- assets/theme-v2/css/status.css
16+
- assets/theme-v2/css/tables.css
17+
- docs_build/dev/reports/coverage_changed_js_guardrail.txt
18+
- docs_build/dev/reports/playwright_v8_coverage_report.txt
19+
20+
## Validation
21+
- PASS: npx playwright test tests/playwright/tools/AdminHealthOperationsPage.spec.mjs --workers=1 --reporter=line
22+
- PASS: git diff --check
23+
24+
## ZIP
25+
- Generated after repair: C:\Users\DavidQ\Documents\GitHub\HTML-JavaScript-Gaming\tmp\PR_26177_CHARLIE_035-system-health-ui-polish_delta.zip

‎docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish_branch-validation.md‎

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
# PR_26177_CHARLIE_035 Branch Validation
2+
3+
Branch: pr/26177-CHARLIE-035-system-health-ui-polish
4+
Base: pr/26177-CHARLIE-034-startup-runtime-report-cleanup
5+
6+
## Results
7+
- PASS: Continued the stacked Charlie workstream from PR_26177_CHARLIE_034.
8+
- PASS: Branch is based on repaired PR_26177_CHARLIE_034 branch.
9+
- PASS: Rebase conflict scope was generated report artifacts only.
10+
- PASS: Worktree was clean before edits.
11+
- PASS: Changes are limited to Theme V2 visual polish and reports.
12+
- PASS: No start_of_day files were modified.
13+
- PASS: No direct commit to main was made.
14+
- PASS: PR branch will be pushed for draft PR creation.

‎docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish_manual-validation-notes.md‎

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
# PR_26177_CHARLIE_035 Manual Validation Notes
2+
3+
- Confirmed System Health tables use scoped Theme V2 table wrapper and caption polish.
4+
- Confirmed status indicators have distinct PASS, WARN/PENDING, FAIL, INFO, and SKIP treatments.
5+
- Confirmed the page still uses external Theme V2 CSS and JavaScript only.
6+
- Confirmed no runtime behavior, API contract, database contract, or storage contract was changed.
7+
- Confirmed branch repair conflict was limited to generated report artifacts.
8+
- Confirmed no start_of_day files changed.

‎docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish_requirements-checklist.md‎

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
# PR_26177_CHARLIE_035 Requirement Checklist
2+
3+
- PASS: Theme V2 polish only.
4+
- PASS: Improve System Health layout/readability/status indicators.
5+
- PASS: No new functionality beyond visual polish.
6+
- PASS: Use existing Theme V2 CSS assets.
7+
- PASS: No inline styles, style blocks, script blocks, inline handlers, or page-local CSS.
8+
- PASS: Preserve existing behavior.
9+
- PASS: Do not modify unrelated files.
10+
- PASS: Do not modify start_of_day folders.
11+
- PASS: Do not introduce MEM DB, fake-login, silent fallbacks, or browser-owned infrastructure state.
12+
- PASS: Rebased onto repaired PR_26177_CHARLIE_034 branch.

‎docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish_validation-lane.md‎

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
# PR_26177_CHARLIE_035 Validation Lane
2+
3+
## Commands
4+
- PASS: npx playwright test tests/playwright/tools/AdminHealthOperationsPage.spec.mjs --workers=1 --reporter=line
5+
- PASS: git diff --check
6+
7+
## Notes
8+
- `git diff --check` reported only expected Windows LF-to-CRLF working-copy warnings.
9+
- No API/unit tests were required because this PR changes only Theme V2 CSS.
10+
- Full samples smoke was not run because this PR is limited to Admin System Health UI polish.

‎docs_build/dev/reports/codex_changed_files.txt‎

Lines changed: 24 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -1,38 +1,31 @@
1-
# git diff --name-only pr/26177-CHARLIE-032-runtime-health-json-endpoints --
2-
docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup.md
3-
docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_branch-validation.md
4-
docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_manual-validation-notes.md
5-
docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_requirements-checklist.md
6-
docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_validation-lane.md
1+
# git diff --name-only pr/26177-CHARLIE-034-startup-runtime-report-cleanup --
2+
assets/theme-v2/css/status.css
3+
assets/theme-v2/css/tables.css
4+
docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish.md
5+
docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish_branch-validation.md
6+
docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish_manual-validation-notes.md
7+
docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish_requirements-checklist.md
8+
docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish_validation-lane.md
79
docs_build/dev/reports/codex_changed_files.txt
810
docs_build/dev/reports/codex_review.diff
11+
docs_build/dev/reports/coverage_changed_js_guardrail.txt
912
docs_build/dev/reports/playwright_v8_coverage_report.txt
10-
scripts/start-local-api-server.mjs
11-
src/dev-runtime/server/local-api-router.mjs
12-
tests/dev-runtime/AdminHealthOperations.test.mjs
13-
tests/dev-runtime/LocalApiStartupLogging.test.mjs
14-
tests/playwright/tools/AdminHealthOperationsPage.spec.mjs
1513

1614
# git status --short
17-
M docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup.md
18-
M docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_branch-validation.md
19-
M docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_manual-validation-notes.md
20-
M docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_requirements-checklist.md
15+
M docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish.md
2116
M docs_build/dev/reports/codex_review.diff
22-
M docs_build/dev/reports/playwright_v8_coverage_report.txt
17+
M docs_build/dev/reports/coverage_changed_js_guardrail.txt
2318

24-
# git diff --stat pr/26177-CHARLIE-032-runtime-health-json-endpoints --
25-
...7_CHARLIE_034-startup-runtime-report-cleanup.md | 35 ++
26-
...tup-runtime-report-cleanup_branch-validation.md | 14 +
27-
...ntime-report-cleanup_manual-validation-notes.md | 9 +
28-
...untime-report-cleanup_requirements-checklist.md | 14 +
29-
...artup-runtime-report-cleanup_validation-lane.md | 14 +
30-
docs_build/dev/reports/codex_changed_files.txt | 59 ++-
31-
docs_build/dev/reports/codex_review.diff | 479 ++++++++++++++++++++-
32-
.../dev/reports/playwright_v8_coverage_report.txt | 3 +-
33-
scripts/start-local-api-server.mjs | 34 +-
34-
src/dev-runtime/server/local-api-router.mjs | 101 ++++-
35-
tests/dev-runtime/AdminHealthOperations.test.mjs | 7 +
36-
tests/dev-runtime/LocalApiStartupLogging.test.mjs | 32 ++
37-
.../tools/AdminHealthOperationsPage.spec.mjs | 6 +
38-
13 files changed, 771 insertions(+), 36 deletions(-)
19+
# git diff --stat pr/26177-CHARLIE-034-startup-runtime-report-cleanup --
20+
assets/theme-v2/css/status.css | 52 ++
21+
assets/theme-v2/css/tables.css | 37 ++
22+
...PR_26177_CHARLIE_035-system-health-ui-polish.md | 25 +
23+
...35-system-health-ui-polish_branch-validation.md | 14 +
24+
...tem-health-ui-polish_manual-validation-notes.md | 8 +
25+
...stem-health-ui-polish_requirements-checklist.md | 12 +
26+
..._035-system-health-ui-polish_validation-lane.md | 10 +
27+
docs_build/dev/reports/codex_changed_files.txt | 57 +-
28+
docs_build/dev/reports/codex_review.diff | 617 +++++++--------------
29+
.../dev/reports/coverage_changed_js_guardrail.txt | 4 +-
30+
.../dev/reports/playwright_v8_coverage_report.txt | 10 +-
31+
11 files changed, 382 insertions(+), 464 deletions(-)

0 commit comments

Comments
 (0)