Skip to content

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

Closed
ToolboxAid wants to merge 2 commits into
pr/26174-ALFA-015-game-hub-actions-and-setup-cleanupfrom
pr/26174-ALFA-016-game-hub-row-edit-add-selected-state
Closed

PR_26174_ALFA_016-game-hub-row-edit-add-selected-state#110
ToolboxAid wants to merge 2 commits into
pr/26174-ALFA-015-game-hub-actions-and-setup-cleanupfrom
pr/26174-ALFA-016-game-hub-row-edit-add-selected-state

Conversation

@ToolboxAid

Copy link
Copy Markdown
Owner

Summary

  • Move Game Hub add/edit behavior into table rows with Save/Cancel actions.
  • Put the active-game selected state on the Game column cell/button instead of the Actions edit button.
  • Add a bottom Add Game row that expands into Game, Purpose, and Status fields.
  • Preserve 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

Notes

  • Draft stacked on pr/26174-ALFA-015-game-hub-actions-and-setup-cleanup.
  • Game textbox is visible during edit and read-only because the current Game Hub API/service contract has purpose/status update methods but no rename method.

@ToolboxAid

Copy link
Copy Markdown
Owner Author

OWNER-approved closure: this historical Alfa PR is superseded by merged consolidated PR #149. No branch deletion requested.

@ToolboxAid ToolboxAid closed this Jun 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants