[useScrollTrigger] Add passive flag to scroll trigger event listener #…

…32437 (#33749)
mui · Aug 8, 2022 · a804ca8 · a804ca8
1 parent 764a6a8
commit a804ca8
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/packages/mui-material/src/useScrollTrigger/useScrollTrigger.js b/packages/mui-material/src/useScrollTrigger/useScrollTrigger.js
@@ -31,9 +31,9 @@ export default function useScrollTrigger(options = {}) {
     };
 
     handleScroll(); // Re-evaluate trigger when dependencies change
-    target.addEventListener('scroll', handleScroll);
+    target.addEventListener('scroll', handleScroll, { passive: true });
     return () => {
-      target.removeEventListener('scroll', handleScroll);
+      target.removeEventListener('scroll', handleScroll, { passive: true });
     };
     // See Option 3. https://github.com/facebook/react/issues/14476#issuecomment-471199055
     // eslint-disable-next-line react-hooks/exhaustive-deps