fix NumberInputBox cursor pos reset when typing

Author: aj3423

app/src/main/java/spam/blocker/ui/setting/Testing.kt

@@ -26,6 +26,7 @@ import spam.blocker.service.SmsReceiver
 import spam.blocker.ui.theme.DarkOrange
 import spam.blocker.ui.theme.LocalPalette
 import spam.blocker.ui.theme.Teal200
+import spam.blocker.ui.widgets.AnimatedVisibleV
 import spam.blocker.ui.widgets.BalloonQuestionMark
 import spam.blocker.ui.widgets.GreyLabel
 import spam.blocker.ui.widgets.PopupDialog
@@ -158,7 +159,7 @@ fun PopupTesting(
                     },
                 )
                 // SMS content
-                if (vm.selectedType.intValue != Def.ForNumber) {
+                AnimatedVisibleV(vm.selectedType.intValue != Def.ForNumber) {
                     StrInputBox(
                         text = vm.sms.value,
                         label = { GreyLabel(Str(R.string.sms_content)) },

app/src/main/java/spam/blocker/ui/widgets/InputBox.kt

@@ -36,6 +36,7 @@ import androidx.compose.ui.layout.onGloballyPositioned
 import androidx.compose.ui.layout.positionOnScreen
 import androidx.compose.ui.platform.LocalContext
 import androidx.compose.ui.semantics.semantics
+import androidx.compose.ui.text.TextRange
 import androidx.compose.ui.text.TextStyle
 import androidx.compose.ui.text.font.FontWeight
 import androidx.compose.ui.text.input.KeyboardType
@@ -241,11 +242,21 @@ fun NumberInputBox(
     helpTooltip: @Composable (() -> Unit)? = null,
 ) {
     // Code learned from the built-in BasicTextField.kt
-    var state by remember(intValue) {
+    var state by remember {
         mutableStateOf(
             TextFieldValue(text = intValue?.toString() ?: "")
         )
     }
+    LaunchedEffect(intValue) {
+        if (intValue?.toString() != state.text) {
+            state = TextFieldValue(
+                text = intValue?.toString() ?: "",
+                selection = TextRange(intValue?.toString()?.length ?: 0)
+            )
+        }
+    }
+
+
     // Last String value that either text field was recomposed with or updated in the onValueChange
     // callback. We keep track of it to prevent calling onValueChange(String) for same String when
     // CoreTextField's onValueChange is called multiple times without recomposition in between.