Clean up Colors terminology fullscreen titles and card spacing - PR_26159_051-colors-terminology-fullscreen-spacing

Author: ToolboxAid

assets/theme-v2/css/layout.css

@@ -255,6 +255,11 @@ body.tool-focus-mode .tool-center-panel {
   grid-row: 2
 }
 
+body.tool-focus-mode .tool-center-panel>h2,
+body.tool-focus-mode .tool-center-panel>h2+p {
+  display: none
+}
+
 body.tool-focus-mode .tool-column:last-of-type {
   grid-column: 3;
   grid-row: 1 / span 2

assets/theme-v2/css/panels.css

@@ -38,6 +38,10 @@
   padding: var(--space-18)
 }
 
+.tool-workspace article.card>.card-body.content-stack {
+  margin: var(--space-8)
+}
+
 .surface {
   border: var(--border-standard);
   border-radius: var(--radius-lg);

docs_build/dev/reports/colors-terminology-fullscreen-spacing-report.md

@@ -0,0 +1,73 @@
+# PR_26159_051 Colors Terminology, Fullscreen, And Spacing Report
+
+## Branch Guard
+
+| Check | Status | Evidence |
+| --- | --- | --- |
+| Current branch is `main` before build | PASS | `git branch --show-current` returned `main`. |
+
+## Requirement Checklist
+
+| Requirement | Status | Evidence |
+| --- | --- | --- |
+| Read `docs_build/dev/PROJECT_INSTRUCTIONS.md` first | PASS | Read before edits; main-branch guard followed. |
+| Hard stop if branch is not `main` | PASS | Branch validated as `main`; build continued. |
+| Colors page top title says `Colors`, not `Palette` | PASS | `toolbox/colors/index.html`; Playwright title/heading assertions in `PaletteToolMockRepository.spec.mjs`. |
+| Replace stale user-facing Palette wording | PASS | Replaced page title, lede, headings, validation copy, status/log copy, table display labels, and repository-surfaced messages. |
+| Keep `Palette Type` only for the selector type dropdown | PASS | `toolbox/colors/index.html` label for `paletteGeneratorType`; tooltips retain `Palette Type` as requested. |
+| Preserve internal `palette-browser` compatibility | PASS | Internal constants/data attributes remain in `palette-api-client.js`, `colors.js`, and `palette-workspace-repository.js`; visible `tools.palette-browser.swatches` text removed from Colors UI. |
+| Fullscreen hides tool H2 and descriptive text | PASS | `assets/theme-v2/css/layout.css`; `ToolDisplayModeNavigation.spec.mjs` asserts hidden in `tool-focus-mode`. |
+| Fullscreen exit restores H2 and descriptive text | PASS | `ToolDisplayModeNavigation.spec.mjs` toggles fullscreen off and asserts restored visibility. |
+| Card body spacing adds 8px reusable tool spacing | PASS | `assets/theme-v2/css/panels.css`; `GameConfigurationMockRepository.spec.mjs` asserts 8px margins on `data-game-configuration-form-card` body. |
+| No page-local/tool-local CSS, inline styles, or inline handlers | PASS | Shared Theme V2 CSS only; `rg` check for inline style/script/event handlers returned no matches in changed active files. |
+| No broken internal storage/contracts | PASS | Repository unit/Playwright lane passed; internal contract names preserved while visible labels are mapped. |
+| No console errors | PASS | Targeted Playwright lanes collect console errors and passed. |
+| Playwright impacted | PASS | Targeted Colors, shared display mode, fullscreen accordion, and Game Configuration lanes passed. |
+
+## User-Facing Terms Changed
+
+- `Palette Tool` -> `Colors`
+- `Palette` page/column/center heading -> `Colors`
+- `Manage the active project palette at tools.palette-browser.swatches.` -> `Manage active project swatches for tool and asset color handoff.`
+- `Selected palette swatch tag editor fields` -> `Project Swatches tag editor fields`
+- `Selected palette color tags` -> `Project Swatches color tags`
+- `Palette summary` -> `Colors summary`
+- `palette swatches` / `project palette colors` -> `project swatches` / `Project Swatches colors`
+- `Source Palette Closest Match` -> `Current Source Closest Match`
+- `All Palettes Closest Match` -> `All Sources Closest Match`
+- `Validation` / `Palette Validation` -> `Swatch Validation`
+- `tools.palette-browser.swatches` visible status -> `Colors storage contract active.`
+- Raw `palette_*` table labels in the Colors UI -> `Project Swatches`, `Source Swatches`, `Swatch Usage`, `Project Swatch Settings`
+- `Palette generator` runtime messages -> `Picker Preview` / `Picker controls`
+- `Invalid palette payload` surfaced messages -> `Invalid Colors payload`
+
+## Internal Keys Preserved
+
+- `PALETTE_TOOL_KEY = "palette-browser"`
+- `PALETTE_WORKSPACE_PATH = tools.palette-browser.swatches`
+- `data-palette-*` attributes
+- `palette_colors`
+- `palette_source_swatches`
+- `palette_swatch_usages`
+- `project_workspace_palette_globals`
+- `palette-api-client.js` and repository function names needed by current compatibility contracts
+
+## Validation Evidence
+
+| Lane | Status | Evidence |
+| --- | --- | --- |
+| Syntax checks | PASS | `node --check toolbox/colors/colors.js`; `node --check src/dev-runtime/persistence/tool-repositories/palette-workspace-repository.js`; `node --check assets/theme-v2/js/tool-display-mode.js`. |
+| Static diff check | PASS | `git diff --check` passed with line-ending warnings only. |
+| Static terminology sweep | PASS | `rg` for targeted stale Palette UI phrases returned no matches in Colors active files and repository-surfaced messages. |
+| Inline style/script/event handler sweep | PASS | `rg` for inline style/script/event handlers returned no matches in changed active files. |
+| Colors runtime/UI | PASS | `npx playwright test tests/playwright/tools/PaletteToolMockRepository.spec.mjs` -> 9 passed. |
+| Shared display mode | PASS | `npx playwright test tests/playwright/tools/ToolDisplayModeNavigation.spec.mjs` -> 5 passed. |
+| Tool center fullscreen accordions | PASS | `npx playwright test tests/playwright/tools/ToolCenterFullscreenAccordion.spec.mjs` -> 1 passed. |
+| Game Configuration spacing | PASS | `npx playwright test tests/playwright/tools/GameConfigurationMockRepository.spec.mjs` -> 4 passed. |
+| Playwright V8 coverage | PASS | `docs_build/dev/reports/playwright_v8_coverage_report.txt`; Colors runtime collected at 86% advisory coverage. |
+
+## Skipped Lanes
+
+- Full samples validation: skipped per request.
+- Broad repo-wide Playwright: skipped because the PR touched Colors, shared display-mode CSS, shared card spacing CSS, and targeted tests covered those surfaces directly.
+

docs_build/dev/reports/coverage_changed_js_guardrail.txt

@@ -6,8 +6,9 @@ Missing changed runtime JS files are WARN, not FAIL.
 Source: Playwright/Chromium built-in V8 coverage from the active Playwright run.
 
 Changed runtime JS files considered:
-(86%) toolbox/colors/colors.js - executed lines 2127/2127; executed functions 182/211
+(0%) src/dev-runtime/persistence/tool-repositories/palette-workspace-repository.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
+(86%) toolbox/colors/colors.js - executed lines 2136/2136; executed functions 183/212
 (92%) assets/theme-v2/js/account-controls.js - executed lines 47/47; executed functions 12/13
 
 Guardrail warnings:
-(100%) none - no changed runtime JS coverage warnings
+(0%) src/dev-runtime/persistence/tool-repositories/palette-workspace-repository.js - WARNING: changed runtime JS file missing from coverage; advisory only

docs_build/dev/reports/playwright_v8_coverage_report.txt

@@ -17,23 +17,26 @@ Exercised tool entry points detected:
 (77%) Theme V2 Shared JS - exercised 3 runtime JS files
 
 Changed runtime JS files covered:
-(86%) toolbox/colors/colors.js - executed lines 2127/2127; executed functions 182/211
+(0%) src/dev-runtime/persistence/tool-repositories/palette-workspace-repository.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
+(86%) toolbox/colors/colors.js - executed lines 2136/2136; executed functions 183/212
 (92%) assets/theme-v2/js/account-controls.js - executed lines 47/47; executed functions 12/13
 
 Files with executed line/function counts where available:
 (58%) src/engine/api/server-api-client.js - executed lines 159/159; executed functions 11/19
 (64%) assets/theme-v2/js/tool-display-mode.js - executed lines 201/201; executed functions 9/14
 (76%) assets/theme-v2/js/gamefoundry-partials.js - executed lines 525/525; executed functions 35/46
 (78%) toolbox/tool-registry-api-client.js - executed lines 148/148; executed functions 21/27
-(86%) toolbox/colors/colors.js - executed lines 2127/2127; executed functions 182/211
+(86%) toolbox/colors/colors.js - executed lines 2136/2136; executed functions 183/212
 (92%) assets/theme-v2/js/account-controls.js - executed lines 47/47; executed functions 12/13
 (100%) toolbox/colors/palette-api-client.js - executed lines 19/19; executed functions 4/4
 
 Uncovered or low-coverage changed JS files:
-(100%) none - no low-coverage changed runtime JS files
+(0%) src/dev-runtime/persistence/tool-repositories/palette-workspace-repository.js - WARNING: uncovered changed runtime JS file; advisory only
 
 Changed JS files considered:
-(0%) tests/helpers/playwrightV8CoverageReporter.mjs - changed JS file not collected as browser runtime coverage
+(0%) src/dev-runtime/persistence/tool-repositories/palette-workspace-repository.js - changed JS file not collected as browser runtime coverage
+(0%) tests/playwright/tools/GameConfigurationMockRepository.spec.mjs - changed JS file not collected as browser runtime coverage
 (0%) tests/playwright/tools/PaletteToolMockRepository.spec.mjs - changed JS file not collected as browser runtime coverage
+(0%) tests/playwright/tools/ToolDisplayModeNavigation.spec.mjs - changed JS file not collected as browser runtime coverage
 (86%) toolbox/colors/colors.js - changed JS file with browser V8 coverage
 (92%) assets/theme-v2/js/account-controls.js - changed JS file with browser V8 coverage