Started code review + some fixes (unfinished)

[vchess.git] / client / src / App.vue

diff --git a/client/src/App.vue b/client/src/App.vue
index 6183d66..9879e27 100644 (file)
--- a/client/src/App.vue
+++ b/client/src/App.vue
@@ -18,12 +18,12 @@
             #leftMenu
               router-link(to="/")
                 | {{ st.tr["Hall"] }}
+              router-link(to="/mygames")
+                | {{ st.tr["My games"] }}
               router-link(to="/variants")
                 | {{ st.tr["Variants"] }}
               router-link(to="/problems")
                 | {{ st.tr["Problems"] }}
-              router-link(to="/mygames")
-                | {{ st.tr["My games"] }}
             #rightMenu
               .clickable(onClick="doClick('modalUser')")
                 | {{ st.user.id > 0 ? (st.user.name || "@nonymous") : "Login" }}
@@ -53,17 +53,17 @@ export default {
     ContactForm,
     Language,
     Settings,
-    UpsertUser,
+    UpsertUser
   },
   data: function() {
     return {
-      st: store.state,
+      st: store.state
     };
   },
   computed: {
     flagImage: function() {
       return `/images/flags/${this.st.lang}.svg`;
-    },
+    }
   },
   mounted: function() {
     let dialogs = document.querySelectorAll("div[role='dialog']");
@@ -73,12 +73,11 @@ export default {
   },
   methods: {
     hideDrawer: function(e) {
-      if (e.target.innerText == "Forum")
-        return; //external link
+      if (e.target.innerText == "Forum") return; //external link
       e.preventDefault(); //TODO: why is this needed?
       document.getElementsByClassName("drawer")[0].checked = false;
-    },
-  },
+    }
+  }
 };
 </script>
 
@@ -103,8 +102,8 @@ body
   // 45px is footer height
   min-height: calc(100vh - 45px)
   overflow: hidden
-  @media screen and (max-width: 767px)
-    padding: 0
+  padding: 0
+  margin: 0
 
 .row > div
   padding: 0
@@ -115,9 +114,6 @@ header
   align-items: center
   justify-content: center
   margin: 0 auto
-  & > img
-    width: 30px
-    height: 30px
 
 .clickable
   cursor: pointer
@@ -128,6 +124,29 @@ header
 .clearer
   clear: both
 
+.button-group
+  margin: 0
+
+input[type="checkbox"]:focus
+  outline: 0
+
+input[type=checkbox]:checked:before
+  top: -5px;
+  height: 18px
+
+table
+  display: block
+  padding: 0
+  tr > td
+    cursor: pointer
+  th, td
+    padding: 5px
+
+@media screen and (max-width: 767px)
+  table
+    tr > th, td
+      font-size: 14px
+
 nav
   width: 100%
   margin: 0
@@ -163,6 +182,7 @@ nav
             height: 27px
     @media screen and (max-width: 767px)
       & > #leftMenu
+        margin-top: 42px
         padding-bottom: 5px
         & > a
           color: #2c3e50
@@ -181,17 +201,28 @@ nav
 
 @media screen and (max-width: 767px)
   nav
-    height: 32px
+    height: 42px
     border: none
     & > label.drawer-toggle
-      font-size: 1.2rem
+      cursor: pointer
+      font-size: 32px
       position: absolute
-      top: -12px
+      top: -22px
       //padding: -5px 0 0 10px
+    & > #menuBar
+      z-index: 5000 //to hide currently selected piece if any
 
 [type="checkbox"].drawer+*
   right: -767px
 
+[type=checkbox].drawer+* .drawer-close
+  top: -10px
+  left: var(--universal-margin)
+  right: 0
+
+[type=checkbox].drawer+* .drawer-close:before
+  font-size: 50px
+
 @media screen and (max-width: 767px)
   .button-group
     flex-direction: row