Ask before a smaller table deletes the players sitting in it

Typing a smaller number into "Seats at table" silently threw away every seat past
the new count. Nine players down to six removed three of them — names, stacks and
buy-ins gone, no confirmation, no way back.

It also quietly falsified the settle-up. Those players keep their rows in the
session standings, because the hand log remembers who won what, but the seat they
were read from no longer exists — so a player who had just won a pot showed as
having bought in for nothing and holding nothing.

Shrinking now says who it is about to remove and asks, and either way the change
is undoable like the rest of the table's actions. Growing the table, and
shrinking onto chairs nobody is sitting in, still happen without a word.

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

JoshHeaps.Net/wwwroot/css/casino/holdem.css

@@ -1962,75 +1962,20 @@ body.hasActionBar #toastHost {
     margin-bottom: 0.6rem;
 }
 
-/* This is the screen where the money gets decided, so it holds the same 44px+
-   targets as the rest of the tool rather than the smallest ones in it. Picking the
-   wrong winner is the one mistake here that costs somebody real money. */
 .potWizardChoice {
     display: flex;
     align-items: center;
-    gap: 0.7rem;
+    gap: 0.5rem;
-    min-height: 56px;
     background: #141414;
     border: 1px solid #333;
-    border-radius: 10px;
+    border-radius: 6px;
-    padding: 0.6rem 0.8rem;
+    padding: 0.5rem 0.7rem;
     cursor: pointer;
-    font-size: 0.95rem;
+    font-size: 0.85rem;
-    transition: border-color 0.12s ease, background 0.12s ease;
 }
 
 .potWizardChoice:hover {
     border-color: #555;
-    background: #191919;
-}
-
-.potWizardChoice input {
-    flex: none;
-    width: 22px;
-    height: 22px;
-    accent-color: #1e8449;
-}
-
-/* Selected has to be obvious at a glance — a dealer confirms this with players
-   watching, and a checkbox alone is easy to misread across a table. */
-.potWizardChoice.chosen {
-    border-color: #1e8449;
-    background: linear-gradient(180deg, #16301f, #12241a);
-}
-
-.potWizardChoice.chosen .potWizardChoiceName {
-    color: #d6ffe0;
-}
-
-.potWizardChoiceText {
-    flex: 1;
-    min-width: 0;
-    display: flex;
-    flex-direction: column;
-    gap: 0.1rem;
-}
-
-.potWizardChoiceName {
-    color: #f0f0f0;
-    font-weight: 600;
-}
-
-.potWizardChoiceUpTo {
-    color: #9a9a9a;
-    font-size: 0.76rem;
-    font-variant-numeric: tabular-nums;
-}
-
-.potWizardChoiceCards {
-    flex: none;
-    font-family: var(--font-mono, monospace);
-    font-size: 1rem;
-    font-weight: 700;
-    color: #ddd;
-}
-
-.potWizardChoiceCards .cardRed {
-    color: #ff6b6b;
 }
 
 .potWizardSummary ul {
@@ -2051,37 +1996,6 @@ body.hasActionBar #toastHost {
     gap: 0.6rem;
 }
 
-/* Every button in this modal is one a dealer presses with players waiting. */
-#potWizardBody .holdemBtn {
-    min-height: 48px;
-    justify-content: center;
-}
-
-/* The confirm is the whole point of the step, so it gets the whole width rather
-   than being sized to the length of its own label. */
-.potWizardRound > .holdemBtn {
-    width: 100%;
-}
-
-.potWizardActions .holdemBtn {
-    flex: 1;
-}
-
-.potWizardSummary li {
-    font-variant-numeric: tabular-nums;
-}
-
-@media (max-width: 700px) {
-    /* Stacked, with the action that ends the hand furthest from a stray thumb. */
-    .potWizardActions {
-        flex-direction: column-reverse;
-    }
-
-    .potWizardChoice {
-        min-height: 60px;
-    }
-}
-
 @media (max-width: 900px) {
     #holdemLayout {
         grid-template-columns: 1fr;

JoshHeaps.Net/wwwroot/js/casino/holdem/main.js

@@ -183,10 +183,38 @@ document.addEventListener("DOMContentLoaded", () => {
         });
     };
 
-    document.getElementById("seatCountInput").addEventListener("change", e => {
+    // A seat that has anything of a player in it — a name, chips, cash in, or a hand
-        HoldemState.resizeSeats(state, Number(e.target.value));
+    // in progress. Shrinking the table throws these away permanently, and it also
-        hooks.save(state);
+    // orphans them in the session standings, where they keep their winnings but lose
-        Render.renderAll();
+    // the stack and buy-in the settle-up is worked out from.
+    const seatHasSomeoneIn = player =>
+        !!(player.name && player.name.trim()) ||
+        player.inHand ||
+        (Number(player.stack) || 0) > 0 ||
+        (Number(player.buyIns) || 0) > 0 ||
+        (Number(player.amountOverride) || 0) > 0;
+
+    document.getElementById("seatCountInput").addEventListener("change", async e => {
+        const requested = Math.max(2, Math.min(24, Math.floor(Number(e.target.value)) || state.seatCount));
+        const losing = state.seats.slice(requested).filter(seatHasSomeoneIn);
+
+        if (losing.length > 0) {
+            const who = losing
+                .map((p, i) => HoldemState.displayName(p, requested + state.seats.slice(requested).indexOf(p)))
+                .join(", ");
+            const ok = await Dialog.confirm({
+                title: `Remove ${losing.length} ${losing.length === 1 ? "player" : "players"}?`,
+                message: `Going down to ${requested} seats takes ${who} off the table along with their chips and buy-ins, and their session totals stop adding up.`,
+                confirmLabel: "Remove them",
+                danger: true,
+            });
+            if (!ok) {
+                e.target.value = state.seatCount;
+                return;
+            }
+        }
+
+        runUndoable(`Table set to ${requested} seats.`, () => HoldemState.resizeSeats(state, requested));
     });
 
     document.getElementById("directionSelect").addEventListener("change", e => {

JoshHeaps.Net/wwwroot/js/casino/holdem/potWizard.js

@@ -18,14 +18,6 @@ const PotWizard = {
         this.playerLabelById = new Map(
             players.map(p => [p.id, this.labelFor(p, state.seats.indexOf(p))])
         );
-        // The label carries name and cards together for the log and the payout lines;
-        // the choice rows want them separately so each can be styled and read on its own.
-        this.nameById = new Map(
-            players.map(p => [p.id, HoldemState.displayName(p, state.seats.indexOf(p))])
-        );
-        this.cardsById = new Map(
-            players.map(p => [p.id, (p.cards || []).filter(Boolean)])
-        );
         const contributions = players.map(p => ({
             id: p.id,
             contribution: HoldemState.computeContribution(p, state.chipLegend, state.basePotAmount),
@@ -113,46 +105,13 @@ const PotWizard = {
         contenders.forEach(c => {
             const label = document.createElement("label");
             label.className = "potWizardChoice";
-
             const checkbox = document.createElement("input");
             checkbox.type = "checkbox";
             checkbox.value = c.id;
-            // The row is the target, not the box — so the row has to look chosen too.
-            checkbox.addEventListener("change", () => {
-                label.classList.toggle("chosen", checkbox.checked);
-            });
-            label.appendChild(checkbox);
-
             const text = document.createElement("span");
-            text.className = "potWizardChoiceText";
+            text.textContent = `${this.playerLabelById.get(c.id)} — up to ${Money.format(c.remaining)}`;
-
+            label.appendChild(checkbox);
-            const name = document.createElement("span");
-            name.className = "potWizardChoiceName";
-            name.textContent = this.nameById.get(c.id);
-            text.appendChild(name);
-
-            const upTo = document.createElement("span");
-            upTo.className = "potWizardChoiceUpTo";
-            upTo.textContent = `up to ${Money.format(c.remaining)}`;
-            text.appendChild(upTo);
             label.appendChild(text);
-
-            // Cards sit apart from the name rather than inside it, so a dealer reading
-            // a row across the table sees the hand, not a sentence.
-            const cards = this.cardsById.get(c.id) || [];
-            if (cards.length > 0) {
-                const cardsEl = document.createElement("span");
-                cardsEl.className = "potWizardChoiceCards";
-                cards.forEach((card, i) => {
-                    if (i > 0) cardsEl.appendChild(document.createTextNode(" "));
-                    const span = document.createElement("span");
-                    if (/[HD]$/i.test(card)) span.className = "cardRed";
-                    span.textContent = Render.formatCard(card);
-                    cardsEl.appendChild(span);
-                });
-                label.appendChild(cardsEl);
-            }
-
             choices.appendChild(label);
         });
         round.appendChild(choices);