Fixes
[vchess.git] / client / src / components / MoveList.vue
index 30ed233..48f082f 100644 (file)
@@ -4,58 +4,6 @@ export default {
   name: "my-move-list",
   props: ["moves", "cursor", "score", "message", "firstNum"],
   render(h) {
-    if (this.moves.length == 0) return h("div");
-    let tableContent = [];
-    let moveCounter = 0;
-    let tableRow = undefined;
-    let moveCells = undefined;
-    let curCellContent = "";
-    let firstIndex = 0;
-    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 = "";
-          firstIndex = i;
-        }
-      }
-      // Next condition is fine because even if the first move is black,
-      // there will be the "..." which count as white move.
-      else if (this.moves[i].color == "b" && this.moves[i - 1].color == "w")
-        firstIndex = i;
-      curCellContent += this.moves[i].notation;
-      if (
-        i < this.moves.length - 1 &&
-        this.moves[i + 1].color == this.moves[i].color
-      )
-        curCellContent += ",";
-      //color change
-      else {
-        moveCells.push(
-          h("td", {
-            domProps: { innerHTML: curCellContent },
-            on: { click: () => this.gotoMove(i) },
-            class: {
-              "highlight-lm": this.cursor >= firstIndex && this.cursor <= i
-            }
-          })
-        );
-        curCellContent = "";
-      }
-    }
-    // Complete last row, which might not be full:
-    if (moveCells.length - 1 == 1) {
-      moveCells.push(h("td", { domProps: { innerHTML: "" } }));
-    }
-    tableRow.children = moveCells;
-    tableContent.push(tableRow);
     let rootElements = [];
     if (!!this.score && this.score != "*") {
       const scoreDiv = h(
@@ -70,37 +18,90 @@ export default {
       );
       rootElements.push(scoreDiv);
     }
-    rootElements.push(
-      h(
-        "table",
-        {
-          class: {
-            "moves-list": true
+    if (this.moves.length > 0) {
+      let tableContent = [];
+      let moveCounter = 0;
+      let tableRow = undefined;
+      let moveCells = undefined;
+      let curCellContent = "";
+      let firstIndex = 0;
+      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(
+                "div",
+                {
+                  "class": {td: true},
+                  domProps: { innerHTML: ++moveCounter + "." }
+                }
+              )
+            ];
+            tableRow = h("div", {"class": {tr: true}});
+            curCellContent = "";
+            firstIndex = i;
           }
-        },
-        tableContent
-      )
-    );
+        }
+        // Next condition is fine because even if the first move is black,
+        // there will be the "..." which count as white move.
+        else if (this.moves[i].color == "b" && this.moves[i - 1].color == "w")
+          firstIndex = i;
+        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(
+              "div",
+              {
+                "class": {
+                  td: true,
+                  "highlight-lm": this.cursor >= firstIndex && this.cursor <= i
+                },
+                domProps: { innerHTML: curCellContent },
+                on: { click: () => this.gotoMove(i) }
+              }
+            )
+          );
+          curCellContent = "";
+        }
+      }
+      // Complete last row, which might not be full:
+      if (moveCells.length - 1 == 1) {
+        moveCells.push(h("div", {"class": {td: true}}));
+      }
+      tableRow.children = moveCells;
+      tableContent.push(tableRow);
+      rootElements.push(
+        h(
+          "div",
+          {
+            class: {
+              "moves-list": true
+            }
+          },
+          tableContent
+        )
+      );
+    }
     return h("div", {}, rootElements);
   },
   watch: {
     cursor: function(newCursor) {
       if (window.innerWidth <= 767) return; //scrolling would hide chessboard
-      // Count grouped moves until the cursor (if multi-moves):
-      let groupsCount = 0;
-      let curCol = undefined;
-      for (let i = 0; i < newCursor; i++) {
-        const m = this.moves[i];
-        if (m.color != curCol) {
-          groupsCount++;
-          curCol = m.color;
-        }
-      }
       // $nextTick to wait for table > tr to be rendered
       this.$nextTick(() => {
-        let rows = document.querySelectorAll("#movesList tr");
-        if (rows.length > 0) {
-          rows[Math.floor(groupsCount / 2)].scrollIntoView({
+        let curMove = document.querySelector(".td.highlight-lm");
+        if (curMove) {
+          curMove.scrollIntoView({
             behavior: "auto",
             block: "nearest"
           });
@@ -118,67 +119,28 @@ export default {
 
 <style lang="sass" scoped>
 .moves-list
-  min-width: 250px
+  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
+.td.highlight-lm
   background-color: plum
 </style>
-
-<!-- TODO: use template function + multi-moves: much easier
-<template lang="pug">
-div
-  #scoreInfo(v-if="score!='*'")
-    p {{ score }}
-    p {{ message }}
-  table.moves-list
-    tbody
-      tr(v-for="moveIdx in evenNumbers")
-        td {{ firstNum + moveIdx / 2 + 1 }}
-        td(:class="{'highlight-lm': cursor == moveIdx}"
-            @click="() => gotoMove(moveIdx)")
-          | {{ moves[moveIdx].notation }}
-        td(v-if="moveIdx < moves.length-1"
-            :class="{'highlight-lm': cursor == moveIdx+1}"
-            @click="() => gotoMove(moveIdx+1)")
-          | {{ moves[moveIdx+1].notation }}
-        // Else: just add an empty cell
-        td(v-else)
-</template>
-
-<script>
-// Component for moves list on the right
-export default {
-  name: 'my-move-list',
-       props: ["moves","cursor","score","message","firstNum"],
-  watch: {
-    cursor: function(newValue) {
-      if (window.innerWidth <= 767)
-        return; //moves list is below: scrolling would hide chessboard
-      if (newValue < 0)
-        newValue = 0; //avoid rows[-1] => error
-      // $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({
-            behavior: "auto",
-            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);
-               },
-       },
-};
-</script>
--->