Give the pot wizard the same tap targets as everything else #34

2026-09-01T00:22:35-06:00

jheaps commented

2026-09-01 00:22:35 -06:00

Branched from master. Touches only potWizard.js and holdem.css — no overlap with the open #33, so the two merge in any order.

I measured the screen where the money actually gets decided and found the smallest targets in the tool:

	before	after
Winner rows	37px	81px
Checkboxes	13×13px	22×22px
Confirm button	36px	48px, full width

Every other surface has been held to 44px and up for several rounds — the one place where a mis-tap pays the wrong player was the one place that had not been.

Rows get a green fill when chosen, because a dealer confirms this with players watching and a bare checkbox is easy to misread across a table. The row was always the tap target; now it looks like one.
Each row splits into name / what they can win / their cards as separate pieces rather than one run-on sentence, so the hand reads at a glance (A♥ K♦ sits at the right, in red).
On a phone the summary buttons stack, with the one that ends the hand furthest from a stray thumb.

Validation (Playwright, headed)

The risk in restyling this screen is breaking the money, so the behaviour was re-checked, not just the pixels:

Selection: tapping the row (not the box) selects and shows chosen; tapping again clears both; two selected gives a tie.
The split is still right: $240 pot, two winners → $120.00 each.
The side-pot walkthrough still works end to end: 50 / 100 / 200 announced as main $150 / side $100 / side $100; round one offers all three, round two correctly offers only Ben and Cass once Ann is capped out; final payouts Ann $150, Ben $100, Cass $100 — $350, exactly what went in.
Rows measure 81px at 390px and 77px at 1280px; modal fits the viewport at both; no console errors.

One thing the screenshot caught: the confirm button was sized to its own label while everything around it was full width. Fixed and verified — it now matches the body width exactly (280px of 280px).

Branched from `master`. Touches only `potWizard.js` and `holdem.css` — **no overlap with the open #33**, so the two merge in any order. I measured the screen where the money actually gets decided and found the **smallest targets in the tool**: | | before | after | |---|---|---| | Winner rows | 37px | **81px** | | Checkboxes | 13×13px | **22×22px** | | Confirm button | 36px | **48px**, full width | Every other surface has been held to 44px and up for several rounds — the one place where a mis-tap pays the wrong player was the one place that had not been. - Rows get a **green fill when chosen**, because a dealer confirms this with players watching and a bare checkbox is easy to misread across a table. The row was always the tap target; now it looks like one. - Each row splits into **name / what they can win / their cards** as separate pieces rather than one run-on sentence, so the hand reads at a glance (`A♥ K♦` sits at the right, in red). - On a phone the summary buttons stack, with the one that ends the hand furthest from a stray thumb. ## Validation (Playwright, headed) The risk in restyling this screen is breaking the money, so the behaviour was re-checked, not just the pixels: - **Selection**: tapping the row (not the box) selects and shows chosen; tapping again clears both; two selected gives a tie. - **The split is still right**: $240 pot, two winners → **$120.00 each**. - **The side-pot walkthrough still works end to end**: 50 / 100 / 200 announced as main $150 / side $100 / side $100; round one offers all three, round two correctly offers **only Ben and Cass** once Ann is capped out; final payouts Ann $150, Ben $100, Cass $100 — $350, exactly what went in. - Rows measure 81px at 390px and 77px at 1280px; modal fits the viewport at both; no console errors. **One thing the screenshot caught**: the confirm button was sized to its own label while everything around it was full width. Fixed and verified — it now matches the body width exactly (280px of 280px).

jheaps added 1 commit 2026-09-01 00:22:35 -06:00

Give the pot wizard the same tap targets as everything else 38488f2978

Measured the screen where the money actually gets decided and found the smallest
targets in the tool: 37px winner rows, 13px checkboxes, a 36px confirm. Every
other surface has been held to 44px and up for several rounds — the one place a
mis-tap pays the wrong player was the one place that hadn't been.

Winner rows are now 56px and up, with a 22px box and a green fill when chosen,
because a dealer confirms this with players watching and a small checkbox is
easy to misread across a table. The row already was the tap target; now it looks
like one.

Each row also splits into name, what they can win, and their cards as separate
pieces rather than one run-on sentence, so the hand can be read at a glance. The
confirm takes the full width, and on a phone the summary's buttons stack with
the one that ends the hand furthest from a stray thumb.

Co-Authored-By: Claude Sonnet 5 <[email protected]>
Claude-Session: https://claude.ai/code/session_015bnp7xKUPFFSpwkYNFaPgb

jheaps referenced this pull request

2026-09-01 00:38:51 -06:00

Bring every phone target up to size, not just the ones I happened to check #35

jheaps referenced this pull request

2026-09-01 01:06:45 -06:00

Lift the faint greys to something readable in a dim room #36

jheaps referenced this pull request

2026-09-01 02:05:59 -06:00

Give the odd chip to the player the rule says gets it #40

jheaps referenced this pull request

2026-09-01 02:41:18 -06:00

Hold'em dealer: five correctness fixes and four polish passes (replaces #33–#41) #42