Skip to content

Commit 325ca63

Browse files
committed
refactor: simplify modal hide logic in ConfirmModalWrapper by removing useEffectEvent
1 parent 1e82a79 commit 325ca63

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, {useEffectEvent, useState} from 'react';
1+
import React, {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 = useEffectEvent(() => {
32+
const handleModalHide = () => {
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)