cm-chessboard 7.10.0 → 7.11.0

package/examples/validate-moves.html

@@ -19,7 +19,7 @@
 <script type="module">
     import {INPUT_EVENT_TYPE, COLOR, Chessboard, BORDER_TYPE} from "../src/Chessboard.js"
     import {MARKER_TYPE, Markers} from "../src/extensions/markers/Markers.js"
-    import {PromotionDialog} from "../src/extensions/promotion-dialog/PromotionDialog.js"
+    import {PROMOTION_DIALOG_RESULT_TYPE, PromotionDialog} from "../src/extensions/promotion-dialog/PromotionDialog.js"
     import {Accessibility} from "../src/extensions/accessibility/Accessibility.js"
     import {Chess} from "https://cdn.jsdelivr.net/npm/chess.mjs@1/src/chess.mjs/Chess.js"
 
@@ -73,7 +73,7 @@
                     if (possibleMove.promotion && possibleMove.to === event.squareTo) {
                         event.chessboard.showPromotionDialog(event.squareTo, COLOR.white, (result) => {
                             console.log("promotion result", result)
-                            if (result) {
+                            if (result.type === PROMOTION_DIALOG_RESULT_TYPE.pieceSelected) {
                                 chess.move({from: event.squareFrom, to: event.squareTo, promotion: result.piece.charAt(1)})
                                 event.chessboard.setPosition(chess.fen(), true)
                                 makeEngineMove(event.chessboard)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cm-chessboard",
-  "version": "7.10.0",
+  "version": "7.11.0",
   "description": "A JavaScript chessboard which is lightweight, ES6 module based, responsive, SVG rendered and without dependencies.",
   "keywords": [
     "chess",

package/src/extensions/promotion-dialog/PromotionDialog.js

@@ -14,6 +14,11 @@ const DISPLAY_STATE = {
     shown: "shown"
 }
 
+export const PROMOTION_DIALOG_RESULT_TYPE = {
+    pieceSelected: "pieceSelected",
+    canceled: "canceled"
+}
+
 export class PromotionDialog extends Extension {
 
     /** @constructor */
@@ -138,7 +143,13 @@ export class PromotionDialog extends Extension {
     promotionDialogOnClickPiece(event) {
         if (event.button !== 2) {
             if (event.target.dataset.piece) {
-                this.state.callback({square: this.state.dialogParams.square, piece: event.target.dataset.piece})
+                if(this.state.callback) {
+                    this.state.callback({
+                        type: PROMOTION_DIALOG_RESULT_TYPE.pieceSelected,
+                        square: this.state.dialogParams.square,
+                        piece: event.target.dataset.piece
+                    })
+                }
                 this.setDisplayState(DISPLAY_STATE.hidden)
             } else {
                 this.promotionDialogOnCancel(event)
@@ -150,14 +161,18 @@ export class PromotionDialog extends Extension {
         if (this.state.displayState === DISPLAY_STATE.shown) {
             event.preventDefault()
             this.setDisplayState(DISPLAY_STATE.hidden)
-            this.state.callback(null)
+            if(this.state.callback) {
+                this.state.callback({type: PROMOTION_DIALOG_RESULT_TYPE.canceled})
+            }
         }
     }
 
     contextMenu(event) {
         event.preventDefault()
         this.setDisplayState(DISPLAY_STATE.hidden)
-        this.state.callback(null)
+        if(this.state.callback) {
+            this.state.callback({type: PROMOTION_DIALOG_RESULT_TYPE.canceled})
+        }
     }
 
     setDisplayState(displayState) {

package/src/view/VisualMoveInput.js

@@ -65,8 +65,8 @@ export class VisualMoveInput {
     }
 
     moveInputCanceledCallback(fromSquare, toSquare, reason) {
-            this.view.moveInputCanceledCallback(fromSquare, toSquare, reason)
-            this.chessboard.state.moveInputProcess.resolve()
+        this.view.moveInputCanceledCallback(fromSquare, toSquare, reason)
+        this.chessboard.state.moveInputProcess.resolve()
     }
 
     setMoveInputState(newState, params = undefined) {
@@ -240,14 +240,15 @@ export class VisualMoveInput {
 
     onPointerDown(e) {
         if (!(e.type === "mousedown" && e.button === 0 || e.type === "touchstart")) {
-            return;
+            return
+        }
+        if(e.type === "mousedown") { // another try to prevent strange safari desktop select all bug
+            e.preventDefault()
         }
-
         const square = e.target.getAttribute("data-square")
         if (!square) { // pointer on square
-            return;
+            return
         }
-
         const pieceName = this.chessboard.getPiece(square)
         let color
         if (pieceName) {