From a608e90f7ad5f66a34050ad855db08b1d3365922 Mon Sep 17 00:00:00 2001
From: Michel Wilhelm <michelwilhelm@gmail.com>
Date: Sat, 7 Nov 2020 20:00:16 -0300
Subject: [PATCH] Visual adjustments

---
 src/App.js                        |  3 ++
 src/assets/scss/styles.scss       | 19 ++++++---
 src/components/Board/Board.js     |  2 -
 src/components/Board/styles.scss  | 14 +------
 src/components/Header/Header.js   | 66 +++++++++++++++++++------------
 src/components/Header/styles.scss | 56 ++++++++++++++++++++++----
 6 files changed, 107 insertions(+), 53 deletions(-)

diff --git a/src/App.js b/src/App.js
index 6659750..4509176 100644
--- a/src/App.js
+++ b/src/App.js
@@ -123,6 +123,7 @@ function App() {
     gameClient.createNewGame(rows, cols, mines).then(response => {
       setActiveGame(response);
       setFlags(mines);
+      document.getElementsByClassName('reset')[0].innerHTML = '🙂';
       generateDummyBoard();
     });
   };
@@ -133,6 +134,7 @@ function App() {
 
   return (
     <div className="app">
+      <div className="app-col-spacer"> </div>
       <div className="canvas">
         <Header
           cols={cols}
@@ -154,6 +156,7 @@ function App() {
           rows={rows}
         />
       </div>
+      <div className="app-col-spacer"> </div>
     </div>
   );
 }
diff --git a/src/assets/scss/styles.scss b/src/assets/scss/styles.scss
index 1a0fe73..a154c05 100644
--- a/src/assets/scss/styles.scss
+++ b/src/assets/scss/styles.scss
@@ -8,15 +8,24 @@ body {
   background-color: $gray-mid;
 }
 
+.app {
+  display: flex;
+
+  &-col-spacer {
+    flex: 1 1 auto;
+  }
+}
+
 .canvas {
-  max-width: 1000px;
-  width: 100%;
-  min-width: 500px;
-  margin: 0 auto;
-  border-radius: 3px;
   display: flex;
   justify-content: center;
   flex-direction: column;
+
+  flex: 0 0 auto;
+  border-width: 3px;
+  border-style: solid;
+  border-color: darken($gray, 20%) lighten($gray, 20%) lighten($gray, 20%)
+    darken($gray, 20%);
 }
 
 .box {
diff --git a/src/components/Board/Board.js b/src/components/Board/Board.js
index 31cca5c..a340702 100644
--- a/src/components/Board/Board.js
+++ b/src/components/Board/Board.js
@@ -6,7 +6,6 @@ import './styles.scss';
 const Board = ({ dummyBoard, handleCellClick, handleCellContextMenu }) => {
   return (
     <div className="board">
-      <div className="board-col-left"> </div>
       <div className="board-col-center">
         {dummyBoard.map((cols, rowId) => (
           <div className="board-row" key={`row_${rowId}`}>
@@ -25,7 +24,6 @@ const Board = ({ dummyBoard, handleCellClick, handleCellContextMenu }) => {
           </div>
         ))}
       </div>
-      <div className="board-col-right"> </div>
     </div>
   );
 };
diff --git a/src/components/Board/styles.scss b/src/components/Board/styles.scss
index 51e042a..4248bd4 100644
--- a/src/components/Board/styles.scss
+++ b/src/components/Board/styles.scss
@@ -2,19 +2,7 @@
 
 .board {
   display: flex;
-
-  &-col-left,
-  &-col-right {
-    flex: 1 1 auto;
-  }
-
-  &-col-center {
-    flex: 0 0 auto;
-    border-width: 3px;
-    border-style: solid;
-    border-color: darken($gray, 20%) lighten($gray, 20%) lighten($gray, 20%)
-      darken($gray, 20%);
-  }
+  justify-content: center;
 
   &-row {
     display: flex;
diff --git a/src/components/Header/Header.js b/src/components/Header/Header.js
index 691889f..424bd73 100644
--- a/src/components/Header/Header.js
+++ b/src/components/Header/Header.js
@@ -17,32 +17,46 @@ const Header = ({
   return (
     <div className="header">
       <div className="settings">
-        <input
-          className="box"
-          onChange={event => setRows(parseInt(event.target.value))}
-          type="number"
-          value={rows}
-        />
-        <input
-          className="box"
-          onChange={event => setCols(parseInt(event.target.value))}
-          type="number"
-          value={cols}
-        />
-        <input
-          className="box"
-          onChange={event => handleSetMines(event)}
-          type="number"
-          value={mines}
-        />
-        <button
-          className="button box"
-          onClick={() => setUpdateDummyBoard(true)}>
-          Apply
-        </button>
+        <div className="form-control">
+          <label for="rows">Rows:</label>
+          <input
+            className="box"
+            onChange={event => setRows(parseInt(event.target.value))}
+            type="number"
+            value={rows}
+            id="rows"
+          />
+        </div>
+        <div className="form-control">
+          <label for="cols">Columns:</label>
+          <input
+            className="box"
+            onChange={event => setCols(parseInt(event.target.value))}
+            type="number"
+            value={cols}
+            id="cols"
+          />
+        </div>
+        <div className="form-control">
+          <label for="mines">Mines:</label>
+          <input
+            className="box"
+            onChange={event => handleSetMines(event)}
+            type="number"
+            value={mines}
+            id="mines"
+          />
+        </div>
+        <div className="form-control">
+          <button
+            className="button box"
+            onClick={() => setUpdateDummyBoard(true)}>
+            Apply
+          </button>
+        </div>
       </div>
-      <div className="canvas">
-        <div className="flags led-panel box">{flags}</div>
+      <div className="header-actions">
+        <div className="flags led-panel">{flags}</div>
         <div
           className="reset box"
           onMouseDown={event => {
@@ -54,7 +68,7 @@ const Header = ({
           }}>
           🙂
         </div>
-        <div className="timer led-panel box">000</div>
+        <div className="timer led-panel">000</div>
       </div>
     </div>
   );
diff --git a/src/components/Header/styles.scss b/src/components/Header/styles.scss
index a23d641..ba45076 100644
--- a/src/components/Header/styles.scss
+++ b/src/components/Header/styles.scss
@@ -1,19 +1,50 @@
 @import '../../assets/scss/variables';
 
 .header {
-  max-width: 500px;
+  max-width: 400px;
   width: 100%;
-  padding: 5px;
   margin: 0 auto;
   display: flex;
   justify-content: space-between;
   flex-direction: column;
 
+  &-col-left,
+  &-col-right {
+    flex: 1 1 auto;
+  }
+
+  &-col-center {
+    flex: 0 0 auto;
+    border-width: 3px;
+    border-style: solid;
+    border-color: darken($gray, 20%) lighten($gray, 20%) lighten($gray, 20%)
+      darken($gray, 20%);
+  }
+
   .settings {
-    max-width: 221px;
     display: flex;
     justify-content: center;
-    margin: 10px auto;
+    border-width: 3px;
+    border-style: solid;
+    border-color: lighten($gray, 20%) darken($gray, 20%) darken($gray, 20%)
+      lighten($gray, 20%);
+    padding-bottom: 5px;
+
+    .form-control {
+      display: flex;
+      flex-direction: column;
+
+      &:last-child {
+        align-items: flex-end;
+        justify-content: flex-end;
+      }
+
+      label {
+        font-size: 75%;
+        text-transform: uppercase;
+        padding: 5px;
+      }
+    }
 
     input {
       width: 40px;
@@ -22,18 +53,28 @@
       text-align: center;
       background: $gray-light;
       margin: 0 2px;
+      border-color: darken($gray, 20%) lighten($gray, 20%) lighten($gray, 20%)
+        darken($gray, 20%);
+      font-size: 120%;
     }
 
     .button {
       cursor: pointer;
       margin: 0 2px;
+      height: 38px;
+      border-color: lighten($gray, 20%) darken($gray, 20%) darken($gray, 20%)
+        lighten($gray, 20%);
     }
   }
 
-  .canvas {
+  &-actions {
     display: flex;
     justify-content: space-between;
     flex-direction: row;
+    border-width: 3px;
+    border-style: solid;
+    border-color: lighten($gray, 20%) darken($gray, 20%) darken($gray, 20%)
+      lighten($gray, 20%);
 
     .led-panel {
       font-family: 'Space Mono', monospace;
@@ -47,10 +88,11 @@
     }
 
     .reset {
-      width: 30px;
-      line-height: 30px;
+      width: 24px;
+      line-height: 24px;
       text-align: center;
       cursor: pointer;
+      margin: 3px;
     }
   }
 }