Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
52 changes: 52 additions & 0 deletions assets/theme-v2/css/status.css
Original file line number Diff line number Diff line change
Expand Up @@ -289,3 +289,55 @@ body.tool-focus-mode .tool-center-panel {
max-width: 30vw
}
}

[data-admin-system-health] [data-health-status] {
font-weight: var(--font-weight-heavy);
letter-spacing: 0;
text-transform: uppercase
}

[data-admin-system-health] td[data-health-status] {
width: 1%;
min-width: 7rem;
text-align: center;
white-space: nowrap
}

[data-admin-system-health] p[data-health-status] {
display: inline-flex;
align-items: center;
justify-content: center;
width: fit-content;
max-width: 100%;
min-height: var(--space-28);
padding: var(--space-4) var(--space-10);
border: var(--border-standard);
border-radius: var(--radius-pill);
overflow-wrap: anywhere
}

[data-admin-system-health] [data-health-status="PASS"] {
border-color: color-mix(in srgb, var(--green) 58%, var(--line));
background: color-mix(in srgb, var(--green) 16%, transparent);
color: var(--green)
}

[data-admin-system-health] [data-health-status="WARN"],
[data-admin-system-health] [data-health-status="PENDING"] {
border-color: color-mix(in srgb, var(--gold) 54%, var(--line));
background: color-mix(in srgb, var(--gold) 13%, transparent);
color: var(--gold)
}

[data-admin-system-health] [data-health-status="FAIL"] {
border-color: color-mix(in srgb, var(--red) 58%, var(--line));
background: color-mix(in srgb, var(--red) 15%, transparent);
color: var(--deep-red)
}

[data-admin-system-health] [data-health-status="INFO"],
[data-admin-system-health] [data-health-status="SKIP"] {
border-color: color-mix(in srgb, var(--cyan) 52%, var(--line));
background: color-mix(in srgb, var(--cyan) 12%, transparent);
color: var(--cyan)
}
37 changes: 37 additions & 0 deletions assets/theme-v2/css/tables.css
Original file line number Diff line number Diff line change
Expand Up @@ -178,3 +178,40 @@ td {
max-width: 100%;
width: 100%
}

[data-admin-system-health] .table-wrapper {
border: var(--border-standard);
border-radius: var(--radius-lg);
background: var(--panel-overlay);
box-shadow: var(--shadow-sm)
}

[data-admin-system-health] .data-table {
min-width: 720px;
background: transparent
}

[data-admin-system-health] .data-table caption {
padding: var(--space-12) var(--space-14);
border-bottom: var(--border-standard);
background: var(--panel-soft);
color: var(--gold)
}

[data-admin-system-health] .data-table th,
[data-admin-system-health] .data-table td {
padding: var(--space-10) var(--space-12);
vertical-align: top
}

[data-admin-system-health] .data-table th {
color: var(--cyan)
}

[data-admin-system-health] .data-table tbody tr:nth-child(even) {
background: var(--panel-soft-subtle)
}

[data-admin-system-health] .data-table td {
overflow-wrap: anywhere
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
# PR_26177_CHARLIE_035-system-health-ui-polish

Team: Charlie
Branch: pr/26177-CHARLIE-035-system-health-ui-polish
Base: pr/26177-CHARLIE-034-startup-runtime-report-cleanup
Lifecycle: Build / Validation
Repair: Rebased onto repaired PR_26177_CHARLIE_034 branch on 2026-06-25.

## Scope
- Added Theme V2 styling for System Health table wrappers, captions, row rhythm, and dense table spacing.
- Added Theme V2 status treatment for System Health status indicators.
- Kept the change visual-only with no new API, runtime, UI behavior, or page-local CSS.

## Changed Files
- assets/theme-v2/css/status.css
- assets/theme-v2/css/tables.css
- docs_build/dev/reports/coverage_changed_js_guardrail.txt
- docs_build/dev/reports/playwright_v8_coverage_report.txt

## Validation
- PASS: npx playwright test tests/playwright/tools/AdminHealthOperationsPage.spec.mjs --workers=1 --reporter=line
- PASS: git diff --check

## ZIP
- Generated after repair: C:\Users\DavidQ\Documents\GitHub\HTML-JavaScript-Gaming\tmp\PR_26177_CHARLIE_035-system-health-ui-polish_delta.zip
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
# PR_26177_CHARLIE_035 Branch Validation

Branch: pr/26177-CHARLIE-035-system-health-ui-polish
Base: pr/26177-CHARLIE-034-startup-runtime-report-cleanup

## Results
- PASS: Continued the stacked Charlie workstream from PR_26177_CHARLIE_034.
- PASS: Branch is based on repaired PR_26177_CHARLIE_034 branch.
- PASS: Rebase conflict scope was generated report artifacts only.
- PASS: Worktree was clean before edits.
- PASS: Changes are limited to Theme V2 visual polish and reports.
- PASS: No start_of_day files were modified.
- PASS: No direct commit to main was made.
- PASS: PR branch will be pushed for draft PR creation.
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
# PR_26177_CHARLIE_035 Manual Validation Notes

- Confirmed System Health tables use scoped Theme V2 table wrapper and caption polish.
- Confirmed status indicators have distinct PASS, WARN/PENDING, FAIL, INFO, and SKIP treatments.
- Confirmed the page still uses external Theme V2 CSS and JavaScript only.
- Confirmed no runtime behavior, API contract, database contract, or storage contract was changed.
- Confirmed branch repair conflict was limited to generated report artifacts.
- Confirmed no start_of_day files changed.
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
# PR_26177_CHARLIE_035 Requirement Checklist

- PASS: Theme V2 polish only.
- PASS: Improve System Health layout/readability/status indicators.
- PASS: No new functionality beyond visual polish.
- PASS: Use existing Theme V2 CSS assets.
- PASS: No inline styles, style blocks, script blocks, inline handlers, or page-local CSS.
- PASS: Preserve existing behavior.
- PASS: Do not modify unrelated files.
- PASS: Do not modify start_of_day folders.
- PASS: Do not introduce MEM DB, fake-login, silent fallbacks, or browser-owned infrastructure state.
- PASS: Rebased onto repaired PR_26177_CHARLIE_034 branch.
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
# PR_26177_CHARLIE_035 Validation Lane

## Commands
- PASS: npx playwright test tests/playwright/tools/AdminHealthOperationsPage.spec.mjs --workers=1 --reporter=line
- PASS: git diff --check

## Notes
- `git diff --check` reported only expected Windows LF-to-CRLF working-copy warnings.
- No API/unit tests were required because this PR changes only Theme V2 CSS.
- Full samples smoke was not run because this PR is limited to Admin System Health UI polish.
55 changes: 24 additions & 31 deletions docs_build/dev/reports/codex_changed_files.txt
Original file line number Diff line number Diff line change
@@ -1,38 +1,31 @@
# git diff --name-only pr/26177-CHARLIE-032-runtime-health-json-endpoints --
docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup.md
docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_branch-validation.md
docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_manual-validation-notes.md
docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_requirements-checklist.md
docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_validation-lane.md
# git diff --name-only pr/26177-CHARLIE-034-startup-runtime-report-cleanup --
assets/theme-v2/css/status.css
assets/theme-v2/css/tables.css
docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish.md
docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish_branch-validation.md
docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish_manual-validation-notes.md
docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish_requirements-checklist.md
docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish_validation-lane.md
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
docs_build/dev/reports/coverage_changed_js_guardrail.txt
docs_build/dev/reports/playwright_v8_coverage_report.txt
scripts/start-local-api-server.mjs
src/dev-runtime/server/local-api-router.mjs
tests/dev-runtime/AdminHealthOperations.test.mjs
tests/dev-runtime/LocalApiStartupLogging.test.mjs
tests/playwright/tools/AdminHealthOperationsPage.spec.mjs

# git status --short
M docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup.md
M docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_branch-validation.md
M docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_manual-validation-notes.md
M docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_requirements-checklist.md
M docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish.md
M docs_build/dev/reports/codex_review.diff
M docs_build/dev/reports/playwright_v8_coverage_report.txt
M docs_build/dev/reports/coverage_changed_js_guardrail.txt

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