Skip to content

Commit

Permalink
Fix scroll and button text and secure now goes to next step (#1737)
Browse files Browse the repository at this point in the history
* Fix scroll and button text

* Update tests

* secure now goes to next step

Co-authored-by: Ibrahim Taveras <ibrahimtaveras00@gmail.com>
  • Loading branch information
andrepimenta and ibrahimtaveras00 authored Jul 30, 2020
1 parent 42e68d0 commit d8c629b
Show file tree
Hide file tree
Showing 6 changed files with 13 additions and 10 deletions.
1 change: 1 addition & 0 deletions app/components/Views/AccountBackupStep1/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -185,6 +185,7 @@ const AccountBackupStep1 = props => {
const hideRemindLaterModal = () => {
setToggleSkipCheckbox(false);
setRemindLaterModal(false);
goNext();
};

const skip = async () => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ exports[`ChoosePassword should render correctly 1`] = `
style={
Object {
"flex": 1,
"marginHorizontal": 32,
"marginBottom": 10,
}
}
testID="choose-password-screen"
Expand Down Expand Up @@ -49,6 +49,7 @@ exports[`ChoosePassword should render correctly 1`] = `
style={
Object {
"flex": 1,
"paddingHorizontal": 32,
}
}
viewIsInsideTabBar={false}
Expand Down Expand Up @@ -318,7 +319,6 @@ exports[`ChoosePassword should render correctly 1`] = `
Object {
"height": 18,
"margin": 10,
"marginLeft": -6,
"width": 18,
}
}
Expand Down Expand Up @@ -361,6 +361,7 @@ exports[`ChoosePassword should render correctly 1`] = `
<View
style={
Object {
"flex": 1,
"marginTop": 20,
"paddingHorizontal": 10,
}
Expand All @@ -382,7 +383,7 @@ exports[`ChoosePassword should render correctly 1`] = `
testID="submit-button"
type="blue"
>
NEXT
Create password
</StyledButton>
</View>
</KeyboardAwareScrollViewMock>
Expand Down
9 changes: 5 additions & 4 deletions app/components/Views/ChoosePassword/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -30,10 +30,11 @@ const styles = StyleSheet.create({
},
wrapper: {
flex: 1,
marginHorizontal: 32
marginBottom: 10
},
scrollableWrapper: {
flex: 1
flex: 1,
paddingHorizontal: 32
},
keyboardScrollableWrapper: {
flexGrow: 1
Expand Down Expand Up @@ -91,8 +92,7 @@ const styles = StyleSheet.create({
checkbox: {
width: 18,
height: 18,
margin: 10,
marginLeft: -6
margin: 10
},
label: {
...fontStyles.normal,
Expand All @@ -117,6 +117,7 @@ const styles = StyleSheet.create({
...fontStyles.normal
},
ctaWrapper: {
flex: 1,
marginTop: 20,
paddingHorizontal: 10
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -298,7 +298,7 @@ exports[`ChoosePasswordSimple should render correctly 1`] = `
testID="submit-button"
type="blue"
>
NEXT
Create password
</StyledButton>
</View>
</View>
Expand Down
2 changes: 1 addition & 1 deletion locales/en.json
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,7 @@
"subtitle": "This password will unlock your MetaMask wallet only on this device.",
"password": "New Password",
"confirm_password": "Confirm Password",
"create_button": "NEXT",
"create_button": "Create password",
"import_with_seed_phrase": "Import with seed phrase",
"password_length_error": "The password needs to be at least 8 characters long",
"password_dont_match": "Passwords don't match",
Expand Down
2 changes: 1 addition & 1 deletion locales/es.json
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,7 @@
"subtitle": "Utilizarás esta contraseña para acceder a tu billetera de MetaMask solamente en este dispositivo.",
"password": "Nueva Contraseña",
"confirm_password": "Confirmar Contraseña",
"create_button": "Siguiente",
"create_button": "Crear contraseña",
"import_with_seed_phrase": "Importar con frase semilla",
"password_length_error": "La contraseña debe contener al menos 8 caracteres",
"password_dont_match": "Las contraseñas no coinciden",
Expand Down

0 comments on commit d8c629b

Please sign in to comment.