Skip to content

Commit

Permalink
fix: don't close cards when clicking as its easy to lose info
Browse files Browse the repository at this point in the history
  • Loading branch information
sekwah41 committed May 1, 2022
1 parent 7b4de89 commit 8394249
Showing 1 changed file with 8 additions and 6 deletions.
14 changes: 8 additions & 6 deletions app/renderer/src/hooks/useTargetOutside.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,12 @@ interface TargetOutside {
eventType?: string;
}

export const useTargetOutside = ({
ref,
eventType = "click",
}: TargetOutside) => {
/**
* If you want to listen to clicks outside the element set eventType to 'click'
* @param ref
* @param eventType
*/
export const useTargetOutside = ({ ref, eventType }: TargetOutside) => {
const [state, setState] = useState<any>();

useLayoutEffect(() => {
Expand All @@ -31,11 +33,11 @@ export const useTargetOutside = ({
}

if (state) {
document.addEventListener(eventType, outsideTarget);
if (eventType) document.addEventListener(eventType, outsideTarget);
document.addEventListener("keydown", closeOnEscape);
}
return () => {
document.removeEventListener(eventType, outsideTarget);
if (eventType) document.removeEventListener(eventType, outsideTarget);
document.removeEventListener("keydown", closeOnEscape);
};
}, [state, ref, eventType]);
Expand Down

0 comments on commit 8394249

Please sign in to comment.