From fbffbc173793197daff9ebc7937f58dad3bdd1b6 Mon Sep 17 00:00:00 2001 From: IIITM-Jay Date: Tue, 6 May 2025 23:36:25 +0530 Subject: [PATCH 1/2] Consistent `movedX` and `movedY` behaviour across zoom levels --- src/events/mouse.js | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/src/events/mouse.js b/src/events/mouse.js index 0f3985a2db..4b2446913c 100644 --- a/src/events/mouse.js +++ b/src/events/mouse.js @@ -819,21 +819,27 @@ p5.prototype.mouseIsPressed = false; p5.prototype._updateNextMouseCoords = function(e) { if (this._curElement !== null && (!e.touches || e.touches.length > 0)) { + this._updateMouseCoords(); + const mousePos = getMousePos( this._curElement.elt, this.width, this.height, e ); - this._setProperty('movedX', e.movementX); - this._setProperty('movedY', e.movementY); + this._setProperty('mouseX', mousePos.x); this._setProperty('mouseY', mousePos.y); this._setProperty('winMouseX', mousePos.winX); this._setProperty('winMouseY', mousePos.winY); + + const deltaX = this.mouseX - this.pmouseX; + const deltaY = this.mouseY - this.pmouseY; + this._setProperty('movedX', deltaX); + this._setProperty('movedY', deltaY); } + if (!this._hasMouseInteracted) { - // For first draw, make previous and next equal this._updateMouseCoords(); this._setProperty('_hasMouseInteracted', true); } From 197f3ee1953948803a554d2e064d6461a4c175ec Mon Sep 17 00:00:00 2001 From: IIITM-Jay Date: Tue, 13 May 2025 16:51:24 +0530 Subject: [PATCH 2/2] removed redundant calling of updateMouseCoords() --- src/events/mouse.js | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/events/mouse.js b/src/events/mouse.js index 4b2446913c..536c68f11f 100644 --- a/src/events/mouse.js +++ b/src/events/mouse.js @@ -819,8 +819,6 @@ p5.prototype.mouseIsPressed = false; p5.prototype._updateNextMouseCoords = function(e) { if (this._curElement !== null && (!e.touches || e.touches.length > 0)) { - this._updateMouseCoords(); - const mousePos = getMousePos( this._curElement.elt, this.width,