Merge pull request #3740 from overleaf/ae-safari-remove-event-listener

Add required arguments for removeEventListener in Safari

GitOrigin-RevId: ad1cc79b12e56dc1019920964a93acbe1bec10ff

services/web/frontend/js/ide/SafariScrollPatcher.js

@@ -50,13 +50,25 @@ export default SafariScrollPatcher = class SafariScrollPatcher {
 
       // If elements aren't null, remove existing listeners.
       if (this.pdfDiv != null) {
-        this.pdfDiv.removeEventListener(this.handlePdfDivMouseLeave)
-        this.pdfDiv.removeEventListener(this.handlePdfDivMouseEnter)
+        this.pdfDiv.removeEventListener(
+          'mouseleave',
+          this.handlePdfDivMouseLeave
+        )
+        this.pdfDiv.removeEventListener(
+          'mouseenter',
+          this.handlePdfDivMouseEnter
+        )
       }
 
       if (this.aceDiv != null) {
-        this.aceDiv.removeEventListener(this.handleAceDivMouseLeave)
-        this.aceDiv.removeEventListener(this.handleAceDivMouseEnter)
+        this.aceDiv.removeEventListener(
+          'mouseleave',
+          this.handleAceDivMouseLeave
+        )
+        this.aceDiv.removeEventListener(
+          'mouseenter',
+          this.handleAceDivMouseEnter
+        )
       }
 
       // Grab elements.