Various fixes related to history and showing moves. Also highlight the shown one.
authorSteinar H. Gunderson <sgunderson@bigfoot.com>
Mon, 10 Nov 2014 19:04:04 +0000 (20:04 +0100)
committerSteinar H. Gunderson <sgunderson@bigfoot.com>
Mon, 10 Nov 2014 19:04:04 +0000 (20:04 +0100)
www/css/remoteglot.css
www/index.html
www/js/remoteglot.js

index b0aadab..ca95682 100644 (file)
@@ -120,6 +120,10 @@ a.move {
        color: black;
        text-decoration: none;
 }
+a.move.highlight {
+       color: red;
+       font-weight: bold;
+}
 a.move:hover {
        font-weight: bold;
        text-decoration: underline;
index 5468d60..03ebcbd 100644 (file)
@@ -26,7 +26,7 @@
     |
     <span id="history">No history</span>
     |
-    <span id="linemsg">Click on any move to show it.</span>
+    <span id="linemsg">Click on any move to show it on the board.</span>
     <span id="linenav">
       <span id="prevmove"><a href="javascript:prev_move();">Previous</a></span>,
       <span id="nextmove"><a href="javascript:next_move();">Next</a></span>,
index 3189396..b42aa8e 100644 (file)
@@ -45,6 +45,9 @@ var highlight_from = undefined;
 /** @type {!string|undefined} @private */
 var highlight_to = undefined;
 
+/** @type {?jQuery} @private */
+var highlighted_move = null;
+
 /** @type {!number} @private */
 var unique = Math.random();
 
@@ -57,7 +60,8 @@ var fen = null;
 /** @typedef {{
  *    start_fen: string,
  *    uci_pv: Array.<string>,
- *    pretty_pv: Array.<string>
+ *    pretty_pv: Array.<string>,
+ *    line_num: number
  * }} DisplayLine
  */
 
@@ -386,7 +390,8 @@ var add_pv = function(fen, uci_pv, pretty_pv, move_num, toplay, opt_limit, opt_s
        display_lines.push({
                start_fen: fen,
                uci_pv: uci_pv,
-               pretty_pv: pretty_pv 
+               pretty_pv: pretty_pv,
+               line_number: display_lines.length
        });
        return print_pv(display_lines.length - 1, pretty_pv, move_num, toplay, opt_limit, opt_showlast);
 }
@@ -414,14 +419,14 @@ var print_pv = function(line_num, pretty_pv, move_num, toplay, opt_limit, opt_sh
                }
                move_num += i / 2;
        } else if (toplay == 'B') {
-               var move = "<a class=\"move\" href=\"javascript:show_line(" + line_num + ", " + 0 + ");\">" + pretty_pv[0] + "</a>";
+               var move = "<a class=\"move\" id=\"automove" + line_num + "-0\" href=\"javascript:show_line(" + line_num + ", " + 0 + ");\">" + pretty_pv[0] + "</a>";
                pv = move_num + '. … ' + move;
                toplay = 'W';
                ++i;
                ++move_num;
        }
        for ( ; i < pretty_pv.length; ++i) {
-               var move = "<a class=\"move\" href=\"javascript:show_line(" + line_num + ", " + i + ");\">" + pretty_pv[i] + "</a>";
+               var move = "<a class=\"move\" id=\"automove" + line_num + "-" + i + "\" href=\"javascript:show_line(" + line_num + ", " + i + ");\">" + pretty_pv[i] + "</a>";
 
                if (toplay == 'W') {
                        if (i > opt_limit && !opt_showlast) {
@@ -474,8 +479,8 @@ var update_refutation_lines = function() {
        if (fen === null) {
                return;
        }
-       if (display_lines.length > 1) {
-               display_lines = [ display_lines[0] ];
+       if (display_lines.length > 2) {
+               display_lines = [ display_lines[0], display_lines[1] ];
        }
 
        var tbl = $("#refutationlines");
@@ -608,7 +613,7 @@ var update_board = function(data, num_viewers) {
        if (data['position']['history']) {
                add_pv('start', data['position']['history'], data['position']['pretty_history'], 1, 'W', 8, true);
        } else {
-               displayed_lines.push(null);
+               display_lines.push(null);
        }
        update_history();
 
@@ -731,6 +736,9 @@ var next_move = function() {
 window['next_move'] = next_move;
 
 var update_displayed_line = function() {
+       if (highlighted_move !== null) {
+               highlighted_move.removeClass('highlight'); 
+       }
        if (current_display_line === null) {
                $("#linenav").hide();
                $("#linemsg").show();
@@ -770,6 +778,10 @@ var update_displayed_line = function() {
                        hiddenboard.move("a8-d8", false);
                }
        }
+
+       highlighted_move = $("#automove" + current_display_line.line_number + "-" + current_display_move);
+       highlighted_move.addClass('highlight'); 
+
        board.position(hiddenboard.position());
 }