From a804ca80a136ac4e93667e41715d39b5f433db63 Mon Sep 17 00:00:00 2001 From: Douglas Salazar Date: Sun, 7 Aug 2022 22:28:08 -0700 Subject: [PATCH] [useScrollTrigger] Add passive flag to scroll trigger event listener #32437 (#33749) --- .../mui-material/src/useScrollTrigger/useScrollTrigger.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/mui-material/src/useScrollTrigger/useScrollTrigger.js b/packages/mui-material/src/useScrollTrigger/useScrollTrigger.js index d88baaf06ba1c6..ec582b3913676a 100644 --- 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