From d32926792677bd1f11db2f49dfefdaf12bffba44 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Wed, 28 Jun 2023 12:34:03 +0200 Subject: [PATCH] Use `Element.scrollTo` unconditionally in the `GrabToPan` class According to the MDN compatibility data this is available in all browsers that we currently support; please see https://developer.mozilla.org/en-US/docs/Web/API/Element/scrollTo#browser_compatibility --- web/grab_to_pan.js | 18 ++++++------------ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git a/web/grab_to_pan.js b/web/grab_to_pan.js index a985afc6b..b3f86d728 100644 --- a/web/grab_to_pan.js +++ b/web/grab_to_pan.js @@ -140,18 +140,12 @@ class GrabToPan { } const xDiff = event.clientX - this.clientXStart; const yDiff = event.clientY - this.clientYStart; - const scrollTop = this.scrollTopStart - yDiff; - const scrollLeft = this.scrollLeftStart - xDiff; - if (this.element.scrollTo) { - this.element.scrollTo({ - top: scrollTop, - left: scrollLeft, - behavior: "instant", - }); - } else { - this.element.scrollTop = scrollTop; - this.element.scrollLeft = scrollLeft; - } + this.element.scrollTo({ + top: this.scrollTopStart - yDiff, + left: this.scrollLeftStart - xDiff, + behavior: "instant", + }); + if (!this.overlay.parentNode) { document.body.append(this.overlay); }