From 8f4474c8a7fe252c4a12e606f99c070e50fdb6e8 Mon Sep 17 00:00:00 2001 From: Claude Date: Thu, 30 Apr 2026 20:13:28 +0000 Subject: [PATCH] fix(nests): make Open lobby action a prominent filled button A TextButton in the AppBar actions slot read as too subtle and small next to the title. Switch to a filled Button so the primary CTA is unmistakable. The actions Row already centers children vertically; trim the trailing modifier padding so the button sits flush with the bar's edge. --- .../loggedIn/nests/room/lobby/NestLobbyScreen.kt | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/amethyst/src/main/java/com/vitorpamplona/amethyst/ui/screen/loggedIn/nests/room/lobby/NestLobbyScreen.kt b/amethyst/src/main/java/com/vitorpamplona/amethyst/ui/screen/loggedIn/nests/room/lobby/NestLobbyScreen.kt index e2b79b3bd..81c217c44 100644 --- a/amethyst/src/main/java/com/vitorpamplona/amethyst/ui/screen/loggedIn/nests/room/lobby/NestLobbyScreen.kt +++ b/amethyst/src/main/java/com/vitorpamplona/amethyst/ui/screen/loggedIn/nests/room/lobby/NestLobbyScreen.kt @@ -38,12 +38,13 @@ import androidx.compose.foundation.layout.windowInsetsPadding import androidx.compose.foundation.lazy.LazyColumn import androidx.compose.foundation.lazy.items import androidx.compose.foundation.lazy.rememberLazyListState +import androidx.compose.material3.Button +import androidx.compose.material3.ButtonDefaults import androidx.compose.material3.ExperimentalMaterial3Api import androidx.compose.material3.IconButton import androidx.compose.material3.MaterialTheme import androidx.compose.material3.Scaffold import androidx.compose.material3.Text -import androidx.compose.material3.TextButton import androidx.compose.runtime.Composable import androidx.compose.runtime.LaunchedEffect import androidx.compose.runtime.getValue @@ -283,7 +284,11 @@ private fun OpenNestRoomAction( if (serviceBase.isNullOrBlank() || endpoint.isNullOrBlank() || roomId.isBlank()) return val context = LocalContext.current - TextButton( + // Filled button so the primary CTA is unmistakable; the actions + // Row in the AppBar already centers its children vertically, but + // we trim the default end padding so the button sits flush with + // the bar's trailing edge. + Button( onClick = { NestBridge.set(accountViewModel) NestActivity.launch( @@ -291,6 +296,8 @@ private fun OpenNestRoomAction( addressValue = event.address().toValue(), ) }, + contentPadding = ButtonDefaults.ContentPadding, + modifier = Modifier.padding(end = 8.dp), ) { Text(stringRes(R.string.nest_lobby_open_action)) }