Skip to content

Commit 023df41

Browse files
committed
Fix AddWalletScreen for small displays
Fix success message on feedback submission
1 parent 2e8f25f commit 023df41

File tree

2 files changed

+31
-31
lines changed

2 files changed

+31
-31
lines changed

common/src/commonMain/kotlin/com/blockstream/common/models/support/SupportViewModel.kt

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,12 @@ package com.blockstream.common.models.support
22

33
import blockstream_green.common.generated.resources.Res
44
import blockstream_green.common.generated.resources.id_contact_support
5+
import blockstream_green.common.generated.resources.id_thank_you_for_your_feedback
56
import blockstream_green.common.generated.resources.id_thanks_your_message_has_been_sent
67
import com.blockstream.common.SupportType
7-
import com.blockstream.common.data.SupportData
88
import com.blockstream.common.data.GreenWallet
99
import com.blockstream.common.data.NavData
10+
import com.blockstream.common.data.SupportData
1011
import com.blockstream.common.events.Event
1112
import com.blockstream.common.events.Events
1213
import com.blockstream.common.extensions.isNotBlank
@@ -98,7 +99,7 @@ class SupportViewModel(type: SupportType, supportData: SupportData, greenWalletO
9899
autoRetry = false
99100
)
100101
}, onSuccess = {
101-
postSideEffect(SideEffects.Snackbar(text = StringHolder.create(Res.string.id_thanks_your_message_has_been_sent)))
102+
postSideEffect(SideEffects.Snackbar(text = StringHolder.create(if (type == SupportType.FEEDBACK) Res.string.id_thank_you_for_your_feedback else Res.string.id_thanks_your_message_has_been_sent)))
102103
postSideEffect(SideEffects.NavigateBack())
103104
})
104105
}

compose/src/commonMain/kotlin/com/blockstream/compose/screens/onboarding/phone/AddWalletScreen.kt

Lines changed: 28 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,8 @@ import androidx.compose.foundation.Image
44
import androidx.compose.foundation.layout.Arrangement
55
import androidx.compose.foundation.layout.Box
66
import androidx.compose.foundation.layout.Column
7-
import androidx.compose.foundation.layout.fillMaxHeight
7+
import androidx.compose.foundation.layout.Spacer
8+
import androidx.compose.foundation.layout.fillMaxSize
89
import androidx.compose.foundation.layout.fillMaxWidth
910
import androidx.compose.foundation.layout.padding
1011
import androidx.compose.material3.Text
@@ -60,7 +61,7 @@ fun AddWalletScreen(
6061
) {
6162

6263
EnvironmentBottomSheet.getResult {
63-
if(it >= 0) {
64+
if (it >= 0) {
6465
viewModel.postEvent(
6566
AddWalletViewModel.LocalEvents.SelectEnviroment(
6667
isTestnet = it == 1,
@@ -74,15 +75,14 @@ fun AddWalletScreen(
7475

7576
Column(
7677
modifier = Modifier
77-
.fillMaxWidth()
78-
.fillMaxHeight(),
78+
.fillMaxSize(),
7979
horizontalAlignment = Alignment.CenterHorizontally,
8080
verticalArrangement = Arrangement.SpaceEvenly
8181
) {
8282

8383
Box(
8484
modifier = Modifier
85-
.weight(6f)
85+
.weight(2f)
8686
.padding(horizontal = 24.dp),
8787
) {
8888
Image(
@@ -102,7 +102,6 @@ fun AddWalletScreen(
102102

103103
GreenColumn(
104104
space = 8,
105-
modifier = Modifier.padding(horizontal = 0.dp).weight(4f),
106105
horizontalAlignment = Alignment.CenterHorizontally
107106
) {
108107
Text(
@@ -118,32 +117,32 @@ fun AddWalletScreen(
118117
)
119118
}
120119

121-
Column {
122-
GreenColumn(
123-
padding = 24,
124-
modifier = Modifier,
125-
horizontalAlignment = Alignment.CenterHorizontally,
126-
) {
120+
Spacer(modifier = Modifier.weight(1f))
121+
122+
GreenColumn(
123+
padding = 0,
124+
modifier = Modifier.padding(horizontal = 24.dp).padding(bottom = 24.dp),
125+
horizontalAlignment = Alignment.CenterHorizontally,
126+
) {
127127

128-
GreenButton(
129-
stringResource(Res.string.id_new_wallet),
130-
modifier = Modifier.fillMaxWidth(),
131-
size = GreenButtonSize.BIG,
132-
) {
133-
viewModel.postEvent(AddWalletViewModel.LocalEvents.NewWallet)
134-
}
135-
136-
GreenButton(
137-
stringResource(Res.string.id_restore_wallet),
138-
modifier = Modifier.fillMaxWidth(),
139-
type = GreenButtonType.OUTLINE,
140-
color = GreenButtonColor.WHITE,
141-
size = GreenButtonSize.BIG,
142-
) {
143-
viewModel.postEvent(AddWalletViewModel.LocalEvents.RestoreWallet)
144-
}
128+
GreenButton(
129+
stringResource(Res.string.id_new_wallet),
130+
modifier = Modifier.fillMaxWidth(),
131+
size = GreenButtonSize.BIG,
132+
) {
133+
viewModel.postEvent(AddWalletViewModel.LocalEvents.NewWallet)
134+
}
145135

136+
GreenButton(
137+
stringResource(Res.string.id_restore_wallet),
138+
modifier = Modifier.fillMaxWidth(),
139+
type = GreenButtonType.OUTLINE,
140+
color = GreenButtonColor.WHITE,
141+
size = GreenButtonSize.BIG,
142+
) {
143+
viewModel.postEvent(AddWalletViewModel.LocalEvents.RestoreWallet)
146144
}
145+
147146
}
148147
}
149148

0 commit comments

Comments
 (0)