Skip to content

fix(golf): keep drawn card in-place and fix double-click selection#190

Merged
aaylward merged 1 commit into
mainfrom
golf_ux
Mar 8, 2026
Merged

fix(golf): keep drawn card in-place and fix double-click selection#190
aaylward merged 1 commit into
mainfrom
golf_ux

Conversation

@aaylward

@aaylward aaylward commented Mar 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Drawn card stays on source pile — drawing from deck flips the card face-up on the draw pile with gold highlight; taking from discard keeps the card decorated in-place. No third pile ever appears.
  • Fix double-click to select — use outline instead of border for card selection (no box model change), remove CSS animation that could cause compositor issues on first click.

Test plan

  • Draw from deck — card flips face-up on the draw pile with gold highlight, no layout shift
  • Take from discard — card stays highlighted on discard pile, no new pile appears
  • Select face-down card for swap — gold outline on first click, no double-click needed
  • Test on mobile Safari, desktop Safari, Firefox

🤖 Generated with Claude Code

@aaylward aaylward enabled auto-merge (squash) March 8, 2026 21:26
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Mar 8, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
muchq 6798074 Commit Preview URL

Branch Preview URL
Mar 08 2026, 09:29 PM

…ck selection

Bug 1: Drawn card now stays visually on its source pile (draw or
discard) with a gold highlight, instead of appearing as a third pile.
Track source via drawnFrom state, reset when drawnCard clears.

Bug 2: Switch .card.selected from border to outline (no box model
change avoids layout shift in the card grid). Drop CSS animation
that could cause compositor layer promotion issues on first click.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@aaylward aaylward merged commit 3a59052 into main Mar 8, 2026
2 checks passed
@aaylward aaylward deleted the golf_ux branch March 8, 2026 21:29
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