Skip to content

Commit 6655092

Browse files
committed
fix: πŸ› make sure all paths in usePageLeave return
1 parent 33ac91b commit 6655092

File tree

1 file changed

+11
-11
lines changed

1 file changed

+11
-11
lines changed

β€Žsrc/usePageLeave.ts

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -2,18 +2,18 @@ import {useEffect} from 'react';
22

33
const usePageLeave = (onPageLeave, args = []) => {
44
useEffect(() => {
5-
if (onPageLeave) {
6-
const handler = (event) => {
7-
event = event ? event : (window.event as any);
8-
const from = event.relatedTarget || event.toElement;
9-
if (!from || (from as any).nodeName === 'HTML') onPageLeave();
10-
};
5+
if (!onPageLeave) return;
116

12-
document.addEventListener('mouseout', handler);
13-
return () => {
14-
document.removeEventListener('mouseout', handler);
15-
};
16-
}
7+
const handler = (event) => {
8+
event = event ? event : (window.event as any);
9+
const from = event.relatedTarget || event.toElement;
10+
if (!from || (from as any).nodeName === 'HTML') onPageLeave();
11+
};
12+
13+
document.addEventListener('mouseout', handler);
14+
return () => {
15+
document.removeEventListener('mouseout', handler);
16+
};
1717
}, args);
1818
};
1919

0 commit comments

Comments
Β (0)