From aba0a4b7dac7b517684f73e92be27a298c1fab60 Mon Sep 17 00:00:00 2001
From: "John \"Elwin\" Edwards" <elwin@rlgallery.org>
Date: Sat, 9 Jun 2012 17:00:25 -0700
Subject: [PATCH] rlgterm.js: make the options table functional.

Enable launching games from the options table, and replace the old
select dialog.
---
 index-rlg.html | 17 +----------------
 rlgterm.js     | 22 +++++++++++++++++++---
 2 files changed, 20 insertions(+), 19 deletions(-)

diff --git a/index-rlg.html b/index-rlg.html
index 1acd296..24f3108 100644
--- a/index-rlg.html
+++ b/index-rlg.html
@@ -86,22 +86,7 @@
 </div>
 </div>
 <div class="modal" id="startgame">
-<form id="startform" action="/play" method="post">
-<div>
-Choose game: <select name="game" id="input_game">
-  <option label="Rogue V3" value="rogue3">Rogue V3</option>
-  <option label="Rogue V4" value="rogue4">Rogue V4</option>
-  <option label="Rogue V5" value="rogue5">Rogue V5</option>
-  <option label="Super-Rogue" value="srogue">Super-Rogue</option>
-</select>
-</div>
-<div>
-<input type="submit" value="Play" onclick="startgame(event)">
-</div>
-</form>
-<div id="opttable">
-  <div><div>Game</div><div>Actions</div></div>
-</div>
+<div id="opttable"><div><div>Game</div><div>Actions</div></div></div>
 </div>
 <div class="modal" id="login">
 <form id="loginform" action="/login" method="post">
diff --git a/rlgterm.js b/rlgterm.js
index 425cd99..95174c8 100644
--- a/rlgterm.js
+++ b/rlgterm.js
@@ -440,6 +440,9 @@ function getchoices() {
       var button = document.createElement("span");
       button.appendChild(document.createTextNode(acttext));
       button.className = "ibutton";
+      if ("s0".indexOf(reply.stat[gname]) >= 0) {
+        button.onclick = makeStarter(gname);
+      }
       var actdiv = document.createElement("div");
       actdiv.appendChild(button);
       var gamediv = document.createElement("div");
@@ -455,13 +458,26 @@ function getchoices() {
   return;
 }
 
-function startgame(ev) {
-  ev.preventDefault();
+/* This can't be in the loop in getchoices(), or the closure's scope will
+ * get overwritten on the next iteration, and then all the games end up 
+ * being Super-Rogue.
+ */
+function makeStarter(gname) {
+  if (!(gname in games))
+    return null;
+  var game = games[gname];
+  function starter(ev) {
+    startgame(game);
+  }
+  return starter;
+}
+
+function startgame(game) {
   if (termemu.sessid != null || !lcred)
     return;
   var smsg = {};
   smsg["key"] = lcred;
-  smsg["game"] = document.getElementById("input_game").value;
+  smsg["game"] = game.uname;
   smsg["h"] = 24;
   smsg["w"] = 80;
   var req = new XMLHttpRequest();