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

[RFC] Support pre promotions #2515

Merged
merged 1 commit into from Jan 13, 2017
Merged

[RFC] Support pre promotions #2515

merged 1 commit into from Jan 13, 2017

Conversation

niklasf
Copy link
Member

@niklasf niklasf commented Jan 12, 2017

Premove a pawn to the eighth rank.

Previously: Wait for the reply. Then the promotion dialog pops up.

Now: You select the promotion piece type and it gets played once there
is a reponse.

image

@niklasf niklasf changed the title Support pre promotions [RFC] Support pre promotions Jan 12, 2017
Premove a pawn to the eighth rank.

Previously: Wait for the reply. Then the promotion dialog pops up.

Now: You select the promotion piece type and it gets played once there
is a reponse.
@ornicar
Copy link
Collaborator

ornicar commented Jan 13, 2017

Very nice. I tested it and couldn't find a bug. But I noticed a behavior that might be improved.

  • disable auto-promotion
  • premove pawn to first/eighth rank
  • the promotion choices are displayed
  • if the opponent now moves, the uncomplete premove is discarded, the choices disappear

Can this be handled better?

@ornicar
Copy link
Collaborator

ornicar commented Jan 13, 2017

I'm merging it anyway, since it's better than master.

@ornicar ornicar merged commit 7b495eb into lichess-org:master Jan 13, 2017
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.

None yet

2 participants