Implemented multi-move possibility in a moves list => better support for multi-moves...

[vchess.git] / client / src / components / MoveList.vue

diff --git a/client/src/components/MoveList.vue b/client/src/components/MoveList.vue
index c63f2cd..b238758 100644 (file)
--- a/client/src/components/MoveList.vue
+++ b/client/src/components/MoveList.vue
@@ -3,59 +3,85 @@ div
   #scoreInfo(v-if="score!='*'")
     p {{ score }}
     p {{ message }}
-  table#movesList
-    tbody
-      tr(v-for="moveIdx in evenNumbers")
-        td {{ firstNum + moveIdx / 2 + 1 }}
-        td(:class="{'highlight-lm': cursor == moveIdx}"
-            data-label="White move" @click="() => gotoMove(moveIdx)")
-          | {{ moves[moveIdx].notation }}
-        td(v-if="moveIdx < moves.length-1"
-            :class="{'highlight-lm': cursor == moveIdx+1}"
-            data-label="Black move" @click="() => gotoMove(moveIdx+1)")
-          | {{ moves[moveIdx+1].notation }}
-        // Else: just add an empty cell
-        td(v-else)
+  .moves-list
+    .tr(v-for="moveIdx in evenNumbers")
+      .td {{ firstNum + moveIdx / 2 + 1 }}
+      .td(v-if="moveIdx < moves.length-1 || show == 'all'"
+        :class="{'highlight-lm': cursor == moveIdx}"
+        @click="() => gotoMove(moveIdx)"
+      )
+        | {{ notation(moves[moveIdx]) }}
+      .td(
+        v-if="moveIdx < moves.length-1"
+        :class="{'highlight-lm': cursor == moveIdx+1}"
+        @click="() => gotoMove(moveIdx+1)"
+      )
+        | {{ notation(moves[moveIdx+1]) }}
+      // Else: just add an empty cell
+      //.td(v-else)
 </template>
 
 <script>
-// Component for moves list on the right
+import { store } from "@/store";
+import { getFullNotation } from "@/utils/notation";
 export default {
-  name: 'my-move-list',
-       props: ["moves","cursor","score","message","firstNum"],
+  name: "my-move-list",
+  props: ["moves", "show", "cursor", "score", "message", "firstNum"],
   watch: {
-    cursor: function(newValue) {
-      if (newValue < 0)
-        newValue = 0; //avoid rows[-1] --> error
+    cursor: function(newCursor) {
+      if (window.innerWidth <= 767) return; //scrolling would hide chessboard
       // $nextTick to wait for table > tr to be rendered
-      this.$nextTick( () => {
-        let rows = document.querySelectorAll('#movesList tr');
-        if (rows.length > 0)
-        {
-          rows[Math.floor(newValue/2)].scrollIntoView({
+      this.$nextTick(() => {
+        let curMove = document.querySelector(".td.highlight-lm");
+        if (curMove) {
+          curMove.scrollIntoView({
             behavior: "auto",
-            block: "nearest",
+            block: "nearest"
           });
         }
       });
-    },
+    }
   },
   computed: {
     evenNumbers: function() {
       return [...Array(this.moves.length).keys()].filter(i => i%2==0);
-    },
+    }
   },
   methods: {
-               gotoMove: function(index) {
-                       this.$emit("goto-move", index);
-               },
-       },
+    notation: function(move) {
+      return getFullNotation(move);
+    },
+    gotoMove: function(index) {
+      this.$emit("goto-move", index);
+    }
+  }
 };
 </script>
 
 <style lang="sass" scoped>
 .moves-list
-  min-width: 250px
-td.highlight-lm
+  cursor: pointer
+  min-height: 1px
+  max-height: 500px
+  overflow: auto
+  background-color: white
+  width: 280px
+  & > .tr
+    clear: both
+    border-bottom: 1px solid lightgrey
+    & > .td
+      float: left
+      padding: 2% 0 2% 1%
+      &:first-child
+        color: grey
+        width: 15%
+      &:not(first-child)
+        width: 41%
+
+@media screen and (max-width: 767px)
+  .moves-list
+    width: 100%
+
+.td.highlight-lm
   background-color: plum
 </style>