From: Benjamin Auder Date: Thu, 28 Jan 2021 18:25:51 +0000 (+0100) Subject: Improve arrows behavior X-Git-Url: https://git.auder.net/app_dev.php/%7B%7B%20asset%28%27mixstore/images/common.css?a=commitdiff_plain;h=2c33215f7c5ae03ab340f6a6e5555790d75762cf;p=vchess.git Improve arrows behavior --- diff --git a/client/src/components/Board.vue b/client/src/components/Board.vue index 9e23fc56..f20e7da6 100644 --- a/client/src/components/Board.vue +++ b/client/src/components/Board.vue @@ -546,15 +546,20 @@ export default { } }, addArrow: function(arrow) { - this.arrows.push(arrow); - // Also add to DOM: - const boardElt = document.getElementById("gamePosition"); - const squareWidth = boardElt.offsetWidth / V.size.y; - const bPos = boardElt.getBoundingClientRect(); - const newArrow = - this.getSvgArrow(arrow, bPos.top, bPos.left, squareWidth); - document.getElementById("arrowCanvas") - .insertAdjacentElement("beforeend", newArrow); + const arrowIdx = this.arrows.findIndex(a => { + return ( + a.start[0] == arrow.start[0] && a.start[1] == arrow.start[1] && + a.end[0] == arrow.end[0] && a.end[1] == arrow.end[1] + ); + }); + if (arrowIdx >= 0) + // Erase the arrow + this.arrows.splice(arrowIdx, 1); + else + // Add to arrows vector: + this.arrows.push(arrow); + // NOTE: no need to draw here, will be re-draw + // by updated() hook callong re_setDrawings() }, getSvgArrow: function(arrow, top, left, squareWidth) { const aStart =