Skip to content

Commit 65a2f9b

Browse files
committed
fix: refactor
1 parent 1fd92bd commit 65a2f9b

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

src/components/Modal/BaseModal.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ type ModalComponentProps = (ReactNativeModalProps | ReanimatedModalProps) & {
4040
shouldUseReanimatedModal?: boolean;
4141
shouldPreventScrollOnFocus?: boolean;
4242
initialFocus?: FocusTrapOptions['initialFocus'];
43+
isVisible: boolean;
4344
};
4445

4546
function ModalComponent({type, shouldUseReanimatedModal, isVisible, shouldPreventScrollOnFocus, initialFocus, children, ...props}: ModalComponentProps) {
@@ -66,7 +67,7 @@ function ModalComponent({type, shouldUseReanimatedModal, isVisible, shouldPreven
6667
isVisible={isVisible}
6768
>
6869
<FocusTrapForModal
69-
active={isVisible ?? false}
70+
active={isVisible}
7071
initialFocus={initialFocus}
7172
shouldPreventScroll={shouldPreventScrollOnFocus}
7273
>

src/components/Modal/ReanimatedModal/index.tsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -187,13 +187,14 @@ function ReanimatedModal({
187187
);
188188
}
189189
const isBackdropMounted = isVisibleState || ((isTransitioning || isContainerOpen !== isVisibleState) && getPlatform() === CONST.PLATFORM.WEB);
190+
const modalVisibility = isVisibleState || isTransitioning || isContainerOpen !== isVisibleState;
190191
return (
191192
<LayoutAnimationConfig skipExiting={getPlatform() !== CONST.PLATFORM.WEB}>
192193
<Modal
193194
transparent
194195
animationType="none"
195196
// eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
196-
visible={isVisibleState || isTransitioning || isContainerOpen !== isVisibleState}
197+
visible={modalVisibility}
197198
onRequestClose={onBackButtonPress}
198199
statusBarTranslucent={statusBarTranslucent}
199200
testID={testID}
@@ -217,7 +218,7 @@ function ReanimatedModal({
217218
</KeyboardAvoidingView>
218219
) : (
219220
<FocusTrapForModal
220-
active={isVisibleState || isTransitioning || isContainerOpen !== isVisibleState}
221+
active={modalVisibility}
221222
initialFocus={initialFocus}
222223
shouldPreventScroll={shouldPreventScrollOnFocus}
223224
>

0 commit comments

Comments
 (0)