Skip to content

Commit 825b252

Browse files
committed
PR_26174_ALFA_021-idea-board-status-filter-table-polish
1 parent e59f0fc commit 825b252

12 files changed

Lines changed: 370 additions & 573 deletions

assets/theme-v2/css/tables.css

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -116,26 +116,26 @@ td {
116116
}
117117

118118
.idea-board-idea-label {
119-
display: inline-flex;
120-
align-items: center;
121-
gap: .35em;
119+
display: inline;
122120
color: inherit;
123121
font: inherit;
124122
line-height: var(--line-height-single);
125123
vertical-align: baseline;
126-
white-space: nowrap
124+
white-space: normal
127125
}
128126

129127
.idea-board-idea-label__text {
130-
line-height: var(--line-height-single)
128+
overflow-wrap: anywhere;
129+
line-height: inherit
131130
}
132131

133132
.idea-board-idea-chevron {
134133
display: inline-block;
135134
width: 1em;
136135
height: 1em;
137-
flex: 0 0 1em;
136+
margin-right: .35em;
138137
background: currentColor;
138+
vertical-align: -0.125em;
139139
-webkit-mask-position: center;
140140
mask-position: center;
141141
-webkit-mask-repeat: no-repeat;
Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
# Branch Validation
2+
3+
PR: PR_26174_ALFA_021-idea-board-status-filter-table-polish
4+
5+
Status: PASS with documented legacy-lane warning
6+
7+
## Checks
8+
9+
- PASS: Current branch is `pr/26174-ALFA-021-idea-board-status-filter-table-polish`.
10+
- PASS: Started from the current stacked Alfa branch.
11+
- PASS: No main merge was performed.
12+
- PASS: Work stayed inside the requested Idea Board table/filter, shared table CSS, impacted tests, and required report scope.
13+
- PASS: Generated required reports and ZIP artifact.
14+
15+
## Warning
16+
17+
- WARN: The focused Toolbox launch-route check reached the updated Idea Board assertions, then failed on the existing `500 /api/game-journey/completion-metrics` request. The scoped Idea Board Playwright lane passed.
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
# Manual Validation Notes
2+
3+
- Confirmed the Status Filter checkbox group is placed in the Idea Board themed control group.
4+
- Confirmed the parent table columns are Idea, Pitch, Status, Notes, and Actions.
5+
- Confirmed expanded notes rows still render under the parent idea row with the adjusted colspan.
6+
- Confirmed editable Status dropdown options are New, Exploring, Refining, and Ready only.
7+
- Confirmed Project and Archived remain in the status filter list for filtering existing lifecycle states.
Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
# PR_26174_ALFA_021-idea-board-status-filter-table-polish
2+
3+
## Summary
4+
5+
Polished the Idea Board status filter and parent table display while preserving the existing table-first notes model.
6+
7+
## Changes
8+
9+
- Applied the Idea Board theme treatment to the status filter checkbox group.
10+
- Removed the Updated column from the Idea Board parent table and adjusted expanded-row colspans.
11+
- Updated the Idea column label styling so long ideas can wrap.
12+
- Removed Project and Archived from editable Status dropdowns while leaving them available in the filter.
13+
- Updated impacted Playwright expectations for the new table columns, wrapping behavior, filter theme, and editable status options.
14+
15+
## Notes
16+
17+
- No API/service contract changes were made.
18+
- No browser-owned product data or fallback arrays were added.
19+
- Notes child rows and inline edit Save/Cancel behavior remain intact.
Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
# Requirement Checklist
2+
3+
- PASS: Idea Board Status Filter checkboxes use the Idea Board theme treatment.
4+
- PASS: Removed Updated column from the Idea Board table.
5+
- PASS: Idea column can word wrap.
6+
- PASS: Removed Project from the editable Status dropdown.
7+
- PASS: Removed Archived from the editable Status dropdown.
8+
- PASS: Project and Archived remain available for status filtering.
9+
- PASS: Table-first parent/child pattern preserved.
10+
- PASS: Inline edit Save/Cancel behavior preserved.
11+
- PASS: Notes child rows preserved.
12+
- PASS: No browser-owned product data added.
13+
- PASS: No silent fallbacks added.
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
# Validation Lane
2+
3+
## PASS
4+
5+
- `node --check toolbox/idea-board/index.js`
6+
- `git diff --check`
7+
- `npx playwright test tests/playwright/tools/IdeaBoardTableNotes.spec.mjs --workers=1`
8+
9+
## WARN
10+
11+
- `npx playwright test tests/playwright/tools/ToolboxRoutePages.spec.mjs --workers=1 --grep "Idea Board launches"` failed after the updated Idea Board assertions because the page recorded `500 http://127.0.0.1:63001/api/game-journey/completion-metrics`. This is outside the scoped Idea Board status/table polish.
Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,5 @@
1-
docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md
2-
src/shared/toolbox/tool-metadata-inventory.js
3-
tests/playwright/tools/BuildPathProgressSimplification.spec.mjs
4-
tests/playwright/tools/GameHubMockRepository.spec.mjs
1+
assets/theme-v2/css/tables.css
52
tests/playwright/tools/IdeaBoardTableNotes.spec.mjs
6-
tests/playwright/tools/ToolboxAdminMetadataSsot.spec.mjs
73
tests/playwright/tools/ToolboxRoutePages.spec.mjs
8-
toolbox/game-hub/game-hub.js
9-
toolbox/game-hub/index.html
104
toolbox/idea-board/index.html
5+
toolbox/idea-board/index.js

0 commit comments

Comments
 (0)