Browse Source

Move some README notes into TODOs

Kirk Trombley 4 years ago
parent
commit
73d3682cf4

+ 0 - 3
README.md

@@ -149,9 +149,6 @@ None currently! Submit ideas!
 
 #### UI/UX
 
-- Clean up rankings screen
-    - Show countries of guesses/results (maybe with flags)
-- Auto-zoom guess map to country in a country-specific game
 - Rotation in Frozen mode
 - Improved alert of cut off in Race mode
 - Improve error handling in UI

+ 1 - 0
client/src/components/screens/GamePanel/GuessPane/ClickMarkerMap.jsx

@@ -7,6 +7,7 @@ const ClickMarkerMap = ({ onMarkerMoved }) => {
   const mapDivRef = useRef(null);
   const mapRef = useMap(mapDivRef);
   useClickMarker(mapRef, onMarkerMoved);
+  // TODO allow setting of map bounds for country locked games
   return <div className={styles.map} ref={mapDivRef} />;
 };
 

+ 2 - 0
client/src/components/screens/GameSummary/ScoreBoard/ScoreBoard.jsx

@@ -1,5 +1,7 @@
 import styles from './ScoreBoard.module.css';
 
+// TODO rewrite this to be cleaner, include flags for each round and guess
+
 const PlayerScoreTile = ({ name, guesses, totalScore }) => (
   <div className={styles.tile}>
     <span className={styles.name}>{name}</span>

+ 1 - 0
client/src/components/util/GameCreationForm/GameCreationForm.jsx

@@ -89,6 +89,7 @@ const GameCreationForm = ({ afterCreate }) => {
             <Item value={5}>5 Rounds</Item>
             <Item value={10}>10 Rounds</Item>
           </Dropdown>
+          {/* TODO base this dynamically on available generators */}
           <Dropdown selected={onlyAmerica} onSelect={setOnlyAmerica} open='america'>
             <Item value={false} display='🌎'>All Countries</Item>
             <Item value={true} display='🇺🇸'>Just America</Item>