inda20plusplus/axenil-chess

axenil-chess

axenil-chess's Issues

Add function that checks if promotion is available before moving

When moving the pawn to the last rank you currently have to add the piece that it promotes to as input for the move. For example if you want to promote a pawn to a queen you have to call make_move(“c7 c8=q”). But if you call make_move(“c7 c8”) without =q you get invalid move as a response. It could therefore be convenient to add a function that checks whether a piece is a pawn or not and if so, check if it’s on the 7th or 1st rank.

Restructure

Add win/Loss screen

As title indicates, a win/loss screen should pop-up in the case of a check-mate or tie and possibly direct the player to shut down the program or possibly (needing more effort) start a new game. This by adding an else clause in chess_controllers event to be run when self.game_over is true

Resize window according to board and UI size

Currently the application window is a fixed size disregarding the size of the contents giving black space which should not be shown unless it is used by UI-elements (such as a future turn indicator and such on the right hand side of the board).

This should be labeled an enhancement

Add promotion to gui

Fix precision of mouse controls and piece-selection

Currently the mouse controls seem to be a bit imprecise, especially when not clicking at the centre of a piece's space. This is probably due to errors in the chess_controller event() function, either due to the mouse position being updated in an incorrect manner or due to rounding errors in the conversion from float mouse_pos values to u8 boardspace coordinates

Label this a bug

Moves resulting in checkmate are returned from get_moves

A move is shown as possible in the gui even if it results in checkmate. But when trying to make the move it’s not possible. This happens since the function get_moves() isn’t checking for checkmate. To fix this call check_if_legal_move() on each move in get_moves() before adding them to the vector.