-
Notifications
You must be signed in to change notification settings - Fork 0
PR_26175_ALFA_050-theme-v2-layout-utility-icons #169
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
ToolboxAid
merged 1 commit into
main
from
codex/pr-26175-alfa-050-theme-v2-layout-utility-icons
Jun 24, 2026
Merged
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
11 changes: 11 additions & 0 deletions
11
...orts/PR_26175_ALFA_050-theme-v2-layout-utility-icons_manual-validation-notes.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,11 @@ | ||
| # PR_26175_ALFA_050 Manual Validation Notes | ||
|
|
||
| ## Notes | ||
| - Manual validation was performed through the required targeted Playwright lanes rather than an ad hoc browser session. | ||
| - Route tests pin the API/site URL to the repo test server so the pages do not drift to a local dev endpoint. | ||
| - Toolbox vote route assertions were made deterministic against the configured product-data provider by reading current vote state, asserting state transitions, and restoring touched Colors metadata/order. | ||
| - No visual redesign was performed; changes are limited to replacing utility text/placeholders with shared Theme V2 SVG icon nodes and matching compact CSS. | ||
| - No files under `start_of_day` were read or modified. | ||
|
|
||
| ## Residual Risk | ||
| - Full cross-browser or mobile manual review was not run beyond the existing targeted Playwright coverage. |
40 changes: 40 additions & 0 deletions
40
docs_build/dev/reports/PR_26175_ALFA_050-theme-v2-layout-utility-icons_report.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,40 @@ | ||
| # PR_26175_ALFA_050-theme-v2-layout-utility-icons Report | ||
|
|
||
| ## Summary | ||
| - Branch validation: PASS. | ||
| - Base branch state: `5426785fc` (`Merge PR #168: PR_26175_CHARLIE_028-team-charlie-final-closeout`). | ||
| - Scope: Theme V2 layout utility controls now use shared registry SVG icons for fullscreen mode, previous/next navigation, horizontal column toggles, and return-to-top. | ||
| - Runtime/UI scope: no broad redesign, no engine core changes, no `start_of_day` changes, and no page-local inline styles or style blocks introduced. | ||
|
|
||
| ## Changed Files | ||
| - `assets/theme-v2/css/buttons.css` | ||
| - `assets/theme-v2/css/icons.css` | ||
| - `assets/theme-v2/css/panels.css` | ||
| - `assets/theme-v2/js/gamefoundry-partials.js` | ||
| - `assets/theme-v2/js/tool-display-mode.js` | ||
| - `tests/playwright/tools/ThemeV2SvgIconRegistry.spec.mjs` | ||
| - `tests/playwright/tools/ToolboxRoutePages.spec.mjs` | ||
| - `tests/playwright/tools/ToolboxSelectedGameStatusBar.spec.mjs` | ||
| - `docs_build/dev/reports/PR_26175_ALFA_050-theme-v2-layout-utility-icons_report.md` | ||
| - `docs_build/dev/reports/PR_26175_ALFA_050-theme-v2-layout-utility-icons_validation-lane.md` | ||
| - `docs_build/dev/reports/PR_26175_ALFA_050-theme-v2-layout-utility-icons_requirements-checklist.md` | ||
| - `docs_build/dev/reports/PR_26175_ALFA_050-theme-v2-layout-utility-icons_manual-validation-notes.md` | ||
| - `docs_build/dev/reports/codex_review.diff` | ||
| - `docs_build/dev/reports/codex_changed_files.txt` | ||
|
|
||
| ## Implementation Notes | ||
| - `tool-display-mode.js` renders shared fullscreen/exit-fullscreen icons in the mode summary and shared chevron icons for previous/next controls while preserving existing labels and links. | ||
| - `gamefoundry-partials.js` loads the Theme V2 icon registry for shared partial utilities, replacing account side-nav text glyphs and return-to-top placeholders with registry icons. | ||
| - `icons.css`, `buttons.css`, and `panels.css` add compact shared layout-icon sizing and layout utility presentation without inline styles. | ||
| - Tests assert the new layout utility icons and keep route coverage deterministic against the configured product-data provider. | ||
|
|
||
| ## Validation Summary | ||
| - PASS: syntax checks for the touched Theme V2 scripts. | ||
| - PASS: targeted Playwright registry, selected-game status bar, and route suites. | ||
| - PASS: inline style/style-block scan returned no matches. | ||
| - PASS: `git diff --check`. | ||
|
|
||
| ## Branch Validation | ||
| - PASS: Branch is `codex/pr-26175-alfa-050-theme-v2-layout-utility-icons`. | ||
| - PASS: Changes are limited to ALFA_050 target implementation, tests, and required reports. | ||
| - PASS: Repo-structured ZIP will be emitted under `tmp/` and not staged. |
15 changes: 15 additions & 0 deletions
15
...ports/PR_26175_ALFA_050-theme-v2-layout-utility-icons_requirements-checklist.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| # PR_26175_ALFA_050 Requirements Checklist | ||
|
|
||
| | Requirement | Status | Evidence | | ||
| | --- | --- | --- | | ||
| | Use shared icons for fullscreen enter/exit controls. | PASS | `tool-display-mode.js` adds fullscreen and exit-fullscreen registry icons; status-bar Playwright asserts both files. | | ||
| | Use shared icons for previous/next navigation. | PASS | Tool display navigation prepends shared chevron-left/chevron-right icons; Playwright asserts both files. | | ||
| | Use shared icons for column collapse/expand controls. | PASS | Horizontal accordion toggles render shared chevron icons through the Theme V2 helper path. | | ||
| | Use shared icons for return-to-top controls. | PASS | Shared partials replace return-to-top content with the registry chevron-up icon; route Playwright asserts it. | | ||
| | Preserve accessible names, roles, and keyboard behavior. | PASS | Existing button/link labels and aria attributes are retained while icons are `aria-hidden`. | | ||
| | Preserve fullscreen bottom status bar anchoring and content reserve. | PASS | Selected-game status bar Playwright suite passed. | | ||
| | Keep utility controls compact and stable. | PASS | Shared `.layout-icon` sizing and navigation-link styles were added without layout refactors. | | ||
| | Avoid page-local utility icon markup when shared helpers can own it. | PASS | Shared partial/helper paths own return-to-top and horizontal toggle icon replacement. | | ||
| | No inline styles, style blocks, or page-local CSS introduced. | PASS | Targeted `rg` scan returned no matches. | | ||
| | No unrelated page/tool redesign. | PASS | Changes are limited to layout utility icon rendering, compact CSS, tests, and reports. | | ||
| | No engine core or `start_of_day` changes. | PASS | No engine or `start_of_day` files changed. | |
18 changes: 18 additions & 0 deletions
18
.../dev/reports/PR_26175_ALFA_050-theme-v2-layout-utility-icons_validation-lane.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,18 @@ | ||
| # PR_26175_ALFA_050 Validation Lane | ||
|
|
||
| ## Static Checks | ||
| - PASS: `node --check assets/theme-v2/js/theme-icons.js` | ||
| - PASS: `node --check assets/theme-v2/js/tool-display-mode.js` | ||
| - PASS: `node --check assets/theme-v2/js/gamefoundry-partials.js` | ||
| - PASS: `rg -n "<[s]tyle|[s]tyle=" assets/theme-v2/js/theme-icons.js assets/theme-v2/js/tool-display-mode.js assets/theme-v2/js/gamefoundry-partials.js assets/theme-v2/css/icons.css assets/theme-v2/css/layout.css assets/theme-v2/css/buttons.css assets/theme-v2/css/panels.css tests/playwright/tools/ThemeV2SvgIconRegistry.spec.mjs tests/playwright/tools/ToolboxSelectedGameStatusBar.spec.mjs tests/playwright/tools/ToolboxRoutePages.spec.mjs` returned no matches. | ||
| - PASS: `git diff --check` | ||
|
|
||
| ## Playwright | ||
| - PASS: `npx playwright test tests/playwright/tools/ThemeV2SvgIconRegistry.spec.mjs --workers=1` (8 passed) | ||
| - PASS: `npx playwright test tests/playwright/tools/ToolboxSelectedGameStatusBar.spec.mjs --workers=1` (7 passed) | ||
| - PASS: `npx playwright test tests/playwright/tools/ToolboxRoutePages.spec.mjs --workers=1` (11 passed) | ||
|
|
||
| ## Branch Validation | ||
| - PASS: Built from refreshed main after latest main refresh (`5426785fc`). | ||
| - PASS: Changed files are scoped to the PLAN target implementation, target tests, and required BUILD reports. | ||
| - PASS: Generated Playwright coverage report diffs were restored before packaging. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,7 +1,14 @@ | ||
| docs_build/dev/reports/codex_changed_files.txt | ||
| assets/theme-v2/css/buttons.css | ||
| assets/theme-v2/css/icons.css | ||
| assets/theme-v2/css/panels.css | ||
| assets/theme-v2/js/gamefoundry-partials.js | ||
| assets/theme-v2/js/tool-display-mode.js | ||
| docs_build/dev/reports/PR_26175_ALFA_050-theme-v2-layout-utility-icons_manual-validation-notes.md | ||
| docs_build/dev/reports/PR_26175_ALFA_050-theme-v2-layout-utility-icons_report.md | ||
| docs_build/dev/reports/PR_26175_ALFA_050-theme-v2-layout-utility-icons_requirements-checklist.md | ||
| docs_build/dev/reports/PR_26175_ALFA_050-theme-v2-layout-utility-icons_validation-lane.md | ||
| docs_build/dev/reports/codex_changed_files.txt | ||
| docs_build/dev/reports/codex_review.diff | ||
| docs_build/dev/reports/PR_26175_CHARLIE_028-team-charlie-final-closeout.md | ||
| docs_build/dev/reports/PR_26175_CHARLIE_028-team-charlie-final-closeout-branch-validation.md | ||
| docs_build/dev/reports/PR_26175_CHARLIE_028-team-charlie-final-closeout-manual-validation-notes.md | ||
| docs_build/dev/reports/PR_26175_CHARLIE_028-team-charlie-final-closeout-requirement-checklist.md | ||
| docs_build/dev/reports/PR_26175_CHARLIE_028-team-charlie-final-closeout-validation.md | ||
| tests/playwright/tools/ThemeV2SvgIconRegistry.spec.mjs | ||
| tests/playwright/tools/ToolboxRoutePages.spec.mjs | ||
| tests/playwright/tools/ToolboxSelectedGameStatusBar.spec.mjs |
Binary file not shown.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When the browser Fullscreen API succeeds, this call runs before
exitFullscreen()resolves, sodocument.fullscreenElementis still set andupdateToolDisplayModeModeIcon()keeps rendering the exit-fullscreen icon even thoughtool-focus-modehas already been removed. Thefullscreenchangehandler does not refresh this click path because the body class is gone by then, so users who exit via the control keep seeing the wrong mode icon until they re-enter focus mode.Useful? React with 👍 / 👎.