Fix issue where the arrow keys could cause us to go out of bounds.

[remoteglot] / www / js / remoteglot.js

diff --git a/www/js/remoteglot.js b/www/js/remoteglot.js
index 936c336aa4492d68786ddf710915670baeb13ded..cb38ebe91e5ea6413a236277effd25475e1d30c4 100644 (file)
--- a/www/js/remoteglot.js
+++ b/www/js/remoteglot.js
@@ -34,7 +34,7 @@ var toplay = 'W';
 var ims = 0;
 
 /** @type {boolean} @private */
-var sort_refutation_lines_by_score = false;
+var sort_refutation_lines_by_score = true;
 
 /** @type {!string|undefined} @private */
 var highlight_from = undefined;
@@ -518,6 +518,11 @@ var update_board = function(data, num_viewers) {
                $("#numviewers").text(num_viewers + " current viewers");
        }
 
+       // The engine id.
+       if (data['id'] && data['id']['name'] !== null) {
+               $("#engineid").text(data['id']['name']);
+       }
+
        // The score.
        if (data['score'] !== null) {
                $("#score").text(data['score']);
@@ -646,13 +651,17 @@ var show_line = function(line_num, move_num) {
 window['show_line'] = show_line;
 
 var prev_move = function() {
-       --current_display_move;
+       if (current_display_move > 0) {
+               --current_display_move;
+       }
        update_displayed_line();
 }
 window['prev_move'] = prev_move;
 
 var next_move = function() {
-       ++current_display_move;
+       if (current_display_line && current_display_move < current_display_line.pretty_pv.length - 1) {
+               ++current_display_move;
+       }
        update_displayed_line();
 }
 window['next_move'] = next_move;
@@ -684,6 +693,18 @@ var update_displayed_line = function() {
                var move = current_display_line.uci_pv[i];
                move = move.substr(0, 2) + "-" + move.substr(2, 4);
                hiddenboard.move(move, false);
+
+               // chessboard.js does not automatically move the rook on castling
+               // (issue #51; marked as won't fix), so update it ourselves.
+               if (move == "e1-g1" && hiddenboard.position().g1 == "wK") {  // white O-O
+                       hiddenboard.move("h1-f1", false);
+               } else if (move == "e1-c1" && hiddenboard.position().c1 == "wK") {  // white O-O-O
+                       hiddenboard.move("a1-d1", false);
+               } else if (move == "e8-g8" && hiddenboard.position().g8 == "bK") {  // black O-O
+                       hiddenboard.move("h8-f8", false);
+               } else if (move == "e8-c8" && hiddenboard.position().c8 == "bK") {  // black O-O-O
+                       hiddenboard.move("a8-d8", false);
+               }
        }
        board.position(hiddenboard.position());
 }
@@ -699,6 +720,13 @@ var init = function() {
                update_highlight();
                redraw_arrows();
        });
+       $(window).keyup(function(event) {
+               if (event.which == 39) {
+                       next_move();
+               } else if (event.which == 37) {
+                       prev_move();
+               }
+       });
 };
 $(document).ready(init);