Merge pull request #1563 from rx-angular/fix/rm-scroll

edbzn · web-flow · commit 944a489fa940 · 2023-05-28T10:19:47.000+02:00
fix(template): remove `scroll` listener
diff --git a/libs/template/experimental/virtual-scrolling/src/lib/virtual-scroll-viewport.component.ts b/libs/template/experimental/virtual-scrolling/src/lib/virtual-scroll-viewport.component.ts
@@ -29,6 +29,9 @@ declare const ngDevMode: boolean;
 
 const NG_DEV_MODE = typeof ngDevMode === 'undefined' || !!ngDevMode;
 
+/** Options used to bind passive event listeners. */
+const passiveEventListenerOptions = { passive: true };
+
 /**
  * @Component RxVirtualScrollViewport
  *
@@ -184,9 +187,7 @@ export class RxVirtualScrollViewportComponent
       this.scrollContainer(),
       'scroll',
       this.scrollListener,
-      {
-        passive: true,
-      }
+      passiveEventListenerOptions
     );
   }
 
@@ -208,7 +209,8 @@ export class RxVirtualScrollViewportComponent
     getZoneUnPatchedApi(this.scrollContainer(), 'removeEventListener').call(
       this.scrollContainer(),
       'scroll',
-      this.scrollListener
+      this.scrollListener,
+      passiveEventListenerOptions
     );
     this.destroy$.next();
     this.scrollStrategy.detach();
