blackjack: stacked split hands, dealer-flanking rules, Perfect Pairs#80
Merged
Conversation
- Split hands now render as stacked, compact one-line-per-hand entries with the hand on turn marked, instead of collapsing later hands to a bare "+". Fixes split cards being hidden and acting on a hand you can not see. - Move the table rules out of the mid-felt banner to flank the DEALER label (left/right), keeping the centre of the felt clear for cards. - Add a Perfect Pairs side bet: adjustable on the Left/Right betting axis (tiers 0/10/25/50/100), paying 25:1 perfect / 12:1 colored / 6:1 mixed. Resolved at the deal and folded into the round net. - Make whose-bet-is-whose unambiguous: each seat's bet and "+pairs N" form one contiguous block, and the side-bet line carries the placing player's character tile. smoke.yaml reseeded (58) to exercise the side bet, both pairs wins, and a split. New unit and render tests cover the pairs evaluation/credit, the betting nav, the stacked split layout, and the relocated rules. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Smoke screensDeterministic scripted screens for
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Three changes to
games/bcook/blackjack, plus a UX clarity follow-up:1. Split hands no longer hide their cards
The 15-col seat slot can't fit two card boxes side by side, so the old layout collapsed a seat's 2nd+ split hands to a bare
+— you couldn't see the cards, and the turn would advance to a hand you couldn't see (the "controls feel broken" report). Split seats now render one compact line per hand, stacked, with the hand on turn marked (►, active colour) and busts in red. Single (un-split) hands keep the full card box.2. Rules moved out of the middle of the felt
~ blackjack pays 3:2 · dealer stands on 17 ~was a banner across mid-felt. It's now split to flank the DEALER label (blackjack pays 3:2…dealer stands on 17), as subtle signage, freeing the centre for cards.3. Perfect Pairs side bet
A side bet on your first two cards being a pair:
0/10/25/50/100); Up/Down still drives the main bet.Whose-bet-is-whose clarity
Each seat's
betand+pairs Nare grouped on adjacent rows as one block, and the side-bet line carries the placing player's character tile so it reads by face, not just column.Tests
perfectPairsOutcome,pairsCreditFor.smoke.yamlreseeded (58) to walk: relocated signage → side-bet placement → both seats winning Perfect Pairs → a split (stacked-hands shot) → payout/ready.go test ./...green,go vetclean.🤖 Generated with Claude Code