Skip to content

Commit

Permalink
fix(fiatconnect): clear button position on fiat details screen (#2775)
Browse files Browse the repository at this point in the history
### Description

Before:
<img alt="fiat-details-clear-before" src="https://user-images.githubusercontent.com/8432644/184025505-8b226d26-0ac2-48c5-91c3-23674fa7ab1a.png" width="200">

After:
<img alt="fiat-details-clear" src="https://user-images.githubusercontent.com/5062591/184043480-3d6ac245-a933-4373-9186-700f8ed3849e.png" width="200">


### Other changes

N/A

### Tested

Manual

### How others should test

1. Ensure you've not already linked a bank account with Valora test provider
2. Navigate to Withdraw flow on Alfajores
3. Select Bank Account with Valora test provider and click on Continue
4. Type on the Bank Name and Account Number text fields and ensure clear button is aligned to the center of the field

### Related issues

- Fixes ACT-181

### Backwards compatibility

Yes
  • Loading branch information
satish-ravi authored Aug 11, 2022
1 parent d3e077a commit 1922a6c
Showing 1 changed file with 7 additions and 6 deletions.
13 changes: 7 additions & 6 deletions src/fiatconnect/FiatDetailsScreen.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -395,7 +395,8 @@ function FormField({
) : (
<TextInput
testID={`input-${field.name}`}
style={styles.formInput}
style={styles.formInputContainer}
inputStyle={styles.formInput}
value={value}
placeholder={field.placeholderText}
onChangeText={onInputChange}
Expand Down Expand Up @@ -438,24 +439,24 @@ const styles = StyleSheet.create({
inputView: {
paddingVertical: 12,
},
formInput: {
...fontStyles.regular,
formInputContainer: {
borderRadius: 4,
borderWidth: 1.5,
borderColor: colors.gray2,
marginBottom: 4,
color: colors.dark,
alignItems: 'flex-start',
paddingHorizontal: 8,
},
formInput: {
...fontStyles.regular,
color: colors.dark,
},
formSelectInput: {
...fontStyles.regular,
borderRadius: 4,
borderWidth: 1.5,
borderColor: colors.gray2,
marginBottom: 4,
color: colors.dark,
alignItems: 'flex-start',
paddingHorizontal: 8,
paddingVertical: 12,
lineHeight: LINE_HEIGHT,
Expand Down

0 comments on commit 1922a6c

Please sign in to comment.