Skip to content

Add piece maneuver arrows for top engine line - #19218

Merged
ornicar merged 5 commits into
lichess-org:masterfrom
dav1312:maneuver-arrows-rebase-sq
Jan 27, 2026
Merged

Add piece maneuver arrows for top engine line#19218
ornicar merged 5 commits into
lichess-org:masterfrom
dav1312:maneuver-arrows-rebase-sq

Conversation

@dav1312

@dav1312 dav1312 commented Jan 26, 2026

Copy link
Copy Markdown
Contributor

Adds multiple arrows if the engine suggests the same piece should be moved multiple consecutive times, indicating a maneuver.

Up to 3 arrows for piece maneuvers and a setting to disable it.
interferingArrow avoids clutter.

image image image

Closes #10982

@ornicar

ornicar commented Jan 27, 2026

Copy link
Copy Markdown
Collaborator

That's really nice, and I agree that it can be enabled by default.

Not sure we even need a setting to disable it. It would be better if we could do without it.

@dav1312

dav1312 commented Jan 27, 2026

Copy link
Copy Markdown
Contributor Author

Sure, I just wanted to give people the choice just in case they don't like it 😅

@ornicar
ornicar merged commit cbd6fd3 into lichess-org:master Jan 27, 2026
6 checks passed
@QRMLichess

Copy link
Copy Markdown

I don't think this is a great update unfortunately, the second arrow is really dependent on what move the opp plays in response to the first arrow move, there is no guarantee the best move will be played or was played in the game being analysed so the second arrow is presumptive and confusing, and often not even the best move after the engine thinks about it some more. I thought my UI was glitching until I saw this is deliberate. Please reconsider, thank you

@mrobaczyk

Copy link
Copy Markdown

This is a little bit wierd now when the first move is a capture. I was wondering for a while which turn is it and how can the white queen move now. Can I turn this somehow off? There are so many analysis toggles now it would be probably better to move them to preferences.

image

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.

Show following moves arrows

4 participants