front: js readability

webapp/src/App.js

@@ -2,7 +2,7 @@ import './App.css';
 import React, { useState, useEffect, useCallback } from 'react';
 import {
   BrowserRouter,
-  Routes, //replaces "Switch" used till v5
+  Routes,
   Route,
 } from "react-router-dom";
 
@@ -15,18 +15,13 @@ function App() {
   const [polling, setPolling] = useState(false);
 
   const pollGames = useCallback(() => {     
-    console.log('start polling..');
+    if (polling)
-    
-    if (polling) {
-      console.log(' ..already in progress');
       return;
-    }
 
     setPolling(true);
     fetch('/api/gamestate')
       .then(response => response.json())
       .then(data => {
-        console.log('poooled');
         setGames(data);
         setPolling(false);
       })
@@ -34,24 +29,21 @@ function App() {
   }, [polling]);
 
   useEffect(() => {
-    const timer = setInterval(pollGames(), 35 * 1000);
+    const timer = setInterval(pollGames(), 35 * 1000); // <<-- poll new gamestates every 35 sec
     return clearInterval(timer);
   }, [pollGames])
 
-  return (
+  return <div className="App">
-    <div className="App">
+    <BrowserRouter>
-      <BrowserRouter>
+      <Header/>
-        <Header/>
+      <div className="Container">
-        <div className="Container">
+        <Routes>
-          <Routes>
+          <Route path="/leaderboard" element={<Leaderboard/>} />
-            <Route path="/leaderboard" element={<Leaderboard/>} />
+          <Route path="/gameproposal" element={<GameProposal games={games}/>} />
-            <Route path="/gameproposal" element={<GameProposal games={games}/>} />
+        </Routes>
-          </Routes>
+      </div>
-        </div>
+    </BrowserRouter>
-
-      </BrowserRouter>
     </div>
-  );
 }
 
 export default App;

webapp/src/GameProposal.js

@@ -16,39 +16,37 @@ const GameProposal = ({games}) => {
    
   const waitForYou = games
     .filter(game => game.status === State.WaitForYou)
-    .map(game => { return (
+    .map(game => { 
-      <div class="li" key={game.uuid}>
+      return <div class="li" key={game.uuid}>
         <p>
           from {game.opponentName}, opponentColor {game.opponentColor}
           <br/>
           <q>{game.message}</q>
         </p>
-      </div>
+        </div>
-    )});
+    });
 
   const WaitForOpponent = games
     .filter(game => game.status === State.WaitForOpponent)
-    .map(game => { return (
+    .map(game => { 
-      <div class="li" key={game.uuid}>
+      return <div class="li" key={game.uuid}>
         <p> 
           to {game.opponentName}, opponentColor ⛀ ⛂{game.opponentColor}
           <br/>
           <q>{game.message}</q>
         </p>
-      </div>
-    )});
-
-  return (
-    <p className="GameProposal">
-      {waitForYou}
-      {WaitForOpponent.length > 0 && 
-        <div class="separator">
-          waiting for opponent ({WaitForOpponent.length})
         </div>
-      }
+    });
-      {WaitForOpponent}
+
-    </p>
+  return <div className="GameProposal">
-  );
+    {waitForYou}
+    {WaitForOpponent.length > 0 && 
+      <div class="separator">
+        waiting for opponent ({WaitForOpponent.length})
+      </div>
+    }
+    {WaitForOpponent}
+    </div>
 };
 
 export default GameProposal;

webapp/src/Leaderboard.js

@@ -24,40 +24,36 @@ const Leaderboard = () => {
 //
 //    return <li key={playerName}>
 //      {playerName}: played {rank.gamesPlayed}, won {rank.gamesWon}, draw {rank.gamesDraw}
-//    </li>
+//      </li>
 //  });
 //  return <ul>{listItems}</ul>;
 
   const tableRows = Object.keys(data).map(playerName => {
     var rank = data[playerName];
 
-    return (
+    return <tr key={playerName}>
-      <tr key={playerName}>
+      <td>{playerName}</td>
-        <td>{playerName}</td>
+      <td>{rank.gamesPlayed}</td>
-        <td>{rank.gamesPlayed}</td>
+      <td>{rank.gamesWon}</td>
-        <td>{rank.gamesWon}</td>
+      <td>{rank.gamesDraw}</td>
-        <td>{rank.gamesDraw}</td>
       </tr>
-    );
   });
 
-  return (
+  return <div className="Leaderboard">
-    <div className="Leaderboard">
+    <table>
-      <table>
+      <thead>
-        <thead>
+        <tr>
-          <tr>
+          <th></th>
-            <th></th>
+          <th>Played</th>
-            <th>Played</th>
+          <th>Won</th>
-            <th>Won</th>
+          <th>Draw</th>
-            <th>Draw</th>
+        </tr>
-          </tr>
+      </thead>
-        </thead>
+      <tbody>
-        <tbody>
+        { tableRows }
-          { tableRows }
+      </tbody>
-        </tbody>
+    </table>
-      </table>
     </div>
-  );
 };
 
 export default Leaderboard;