From 45fe755470e21757c3237d278e10e5a13d031eb1 Mon Sep 17 00:00:00 2001 From: atsunatsu Date: Wed, 16 Sep 2026 16:53:50 +0800 Subject: [PATCH] ui(dialogs): pad title away from cancel; LoTW buttons bottom corners - SharedDialog title gets start padding when only a cancel button is shown (was flush against it); dialogs with both buttons unchanged. - LoTW dialog: full-sync button bottom-left, incremental-merge bottom-right (SpaceBetween instead of side-by-side). --- .../com/rtbishop/look4sat/core/presentation/Components.kt | 5 ++++- .../com/rtbishop/look4sat/feature/settings/SettingsDialog.kt | 3 ++- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/core/presentation/src/main/java/com/rtbishop/look4sat/core/presentation/Components.kt b/core/presentation/src/main/java/com/rtbishop/look4sat/core/presentation/Components.kt index b22fdeea..998af9aa 100644 --- a/core/presentation/src/main/java/com/rtbishop/look4sat/core/presentation/Components.kt +++ b/core/presentation/src/main/java/com/rtbishop/look4sat/core/presentation/Components.kt @@ -375,7 +375,10 @@ fun SharedDialog( overflow = TextOverflow.Ellipsis, modifier = Modifier .weight(1f) - .padding(horizontal = if (onCancel != null && onAccept != null) padding else 0.dp) + .padding( + start = if (onCancel != null) padding else 0.dp, + end = if (onAccept != null) padding else 0.dp + ) ) if (onAccept != null) { CardButton( diff --git a/feature/settings/src/main/java/com/rtbishop/look4sat/feature/settings/SettingsDialog.kt b/feature/settings/src/main/java/com/rtbishop/look4sat/feature/settings/SettingsDialog.kt index ddf4ca6e..ca8f10a7 100644 --- a/feature/settings/src/main/java/com/rtbishop/look4sat/feature/settings/SettingsDialog.kt +++ b/feature/settings/src/main/java/com/rtbishop/look4sat/feature/settings/SettingsDialog.kt @@ -323,9 +323,10 @@ fun LoTWDialog( ) } Row( - horizontalArrangement = Arrangement.spacedBy(8.dp), + horizontalArrangement = Arrangement.SpaceBetween, modifier = Modifier.fillMaxWidth().padding(horizontal = LocalSpacing.current.large) ) { + // Full sync bottom-left, incremental merge bottom-right. CardButton( onClick = { onSyncFull(com.rtbishop.look4sat.core.domain.model.LoTWSettings(call.value, pass.value)) }, text = stringResource(R.string.lotw_sync_full),