fix: update action paddings on confirm TX screen (#3064)

Emurgo · Jan 30, 2024 · 564972a · 564972a
1 parent 498a8b2
commit 564972a
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 7 deletions.
diff --git a/apps/wallet-mobile/src/components/ScrollView/ScrollView.tsx b/apps/wallet-mobile/src/components/ScrollView/ScrollView.tsx
@@ -14,12 +14,12 @@ export const ScrollView = ({
   return (
     <RNScrollView
       onLayout={(event) => {
-        if (!onScrollBarChange) return
+        if (onScrollBarChange) {
+          const {height} = event.nativeEvent.layout
 
-        const {height} = event.nativeEvent.layout
-
-        const shouldChange = wrapperHeight > Math.trunc(height)
-        onScrollBarChange(shouldChange)
+          const shouldChange = wrapperHeight > Math.trunc(height)
+          onScrollBarChange(shouldChange)
+        }
         props.onLayout?.(event)
       }}
       {...props}

diff --git a/apps/wallet-mobile/src/features/Settings/ManageCollateral/ConfirmTx/ConfirmTxScreen.tsx b/apps/wallet-mobile/src/features/Settings/ManageCollateral/ConfirmTx/ConfirmTxScreen.tsx
@@ -1,7 +1,7 @@
 /* eslint-disable @typescript-eslint/no-explicit-any */
 import React, {useEffect} from 'react'
 import {useIntl} from 'react-intl'
-import {ScrollView, StyleSheet, View, ViewProps} from 'react-native'
+import {Platform, ScrollView, StyleSheet, View, ViewProps} from 'react-native'
 import {SafeAreaView} from 'react-native-safe-area-context'
 
 import {KeyboardAvoidingView, Spacer, ValidatedTextInput} from '../../../../components'
@@ -104,7 +104,7 @@ export const ConfirmTxScreen = () => {
   )
 }
 
-const Actions = (props: ViewProps) => <View {...props} style={{padding: 16}} />
+const Actions = (props: ViewProps) => <View style={styles.actions} {...props} />
 
 const styles = StyleSheet.create({
   root: {
@@ -115,6 +115,11 @@ const styles = StyleSheet.create({
     flex: 1,
     paddingHorizontal: 16,
   },
+  actions: {
+    paddingTop: 16,
+    paddingHorizontal: 16,
+    paddingBottom: Platform.OS === 'ios' ? 25 : 16,
+  },
 })
 
 const useStrings = () => {