Merge pull request #31547 from Expensify/jules-revert30679

[NO QA] Revert PR 30679
Expensify · Nov 20, 2023 · 3ac55d2 · 3ac55d2
2 parents c7729bd + 8b980a6
commit 3ac55d2
Show file tree

Hide file tree

Showing 7 changed files with 2 additions and 30 deletions.
diff --git a/src/CONST.ts b/src/CONST.ts
@@ -2881,15 +2881,7 @@ const CONST = {
 
     /** <input /> types that will show a virtual keyboard in a mobile browser */
     INPUT_TYPES_WITH_KEYBOARD: ['text', 'search', 'tel', 'url', 'email', 'password'],
-    /**
-     * native IDs for close buttons in Overlay component
-     */
-    OVERLAY: {
-        TOP_BUTTON_NATIVE_ID: 'overLayTopButton',
-        BOTTOM_BUTTON_NATIVE_ID: 'overLayBottomButton',
-    },
 
-    BACK_BUTTON_NATIVE_ID: 'backButton',
     REFERRAL_PROGRAM: {
         CONTENT_TYPES: {
             MONEY_REQUEST: 'request',

diff --git a/src/components/Form.js b/src/components/Form.js
@@ -351,17 +351,7 @@ function Form(props) {
                             // We delay the validation in order to prevent Checkbox loss of focus when
                             // the user are focusing a TextInput and proceeds to toggle a CheckBox in
                             // web and mobile web platforms.
-
-                            // Prevents React from resetting its properties
-                            event.persist();
                             setTimeout(() => {
-                                const relatedTargetId = lodashGet(event, 'nativeEvent.relatedTarget.id');
-                                if (
-                                    relatedTargetId &&
-                                    _.includes([CONST.OVERLAY.BOTTOM_BUTTON_NATIVE_ID, CONST.OVERLAY.TOP_BUTTON_NATIVE_ID, CONST.BACK_BUTTON_NATIVE_ID], relatedTargetId)
-                                ) {
-                                    return;
-                                }
                                 setTouchedInput(inputID);
                                 if (props.shouldValidateOnBlur) {
                                     onValidate(inputValues, !hasServerError);

diff --git a/src/components/Form/FormProvider.js b/src/components/Form/FormProvider.js
@@ -266,14 +266,7 @@ function FormProvider({validate, formID, shouldValidateOnBlur, shouldValidateOnC
                         // We delay the validation in order to prevent Checkbox loss of focus when
                         // the user is focusing a TextInput and proceeds to toggle a CheckBox in
                         // web and mobile web platforms.
-
-                        // Prevents React from resetting its properties
-                        event.persist();
                         setTimeout(() => {
-                            const relatedTargetId = lodashGet(event, 'nativeEvent.relatedTarget.id');
-                            if (relatedTargetId && _.includes([CONST.OVERLAY.BOTTOM_BUTTON_NATIVE_ID, CONST.OVERLAY.TOP_BUTTON_NATIVE_ID, CONST.BACK_BUTTON_NATIVE_ID], relatedTargetId)) {
-                                return;
-                            }
                             setTouchedInput(inputID);
                             if (shouldValidateOnBlur) {
                                 onValidate(inputValues, !hasServerError);

diff --git a/src/components/HeaderWithBackButton/index.js b/src/components/HeaderWithBackButton/index.js
@@ -79,7 +79,6 @@ function HeaderWithBackButton({
                             style={[styles.touchableButtonImage]}
                             role="button"
                             accessibilityLabel={translate('common.back')}
-                            nativeID={CONST.BACK_BUTTON_NATIVE_ID}
                         >
                             <Icon
                                 src={Expensicons.BackArrow}

diff --git a/src/components/RoomNameInput/index.js b/src/components/RoomNameInput/index.js
@@ -57,7 +57,7 @@ function RoomNameInput({isFocused, autoFocus, disabled, errorText, forwardedRef,
             onSelectionChange={(event) => setSelection(event.nativeEvent.selection)}
             errorText={errorText}
             autoCapitalize="none"
-            onBlur={(event) => isFocused && onBlur(event)}
+            onBlur={() => isFocused && onBlur()}
             shouldDelayFocus={shouldDelayFocus}
             autoFocus={isFocused && autoFocus}
             maxLength={CONST.REPORT.MAX_ROOM_NAME_LENGTH}

diff --git a/src/components/RoomNameInput/index.native.js b/src/components/RoomNameInput/index.native.js
@@ -41,7 +41,7 @@ function RoomNameInput({isFocused, autoFocus, disabled, errorText, forwardedRef,
             errorText={errorText}
             maxLength={CONST.REPORT.MAX_ROOM_NAME_LENGTH}
             keyboardType={keyboardType} // this is a bit hacky solution to a RN issue https://github.com/facebook/react-native/issues/27449
-            onBlur={(event) => isFocused && onBlur(event)}
+            onBlur={() => isFocused && onBlur()}
             autoFocus={isFocused && autoFocus}
             autoCapitalize="none"
             shouldDelayFocus={shouldDelayFocus}

diff --git a/src/libs/Navigation/AppNavigator/Navigators/Overlay.js b/src/libs/Navigation/AppNavigator/Navigators/Overlay.js
@@ -29,15 +29,13 @@ function Overlay(props) {
                     onPress={props.onPress}
                     accessibilityLabel={translate('common.close')}
                     role={CONST.ACCESSIBILITY_ROLE.BUTTON}
-                    nativeID={CONST.OVERLAY.TOP_BUTTON_NATIVE_ID}
                 />
                 <PressableWithoutFeedback
                     style={[styles.flex1]}
                     onPress={props.onPress}
                     accessibilityLabel={translate('common.close')}
                     role={CONST.ACCESSIBILITY_ROLE.BUTTON}
                     noDragArea
-                    nativeID={CONST.OVERLAY.BOTTOM_BUTTON_NATIVE_ID}
                 />
             </View>
         </Animated.View>