In web client index selection, disallow non-number replies.

[plomrogue2] / rogue_chat.html

diff --git a/rogue_chat.html b/rogue_chat.html
index 3f281579afa8e5dcf0edcaedcc00aad23994303d..d810c2e580aabb3f4cc1443d8ff040b4f657374a 100644 (file)
--- a/rogue_chat.html
+++ b/rogue_chat.html
@@ -1045,7 +1045,7 @@ let tui = {
   },
   pick_selectable: function(task_name) {
       const i = parseInt(this.inputEl.value);
-      if (isNaN(i) || i < 0 || i >= this.selectables.length) {
+      if (isNaN(this.inputEl.value) || i < 0 || i >= this.selectables.length) {
           tui.log_msg('? invalid index, aborted');
       } else {
           server.send(['TASK:' + task_name, tui.selectables[i]]);
@@ -1801,15 +1801,17 @@ for (const move_button of document.querySelectorAll('[id*="_move_"]')) {
     };
     let direction = move_button.id.split('_')[2].toUpperCase();
     let move_repeat;
+    function move() {
+        if (tui.mode.available_actions.includes("move")) {
+            server.send(['TASK:MOVE', direction]);
+        } else if (tui.mode.available_actions.includes("move_explorer")) {
+            explorer.move(direction);
+            tui.full_refresh();
+        };
+    }
     move_button.onmousedown = function() {
-        move_repeat = window.setInterval(function() {
-            if (tui.mode.available_actions.includes("move")) {
-                server.send(['TASK:MOVE', direction]);
-            } else if (tui.mode.available_actions.includes("move_explorer")) {
-                explorer.move(direction);
-                tui.full_refresh();
-            };
-        }, 100);
+        move();
+        move_repeat = window.setInterval(move, 100);
     };
     move_button.onmouseup = function() {
         window.clearInterval(move_repeat);