Skip to content

Commit a444667

Browse files
committed
Align toolbox Idea and Create groups with rainbow color model - PR_26170_005-toolbox-create-idea-color-model
1 parent 95cb1d5 commit a444667

7 files changed

Lines changed: 2039 additions & 922 deletions
Lines changed: 48 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,48 @@
1+
# PR_26170_005-toolbox-create-idea-color-model
2+
3+
## Branch Validation
4+
- PASS: Current branch verified as `main`.
5+
6+
## Requirement Checklist
7+
- PASS: Read and followed `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
8+
- PASS: Created BUILD doc at `docs_build/pr/BUILD_PR_26170_005-toolbox-create-idea-color-model.md`.
9+
- PASS: Moved Tags to the Toolbox `Create` group in `toolbox/tools-page-accordions.js`.
10+
- PASS: Moved Creator Learning to the Toolbox `Idea` group in `toolbox/tools-page-accordions.js`.
11+
- PASS: Preserved existing tool routes, statuses, database behavior, workspace behavior, runtime behavior, and metadata source contracts.
12+
- PASS: Preserved the Toolbox Game Journey group order and unique rainbow colors.
13+
- PASS: Updated Owner `Group Color Model`, `Usage Notes`, and `Color Groups` in `owner/grouping-colors.html`.
14+
- PASS: Added `#RRGGBB` values to Owner group color listings.
15+
- PASS: Kept product changes scoped to Toolbox grouping and Owner color documentation.
16+
17+
## Impacted Lane
18+
- Toolbox grouped-view rendering.
19+
- Owner color reference documentation.
20+
- Targeted validation test updated to assert Tags/Create and Creator Learning/Idea.
21+
22+
## Validation Lane Report
23+
- PASS: `node --check toolbox/tools-page-accordions.js`
24+
- PASS: `node --check tests/playwright/tools/ToolboxRoutePages.spec.mjs`
25+
- PASS: `git diff --check`
26+
- PASS: Targeted inline HTML guard found no inline `<style>`, inline `style=`, or inline event handlers in `toolbox/index.html` or `owner/grouping-colors.html`.
27+
- PASS: Targeted Toolbox Playwright validation:
28+
- `npx playwright test tests/playwright/tools/ToolboxRoutePages.spec.mjs --grep "toolbox grouped view renders Game Journey order" --workers=1 --reporter=line`
29+
- Result: `1 passed`.
30+
- FAIL: Requested broader UI lane `npm run test:workspace-v2`.
31+
- Result: 3 passed, 2 failed.
32+
- Failure 1: `RootToolsFutureState.spec.mjs` expected `Tool Count: 13/41`, page rendered `Tool Count: 13/45`.
33+
- Failure 2: `RootToolsFutureState.spec.mjs` expected signed-out primary nav ending in `Sign In`, page rendered authenticated nav labels `DavidQ`, `Owner`, `Admin`.
34+
- Assessment: Failures are outside this PR's product changes. The targeted Toolbox grouping test passed after validating the requested group order, unique colors, moved tool groups, and preserved tool links.
35+
36+
## Playwright Decision
37+
- Playwright impacted: Yes, Toolbox group rendering changed.
38+
- Ran targeted Toolbox Playwright validation and the requested `npm run test:workspace-v2` lane.
39+
- Did not run full samples.
40+
41+
## Skipped Lanes
42+
- Full samples smoke: skipped per request.
43+
- Unrelated Admin, Owner, database, storage, auth, marketplace, memberships, AI credits, and runtime lanes: skipped because this PR only changes Toolbox grouping display and Owner color documentation.
44+
45+
## Manual Validation Notes
46+
- Confirmed Toolbox Game Journey group order remains `Idea`, `Create`, `Design`, `Graphics`, `Audio`, `Objects`, `Worlds`, `Interface`, `Controls`, `Rules`, `Progression`, `Play Test`, `Publish`, `Share`.
47+
- Confirmed targeted test coverage now verifies `Creator Learning` appears in `Idea` and `Tags` appears in `Create`.
48+
- Confirmed Owner color reference lists the 14 groups in rainbow order with unique swatches and explicit hex values.
Lines changed: 20 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,20 @@
1-
assets/theme-v2/css/colors.css
2-
assets/theme-v2/css/status.css
3-
docs_build/dev/reports/PR_26170_002-idea-board-tool.md
4-
docs_build/dev/reports/PR_26170_003-toolbox-create-group.md
5-
docs_build/dev/reports/codex_changed_files.txt
6-
docs_build/dev/reports/codex_review.diff
7-
docs_build/dev/reports/coverage_changed_js_guardrail.txt
8-
docs_build/dev/reports/playwright_v8_coverage_report.txt
9-
docs_build/pr/BUILD_PR_26170_002-idea-board-tool.md
10-
docs_build/pr/BUILD_PR_26170_003-toolbox-create-group.md
11-
src/shared/toolbox/tool-metadata-inventory.js
12-
tests/playwright/tools/BuildPathProgressSimplification.spec.mjs
13-
tests/playwright/tools/GameJourneyTool.spec.mjs
14-
tests/playwright/tools/RootToolsFutureState.spec.mjs
15-
tests/playwright/tools/ToolboxAdminMetadataSsot.spec.mjs
16-
tests/playwright/tools/ToolboxRoutePages.spec.mjs
17-
toolbox/idea-board/index.html
18-
toolbox/tools-page-accordions.js
1+
# git status --short
2+
M docs_build/dev/reports/codex_changed_files.txt
3+
M docs_build/dev/reports/codex_review.diff
4+
M owner/grouping-colors.html
5+
M tests/playwright/tools/ToolboxRoutePages.spec.mjs
6+
M toolbox/tools-page-accordions.js
7+
?? docs_build/dev/reports/PR_26170_005-toolbox-create-idea-color-model.md
8+
?? docs_build/pr/BUILD_PR_26170_005-toolbox-create-idea-color-model.md
9+
10+
# git ls-files --others --exclude-standard
11+
docs_build/dev/reports/PR_26170_005-toolbox-create-idea-color-model.md
12+
docs_build/pr/BUILD_PR_26170_005-toolbox-create-idea-color-model.md
13+
14+
# git diff --stat
15+
docs_build/dev/reports/codex_changed_files.txt | 34 +-
16+
docs_build/dev/reports/codex_review.diff | 1246 +++++++--------------
17+
owner/grouping-colors.html | 214 +++-
18+
tests/playwright/tools/ToolboxRoutePages.spec.mjs | 33 +-
19+
toolbox/tools-page-accordions.js | 4 +-
20+
5 files changed, 599 insertions(+), 932 deletions(-)

0 commit comments

Comments
 (0)