Skip to content

Allow removing players/CPUs at any count during setup - #31

Merged
lchoward merged 1 commit into
mainfrom
feature/fli-17-remove-player
Feb 19, 2026
Merged

lchoward merged 1 commit into
mainfrom
feature/fli-17-remove-player

Conversation

@lchoward

@lchoward lchoward commented Feb 19, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Removed the game.players.length > 2 guard on the remove button in SetupScreen
  • The × button now appears next to every player/CPU row regardless of player count, so users can always remove a player or CPU during setup

Fixes FLI-17

Test plan

  • Start a new game, add 1 player — × button should appear
  • Add 2 players — both should have × buttons
  • Remove a player with 2 players — should go down to 1
  • Verify "Start Game" still requires 2+ players with names

🤖 Generated with Claude Code

Previously the remove button only appeared when 3+ players were added.
Now it's always visible so users can remove any player or CPU.

Fixes FLI-17

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@linear

linear Bot commented Feb 19, 2026

Copy link
Copy Markdown

@lchoward
lchoward merged commit b677efc into main Feb 19, 2026
1 check passed
@lchoward
lchoward deleted the feature/fli-17-remove-player branch February 19, 2026 09:46
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.

1 participant