Skip to content

Commit ca62687

Browse files
committed
PR_26171_041 polish idea board production behavior
1 parent 9aea9f8 commit ca62687

11 files changed

Lines changed: 431 additions & 483 deletions

File tree

assets/theme-v2/css/tables.css

Lines changed: 68 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,67 @@ td {
3838
color: var(--text)
3939
}
4040

41+
.idea-board-table-caption {
42+
position: relative;
43+
z-index: var(--z-index-sm);
44+
display: flex;
45+
align-items: center;
46+
justify-content: space-between;
47+
gap: var(--space-10);
48+
padding: var(--space-10);
49+
font-weight: var(--font-weight-heavy)
50+
}
51+
52+
.idea-board-show-filter {
53+
position: relative;
54+
z-index: var(--z-index-md);
55+
color: var(--text);
56+
font-size: var(--font-size-xs);
57+
font-weight: var(--font-weight-bold)
58+
}
59+
60+
.idea-board-show-filter summary {
61+
display: inline-flex;
62+
align-items: center;
63+
min-height: var(--space-28);
64+
padding: var(--space-3) var(--space-10);
65+
border: var(--border-standard);
66+
border-radius: var(--radius-md);
67+
background: var(--panel-soft);
68+
cursor: pointer;
69+
list-style: none
70+
}
71+
72+
.idea-board-show-filter summary::-webkit-details-marker {
73+
display: none
74+
}
75+
76+
.idea-board-show-filter__menu {
77+
position: static;
78+
min-width: 12rem;
79+
margin-top: var(--space-6);
80+
padding: var(--space-10);
81+
border: var(--border-standard);
82+
border-radius: var(--radius-md);
83+
background: var(--panel);
84+
box-shadow: var(--shadow-md)
85+
}
86+
87+
.idea-board-show-filter__options {
88+
display: grid;
89+
gap: var(--space-6);
90+
margin-top: var(--space-10)
91+
}
92+
93+
.idea-board-show-filter__option {
94+
display: flex;
95+
align-items: center;
96+
gap: var(--space-8);
97+
color: var(--text);
98+
font-weight: var(--font-weight-regular);
99+
white-space: nowrap
100+
}
101+
41102
.data-table th {
42103
color: var(--gold);
43104
font-size: var(--font-size-sm);
@@ -60,8 +121,13 @@ td {
60121
gap: .35em;
61122
color: inherit;
62123
font: inherit;
63-
line-height: inherit;
64-
vertical-align: baseline
124+
line-height: var(--line-height-single);
125+
vertical-align: baseline;
126+
white-space: nowrap
127+
}
128+
129+
.idea-board-idea-label__text {
130+
line-height: var(--line-height-single)
65131
}
66132

67133
.idea-board-idea-chevron {

assets/theme-v2/js/tool-display-mode.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,7 @@
157157
const diagnostic = document.createElement("p");
158158
diagnostic.className = "status";
159159
diagnostic.setAttribute("role", "status");
160-
diagnostic.textContent = "Tool navigation could not load from the server API. Start the local server API and refresh.";
160+
diagnostic.textContent = "Tool navigation is temporarily unavailable. Refresh the page or try again shortly.";
161161
body.appendChild(diagnostic);
162162
}
163163
}

docs_build/dev/reports/codex_changed_files.txt

Lines changed: 56 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -1,60 +1,83 @@
1-
# Codex Changed Files - PR_26171_040-idea-board-production-copy-cleanup
1+
# Codex Changed Files - PR_26171_041-idea-board-production-polish
22

33
## Git Workflow
4-
- Current branch: `codex/pr-26171-040-idea-board-production-copy-cleanup`.
4+
- Current branch: `codex/pr-26171-041-idea-board-production-polish`.
55
- Expected starting branch: `main` (PASS before branch creation).
6-
- Created branch: `codex/pr-26171-040-idea-board-production-copy-cleanup`.
7-
- Commit before conflict resolution: `acbc5b113e9e90a4051e810897415ea222040ddd`.
8-
- Initial push result: PASS, branch pushed to `origin/codex/pr-26171-040-idea-board-production-copy-cleanup`.
9-
- PR URL: `https://github.com/ToolboxAid/HTML-JavaScript-Gaming/pull/15`.
10-
- Conflict resolution: merged `origin/main` at `7393b650228c84d266701cc3f4ce3696029dd47e`; conflicts were limited to `codex_changed_files.txt` and `codex_review.diff`.
11-
- Conflict resolution push result: pending until after merge-validation commit.
6+
- Created branch: `codex/pr-26171-041-idea-board-production-polish`.
7+
- Push result: pending until after commit.
8+
- PR URL: pending until after push.
129
- Merge result: pending until after PR validation/merge.
1310
- Final main sync: pending until after merge and final pull.
1411

15-
## Changed Files
16-
- docs_build/dev/reports/codex_changed_files.txt
17-
- docs_build/dev/reports/codex_review.diff
18-
- docs_build/dev/reports/coverage_changed_js_guardrail.txt
19-
- docs_build/dev/reports/playwright_v8_coverage_report.txt
20-
- src/dev-runtime/server/local-api-router.mjs
21-
- src/shared/toolbox/tool-metadata-inventory.js
22-
- tests/playwright/tools/IdeaBoardTableNotes.spec.mjs
23-
- tests/playwright/tools/ToolboxRoutePages.spec.mjs
24-
- toolbox/idea-board/index.html
25-
- toolbox/idea-board/index.js
12+
## Git Status
13+
```text
14+
M assets/theme-v2/css/tables.css
15+
M assets/theme-v2/js/tool-display-mode.js
16+
M docs_build/dev/reports/coverage_changed_js_guardrail.txt
17+
M docs_build/dev/reports/playwright_v8_coverage_report.txt
18+
M src/shared/toolbox/tool-metadata-inventory.js
19+
M tests/playwright/tools/IdeaBoardTableNotes.spec.mjs
20+
M tests/playwright/tools/ToolboxRoutePages.spec.mjs
21+
M toolbox/idea-board/index.html
22+
M toolbox/idea-board/index.js
23+
```
24+
25+
## Diff Stat
26+
```text
27+
assets/theme-v2/css/tables.css | 70 +++++++-
28+
assets/theme-v2/js/tool-display-mode.js | 2 +-
29+
.../dev/reports/coverage_changed_js_guardrail.txt | 5 +-
30+
.../dev/reports/playwright_v8_coverage_report.txt | 11 +-
31+
src/shared/toolbox/tool-metadata-inventory.js | 6 +-
32+
.../playwright/tools/IdeaBoardTableNotes.spec.mjs | 71 +++++++-
33+
tests/playwright/tools/ToolboxRoutePages.spec.mjs | 26 ++-
34+
toolbox/idea-board/index.html | 31 ++--
35+
toolbox/idea-board/index.js | 191 ++++++++++++++++++++-
36+
9 files changed, 368 insertions(+), 45 deletions(-)
37+
```
2638

2739
## Requirement Evidence
28-
- PASS: Creator-visible Idea Board page copy no longer uses DB-shaped, in-page data model, ID/flag/metadata/seed/debug/mock/test/internal wording.
29-
- PASS: Shared Toolbox metadata for Idea Board now uses simple creator-facing language for short description, description, and progress checklist text.
30-
- PASS: Existing persisted Toolbox metadata rows now source-sync Idea Board copy so the browser-visible registry summary updates from source-controlled text.
31-
- PASS: Idea Board table workflow, Add Idea, Add Note, Edit/Delete, Save/Cancel, Status dropdown, accordion behavior, notes under idea rows, and existing in-page data structure were preserved.
32-
- PASS: Targeted Playwright production-copy checks verify the Idea Board main surface does not expose the prohibited creator-visible wording.
33-
- PASS: Copied/adapted files verified as not applicable for this copy-only PR; no Tool Template V2 file copy was required.
40+
- PASS: Removed creator-visible Create Project from the right-side Next Steps area; no disabled `data-idea-board-create-project` panel remains.
41+
- PASS: Create Project is shown only in a Ready idea row's Actions column.
42+
- PASS: Clicking Create Project changes the idea status from Ready to Project and replaces Create Project/Delete with Open Project and Archive.
43+
- PASS: Project rows show Open Project and Archive and do not show Delete.
44+
- PASS: Added Archived status; archived rows are hidden by default.
45+
- PASS: Archived rows show Restore and Delete.
46+
- PASS: Restore returns archived ideas to their previous non-archived status, with Refining fallback in runtime.
47+
- PASS: Delete is guarded so Project ideas cannot be deleted unless archived first.
48+
- PASS: Status dropdown uses New, Exploring, Refining, Ready, Project, and Archived.
49+
- PASS: Added compact Show checkbox dropdown beside the visible table description line, with default statuses New, Exploring, Refining, Ready, and Project selected and Archived unselected.
50+
- PASS: Show dropdown supports any checkbox combination plus Select All and Clear All.
51+
- PASS: Updated creator-facing copy to "Capture, compare, and shape game ideas." and "Scan, compare, and update early ideas."
52+
- PASS: Replaced creator-visible Tool Display Mode navigation error with safe copy that does not mention server/API/local server/port/implementation details.
53+
- PASS: Chevron renders to the left of the Idea text, inline, same size, same color, and on the same text line; the whole Idea cell remains the expansion target.
54+
- PASS: Notes count remains informational only.
55+
- PASS: Table-first structure, inline Add Idea/Add Note, row-level Save/Cancel, status dropdown edit mode, notes indentation, and single-open accordion behavior remain covered.
3456

3557
## Validation
3658
- PASS: `node --check toolbox/idea-board/index.js`.
37-
- PASS: `node --check src/dev-runtime/server/local-api-router.mjs`.
59+
- PASS: `node --check assets/theme-v2/js/tool-display-mode.js`.
3860
- PASS: `node --check src/shared/toolbox/tool-metadata-inventory.js`.
3961
- PASS: `node --check tests/playwright/tools/IdeaBoardTableNotes.spec.mjs`.
4062
- PASS: `node --check tests/playwright/tools/ToolboxRoutePages.spec.mjs`.
41-
- PASS: `npx playwright test tests/playwright/tools/IdeaBoardTableNotes.spec.mjs --project=playwright --workers=1 --reporter=line`.
42-
- PASS: `npx playwright test tests/playwright/tools/ToolboxRoutePages.spec.mjs --project=playwright --workers=1 --reporter=line -g "Idea Board launches"`.
63+
- PASS: `npx playwright test tests/playwright/tools/IdeaBoardTableNotes.spec.mjs --project=playwright --workers=1 --reporter=line --timeout=90000`.
64+
- PASS: `npx playwright test tests/playwright/tools/ToolboxRoutePages.spec.mjs --project=playwright --workers=1 --reporter=line -g "Idea Board launches" --timeout=90000`.
65+
- PASS: `npm run test:workspace-v2` (workspace-contract lane, 5 passed).
4366
- PASS: `git diff --check`.
44-
- PASS: Post-conflict rerun completed after merging `origin/main` at `7393b650228c84d266701cc3f4ce3696029dd47e`.
4567
- PASS: Playwright V8 coverage report produced because runtime JavaScript changed.
46-
- WARN: Coverage report marks `src/dev-runtime/server/local-api-router.mjs` and `src/shared/toolbox/tool-metadata-inventory.js` as not collected by browser V8 coverage; advisory only per project instructions.
68+
- WARN: Coverage report marks `src/shared/toolbox/tool-metadata-inventory.js` as not collected by browser V8 coverage; advisory only per project instructions.
4769
- SKIPPED: Full samples smoke was not run per request.
4870

4971
## ZIP
50-
- Path: `tmp/PR_26171_040-idea-board-production-copy-cleanup_delta.zip`.
51-
- Size: final size reported in the delivery summary after conflict-resolution ZIP refresh.
72+
- Path: `tmp/PR_26171_041-idea-board-production-polish_delta.zip`.
73+
- Size: `50677` bytes before final report refresh; final size reported in the delivery summary.
5274
- Contents:
75+
- assets/theme-v2/css/tables.css
76+
- assets/theme-v2/js/tool-display-mode.js
5377
- docs_build/dev/reports/codex_changed_files.txt
5478
- docs_build/dev/reports/codex_review.diff
5579
- docs_build/dev/reports/coverage_changed_js_guardrail.txt
5680
- docs_build/dev/reports/playwright_v8_coverage_report.txt
57-
- src/dev-runtime/server/local-api-router.mjs
5881
- src/shared/toolbox/tool-metadata-inventory.js
5982
- tests/playwright/tools/IdeaBoardTableNotes.spec.mjs
6083
- tests/playwright/tools/ToolboxRoutePages.spec.mjs

0 commit comments

Comments
 (0)