fix: make mousewheel listener passive

Author: aminya

lib/datatip-manager.js

@@ -292,6 +292,14 @@ export class DataTipManager {
     )
   }
 
+  /**
+   * handles the mouse wheel event to enable scrolling over long text
+   * @param evt the mouse wheel event being triggered
+   */
+  onMouseWheel(evt: WheelEvent) {
+    evt.stopPropagation()
+  }
+
   /**
    * the central command event handler
    * @param evt command event
@@ -394,11 +402,6 @@ export class DataTipManager {
 
           const dataTipView = new ViewContainer({ snippet, markdown, className: `datatip-element ${this.glowClass}` })
 
-          // TODO move this code to atom-ide-base
-          dataTipView.element.addEventListener('mousewheel', (e: WheelEvent) => {
-            e.stopPropagation()
-          })
-
           this.dataTipMarkerDisposables = this.mountDataTipWithMarker(editor, datatip.range, position, dataTipView)
         }
       }
@@ -464,6 +467,9 @@ export class DataTipManager {
       this.editorView.addEventListener("mousemove", this.onMouseMoveEvt)
     })
 
+    // TODO move this code to atom-ide-base
+    view.element.addEventListener("mousewheel", this.onMouseWheel, { passive: true })
+
     disposables.add(
       new Disposable(() => {
         this.editorView.addEventListener("mousemove", this.onMouseMoveEvt)