PR_26174_ALFA_016-game-hub-row-edit-add-selected-state

Author: ToolboxAid

assets/theme-v2/css/tables.css

@@ -115,12 +115,11 @@ td {
   cursor: pointer
 }
 
-.data-table [data-game-row][data-game-active="true"] {
+.data-table [data-game-active-cell="true"] {
   background: color-mix(in srgb, var(--gold) 14%, var(--panel))
 }
 
-.data-table [data-game-row][data-game-active="true"] > th,
-.data-table [data-game-row][data-game-active="true"] > td {
+.data-table [data-game-active-cell="true"] {
   border-bottom-color: var(--gold-border-muted)
 }
 

docs_build/dev/reports/PR_26174_ALFA_016-game-hub-row-edit-add-selected-state-branch-validation.txt

@@ -0,0 +1,9 @@
+Branch Validation: PASS
+
+PASS - Current branch is pr/26174-ALFA-016-game-hub-row-edit-add-selected-state.
+PASS - Branch was created from pr/26174-ALFA-015-game-hub-actions-and-setup-cleanup at ef771ff76.
+PASS - Worktree was clean before creating the PR branch.
+PASS - Scope is limited to Game Hub table add/edit selected-state behavior, Theme V2 table styling, targeted Playwright coverage, and required reports.
+PASS - No protected Project Instructions files were modified.
+PASS - No merge to main was performed.
+

docs_build/dev/reports/PR_26174_ALFA_016-game-hub-row-edit-add-selected-state-manual-validation-notes.txt

@@ -0,0 +1,11 @@
+Manual Validation Notes
+
+PASS - Confirmed active-game selected styling is on the Game table cell/button, not the Actions edit button.
+PASS - Confirmed Edit remains a plain action until it opens an inline edit row.
+PASS - Confirmed inline edit row shows Game, Purpose, Status, Owner, Actions columns with Save and Cancel.
+PASS - Confirmed Game textbox is visible during edit and read-only because no rename API exists in the current Game Hub contract.
+PASS - Confirmed Add Game row appears under the game table and expands to Game, Purpose, Status, Owner, Actions with Save and Cancel.
+PASS - Confirmed add and edit saves use the existing repository API/service methods.
+PASS - Confirmed Source Idea and Readiness Output child rows remain under expanded game parent rows.
+PASS - Confirmed guest users can browse/select games but cannot add or edit.
+

docs_build/dev/reports/PR_26174_ALFA_016-game-hub-row-edit-add-selected-state-requirement-checklist.txt

@@ -0,0 +1,15 @@
+Requirement Checklist: PASS
+
+PASS - Selected state now appears on the Game button/cell for the active game.
+PASS - Edit button does not look selected while a row is merely active.
+PASS - Add Game button/row appears under the game table.
+PASS - Add row includes Game textbox, Purpose dropdown, and Status dropdown.
+PASS - Add row actions show Save and Cancel.
+PASS - Edit row actions show Save and Cancel.
+PASS - Columns remain Game, Purpose, Status, Owner, Actions.
+PASS - Source Idea child rows are preserved.
+PASS - Readiness Output child rows are preserved.
+PASS - table_first_ui.md and the Idea Board table-first add-row/edit-row pattern were followed.
+PASS - No browser-owned product data was introduced.
+PASS - No silent fallbacks were introduced.
+

docs_build/dev/reports/PR_26174_ALFA_016-game-hub-row-edit-add-selected-state-validation-lane.txt

@@ -0,0 +1,12 @@
+Validation Lane: PASS
+
+Commands:
+1. npx playwright test tests/playwright/tools/GameHubMockRepository.spec.mjs -g "Game Hub creates, opens, and deletes mock games|Game Hub validates game parent rows and child tables|Game Hub preserves guest browsing and blocks guest saves|Game Hub shows a creator-safe empty state when no projects exist|Game Hub shows a creator-safe unavailable state when project list API fails|Game Hub displays and edits game purpose and member role|Game Hub readiness child rows update from mock game state"
+Result: PASS - 7 passed.
+
+2. npx playwright test tests/playwright/tools/IdeaBoardTableNotes.spec.mjs -g "Idea Board uses accordion table ideas and notes"
+Result: PASS - 1 passed.
+
+3. git diff --check -- assets/theme-v2/css/tables.css toolbox/game-hub/index.html toolbox/game-hub/game-hub.js tests/playwright/tools/GameHubMockRepository.spec.mjs
+Result: PASS.
+

docs_build/dev/reports/PR_26174_ALFA_016-game-hub-row-edit-add-selected-state.md

@@ -0,0 +1,21 @@
+# PR_26174_ALFA_016-game-hub-row-edit-add-selected-state
+
+## Purpose
+
+Move Game Hub add/edit behavior into the game table and correct selected-state styling.
+
+## Summary
+
+- Moved the active-game selected state from the Actions edit button to the Game column cell/button.
+- Kept Edit as a plain row action and rendered Save/Cancel only while a row is editing.
+- Added a bottom Add Game row that expands into Game, Purpose, and Status fields with Save/Cancel actions.
+- Removed the old sidebar add-game form while preserving the existing API/service contract.
+- Preserved Game, Purpose, Status, Owner, and Actions columns.
+- Preserved Source Idea and Readiness Output child rows under game parent rows.
+
+## Validation
+
+PASS - `npx playwright test tests/playwright/tools/GameHubMockRepository.spec.mjs -g "Game Hub creates, opens, and deletes mock games|Game Hub validates game parent rows and child tables|Game Hub preserves guest browsing and blocks guest saves|Game Hub shows a creator-safe empty state when no projects exist|Game Hub shows a creator-safe unavailable state when project list API fails|Game Hub displays and edits game purpose and member role|Game Hub readiness child rows update from mock game state"`
+PASS - `npx playwright test tests/playwright/tools/IdeaBoardTableNotes.spec.mjs -g "Idea Board uses accordion table ideas and notes"`
+PASS - `git diff --check -- assets/theme-v2/css/tables.css toolbox/game-hub/index.html toolbox/game-hub/game-hub.js tests/playwright/tools/GameHubMockRepository.spec.mjs`
+

docs_build/dev/reports/codex_changed_files.txt

@@ -1,5 +1,4 @@
 assets/theme-v2/css/tables.css
 tests/playwright/tools/GameHubMockRepository.spec.mjs
-tests/playwright/tools/IdeaBoardTableNotes.spec.mjs
 toolbox/game-hub/game-hub.js
 toolbox/game-hub/index.html