diff options
| -rw-r--r-- | app/src/main/java/org/terst/listen/ui/SettingRows.kt | 6 | ||||
| -rw-r--r-- | app/src/main/java/org/terst/listen/ui/SettingsScreen.kt | 7 |
2 files changed, 10 insertions, 3 deletions
diff --git a/app/src/main/java/org/terst/listen/ui/SettingRows.kt b/app/src/main/java/org/terst/listen/ui/SettingRows.kt index a1a6db9..b224719 100644 --- a/app/src/main/java/org/terst/listen/ui/SettingRows.kt +++ b/app/src/main/java/org/terst/listen/ui/SettingRows.kt @@ -31,7 +31,11 @@ fun SwitchRow( ) { Row( Modifier.fillMaxWidth().padding(vertical = 4.dp), - verticalAlignment = Alignment.CenterVertically, + // Aligned to the top rather than the middle. Centring looks right while every label + // fits on one line and goes wrong the moment one wraps: the switch drifts down beside + // the middle of the block and stops lining up with the switch above it, which is + // exactly what the settings screen looked like. + verticalAlignment = Alignment.Top, ) { Switch( modifier = Modifier.semantics { contentDescription = description }, diff --git a/app/src/main/java/org/terst/listen/ui/SettingsScreen.kt b/app/src/main/java/org/terst/listen/ui/SettingsScreen.kt index 00f973d..8f4aeca 100644 --- a/app/src/main/java/org/terst/listen/ui/SettingsScreen.kt +++ b/app/src/main/java/org/terst/listen/ui/SettingsScreen.kt @@ -140,8 +140,11 @@ private fun SkippingSection(settings: SkipServerSettings, furniture: FurnitureSk "its break cue and its closing credits.", ) SwitchRow( - label = "Race through breaks at 3x instead of skipping", - description = "Turn racing through breaks at 3x instead of skipping them on or off", + // "instead of skipping" came off the label: it wrapped the line onto two and the + // detail underneath already draws that contrast. The multiplication sign matches + // the speed chip, which has always used one. + label = "Race through breaks at 3\u00d7", + description = "Turn racing through breaks instead of skipping them on or off", checked = furniture.fastPlayBreaks, onCheckedChange = furniture.onFastPlayBreaksChange, // Says "breaks" rather than "ads" deliberately: this chooses what happens to |