最近在做代码迁移的时候出现一个问题,发现之前自己写好的一个自定义滚动条组件有个bug,那就滚动时父容器也会滚动。
看一下代码,代码做了简化
export default ()=>{
return return (
<div
className={classNames(getCls('container'), isDragRef.current ? 'active' : '', className)}
ref={scrollDOMRef}
onWheelCapture={(e: any) => {
e.preventDefault();
if (e.deltaY < 0) {
// 向上
setDragY(dragY - dragSpeed);
} else {
// 向下
setDragY(dragY + dragSpeed);
}
}}>
</div>
);
}
既然父容器会滚动那就阻止默认行为就好e.preventDefault();
,但是没用。
这里我猜测应该因为react的事件是合成事件的缘故,所有事件都注册document上,所以导致阻止的默认行为并没有阻止到父容器上。
那就用原生的呗。
useEffect(() => {
if (scrollDOMRef.current) {
scrollDOMRef.current.addEventListener('wheel', (e) => {
e.preventDefault();
if (e.deltaY < 0) {
// 向上
setDragY(dragY - dragSpeed);
} else {
// 向下
setDragY(dragY + dragSpeed);
}
});
}
}, [scrollDOMRef.current]);
原生事件和react事件一起用时要注意,阻止冒泡要考虑清楚,因为可能会导致react合成事件失效。
那这么做后就可以了吗?
确实
父容器不滚动了
但是又掉进了react的闭包陷阱
注册函数并没有及时更新,dragY 和 dragSpeed的闭包导致出现了bug。
那么又应该怎么做呢?
useEffect(() => {
if (scrollDOMRef.current) {
scrollDOMRef.current.addEventListener('wheel', (e) => {
e.preventDefault();
if (e.deltaY < 0) {
// 向上
setDragY(dragY - dragSpeed);
} else {
// 向下
setDragY(dragY + dragSpeed);
}
});
}
}, [scrollDOMRef.current, dragY, dragSpeed]);
又一个问题出现了
每次应该把之前的事件销毁,然后在注册才对。不然多个事件同时触发导致了bug。
useEffect(() => {
const handle = (e: any) => {
e.preventDefault();
if (e.deltaY < 0) {
// 向上
setDragY(dragY - dragSpeed);
} else {
// 向下
setDragY(dragY + dragSpeed);
}
};
if (scrollDOMRef.current) {
scrollDOMRef.current.addEventListener('wheel', handle, {
passive: false,
});
}
return () => {
if (scrollDOMRef.current) {
scrollDOMRef.current.removeEventListener('wheel', handle);
}
};
}, [scrollDOMRef.current, dragY, dragSpeed]);
passive
passive为false时,浏览器执行完回调函数才知道有没有调用preventDefault,如果没有调用preventDefault,再去执行默认行为,就是滚动。这样就回造成滚动不流畅。
passive为true,就是告诉浏览器不会调用preventDefault,浏览器直接执行滚动就行,不用考虑回调函数了。
这时,即使你在回调函数里调用preventDefault也不会生效。
mdn中说,在有些浏览器(特别是Chrome和Firefox)中,你监听window、document或者document.body上的touchstart和touchmove,会将passive默认设置为true。
还是要提醒大家,在你不需要调用preventDefault的时候,监听scroll或者touchmove,将passive设置为true