From b2bfdc0f4288da17b126ad3ba15ec5472f1939d0 Mon Sep 17 00:00:00 2001 From: Jean-Luc Makiola Date: Sat, 27 Jun 2026 15:36:19 +0200 Subject: [PATCH] refactor(settings): regroup Appearance into theme / calendar / agenda MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Drop the uniform two-per-card grouping for meaningful sections: theme & colour (2), calendar — default view, week start, time format, hour lines (4), and agenda (2). Co-Authored-By: Claude Opus 4.8 (1M context) --- .../calendula/ui/settings/SettingsScreen.kt | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/app/src/main/java/de/jeanlucmakiola/calendula/ui/settings/SettingsScreen.kt b/app/src/main/java/de/jeanlucmakiola/calendula/ui/settings/SettingsScreen.kt index a3decae..449602e 100644 --- a/app/src/main/java/de/jeanlucmakiola/calendula/ui/settings/SettingsScreen.kt +++ b/app/src/main/java/de/jeanlucmakiola/calendula/ui/settings/SettingsScreen.kt @@ -470,7 +470,7 @@ private fun AppearanceScreen( Spacer(Modifier.height(16.dp)) - // Calendar layout + // Calendar — view, week, and timeline formatting GroupedRow( title = stringResource(R.string.settings_default_view), summary = stringResource(state.defaultView.labelRes), @@ -480,17 +480,13 @@ private fun AppearanceScreen( GroupedRow( title = stringResource(R.string.settings_week_start), summary = weekStartLabel(state.weekStart), - position = Position.Bottom, + position = Position.Middle, onClick = { showWeekStart = true }, ) - - Spacer(Modifier.height(16.dp)) - - // Timeline display GroupedRow( title = stringResource(R.string.settings_time_format), summary = timeFormatLabel(state.timeFormat), - position = Position.Top, + position = Position.Middle, onClick = { showTimeFormat = true }, ) GroupedRow(