Track removal of players (by either deregister or by capture)

Currently the capture UI is rather spartan, (the captured name just
disappears—it doesn't appear as a capture next to the capturing
player).

diff --git a/empires-client.js b/empires-client.js
index 9ac3308feff2519c5bb31b03d0f2fec7c9e7f722..5f489f8780c547c6f8af2a743561a965ebdcd07c 100644 (file)
--- a/empires-client.js
+++ b/empires-client.js
@@ -40,7 +40,8 @@ events.addEventListener("players", function(event) {
 
   players_element.innerHTML = '';
   for (const player of players) {
-    var li =document.createElement('li');
+    var li = document.createElement('li');
+    li.id = "player-" + player.id;
     li.innerText = player.name;
     players_element.appendChild(li);
   }
@@ -51,6 +52,23 @@ events.addEventListener("player-register", function(event) {
   const player = JSON.parse(event.data);
 
   const li = document.createElement('li');
+  li.id = "player-" + player.id;
   li.innerText = player.name;
   players.appendChild(li);
 });
+
+events.addEventListener("player-deregister", function(event) {
+  const players = document.getElementById("players");
+  const id = JSON.parse(event.data);
+  const player = document.getElementById("player-" + id.id);
+
+  players.removeChild(player);
+});
+
+events.addEventListener("capture", function(event_string) {
+  const players = document.getElementById("players");
+  const event = JSON.parse(event_string.data);
+  const player = document.getElementById("player-" + event.captee);
+
+  players.removeChild(player);
+});