Fix history display with dropped pieces.

[remoteglot-book] / www / js / book.js

diff --git a/www/js/book.js b/www/js/book.js
index 9ed5ee5c7ae31491bf8a170252d27cf9b8755fff..611678f8f99c52a3e73ea53241e31b88766dd24c 100644 (file)
--- a/www/js/book.js
+++ b/www/js/book.js
@@ -27,6 +27,20 @@ var get_game = function() {
 }
 
 var update = function() {
+       var text = "";
+       for (var i = 0; i < history.length; ++i) {
+               if (i % 2 == 0) {
+                       text += (i/2 + 1) + ". ";
+               }
+               if (i == move_override) {
+                       text += '<strong>' + history[i] + '</strong>';
+               } else {
+                       text += history[i];
+               }
+               text += " ";
+       }
+       $('#gamehistory').html(text);
+
        var game = get_game();
        board.position(game.fen());
        fetch_analysis();
@@ -216,9 +230,14 @@ var show_lines = function(data, game) {
 }
 
 var make_move = function(move) {
-       history.length = move_override;
-       history.push(move);
-       move_override = history.length;
+       if (move_override < history.length && history[move_override] == move) {
+               // User effectively only moved forward in history.
+               ++move_override;
+       } else {
+               history.length = move_override;
+               history.push(move);
+               move_override = history.length;
+       }
        update();
 }
 window['make_move'] = make_move;
@@ -261,8 +280,13 @@ var onDrop = function(source, target) {
        // illegal move
        if (move === null) return 'snapback';
 
-       history = game.history({ verbose: true });
+       var new_history = game.history({ verbose: true });
+       history = [];
+       for (var i = 0; i < new_history.length; ++i) {
+               history.push(new_history[i].san);
+       }
        move_override = history.length;
+       update();
 };
 
 // update the board position after the piece snap