From: Steinar H. Gunderson <sgunderson@bigfoot.com>
Date: Mon, 10 Nov 2014 19:04:04 +0000 (+0100)
Subject: Various fixes related to history and showing moves. Also highlight the shown one.
X-Git-Url: https://git.sesse.net/?p=remoteglot;a=commitdiff_plain;h=aacc612ada3aa00ac9db0bbf785ea7b464deb344

Various fixes related to history and showing moves. Also highlight the shown one.
---

diff --git a/www/css/remoteglot.css b/www/css/remoteglot.css
index b0aadab..ca95682 100644
--- a/www/css/remoteglot.css
+++ b/www/css/remoteglot.css
@@ -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;
diff --git a/www/index.html b/www/index.html
index 5468d60..03ebcbd 100644
--- a/www/index.html
+++ b/www/index.html
@@ -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>,
diff --git a/www/js/remoteglot.js b/www/js/remoteglot.js
index 3189396..b42aa8e 100644
--- a/www/js/remoteglot.js
+++ b/www/js/remoteglot.js
@@ -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());
 }