lichess-org / lichobile

lichess.org mobile application
https://lichess.org/mobile
GNU General Public License v3.0
2.03k stars 318 forks source link

Crazyhouse analysis UI bugs #511

Closed xivaxy closed 7 years ago

xivaxy commented 7 years ago

Hi!

Great job finally making crazyhouse AI working! However, there are several user interface related bugs in the analysis mode in the latest android version:

In other words, whenever a standard chess checkmate is on the board, no further moves can be made, even though the position is not a checkmate in crazyhouse (the king can be saved by dropping a piece). See the first pic, here i am not able to drop a pawn to interpose, as advised by the engine.

I guess in the crazyhouse regime in the current layout there is no space for the little area with this information, but still it would very helpful to see.

The evaluated score can be seen in the moves list in a very small print, but it is shown only in the main line of the game, and not in the variations, which are the purpose of game analysis (see the second pic).

The best move can be seen as an arrow on the board, but it is ambiguous when the move is pawn promotion to queen vs. knight. See the second pic, the engine properly advises to promote a pawn, but there is no indication that I should promote to a knight.

veloce commented 7 years ago

Thanks for the report. I will find the place to add evaluation. To indicate the best promotion is trickier, but I'll see.

veloce commented 7 years ago

All fixed. Best move is shown in pgn notation in the eval box, so you have a way to know to which piece promote.

Maybe I'll find a better system later.