fix(frontend): fix event listener memory leak in useIsDisabled hook (#5027)

fix: optimize event listener management in useIsDisabled hook
This commit is contained in:
Dipanshu Rawat
2026-02-20 01:54:24 +05:30
committed by GitHub
parent eaa35eba61
commit a961fb8f75

View File

@@ -360,20 +360,15 @@ function useIsDisabled() {
*/
useEffect(() => {
if (!window) return;
window.addEventListener(ATTACHMENTS_PROCESSING_EVENT, () =>
setIsDisabled(true)
);
window.addEventListener(ATTACHMENTS_PROCESSED_EVENT, () =>
setIsDisabled(false)
);
const onProcessing = () => setIsDisabled(true);
const onProcessed = () => setIsDisabled(false);
window.addEventListener(ATTACHMENTS_PROCESSING_EVENT, onProcessing);
window.addEventListener(ATTACHMENTS_PROCESSED_EVENT, onProcessed);
return () => {
window?.removeEventListener(ATTACHMENTS_PROCESSING_EVENT, () =>
setIsDisabled(true)
);
window?.removeEventListener(ATTACHMENTS_PROCESSED_EVENT, () =>
setIsDisabled(false)
);
window.removeEventListener(ATTACHMENTS_PROCESSING_EVENT, onProcessing);
window.removeEventListener(ATTACHMENTS_PROCESSED_EVENT, onProcessed);
};
}, []);