ppfun/pixelplanet
13
Fork 18
Code Issues 24 Pull Requests Releases 2 Activity

check for click outside with mousedown instead of click

Browse Source
This commit is contained in:
HF 2022-08-12 21:24:10 +02:00
parent ae1e259bab
commit 43c2f1d012
1 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/components/hooks/clickOutside.js
View File

@ -17,7 +17,6 @@ export function useConditionalClickOutside(insideRefs, active, callback) {
const handleClickOutside = useCallback((event) => {
if (insideRefs.every((ref) => !ref.current
|| !ref.current.contains(event.target))) {
event.stopPropagation();
callback();
}
}, [callback]);
@ -28,12 +27,12 @@ export function useConditionalClickOutside(insideRefs, active, callback) {
useLayoutEffect(() => {
if (active) {
document.addEventListener('click', handleClickOutside, {
document.addEventListener('mousedown', handleClickOutside, {
capture: true,
});
window.addEventListener('resize', handleWindowResize);
} else {
document.removeEventListener('click', handleClickOutside, {
document.removeEventListener('mousedown', handleClickOutside, {
capture: true,
});
window.removeEventListener('resize', handleWindowResize);
@ -51,7 +50,6 @@ export function useClickOutside(insideRefs, callback) {
const handleClickOutside = (event) => {
if (insideRefs.every((ref) => !ref.current
|| !ref.current.contains(event.target))) {
event.stopPropagation();
callback();
}
};
@ -60,12 +58,12 @@ export function useClickOutside(insideRefs, callback) {
callback();
};
document.addEventListener('click', handleClickOutside, {
document.addEventListener('mousedown', handleClickOutside, {
capture: true,
});
window.addEventListener('resize', handleWindowResize);
return () => {
document.removeEventListener('click', handleClickOutside, {
document.removeEventListener('mousedown', handleClickOutside, {
capture: true,
});
window.removeEventListener('resize', handleWindowResize);