Skip to content

Commit 673db19

Browse files
Move the alert banner below the profile card
1 parent 131c9ce commit 673db19

File tree

1 file changed

+8
-8
lines changed
  • gravatar-quickeditor/src/main/java/com/gravatar/quickeditor/ui/avatarpicker

1 file changed

+8
-8
lines changed

gravatar-quickeditor/src/main/java/com/gravatar/quickeditor/ui/avatarpicker/AvatarPicker.kt

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -205,25 +205,25 @@ internal fun AvatarPicker(uiState: AvatarPickerUiState, onEvent: (AvatarPickerEv
205205
.fillMaxWidth()
206206
.padding(bottom = 10.dp),
207207
)
208+
ProfileCard(
209+
profile = uiState.profile,
210+
email = uiState.email,
211+
avatarCacheBuster = uiState.avatarCacheBuster.toString(),
212+
modifier = Modifier.padding(start = 16.dp, end = 16.dp, bottom = 16.dp),
213+
)
208214
AnimatedVisibility(visible = uiState.nonSelectedAvatarAlertVisible) {
209215
AlertBanner(
210216
message = stringResource(id = R.string.gravatar_qe_alert_banner_no_avatar_selected),
211217
onClose = { onEvent(AvatarPickerEvent.AvatarDeleteAlertDismissed) },
212-
modifier = Modifier.padding(start = 16.dp, end = 16.dp, bottom = 24.dp),
218+
modifier = Modifier.padding(horizontal = 16.dp),
213219
)
214220
}
215-
ProfileCard(
216-
profile = uiState.profile,
217-
email = uiState.email,
218-
avatarCacheBuster = uiState.avatarCacheBuster.toString(),
219-
modifier = Modifier.padding(horizontal = 16.dp),
220-
)
221221
Box(
222222
modifier = Modifier
223223
.fillMaxWidth()
224224
.animateContentSize(),
225225
) {
226-
val sectionModifier = Modifier.padding(top = 24.dp, bottom = 10.dp)
226+
val sectionModifier = Modifier.padding(top = 16.dp, bottom = 10.dp)
227227
when {
228228
uiState.isLoading -> Box(
229229
modifier = sectionModifier

0 commit comments

Comments
 (0)