fix: Correct DOM element IDs for move history and captured pieces #5
@ -105,7 +105,8 @@ export class GameController {
|
||||
captured = SpecialMoves.executeEnPassant(this.board, piece, toRow, toCol);
|
||||
} else {
|
||||
// Normal move
|
||||
captured = this.board.movePiece(fromRow, fromCol, toRow, toCol);
|
||||
const moveResult = this.board.movePiece(fromRow, fromCol, toRow, toCol);
|
||||
captured = moveResult.captured;
|
||||
|
||||
// Check for promotion
|
||||
if (specialMoveType === 'promotion' || (piece.type === 'pawn' && piece.canPromote())) {
|
||||
|
||||
48
js/main.js
48
js/main.js
@ -84,10 +84,13 @@ class ChessApp {
|
||||
});
|
||||
|
||||
// Offer Draw
|
||||
document.getElementById('btn-offer-draw').addEventListener('click', () => {
|
||||
this.game.offerDraw();
|
||||
this.showMessage('Draw offered to opponent');
|
||||
});
|
||||
const offerDrawBtn = document.getElementById('btn-offer-draw');
|
||||
if (offerDrawBtn) {
|
||||
offerDrawBtn.addEventListener('click', () => {
|
||||
this.game.offerDraw();
|
||||
this.showMessage('Draw offered to opponent');
|
||||
});
|
||||
}
|
||||
|
||||
// Resign
|
||||
document.getElementById('btn-resign').addEventListener('click', () => {
|
||||
@ -172,9 +175,13 @@ class ChessApp {
|
||||
* Update turn indicator
|
||||
*/
|
||||
updateTurnIndicator() {
|
||||
const indicator = document.getElementById('turn-indicator');
|
||||
const indicator = document.getElementById('current-turn');
|
||||
if (!indicator) {
|
||||
console.error('Turn indicator element not found');
|
||||
return;
|
||||
}
|
||||
const turn = this.game.currentTurn;
|
||||
indicator.textContent = `${turn.charAt(0).toUpperCase() + turn.slice(1)} to move`;
|
||||
indicator.textContent = `${turn.charAt(0).toUpperCase() + turn.slice(1)}'s Turn`;
|
||||
indicator.style.color = turn === 'white' ? '#ffffff' : '#333333';
|
||||
}
|
||||
|
||||
@ -182,7 +189,7 @@ class ChessApp {
|
||||
* Update move history display
|
||||
*/
|
||||
updateMoveHistory() {
|
||||
const moveList = document.getElementById('move-list');
|
||||
const moveList = document.getElementById('move-history');
|
||||
const history = this.game.gameState.moveHistory;
|
||||
|
||||
if (history.length === 0) {
|
||||
@ -211,17 +218,19 @@ class ChessApp {
|
||||
* Update captured pieces display
|
||||
*/
|
||||
updateCapturedPieces() {
|
||||
const whiteCaptured = document.getElementById('white-captured');
|
||||
const blackCaptured = document.getElementById('black-captured');
|
||||
const whiteCaptured = document.getElementById('captured-white-pieces');
|
||||
const blackCaptured = document.getElementById('captured-black-pieces');
|
||||
|
||||
const captured = this.game.gameState.capturedPieces;
|
||||
|
||||
whiteCaptured.innerHTML = captured.black.map(piece =>
|
||||
`<span class="captured-piece black">${piece.getSymbol()}</span>`
|
||||
// "Captured by Black" shows white pieces that black captured
|
||||
whiteCaptured.innerHTML = captured.white.map(piece =>
|
||||
`<span class="captured-piece white">${piece.getSymbol()}</span>`
|
||||
).join('') || '-';
|
||||
|
||||
blackCaptured.innerHTML = captured.white.map(piece =>
|
||||
`<span class="captured-piece white">${piece.getSymbol()}</span>`
|
||||
// "Captured by White" shows black pieces that white captured
|
||||
blackCaptured.innerHTML = captured.black.map(piece =>
|
||||
`<span class="captured-piece black">${piece.getSymbol()}</span>`
|
||||
).join('') || '-';
|
||||
}
|
||||
|
||||
@ -232,6 +241,10 @@ class ChessApp {
|
||||
*/
|
||||
showMessage(message, type = 'info') {
|
||||
const statusMessage = document.getElementById('status-message');
|
||||
if (!statusMessage) {
|
||||
console.warn('Status message element not found, using console:', message);
|
||||
return;
|
||||
}
|
||||
statusMessage.textContent = message;
|
||||
statusMessage.style.display = 'block';
|
||||
|
||||
@ -250,7 +263,14 @@ class ChessApp {
|
||||
const overlay = document.getElementById('promotion-overlay');
|
||||
const dialog = document.getElementById('promotion-dialog');
|
||||
|
||||
overlay.style.display = 'block';
|
||||
if (!dialog) {
|
||||
console.error('Promotion dialog not found');
|
||||
return;
|
||||
}
|
||||
|
||||
if (overlay) {
|
||||
overlay.style.display = 'block';
|
||||
}
|
||||
dialog.style.display = 'block';
|
||||
|
||||
// Update symbols for current color
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user