Skip to content

Commit 82b1c88

Browse files
committed
fix: update ConfirmModalWrapper to use useEffectEvent for handling modal hide logic
1 parent 85b8e43 commit 82b1c88

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

src/components/Modal/Global/ConfirmModalWrapper.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import React, {useState} from 'react';
1+
import React, {useEffectEvent, useState} from 'react';
22
import type {ConfirmModalProps} from '@components/ConfirmModal';
33
import ConfirmModal from '@components/ConfirmModal';
44
import useActiveElementRole from '@hooks/useActiveElementRole';
@@ -29,13 +29,13 @@ function ConfirmModalWrapper({closeModal, onModalHide, ...props}: ConfirmModalWr
2929
setIsVisible(false);
3030
};
3131

32-
const handleModalHide = () => {
32+
const handleModalHide = useEffectEvent(() => {
3333
if (isVisible) {
3434
return;
3535
}
3636
closeModal({action: closeAction});
3737
onModalHide?.();
38-
};
38+
});
3939

4040
const shortcutConfig = {
4141
isActive: activeElementRole !== CONST.ROLE.BUTTON,

0 commit comments

Comments
 (0)