refactor(ContextMenu): simplify useEffect

This commit is contained in:
Tim 2025-03-20 15:30:13 +01:00
parent 43a7e8d856
commit 6a2054372f

View file

@ -58,12 +58,8 @@ const ContextMenu = ({ children, on, autoClose }: Props) => {
};
useEffect(() => {
const containers = on.map((ref) => ref.current).filter((element) => !!element);
containers.forEach((container) => container.addEventListener('contextmenu', onContextMenu));
return () => {
containers.forEach((container) => container.removeEventListener('contextmenu', onContextMenu));
};
on.forEach((ref) => ref.current && ref.current.addEventListener('contextmenu', onContextMenu));
return () => on.forEach((ref) => ref.current && ref.current.removeEventListener('contextmenu', onContextMenu));
}, [on]);
return active && createPortal((