avi/S.H.O.N.A.R._Desktop_Companion
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Settings screen: make root column scrollable

Browse source 
Model cards + folder row overflowed in short windows (split-screen) with
no scroll path — the only screen where content was truly unreachable.
Library/Detail already scroll via LazyColumns; verified compileKotlin +
39 unit tests green.
This commit is contained in:
avi 2026-09-15 14:00:25 -05:00
commit 0d8f3be614
1 changed files with 6 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

7
app/src/main/kotlin/com/shonar/desktop/Screens.kt
View file

@ -14,6 +14,8 @@ import androidx.compose.foundation.layout.size
import androidx.compose.foundation.layout.width
import androidx.compose.foundation.lazy.LazyColumn
import androidx.compose.foundation.lazy.items
import androidx.compose.foundation.rememberScrollState
import androidx.compose.foundation.verticalScroll
import androidx.compose.material.icons.Icons
import androidx.compose.material.icons.filled.Delete
import androidx.compose.material.icons.filled.Edit
@ -833,7 +835,10 @@ fun SettingsScreen(state: DesktopState) {
val folder by state.folder.collectAsState()
var showPicker by remember { mutableStateOf(false) }
Column(Modifier.fillMaxSize(), verticalArrangement = Arrangement.spacedBy(12.dp)) {
// Root is scrollable: in split-screen/short windows the model cards
// (and everything below) would otherwise clip with no way to reach them.
Column(Modifier.fillMaxSize().verticalScroll(rememberScrollState()),
verticalArrangement = Arrangement.spacedBy(12.dp)) {
Row(verticalAlignment = Alignment.CenterVertically) {
TextButton({ state.go(DesktopState.Screen.LIBRARY) }) { Text("← Library") }
Text("Settings", style = MaterialTheme.typography.headlineSmall)