Skip to content

Commit a37f086

Browse files
fix: extra padding bottom in bottom modal
1 parent 6000a8c commit a37f086

File tree

2 files changed

+2
-12
lines changed

2 files changed

+2
-12
lines changed

src/components/FeatureTrainingModal.tsx

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ import Lottie from './Lottie';
2323
import LottieAnimations from './LottieAnimations';
2424
import type DotLottieAnimation from './LottieAnimations/types';
2525
import Modal from './Modal';
26-
import SafeAreaConsumer from './SafeAreaConsumer';
2726
import Text from './Text';
2827
import VideoPlayer from './VideoPlayer';
2928

@@ -311,8 +310,6 @@ function FeatureTrainingModal({
311310
useLayoutEffect(parseFSAttributes, []);
312311

313312
return (
314-
<SafeAreaConsumer>
315-
{({safeAreaPaddingBottomStyle}) => (
316313
<Modal
317314
isVisible={isModalVisible}
318315
type={onboardingIsMediumOrLargerScreenWidth ? CONST.MODAL.MODAL_TYPE.CENTERED_UNSWIPEABLE : CONST.MODAL.MODAL_TYPE.BOTTOM_DOCKED}
@@ -334,7 +331,7 @@ function FeatureTrainingModal({
334331
shouldUseNewModal
335332
>
336333
<View
337-
style={[styles.mh100, onboardingIsMediumOrLargerScreenWidth && StyleUtils.getWidthStyle(width), safeAreaPaddingBottomStyle]}
334+
style={[styles.mh100, onboardingIsMediumOrLargerScreenWidth && StyleUtils.getWidthStyle(width)]}
338335
fsClass={CONST.FULL_STORY.UNMASK}
339336
testID={CONST.FULL_STORY.UNMASK}
340337
>
@@ -377,8 +374,6 @@ function FeatureTrainingModal({
377374
</View>
378375
</View>
379376
</Modal>
380-
)}
381-
</SafeAreaConsumer>
382377
);
383378
}
384379

src/components/RequireTwoFactorAuthenticationModal.tsx

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ import Button from './Button';
99
import Lottie from './Lottie';
1010
import LottieAnimations from './LottieAnimations';
1111
import Modal from './Modal';
12-
import SafeAreaConsumer from './SafeAreaConsumer';
1312
import Text from './Text';
1413

1514
type RequireTwoFactorAuthenticationModalProps = {
@@ -39,8 +38,6 @@ function RequireTwoFactorAuthenticationModal({onCancel = () => {}, description,
3938
const StyleUtils = useStyleUtils();
4039

4140
return (
42-
<SafeAreaConsumer>
43-
{({safeAreaPaddingBottomStyle}) => (
4441
<Modal
4542
onClose={onCancel}
4643
isVisible={isVisible}
@@ -49,7 +46,7 @@ function RequireTwoFactorAuthenticationModal({onCancel = () => {}, description,
4946
shouldEnableNewFocusManagement={shouldEnableNewFocusManagement}
5047
shouldUseNewModal
5148
>
52-
<View style={safeAreaPaddingBottomStyle}>
49+
<View >
5350
<View style={[styles.cardSectionIllustration, styles.alignItemsCenter, StyleUtils.getBackgroundColorStyle(LottieAnimations.Safe.backgroundColor)]}>
5451
<Lottie
5552
source={LottieAnimations.Safe}
@@ -74,8 +71,6 @@ function RequireTwoFactorAuthenticationModal({onCancel = () => {}, description,
7471
</View>
7572
</View>
7673
</Modal>
77-
)}
78-
</SafeAreaConsumer>
7974
);
8075
}
8176

0 commit comments

Comments
 (0)