Skip to content

Conversation

@jonathan-irvin
Copy link
Collaborator

Summary

  • Replace 'hide board' behavior with 'GAME CLOSED' message when game is closed
  • Add type hints throughout the codebase with a new types module
  • Fix tests to check for closed message display
  • Keep same styling as header for consistency

Test plan

  • All unit tests pass with updated expectations
  • Game closed message appears in both main and stream views
  • Message uses the same font as header for consistent styling
  • Manual test by running the app and closing the game

🤖 Generated with Claude Code

Replace the 'hide board' behavior with a 'GAME CLOSED' message displayed in the same space.
This provides better visual feedback to users when the game is closed.

- Add new constants for closed message text and color
- Create new build_closed_message function that displays large text
- Update close_game and sync_board_state to show message instead of hiding board
Instead of hiding the board when the game is closed, display a large 'GAME CLOSED'
message in the same space as the board. This provides better visibility and a consistent
user experience across all views including the main.py implementation.

- Add build_closed_message function to both modules
- Update sync_board_state to display message in both modules
- Display message using the header font and free space color
- Add proper typing to game_logic, board_builder, and constants
- Create dedicated types module for shared type definitions
- Fix tests to reflect new closed message functionality
- Add mypy type checking to development dependencies
@jonathan-irvin jonathan-irvin merged commit 812ae28 into main Mar 3, 2025
5 checks passed
@jonathan-irvin jonathan-irvin deleted the feature/closed-message-everywhere branch March 3, 2025 16:15
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