Merge pull request #32887 from Expensify/marcaaron-fixScreenWrapper

[CP Staging] Enable KB Avoiding View + remove padding bottom
Expensify · Dec 12, 2023 · f28ac76 · f28ac76
2 parents bc121d7 + 0132bc3
commit f28ac76
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/src/pages/iou/request/step/IOURequestStepAmount.js b/src/pages/iou/request/step/IOURequestStepAmount.js
@@ -99,6 +99,7 @@ function IOURequestStepAmount({
             onBackButtonPress={navigateBack}
             testID={IOURequestStepAmount.displayName}
             shouldShowWrapper={Boolean(backTo)}
+            includeSafeAreaPaddingBottom
         >
             <MoneyRequestAmountForm
                 isEditing={Boolean(backTo)}

diff --git a/src/pages/iou/request/step/IOURequestStepParticipants.js b/src/pages/iou/request/step/IOURequestStepParticipants.js
@@ -85,6 +85,7 @@ function IOURequestStepParticipants({
             shouldShowWrapper
             testID={IOURequestStepParticipants.displayName}
             onEntryTransitionEnd={() => optionsSelectorRef.current && optionsSelectorRef.current.focus()}
+            includeSafeAreaPaddingBottom
         >
             <MoneyRequestParticipantsSelector
                 ref={(el) => (optionsSelectorRef.current = el)}

diff --git a/src/pages/iou/request/step/StepScreenWrapper.js b/src/pages/iou/request/step/StepScreenWrapper.js
@@ -25,13 +25,17 @@ const propTypes = {
 
     /** An ID used for unit testing */
     testID: PropTypes.string.isRequired,
+
+    /** Whether or not to include safe area padding */
+    includeSafeAreaPaddingBottom: PropTypes.bool,
 };
 
 const defaultProps = {
     onEntryTransitionEnd: () => {},
+    includeSafeAreaPaddingBottom: false,
 };
 
-function StepScreenWrapper({testID, headerTitle, onBackButtonPress, onEntryTransitionEnd, children, shouldShowWrapper}) {
+function StepScreenWrapper({testID, headerTitle, onBackButtonPress, onEntryTransitionEnd, children, shouldShowWrapper, includeSafeAreaPaddingBottom}) {
     const styles = useThemeStyles();
 
     if (!shouldShowWrapper) {
@@ -40,14 +44,13 @@ function StepScreenWrapper({testID, headerTitle, onBackButtonPress, onEntryTrans
 
     return (
         <ScreenWrapper
-            includeSafeAreaPaddingBottom={false}
-            shouldEnableKeyboardAvoidingView={false}
+            includeSafeAreaPaddingBottom={includeSafeAreaPaddingBottom}
             onEntryTransitionEnd={onEntryTransitionEnd}
             testID={testID}
             shouldEnableMaxHeight={DeviceCapabilities.canUseTouchScreen()}
         >
             {({insets, safeAreaPaddingBottomStyle, didScreenTransitionEnd}) => (
-                <View style={[styles.flex1, safeAreaPaddingBottomStyle]}>
+                <View style={[styles.flex1]}>
                     <HeaderWithBackButton
                         title={headerTitle}
                         onBackButtonPress={onBackButtonPress}