From a3eb9879370968faf63ae739c7e3037cc7da343d Mon Sep 17 00:00:00 2001
From: timklge <2026103+timklge@users.noreply.github.com>
Date: Wed, 12 Mar 2025 23:23:16 +0100
Subject: [PATCH] Autoclose tone select dialog on select (#39)

---
 .../karooreminder/screens/DetailScreen.kt     | 29 ++++---------------
 1 file changed, 5 insertions(+), 24 deletions(-)

diff --git a/app/src/main/kotlin/de/timklge/karooreminder/screens/DetailScreen.kt b/app/src/main/kotlin/de/timklge/karooreminder/screens/DetailScreen.kt
index b9ac66e..7d9ccff 100644
--- a/app/src/main/kotlin/de/timklge/karooreminder/screens/DetailScreen.kt
+++ b/app/src/main/kotlin/de/timklge/karooreminder/screens/DetailScreen.kt
@@ -313,8 +313,6 @@ fun DetailScreen(isCreating: Boolean, reminder: Reminder, onSubmit: (updatedRemi
 
             if (toneDialogVisible){
                 Dialog(onDismissRequest = { toneDialogVisible = false }) {
-                    var dialogSelectedTone by remember { mutableStateOf(selectedTone) }
-
                     Card(
                         modifier = Modifier
                             .fillMaxWidth()
@@ -332,11 +330,13 @@ fun DetailScreen(isCreating: Boolean, reminder: Reminder, onSubmit: (updatedRemi
                                 Row(modifier = Modifier
                                     .fillMaxWidth()
                                     .clickable {
-                                        dialogSelectedTone = pattern
+                                        selectedTone = pattern
                                         karooSystem.dispatch(PlayBeepPattern(tones))
+                                        toneDialogVisible = false
                                     }, verticalAlignment = Alignment.CenterVertically) {
-                                    RadioButton(selected = dialogSelectedTone == pattern, onClick = {
-                                        dialogSelectedTone = pattern
+                                    RadioButton(selected = selectedTone == pattern, onClick = {
+                                        selectedTone = pattern
+                                        toneDialogVisible = false
                                         karooSystem.dispatch(PlayBeepPattern(tones))
                                     })
                                     Text(
@@ -345,25 +345,6 @@ fun DetailScreen(isCreating: Boolean, reminder: Reminder, onSubmit: (updatedRemi
                                     )
                                 }
                             }
-
-                            FilledTonalButton(modifier = Modifier
-                                .fillMaxWidth()
-                                .height(50.dp), onClick = {
-                                selectedTone = dialogSelectedTone
-                                toneDialogVisible = false
-                            }) {
-                                Icon(Icons.Default.Done, contentDescription = "Save")
-                                Text("Save")
-                            }
-
-                            FilledTonalButton(modifier = Modifier
-                                .fillMaxWidth()
-                                .height(50.dp), onClick = {
-                                toneDialogVisible = false
-                            }) {
-                                Icon(Icons.Default.Close, contentDescription = "Cancel")
-                                Text("Cancel")
-                            }
                         }
                     }
                 }