Fix multi-hop move animation showing only the last segment

[Copilot]

When a piece is moved multiple times in one turn (e.g. "6/3 3/1"), the animation overlay only played the final segment instead of all hops.

Root cause

parseMove's cancellation logic removes moved[point] for intermediate stops — points where a piece both arrives and departs. This is correct for ghost/moved highlighting, but MoveAnimation.tsx relied solely on .piece.moved elements to locate the animation target, so intermediate destinations had no toEl and the segment was silently skipped.

Fix

Fall back to .piece.ghost elements when no .piece.moved is found at the destination. Ghost pieces survive the cancellation and are rendered at every intermediate point, so their DOM position correctly anchors the animation end for that segment.

// Before
const toEl = movedEls?.[toIdx] ?? null;

// After — ghost fallback for intermediate points
const ghostFallbackEls = points[pair.to as number]?.querySelectorAll<HTMLElement>('.piece.ghost');
const toEl = movedEls?.[toIdx] ?? ghostFallbackEls?.[toIdx] ?? null;

The toIdx counter already tracks how many times each point has been visited as a destination, so it correctly indexes into the ghost list for cases like doubles where the same intermediate point is visited multiple times.

Original prompt

when a piece is moved twice or more in one turn, only the last move is shown in the animation instead of all of the moves

---

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

[github-actions]

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-03-21 22:10 UTC