fix(settings): make whole toggle row tappable; clarify OEM keep-alive button (#22)

Two phone-only UX fixes that the on-device test surfaced.

1. The Material3 Switch widget is small on a real phone, and the
   surrounding ToggleRow had no click handler of its own. Tapping the
   label area did nothing, so the suggestions, network, and update-check
   toggles all felt "stuck in their default position" even though the
   wiring underneath was fine. Use the canonical Compose pattern:
   Modifier.toggleable on the Row (role = Role.Switch), with the inner
   Switch as a visual indicator (onCheckedChange = null) so there are no
   competing click handlers. The whole row is now a single, accessible
   click target.

2. Rename the "Keep it alive on your phone" button to "Open OEM
   keep-alive guide (web)". The old text implied the app could do it
   itself; in reality the button opens the dontkillmyapp.com page in a
   browser. The new label says what it actually does.

Co-authored-by: FlintWave <flintwave@tuta.com>
Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>

Author: 3 people

app/src/main/java/org/searchmob/ui/settings/SettingsScreen.kt

@@ -11,6 +11,7 @@ import androidx.compose.foundation.layout.fillMaxWidth
 import androidx.compose.foundation.layout.padding
 import androidx.compose.foundation.rememberScrollState
 import androidx.compose.foundation.selection.selectable
+import androidx.compose.foundation.selection.toggleable
 import androidx.compose.foundation.verticalScroll
 import androidx.compose.material.icons.Icons
 import androidx.compose.material.icons.automirrored.filled.ArrowBack
@@ -43,6 +44,7 @@ import androidx.compose.ui.Modifier
 import androidx.compose.ui.platform.LocalClipboardManager
 import androidx.compose.ui.platform.LocalContext
 import androidx.compose.ui.platform.testTag
+import androidx.compose.ui.semantics.Role
 import androidx.compose.ui.text.AnnotatedString
 import androidx.compose.ui.text.input.PasswordVisualTransformation
 import androidx.compose.ui.unit.dp
@@ -324,8 +326,20 @@ private fun ToggleRow(
     tag: String,
     onCheckedChange: (Boolean) -> Unit,
 ) {
+    // The whole row is the click target: a phone-finger tap on the label or supporting text flips the
+    // switch, not just a hit on the small Material3 Switch widget. The inner Switch is purely visual
+    // (onCheckedChange = null) so we do not register two competing click handlers on the same row.
     Row(
-        modifier = Modifier.fillMaxWidth().padding(vertical = 4.dp),
+        modifier =
+            Modifier
+                .fillMaxWidth()
+                .toggleable(
+                    value = checked,
+                    role = Role.Switch,
+                    onValueChange = onCheckedChange,
+                )
+                .padding(vertical = 4.dp)
+                .testTag(tag),
         verticalAlignment = Alignment.CenterVertically,
         horizontalArrangement = Arrangement.SpaceBetween,
     ) {
@@ -339,7 +353,7 @@ private fun ToggleRow(
                 )
             }
         }
-        Switch(checked = checked, onCheckedChange = onCheckedChange, modifier = Modifier.testTag(tag))
+        Switch(checked = checked, onCheckedChange = null)
     }
 }
 

app/src/main/res/values/strings.xml

@@ -25,7 +25,7 @@
     <string name="battery_allow">Allow always-on</string>
 
     <!-- OEM guidance -->
-    <string name="oem_guidance_button">Keep it alive on your phone</string>
+    <string name="oem_guidance_button">Open OEM keep-alive guide (web)</string>
     <string name="oem_guidance_warning">Heads up: some manufacturers reset these settings after a firmware/OS update, so you may need to re-apply them.</string>
 
     <!-- Common -->