Auto scroll to end after adding or editing addresses & amounts

Author: paullinator

CHANGELOG.md

@@ -5,8 +5,11 @@
 - added: Add eCash.
 - changed: Auto launch QR scanner for multi-out payments if previously used
 - changed: Auto populate amount for multi-out payments if prior amounts are similar
+- changed: Auto scroll to end of `SendScene2` when addresses or amounts change
 - fixed: No longer show FIO onboarding modal while in Duress Mode.
 - fixed: Prevent pin changes which match duress pin.
+- changed: Auto launch QR scanner for multi-out payments if previously used
+- changed: Auto populate amount for multi-out payments if prior amounts are similar
 
 ## 4.28.0 (staging)
 

src/__tests__/scenes/__snapshots__/SendScene2.ui.test.tsx.snap

@@ -209,6 +209,7 @@ exports[`SendScene2 1 spendTarget 1`] = `
       extraScrollHeight={62}
       getScrollResponder={[Function]}
       handleOnScroll={[Function]}
+      innerRef={[Function]}
       keyboardDismissMode="interactive"
       keyboardOpeningTime={250}
       keyboardSpace={0}
@@ -1605,6 +1606,7 @@ exports[`SendScene2 1 spendTarget with info tiles 1`] = `
       extraScrollHeight={62}
       getScrollResponder={[Function]}
       handleOnScroll={[Function]}
+      innerRef={[Function]}
       keyboardDismissMode="interactive"
       keyboardOpeningTime={250}
       keyboardSpace={0}
@@ -3229,6 +3231,7 @@ exports[`SendScene2 2 spendTargets 1`] = `
       extraScrollHeight={62}
       getScrollResponder={[Function]}
       handleOnScroll={[Function]}
+      innerRef={[Function]}
       keyboardDismissMode="interactive"
       keyboardOpeningTime={250}
       keyboardSpace={0}
@@ -4811,6 +4814,7 @@ exports[`SendScene2 2 spendTargets hide tiles 1`] = `
       extraScrollHeight={62}
       getScrollResponder={[Function]}
       handleOnScroll={[Function]}
+      innerRef={[Function]}
       keyboardDismissMode="interactive"
       keyboardOpeningTime={250}
       keyboardSpace={0}
@@ -6052,6 +6056,7 @@ exports[`SendScene2 2 spendTargets hide tiles 2`] = `
       extraScrollHeight={62}
       getScrollResponder={[Function]}
       handleOnScroll={[Function]}
+      innerRef={[Function]}
       keyboardDismissMode="interactive"
       keyboardOpeningTime={250}
       keyboardSpace={0}
@@ -7269,6 +7274,7 @@ exports[`SendScene2 2 spendTargets hide tiles 3`] = `
       extraScrollHeight={62}
       getScrollResponder={[Function]}
       handleOnScroll={[Function]}
+      innerRef={[Function]}
       keyboardDismissMode="interactive"
       keyboardOpeningTime={250}
       keyboardSpace={0}
@@ -8304,6 +8310,7 @@ exports[`SendScene2 2 spendTargets lock tiles 1`] = `
       extraScrollHeight={62}
       getScrollResponder={[Function]}
       handleOnScroll={[Function]}
+      innerRef={[Function]}
       keyboardDismissMode="interactive"
       keyboardOpeningTime={250}
       keyboardSpace={0}
@@ -9661,6 +9668,7 @@ exports[`SendScene2 2 spendTargets lock tiles 2`] = `
       extraScrollHeight={62}
       getScrollResponder={[Function]}
       handleOnScroll={[Function]}
+      innerRef={[Function]}
       keyboardDismissMode="interactive"
       keyboardOpeningTime={250}
       keyboardSpace={0}
@@ -10976,6 +10984,7 @@ exports[`SendScene2 2 spendTargets lock tiles 3`] = `
       extraScrollHeight={62}
       getScrollResponder={[Function]}
       handleOnScroll={[Function]}
+      innerRef={[Function]}
       keyboardDismissMode="interactive"
       keyboardOpeningTime={250}
       keyboardSpace={0}
@@ -12225,6 +12234,7 @@ exports[`SendScene2 Render SendScene 1`] = `
       extraScrollHeight={62}
       getScrollResponder={[Function]}
       handleOnScroll={[Function]}
+      innerRef={[Function]}
       keyboardDismissMode="interactive"
       keyboardOpeningTime={250}
       keyboardSpace={0}

src/components/scenes/SendScene2.tsx

@@ -69,6 +69,8 @@ import { ErrorTile } from '../tiles/ErrorTile'
 
 // TODO: Check contentPadding
 
+const SCROLL_TO_END_DELAY_MS = 150
+
 interface Props extends EdgeAppSceneProps<'send2'> {}
 
 export interface SendScene2Params {
@@ -128,7 +130,9 @@ const SendComponent = (props: Props) => {
   const theme = useTheme()
   const styles = getStyles(theme)
 
+  const needsScrollToEnd = React.useRef<boolean>(false)
   const makeSpendCounter = React.useRef<number>(0)
+  const scrollViewRef = React.useRef<KeyboardAwareScrollView | null>(null)
 
   const initialMount = React.useRef<boolean>(true)
   const pinInputRef = React.useRef<TextInput>(null)
@@ -269,6 +273,7 @@ const SendComponent = (props: Props) => {
         setMinNativeAmount(parsedUri.minNativeAmount)
         setExpireDate(parsedUri?.expireDate)
         setSpendInfo({ ...spendInfo })
+        needsScrollToEnd.current = true
       }
     }
 
@@ -278,6 +283,7 @@ const SendComponent = (props: Props) => {
     setLastAddressEntryMethod(undefined)
     spendInfo.spendTargets.splice(index, 1)
     setSpendInfo({ ...spendInfo })
+    needsScrollToEnd.current = true
   }
 
   const renderAddressAmountTile = (index: number, spendTarget: EdgeSpendTarget) => {
@@ -357,6 +363,7 @@ const SendComponent = (props: Props) => {
     setSpendInfo({ ...spendInfo })
     setMaxSpendSetter(-1)
     setFieldChanged(newField)
+    needsScrollToEnd.current = true
   }
 
   const handleSetMax = (index: number) => () => {
@@ -482,6 +489,7 @@ const SendComponent = (props: Props) => {
   const handleAddAddress = useHandler(() => {
     spendInfo.spendTargets.push({})
     setSpendInfo({ ...spendInfo })
+    needsScrollToEnd.current = true
   })
 
   const renderAddAddress = () => {
@@ -1139,6 +1147,19 @@ const SendComponent = (props: Props) => {
     backgroundColors[0] = scaledColor
   }
 
+  React.useEffect(() => {
+    // Hack: While you would think to use InteractionManager.runAfterInteractions,
+    // it doesn't work because several renders occur before the full height is
+    // determined and the scrollToEnd call would be effective.
+    const timeout = setTimeout(() => {
+      if (needsScrollToEnd.current) {
+        scrollViewRef.current?.scrollToEnd(true)
+        needsScrollToEnd.current = false
+      }
+    }, SCROLL_TO_END_DELAY_MS)
+    return () => clearTimeout(timeout)
+  })
+
   return (
     <SceneWrapper
       hasNotifications
@@ -1152,6 +1173,10 @@ const SendComponent = (props: Props) => {
       {({ insetStyle }) => (
         <>
           <StyledKeyboardAwareScrollView
+            innerRef={ref => {
+              const kbRef: KeyboardAwareScrollView | null = ref as any
+              scrollViewRef.current = kbRef
+            }}
             contentContainerStyle={{
               ...insetStyle,
               paddingTop: 0,