Skip to content

fix: ground click animation and attack-move pathfinding#1251

Merged
braedonsaunders merged 1 commit into
mainfrom
claude/fix-ground-click-animation-5XNJd
Feb 2, 2026
Merged

fix: ground click animation and attack-move pathfinding#1251
braedonsaunders merged 1 commit into
mainfrom
claude/fix-ground-click-animation-5XNJd

Conversation

@braedonsaunders

Copy link
Copy Markdown
Owner

Two issues fixed:

  1. Ground click animation not showing:

    • GameplayInputHandler emitted command:moveGround on game.eventBus
    • OverlayScene listened on workerBridge.eventBus (different instance)
    • Added event forwarding from game.eventBus to bridge.eventBus for visual feedback events (command:moveGround, command:attackGround)
  2. Attack-move not working (units not moving):

    • CommandInputHandler issued ATTACK command when clicking ground
    • CombatSystem handled it but didn't request pathfinding
    • Changed to issue ATTACK_MOVE command instead, which MovementOrchestrator handles with proper pathfinding

https://claude.ai/code/session_01CFSzAg9SkexYrLRipZFq2v

Two issues fixed:

1. Ground click animation not showing:
   - GameplayInputHandler emitted command:moveGround on game.eventBus
   - OverlayScene listened on workerBridge.eventBus (different instance)
   - Added event forwarding from game.eventBus to bridge.eventBus
     for visual feedback events (command:moveGround, command:attackGround)

2. Attack-move not working (units not moving):
   - CommandInputHandler issued ATTACK command when clicking ground
   - CombatSystem handled it but didn't request pathfinding
   - Changed to issue ATTACK_MOVE command instead, which
     MovementOrchestrator handles with proper pathfinding

https://claude.ai/code/session_01CFSzAg9SkexYrLRipZFq2v
@braedonsaunders braedonsaunders force-pushed the claude/fix-ground-click-animation-5XNJd branch from 61b67d7 to 013c740 Compare February 2, 2026 23:45
@braedonsaunders braedonsaunders merged commit 25f7bf4 into main Feb 2, 2026
4 checks passed
@braedonsaunders braedonsaunders deleted the claude/fix-ground-click-animation-5XNJd branch February 3, 2026 02:52
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.

2 participants