From f9c9dec19b3ee6e272683301d49c3add6fb176f8 Mon Sep 17 00:00:00 2001
From: Carl Worth <cworth@cworth.org>
Date: Sun, 28 Jun 2020 09:08:06 -0700
Subject: [PATCH] Add a containing div element to the "still waiting" component
 chunks

And drop the use of a list.

The list was annoying because it was triggering the React warning:

	Each child in a list should have a unique "key" prop

And in this case, it seemed really silly to have to name/distinguish
the <p> element from the following <ul> element.

And this new <div> does give us a logical place to treat this chunk of
the document as a single element if needed, so this is the logically
correct thing to do.
---
 empathy/empathy.jsx | 84 ++++++++++++++++++++++++---------------------
 1 file changed, 44 insertions(+), 40 deletions(-)

diff --git a/empathy/empathy.jsx b/empathy/empathy.jsx
index e887b1b..163b977 100644
--- a/empathy/empathy.jsx
+++ b/empathy/empathy.jsx
@@ -572,26 +572,28 @@ class Ambiguities extends React.PureComponent {
     let still_waiting = null;
     const judging_players = Object.keys(this.props.players_judging);
     if (judging_players.length) {
-      still_waiting = [
-        <p>
-          Still waiting for the following player
-          {judging_players.length > 1 ? 's' : '' }
-          :
-        </p>,
-        <ul>
-          {judging_players.map(player => {
-            return (
-              <li
-                key={player}
-              >
-                {player}
-                {this.props.players_judging[player] ?
-                 <span className="typing"/> : null }
-              </li>
-            );
-          })}
-        </ul>
-      ];
+      still_waiting = (
+        <div>
+          <p>
+            Still waiting for the following player
+            {judging_players.length > 1 ? 's' : '' }
+            :
+          </p>
+          <ul>
+            {judging_players.map(player => {
+              return (
+                <li
+                  key={player}
+                >
+                  {player}
+                  {this.props.players_judging[player] ?
+                   <span className="typing"/> : null }
+                </li>
+              );
+            })}
+          </ul>
+        </div>
+      );
     }
 
     if (this.props.players_judged.has(this.props.player.name)) {
@@ -745,26 +747,28 @@ class ActivePrompt extends React.PureComponent {
     let still_waiting = null;
     const answering_players = Object.keys(this.props.players_answering);;
     if (answering_players.length) {
-      still_waiting = [
-        <p>
-          Still waiting for the following player
-          {answering_players.length > 1 ? 's' : ''}
-          :
-        </p>,
-        <ul>
-           {answering_players.map(player => {
-             return (
-               <li
-                 key={player}
-               >
-                 {player}
-                 {this.props.players_answering[player] ?
-                  <span className="typing"/> : null }
-               </li>
-             );
-           })}
-        </ul>
-      ];
+      still_waiting = (
+        <div>
+          <p>
+            Still waiting for the following player
+            {answering_players.length > 1 ? 's' : ''}
+            :
+          </p>
+          <ul>
+            {answering_players.map(player => {
+              return (
+                <li
+                  key={player}
+                >
+                  {player}
+                  {this.props.players_answering[player] ?
+                   <span className="typing"/> : null }
+                </li>
+              );
+            })}
+          </ul>
+        </div>
+      );
     }
 
     if (this.props.players_answered.has(this.props.player.name)) {
-- 
2.43.0