Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Valid move snapping for draw arrow not working correctly #307

Closed
KartikWatts opened this issue Jul 23, 2024 · 3 comments
Closed

Valid move snapping for draw arrow not working correctly #307

KartikWatts opened this issue Jul 23, 2024 · 3 comments

Comments

@KartikWatts
Copy link

Description:

During my recent game analysis, I encountered a minor issue with the valid move-snapping functionality for the draw arrow.

Problem:

Even when defaultSnapToValidMove is true, if the right mouse button is pressed and the pointer moves outside of the chessboard before being released, the snapping feature fails to work as expected. This occurs because the cur.snapToValidMove value is not being updated properly.

Steps to Reproduce:

  1. Press and hold the right mouse button to draw an arrow.
  2. Move the pointer outside of the chessboard.
  3. Bring the pointer back onto the chessboard.

Attaching GIF for reference:

chessground_snap

Expected Behavior:

The draw arrow should snap to the valid move after the pointer returns to the chessboard.

Actual Behavior:

The draw arrow does not snap to the valid move because cur.snapToValidMove is not updated.

@KartikWatts
Copy link
Author

Considering it as a bug, a small fix was required, raised a PR to close this: #308. Regards.

@fitztrev
Copy link
Member

Hi, this is actually intended

lichess-org/lila#11713
lichess-org/lila#9095
lichess-org/lila#8313
lichess-org/lila#15073

@fitztrev fitztrev closed this as not planned Won't fix, can't repro, duplicate, stale Jul 23, 2024
@KartikWatts
Copy link
Author

Oops got it 🙈. Thanks.

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 a pull request may close this issue.

2 participants