bug(ui): turn animation ring shows when no hand is active

[bitcoinbrisbane]

Problem

The glowing turn animation ring (ripple effect) appears around a player's seat even when no hand is in progress. For example, if a single player joins and sits in, the ring shows despite nextToAct having no valid target.

Expected Behavior

The turn animation ring should only render when:

• A hand is actively in progress (isGameInProgress === true)
• nextToAct points to a valid seated, active player

Current Behavior

The ring renders based solely on nextToActSeat === index + 1 without checking whether a hand is actually running.

Key Files

• ui/src/components/playPage/Animations/TurnAnimation.tsx — renders rings when nextToActSeat === index + 1
• ui/src/components/playPage/Animations/TurnAnimation.css — ripple animation styles
• ui/src/hooks/game/useNextToActInfo.ts — provides nextToActSeat

Suggested Fix

Add an isGameInProgress check (or verify nextToAct > 0 and game round is not ANTE/idle) before rendering the turn animation rings.

Related

• feat(pvm): Complete sit-in / sit-out / leave table state logic (cash games) #51 — Sit-in/sit-out state logic epic
• feat(pvm): Implement sit-in options on join (Next BB vs Post Now) #50 — Sit-in method persistence

🤖 Generated with Claude Code

[bitcoinbrisbane]

Bug Description

The turn animation ring (circular progress indicator) appears around players even when no active hand is in progress, creating visual confusion.

Current Behavior

• Turn ring animation displays during non-game states
• Ring may appear during lobby, between hands, or when sitting out
• Gives false indication that action is required

Expected Behavior

Turn animation ring should ONLY appear when:

• A hand is actively in progress (gameState.status === ACTIVE or similar)
• It is currently the player's turn to act
• The action timer is counting down

Turn ring should NOT appear when:

• No hand is active (between hands, lobby, sitting out)
• Hand has completed (showdown, all folded)
• Player is not the current actor

Root Cause (Likely)

Missing conditional check for active hand status before rendering turn ring component.

Fix Location

Check these files:

• ui/src/components/player/ - Player turn indicator components
• ui/src/hooks/animations/useTurnAnimation.ts - Turn animation hook
• Conditional rendering should check both hand status AND current turn

Proposed Fix

Add proper game state checks:

• Verify hand is in progress
• Verify player has active turn
• Only then render turn ring animation

Impact

• Confusing for new players
• Visual clutter during non-game time
• May cause players to think they need to act when they do not

Priority: Medium - Visual bug affecting UX clarity
Effort: Small - Add conditional checks to animation rendering

[bitcoinbrisbane]