Move the code into mount of ReportActionMessageEdit

Signed-off-by: Tsaqif <[email protected]>
Expensify · Nov 6, 2023 · 657b79e · 657b79e
1 parent 9402276
commit 657b79e
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 4 deletions.
diff --git a/src/pages/home/report/ReportActionItem.js b/src/pages/home/report/ReportActionItem.js
@@ -194,9 +194,6 @@ function ReportActionItem(props) {
             return;
         }
 
-        if (textInputRef.current) {
-            textInputRef.current.focus();
-        }
         focusTextInputAfterAnimation(textInputRef.current, 100);
     }, [prevDraftMessage, props.draftMessage]);
 

diff --git a/src/pages/home/report/ReportActionItemMessageEdit.js b/src/pages/home/report/ReportActionItemMessageEdit.js
@@ -169,6 +169,15 @@ function ReportActionItemMessageEdit(props) {
         [props.action.reportActionID],
     );
 
+    // Scroll content of textInputRef to bottom
+    useEffect(() => {
+        if (!textInputRef.current) {
+            return;
+        }
+        textInputRef.current.focus();
+        textInputRef.current.scrollTop = textInputRef.current.scrollHeight;
+    }, []);
+
     useEffect(() => {
         // For mobile Safari, updating the selection prop on an unfocused input will cause it to automatically gain focus
         // and subsequent programmatic focus shifts (e.g., modal focus trap) to show the blue frame (:focus-visible style),
@@ -181,7 +190,6 @@ function ReportActionItemMessageEdit(props) {
                 });
                 return prevDraft;
             });
-            textInputRef.current.scrollTop = textInputRef.current.scrollHeight;
         }
 
         return () => {

diff --git a/src/pages/home/report/ReportActionsList.js b/src/pages/home/report/ReportActionsList.js
@@ -429,6 +429,7 @@ function ReportActionsList({
                     keyboardShouldPersistTaps="handled"
                     onLayout={onLayoutInner}
                     onScroll={trackVerticalScrolling}
+                    onScrollToIndexFailed={() => {}}
                     extraData={extraData}
                 />
             </Animated.View>