Implemented multi-move possibility in a moves list => better support for multi-moves...
[vchess.git] / client / src / components / MoveList.vue
index 63e985f..b238758 100644 (file)
@@ -1,98 +1,87 @@
-<!-- TODO: merge Game.vue and MoveList.vue (one logic entity, no ?) -->
+<template lang="pug">
+div
+  #scoreInfo(v-if="score!='*'")
+    p {{ score }}
+    p {{ message }}
+  .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"], //TODO: other props for e.g. players names + connected indicator
-       // --> we could also add turn indicator here
-       data: function() {
-               return {
-                       something: "", //TODO?
-               };
-       },
-       // TODO: extend rendering for more than 2 colors: would be a parameter
-       // in that case some moves for some colors could be just skipped (if a player lost)
-       render(h) {
-               if (this.moves.length == 0)
-                       return;
-               const nbColors = 2;
-               // TODO: name colors "white", "black", "red", "yellow" ?
-               if (this.moves[0].color == "b")
-                       this.moves.unshift({color: "w", notation: "..."});
-               let tableContent = [];
-               let moveCounter = 0;
-               let tableRow = undefined;
-               let moveCells = undefined;
-               let curCellContent = "";
-               for (let i=0; i<this.moves.length; i++)
-               {
-                       if (this.moves[i].color == "w")
-                       {
-                               if (i == 0 || i>0 && this.moves[i-1].color=="b")
-                               {
-                                       if (!!tableRow)
-                                       {
-                                               tableRow.children = moveCells;
-                                               tableContent.push(tableRow);
-                                       }
-                                       moveCells = [
-                                               h(
-                                                       "td",
-                                                       { domProps: { innerHTML: (++moveCounter) + "." } }
-                                               )
-                                       ];
-                                       tableRow = h(
-                                               "tr",
-                                               { }
-                                       );
-                                       curCellContent = "";
-                               }
-                       }
-                       curCellContent += this.moves[i].notation;
-                       if (i < this.moves.length-1 && this.moves[i+1].color == this.moves[i].color)
-                               curCellContent += ",";
-                       else //color change
-                       {
-                               moveCells.push(
-                                       h(
-                                               "td",
-                                               {
-                                                       domProps: { innerHTML: curCellContent },
-                                                       on: { click: () => this.gotoMove(i) },
-                                                       "class": { "highlight-lm": this.cursor == i },
-                                               }
-                                       )
-                               );
-                               curCellContent = "";
-                       }
-               }
-               // Complete last row, which might not be full:
-               if (moveCells.length-1 < nbColors)
-               {
-                       const delta = nbColors - (moveCells.length-1);
-                       for (let i=0; i<delta; i++)
-                       {
-                               moveCells.push(
-                                       h(
-                                               "td",
-                                               { domProps: { innerHTML: "" } }
-                                       )
-                               );
-                       }
-               }
-               tableRow.children = moveCells;
-               tableContent.push(tableRow);
-               const movesTable = h(
-                       "table",
-                       { },
-                       tableContent
-               );
-               return movesTable;
-       },
-       methods: {
-               gotoMove: function(index) {
-                       this.$emit("goto-move", index);
-               },
-       },
+  name: "my-move-list",
+  props: ["moves", "show", "cursor", "score", "message", "firstNum"],
+  watch: {
+    cursor: function(newCursor) {
+      if (window.innerWidth <= 767) return; //scrolling would hide chessboard
+      // $nextTick to wait for table > tr to be rendered
+      this.$nextTick(() => {
+        let curMove = document.querySelector(".td.highlight-lm");
+        if (curMove) {
+          curMove.scrollIntoView({
+            behavior: "auto",
+            block: "nearest"
+          });
+        }
+      });
+    }
+  },
+  computed: {
+    evenNumbers: function() {
+      return [...Array(this.moves.length).keys()].filter(i => i%2==0);
+    }
+  },
+  methods: {
+    notation: function(move) {
+      return getFullNotation(move);
+    },
+    gotoMove: function(index) {
+      this.$emit("goto-move", index);
+    }
+  }
 };
 </script>
+
+<style lang="sass" scoped>
+.moves-list
+  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>