Put the limit for showing seconds at 20 minutes instead of 10.

[remoteglot] / www / js / remoteglot.js

diff --git a/www/js/remoteglot.js b/www/js/remoteglot.js
index b049c32ea81bc52c63b1cbb19172d7c769778b21..34e1863624ffdc7afd34a7913e54acd35bbc6028 100644 (file)
--- a/www/js/remoteglot.js
+++ b/www/js/remoteglot.js
@@ -7,7 +7,7 @@
  * @type {Number}
  * @const
  * @private */
-var SCRIPT_VERSION = 2016091401;
+var SCRIPT_VERSION = 2016113002;
 
 /**
  * The current backend URL.
@@ -1053,15 +1053,27 @@ var update_board = function() {
        if (displayed_analysis_data) {
                // Displaying some non-current position, pick out the last move
                // from the history. This will work even if the fetch failed.
-               last_move = format_halfmove_with_number(
-                       current_display_line.pv[current_display_move],
-                       current_display_move + 1);
-               headline += ' after ' + last_move;
+               if (current_display_move !== -1) {
+                       last_move = format_halfmove_with_number(
+                               current_display_line.pv[current_display_move],
+                               current_display_move);
+                       headline += ' after ' + last_move;
+               }
        } else if (data['position']['last_move'] !== 'none') {
+               // Find the previous move.
+               var previous_move_num, previous_toplay;
+               if (data['position']['toplay'] == 'B') {
+                       previous_move_num = data['position']['move_num'];
+                       previous_toplay = 'W';
+               } else {
+                       previous_move_num = data['position']['move_num'] - 1;
+                       previous_toplay = 'B';
+               }
+
                last_move = format_move_with_number(
                        data['position']['last_move'],
-                       data['position']['move_num'],
-                       data['position']['toplay'] == 'W');
+                       previous_move_num,
+                       previous_toplay == 'W');
                headline += ' after ' + last_move;
        } else {
                last_move = null;
@@ -1196,7 +1208,9 @@ var update_board = function() {
                for (var i = 1; i < alt_moves.length && i < 3; ++i) {
                        hiddenboard = new Chess(base_fen);
                        var move = hiddenboard.move(alt_moves[i]);
-                       create_arrow(move.from, move.to, '#f66', 1, 10);
+                       if (move !== null) {
+                               create_arrow(move.from, move.to, '#f66', 1, 10);
+                       }
                }
        }
 
@@ -1298,7 +1312,16 @@ var update_sparkline = function(data) {
                                chartRangeMin: min_score,
                                chartRangeMax: max_score,
                                tooltipFormatter: function(sparkline, options, fields) {
-                                       return format_tooltip(data, fields[0].offset + first_move_num);
+                                       // score_history contains the Nth _position_, but format_tooltip
+                                       // wants to format the Nth _move_; thus the -1.
+                                       return format_tooltip(data, fields[0].offset + first_move_num - 1);
+                               }
+                       });
+                       $('#scorespark').bind('sparklineClick', function(event) {
+                               var sparkline = event.sparklines[0];
+                               var region = sparkline.getCurrentRegionFields();
+                               if (region[0].offset !== undefined) {
+                                       show_line(0, first_move_num + region[0].offset - 1);
                                }
                        });
                } else {
@@ -1395,8 +1418,9 @@ var update_clock = function() {
                return;
        }
 
-       // If either player has ten minutes or less left, add the second counters.
-       var show_seconds = (white_clock_ms < 60 * 10 * 1000 || black_clock_ms < 60 * 10 * 1000);
+       // If either player has twenty minutes or less left, add the second counters.
+       // This matches what DGT clocks do.
+       var show_seconds = (white_clock_ms < 60 * 20 * 1000 || black_clock_ms < 60 * 20 * 1000);
 
        if (color) {
                // See when the clock will change next, and update right after that.
@@ -1452,15 +1476,15 @@ var format_2d = function(x) {
 
 /**
  * @param {string} move
- * @param {Number} move_num
- * @param {boolean} white_to_play
+ * @param {Number} move_num Move number of this move.
+ * @param {boolean} white_to_play Whether white is to play this move.
  */
 var format_move_with_number = function(move, move_num, white_to_play) {
        var ret;
        if (white_to_play) {
-               ret = (move_num - 1) + '… ';
-       } else {
                ret = move_num + '. ';
+       } else {
+               ret = move_num + '… ';
        }
        ret += move;
        return ret;
@@ -1468,7 +1492,8 @@ var format_move_with_number = function(move, move_num, white_to_play) {
 
 /**
  * @param {string} move
- * @param {Number} halfmove_num
+ * @param {Number} halfmove_num Half-move number that is to be played,
+ *   starting from 0.
  */
 var format_halfmove_with_number = function(move, halfmove_num) {
        return format_move_with_number(
@@ -1482,25 +1507,34 @@ var format_halfmove_with_number = function(move, halfmove_num) {
  * @param {Number} halfmove_num
  */
 var format_tooltip = function(data, halfmove_num) {
-       if (data['score_history'][halfmove_num] ||
-           halfmove_num === data['position']['history'].length) {
+       if (data['score_history'][halfmove_num + 1] ||
+           (halfmove_num + 1) === data['position']['history'].length) {
+               // Position is in the history, or it is the current position
+               // (which is implicitly tacked onto the history).
                var move;
                var short_score;
-               if (halfmove_num === data['position']['history'].length) {
+               if ((halfmove_num + 1) === data['position']['history'].length) {
                        move = data['position']['last_move'];
                        short_score = format_short_score(data['score']);
                } else {
                        move = data['position']['history'][halfmove_num];
-                       short_score = format_short_score(data['score_history'][halfmove_num]);
+                       short_score = format_short_score(data['score_history'][halfmove_num + 1]);
+               }
+               if (halfmove_num === -1) {
+                       return "Start position: " + short_score;
+               } else {
+                       var move_with_number = format_halfmove_with_number(move, halfmove_num);
+                       return "After " + move_with_number + ": " + short_score;
                }
-               var move_with_number = format_halfmove_with_number(move, halfmove_num);
-
-               return "After " + move_with_number + ": " + short_score;
        } else {
-               for (var i = halfmove_num; i --> 0; ) {
+               for (var i = halfmove_num; i --> -1; ) {
                        if (data['score_history'][i]) {
                                var move = data['position']['history'][i];
-                               return "[Analysis kept from " + format_halfmove_with_number(move, i) + "]";
+                               if (i === -1) {
+                                       return "[Analysis kept from start position]";
+                               } else {
+                                       return "[Analysis kept from " + format_halfmove_with_number(move, i) + "]";
+                               }
                        }
                }
        }