From 601e9a3292604da38e54be5231b1e198fc5d3130 Mon Sep 17 00:00:00 2001
From: Carl Worth <cworth@cworth.org>
Date: Wed, 7 Dec 2022 23:00:15 -0800
Subject: [PATCH] Add dialog for setting the user's name

And force it to be shown when clicking the comment field, (if the name
has not been set previously).
---
 index.html | 37 +++++++++++++++++++++++++++++++++----
 index.js   | 14 ++++++++------
 2 files changed, 41 insertions(+), 10 deletions(-)

diff --git a/index.html b/index.html
index ad53d86..a54a090 100644
--- a/index.html
+++ b/index.html
@@ -28,6 +28,20 @@
       all. The only limit is yourself!
     </p>
 
+    <dialog id="name-dialog">
+      <form method="dialog">
+        <p>
+          <label>Your name:
+            <input id="name" type="text">
+          </label>
+        </p>
+        <div>
+          <button value="cancel">Cancel</button>
+          <button id="confirm" value="default">Confirm</button>
+        </div>
+      </form>
+    </dialog>
+
     <p>
       <div id="spinner" align="center">
         <div class="animate-flicker">
@@ -102,21 +116,32 @@ mute.addEventListener("click", () => {
   <script>
     var socket = io();
 
+    const name = document.querySelector("#name");
+    const name_dialog = document.querySelector("#name-dialog");
     const images = document.querySelector("#images");
-
     const comments = document.querySelector("#comments");
     const comment_form = document.querySelector("#comment-form");
     const comment = document.querySelector("#comment");
-
     const zombo_form = document.querySelector("#zombo-form");
     const prompt = document.querySelector("#prompt");
     const code = document.querySelector("#code");
-
     const safety= document.querySelector("#safety");
-
     const spinner = document.querySelector("#spinner");
     var spinner_timeout;
 
+    comment.addEventListener('focus', () => {
+        /* Do nothing if name is already set. */
+        if (name.value)
+            return;
+
+        /* Otherwise, bring up the modal dialog to set the name. */
+        name_dialog.showModal();
+    });
+
+    name_dialog.addEventListener('close', () => {
+        socket.emit('set-name', name.value);
+    });
+
     comment_form.addEventListener('submit', function(e) {
         e.preventDefault();
         if (comment.value) {
@@ -131,6 +156,10 @@ mute.addEventListener("click", () => {
         comments.appendChild(item);
     });
 
+    socket.on('inform-name', (name) => {
+        name.value = name;
+    });
+
     socket.on('reset', () => {
         images.replaceChildren();
     });
diff --git a/index.js b/index.js
index 9d4028f..2c980dc 100644
--- a/index.js
+++ b/index.js
@@ -65,17 +65,15 @@ process.on('SIGINT', () => {
 });
 
 app.get('/index.html', (req, res) => {
-    if (req.session.views) {
-        req.session.views++;
-    } else {
-        req.session.views = 1;
-    }
     res.sendFile(__dirname + '/index.html');
 });
 
 io.on('connection', (socket) => {
 
-    console.log("Connection from client with " + socket.request.session.views + " views.");
+    // First things first, tell the client their name (if any)
+    if (socket.request.session.name) {
+        socket.emit('inform-name', socket.request.session.name);
+    }
 
     // Replay old comments and images to a newly-joining client
     socket.emit('reset');
@@ -86,6 +84,10 @@ io.on('connection', (socket) => {
         socket.emit('image', image)
     });
 
+    socket.on('set-name', (name) => {
+        socket.request.session.name = name;
+    });
+
     // When any client comments, send that to all clients (including sender)
     socket.on('comment', (comment) => {
         io.emit('comment', comment);
-- 
2.43.0