Skip to content

Commit db07f77

Browse files
ToolboxAidToolboxAid
committed
Polish System Health Theme V2 UI
1 parent 5a1df79 commit db07f77

10 files changed

Lines changed: 322 additions & 464 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: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
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+
8+
## Scope
9+
- Added Theme V2 styling for System Health table wrappers, captions, row rhythm, and dense table spacing.
10+
- Added Theme V2 status treatment for System Health status indicators.
11+
- Kept the change visual-only with no new API, runtime, UI behavior, or page-local CSS.
12+
13+
## Changed Files
14+
- assets/theme-v2/css/status.css
15+
- assets/theme-v2/css/tables.css
16+
- docs_build/dev/reports/playwright_v8_coverage_report.txt
17+
18+
## Validation
19+
- PASS: npx playwright test tests/playwright/tools/AdminHealthOperationsPage.spec.mjs --workers=1 --reporter=line
20+
- PASS: git diff --check
21+
22+
## ZIP
23+
- Generated after commit: 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: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
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: Worktree was clean before edits.
9+
- PASS: Changes are limited to Theme V2 visual polish and reports.
10+
- PASS: No start_of_day files were modified.
11+
- PASS: No direct commit to main was made.
12+
- 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: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
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.

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

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
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.

‎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: 16 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -1,30 +1,22 @@
11
# git status --short
2-
M docs_build/dev/reports/coverage_changed_js_guardrail.txt
2+
M assets/theme-v2/css/status.css
3+
M assets/theme-v2/css/tables.css
34
M docs_build/dev/reports/playwright_v8_coverage_report.txt
4-
M scripts/start-local-api-server.mjs
5-
M src/dev-runtime/server/local-api-router.mjs
6-
M tests/dev-runtime/AdminHealthOperations.test.mjs
7-
M tests/dev-runtime/LocalApiStartupLogging.test.mjs
8-
M tests/playwright/tools/AdminHealthOperationsPage.spec.mjs
9-
?? docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup.md
10-
?? docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_branch-validation.md
11-
?? docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_manual-validation-notes.md
12-
?? docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_requirements-checklist.md
13-
?? docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_validation-lane.md
5+
?? docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish.md
6+
?? docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish_branch-validation.md
7+
?? docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish_manual-validation-notes.md
8+
?? docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish_requirements-checklist.md
9+
?? docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish_validation-lane.md
1410

1511
# git ls-files --others --exclude-standard
16-
docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup.md
17-
docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_branch-validation.md
18-
docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_manual-validation-notes.md
19-
docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_requirements-checklist.md
20-
docs_build/dev/reports/PR_26177_CHARLIE_034-startup-runtime-report-cleanup_validation-lane.md
12+
docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish.md
13+
docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish_branch-validation.md
14+
docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish_manual-validation-notes.md
15+
docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish_requirements-checklist.md
16+
docs_build/dev/reports/PR_26177_CHARLIE_035-system-health-ui-polish_validation-lane.md
2117

2218
# git diff --stat
23-
.../dev/reports/coverage_changed_js_guardrail.txt | 1 -
24-
.../dev/reports/playwright_v8_coverage_report.txt | 4 +-
25-
scripts/start-local-api-server.mjs | 34 ++++++-
26-
src/dev-runtime/server/local-api-router.mjs | 101 ++++++++++++++++++++-
27-
tests/dev-runtime/AdminHealthOperations.test.mjs | 7 ++
28-
tests/dev-runtime/LocalApiStartupLogging.test.mjs | 32 +++++++
29-
.../tools/AdminHealthOperationsPage.spec.mjs | 6 ++
30-
7 files changed, 179 insertions(+), 6 deletions(-)
19+
assets/theme-v2/css/status.css | 52 ++++++++++++++++++++++
20+
assets/theme-v2/css/tables.css | 37 +++++++++++++++
21+
.../dev/reports/playwright_v8_coverage_report.txt | 1 -
22+
3 files changed, 89 insertions(+), 1 deletion(-)

0 commit comments

Comments
 (0)