Skip to content

Commit 321c68c

Browse files
committed
fix: fix
1 parent b530fe9 commit 321c68c

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed

src/stickyScrollBar.tsx

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -142,20 +142,18 @@ const StickyScrollBar: React.ForwardRefRenderFunction<unknown, StickyScrollBarPr
142142
document.body.addEventListener(MOUSEMOVE_EVENT, onMouseMove, false);
143143
checkScrollBarVisible();
144144
return () => {
145-
document.body.removeEventListener(MOUSEUP_EVENT, onMouseUp, false);
146-
document.body.removeEventListener(MOUSEMOVE_EVENT, onMouseMove, false);
145+
document.body.removeEventListener(MOUSEUP_EVENT, onMouseUp);
146+
document.body.removeEventListener(MOUSEMOVE_EVENT, onMouseMove);
147147
};
148-
// eslint-disable-next-line react-hooks/exhaustive-deps
149-
}, [scrollBarWidth, isActive, checkScrollBarVisible]);
148+
}, [scrollBarWidth, isActive]);
150149

151150
React.useEffect(() => {
152151
container.addEventListener(SCROLL_EVENT, checkScrollBarVisible, false);
153152
window.addEventListener(RESIZE_EVENT, checkScrollBarVisible, false);
154153
return () => {
155-
container.removeEventListener(SCROLL_EVENT, checkScrollBarVisible, false);
156-
window.removeEventListener(RESIZE_EVENT, checkScrollBarVisible, false);
154+
container.removeEventListener(SCROLL_EVENT, checkScrollBarVisible);
155+
window.removeEventListener(RESIZE_EVENT, checkScrollBarVisible);
157156
};
158-
// eslint-disable-next-line react-hooks/exhaustive-deps
159157
}, [container]);
160158

161159
React.useEffect(() => {

0 commit comments

Comments
 (0)