TODO: fix draw logic

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

diff --git a/client/src/components/MoveList.vue b/client/src/components/MoveList.vue
index 70756cc..28d0d00 100644 (file)
--- a/client/src/components/MoveList.vue
+++ b/client/src/components/MoveList.vue
@@ -3,16 +3,16 @@ div
   #scoreInfo(v-if="score!='*'")
     p {{ score }}
     p {{ message }}
-  table#movesList
+  table.moves-list
     tbody
       tr(v-for="moveIdx in evenNumbers")
-        td {{ moveIdx / 2 + 1 }}
+        td {{ firstNum + moveIdx / 2 + 1 }}
         td(:class="{'highlight-lm': cursor == moveIdx}"
-            data-label="White move" @click="() => gotoMove(moveIdx)")
+            @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)")
+            @click="() => gotoMove(moveIdx+1)")
           | {{ moves[moveIdx+1].notation }}
         // Else: just add an empty cell
         td(v-else)
@@ -22,17 +22,21 @@ div
 // Component for moves list on the right
 export default {
   name: 'my-move-list',
-       props: ["moves","cursor","score","message"],
+       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: 'smooth',
-            block: 'center'
+            behavior: "auto",
+            block: "nearest",
           });
         }
       });
@@ -54,101 +58,14 @@ export default {
 <style lang="sass" scoped>
 .moves-list
   min-width: 250px
+@media screen and (max-width: 767px)
+  .moves-list
+    tr
+      display: flex
+      margin: 0
+      padding: 0
+      td
+        text-align: left
 td.highlight-lm
   background-color: plum
 </style>
-
-<!-- Old render method:
-  render(h) {
-               if (this.moves.length == 0)
-                       return;
-               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 == 1)
-               {
-      moveCells.push(
-        h(
-          "td",
-          { domProps: { innerHTML: "" } }
-        )
-      );
-               }
-               tableRow.children = moveCells;
-               tableContent.push(tableRow);
-    const scoreDiv = h("div",
-      {
-        id: "scoreInfo",
-        style: {
-          display: this.score!="*" ? "block" : "none",
-        },
-      },
-      [
-        h("p", this.score),
-        h("p", this.message),
-      ]
-    );
-               const movesTable = h(
-      "div",
-      { },
-      [
-        scoreDiv,
-        h(
-          "table",
-          {
-            "class": {
-              "moves-list": true,
-            },
-          },
-          tableContent
-                   )
-      ]
-    );
-               return movesTable;
-       },
--->