From d221dc069850aa47269443f22e99fe58b33d74de Mon Sep 17 00:00:00 2001
From: Juha Heinanen <jh@tutpro.com>
Date: Sun, 8 Feb 2026 06:22:02 +0200
Subject: [PATCH] Simplified vertical scrollbar usage

---
 .../main/kotlin/com/tutpro/baresip/AccountsScreen.kt  |  6 +-----
 .../main/kotlin/com/tutpro/baresip/BlockedScreen.kt   |  6 +-----
 .../kotlin/com/tutpro/baresip/CallDetailsScreen.kt    |  5 +----
 app/src/main/kotlin/com/tutpro/baresip/CallsScreen.kt |  6 +-----
 app/src/main/kotlin/com/tutpro/baresip/ChatScreen.kt  |  6 +-----
 app/src/main/kotlin/com/tutpro/baresip/ChatsScreen.kt | 11 ++---------
 .../main/kotlin/com/tutpro/baresip/CodecsScreen.kt    |  5 +----
 .../main/kotlin/com/tutpro/baresip/ContactsScreen.kt  |  6 +-----
 .../main/kotlin/com/tutpro/baresip/CustomElements.kt  |  8 +++-----
 app/src/main/kotlin/com/tutpro/baresip/MainScreen.kt  |  7 ++-----
 10 files changed, 14 insertions(+), 52 deletions(-)

diff --git a/app/src/main/kotlin/com/tutpro/baresip/AccountsScreen.kt b/app/src/main/kotlin/com/tutpro/baresip/AccountsScreen.kt
index b97dabac..f68dc9ea 100644
--- a/app/src/main/kotlin/com/tutpro/baresip/AccountsScreen.kt
+++ b/app/src/main/kotlin/com/tutpro/baresip/AccountsScreen.kt
@@ -139,11 +139,7 @@ fun AccountsContent(contentPadding: PaddingValues, navController: NavController)
                 .fillMaxWidth()
                 .padding(contentPadding)
                 .padding(start = 16.dp, end = 4.dp, top = 8.dp, bottom = 8.dp)
-                .verticalScrollbar(
-                    state = lazyListState,
-                    width = 4.dp,
-                    color = MaterialTheme.colorScheme.outlineVariant
-                ),
+                .verticalScrollbar(state = lazyListState),
             verticalArrangement = Arrangement.spacedBy(4.dp),
         ) {
             items(BaresipService.uas.value) { ua ->
diff --git a/app/src/main/kotlin/com/tutpro/baresip/BlockedScreen.kt b/app/src/main/kotlin/com/tutpro/baresip/BlockedScreen.kt
index 6b876297..cb8204cf 100644
--- a/app/src/main/kotlin/com/tutpro/baresip/BlockedScreen.kt
+++ b/app/src/main/kotlin/com/tutpro/baresip/BlockedScreen.kt
@@ -268,11 +268,7 @@ private fun Blocked(ctx: Context, navController: NavController, blocked: Mutable
         modifier = Modifier
             .fillMaxWidth()
             .padding(start = 16.dp, end = 4.dp)
-            .verticalScrollbar(
-                state = lazyListState,
-                width = 4.dp,
-                color = MaterialTheme.colorScheme.outlineVariant
-            )
+            .verticalScrollbar(state = lazyListState)
             .background(MaterialTheme.colorScheme.background),
         state = lazyListState,
         verticalArrangement = Arrangement.spacedBy(12.dp),
diff --git a/app/src/main/kotlin/com/tutpro/baresip/CallDetailsScreen.kt b/app/src/main/kotlin/com/tutpro/baresip/CallDetailsScreen.kt
index 14eab463..4af77a75 100644
--- a/app/src/main/kotlin/com/tutpro/baresip/CallDetailsScreen.kt
+++ b/app/src/main/kotlin/com/tutpro/baresip/CallDetailsScreen.kt
@@ -200,10 +200,7 @@ private fun Details(ctx: Context, details: SnapshotStateList<Details>, onDelete:
     LazyColumn(
         modifier = Modifier
             .fillMaxWidth()
-            .verticalScrollbar(
-                state = lazyListState,
-                width = 4.dp
-            )
+            .verticalScrollbar(state = lazyListState)
             .background(MaterialTheme.colorScheme.background),
         state = lazyListState,
         verticalArrangement = Arrangement.spacedBy(12.dp),
diff --git a/app/src/main/kotlin/com/tutpro/baresip/CallsScreen.kt b/app/src/main/kotlin/com/tutpro/baresip/CallsScreen.kt
index 1aafe3f4..09c9ce9f 100644
--- a/app/src/main/kotlin/com/tutpro/baresip/CallsScreen.kt
+++ b/app/src/main/kotlin/com/tutpro/baresip/CallsScreen.kt
@@ -297,11 +297,7 @@ private fun Calls(
         modifier = Modifier
             .fillMaxWidth()
             .padding(start = 16.dp, end = 4.dp)
-            .verticalScrollbar(
-                state = lazyListState,
-                width = 4.dp,
-                color = MaterialTheme.colorScheme.outlineVariant
-            )
+            .verticalScrollbar(state = lazyListState)
             .background(MaterialTheme.colorScheme.background),
         state = lazyListState,
         verticalArrangement = Arrangement.spacedBy(12.dp),
diff --git a/app/src/main/kotlin/com/tutpro/baresip/ChatScreen.kt b/app/src/main/kotlin/com/tutpro/baresip/ChatScreen.kt
index 71d1b318..d2a47af9 100644
--- a/app/src/main/kotlin/com/tutpro/baresip/ChatScreen.kt
+++ b/app/src/main/kotlin/com/tutpro/baresip/ChatScreen.kt
@@ -347,11 +347,7 @@ private fun Messages(
         modifier = Modifier
             .fillMaxWidth()
             .padding(start = 16.dp, end = 2.dp)
-            .verticalScrollbar(
-                state = lazyListState,
-                width = 4.dp,
-                color = MaterialTheme.colorScheme.outlineVariant
-            )
+            .verticalScrollbar(state = lazyListState)
             .background(MaterialTheme.colorScheme.background),
         reverseLayout = true,
         state = lazyListState,
diff --git a/app/src/main/kotlin/com/tutpro/baresip/ChatsScreen.kt b/app/src/main/kotlin/com/tutpro/baresip/ChatsScreen.kt
index f29724e9..87fc1b8e 100644
--- a/app/src/main/kotlin/com/tutpro/baresip/ChatsScreen.kt
+++ b/app/src/main/kotlin/com/tutpro/baresip/ChatsScreen.kt
@@ -292,11 +292,7 @@ private fun Chats(
         modifier = Modifier
             .fillMaxWidth()
             .padding(start = 8.dp, end = 4.dp)
-            .verticalScrollbar(
-                state = lazyListState,
-                width = 4.dp,
-                color = MaterialTheme.colorScheme.outlineVariant
-            )
+            .verticalScrollbar(state = lazyListState)
             .background(MaterialTheme.colorScheme.background),
         reverseLayout = true,
         state = lazyListState,
@@ -513,10 +509,7 @@ private fun NewChatPeer(ctx: Context, navController: NavController, account: Acc
                             modifier = Modifier
                                 .fillMaxWidth()
                                 .heightIn(max = 180.dp)
-                                .verticalScrollbar(
-                                    state = lazyListState,
-                                    color = MaterialTheme.colorScheme.outlineVariant
-                                ),
+                                .verticalScrollbar(state = lazyListState, width = 6.dp),
                             horizontalAlignment = Alignment.Start,
                             state = lazyListState
                         ) {
diff --git a/app/src/main/kotlin/com/tutpro/baresip/CodecsScreen.kt b/app/src/main/kotlin/com/tutpro/baresip/CodecsScreen.kt
index 552bcf73..b3676072 100644
--- a/app/src/main/kotlin/com/tutpro/baresip/CodecsScreen.kt
+++ b/app/src/main/kotlin/com/tutpro/baresip/CodecsScreen.kt
@@ -218,10 +218,7 @@ private fun Codecs(codecs: SnapshotStateList<Codec>, onUpdateCodecs: (List<Codec
     LazyColumn(
         modifier = Modifier
             .padding(end = 4.dp)
-            .verticalScrollbar(
-                state = draggableState.listState,
-                width = 4.dp
-            ),
+            .verticalScrollbar(state = draggableState.listState),
         state = draggableState.listState,
         contentPadding = PaddingValues(start = 12.dp, end = 12.dp),
         //verticalArrangement = Arrangement.spacedBy(8.dp)
diff --git a/app/src/main/kotlin/com/tutpro/baresip/ContactsScreen.kt b/app/src/main/kotlin/com/tutpro/baresip/ContactsScreen.kt
index ed09cce7..52fa8fb3 100644
--- a/app/src/main/kotlin/com/tutpro/baresip/ContactsScreen.kt
+++ b/app/src/main/kotlin/com/tutpro/baresip/ContactsScreen.kt
@@ -263,11 +263,7 @@ private fun ContactsContent(
             .background(MaterialTheme.colorScheme.background)
             .padding(contentPadding)
             .padding(start = 16.dp, end = 4.dp, top = 16.dp, bottom = 10.dp)
-            .verticalScrollbar(
-                state = lazyListState,
-                width = 4.dp,
-                color = MaterialTheme.colorScheme.outlineVariant
-            ),
+            .verticalScrollbar(state = lazyListState),
         state = lazyListState,
         verticalArrangement = Arrangement.spacedBy(10.dp),
     ) {
diff --git a/app/src/main/kotlin/com/tutpro/baresip/CustomElements.kt b/app/src/main/kotlin/com/tutpro/baresip/CustomElements.kt
index 947b02d7..399bc879 100644
--- a/app/src/main/kotlin/com/tutpro/baresip/CustomElements.kt
+++ b/app/src/main/kotlin/com/tutpro/baresip/CustomElements.kt
@@ -200,15 +200,13 @@ object CustomElements {
     @Composable
     fun Modifier.verticalScrollbar(
         state: LazyListState,
-        width: Dp = 8.dp,
+        width: Dp = 4.dp,
         alwaysShow: Boolean = true,
         color: Color = MaterialTheme.colorScheme.outlineVariant
     ): Modifier {
-        val targetAlpha = if (state.isScrollInProgress || alwaysShow) 1f else 0f
-        val duration = if (state.isScrollInProgress) 150 else 500
         val alpha by animateFloatAsState(
-            targetValue = targetAlpha,
-            animationSpec = tween(durationMillis = duration)
+            targetValue = if (state.isScrollInProgress || alwaysShow) 1f else 0f,
+            animationSpec = tween(durationMillis = if (state.isScrollInProgress) 150 else 500)
         )
         return drawWithContent {
             drawContent()
diff --git a/app/src/main/kotlin/com/tutpro/baresip/MainScreen.kt b/app/src/main/kotlin/com/tutpro/baresip/MainScreen.kt
index 2c2bdb36..c2c45a7f 100644
--- a/app/src/main/kotlin/com/tutpro/baresip/MainScreen.kt
+++ b/app/src/main/kotlin/com/tutpro/baresip/MainScreen.kt
@@ -1289,10 +1289,7 @@ private fun CallUriRow(
                         LazyColumn(
                             modifier = Modifier
                                 .fillMaxWidth()
-                                .verticalScrollbar(
-                                    state = lazyListState,
-                                    color = MaterialTheme.colorScheme.outlineVariant
-                                ),
+                                .verticalScrollbar(state = lazyListState, width = 6.dp),
                             horizontalAlignment = Alignment.Start,
                             state = lazyListState
                         ) {
@@ -1684,7 +1681,7 @@ private fun CallRow(
                                                         .fillMaxWidth()
                                                         .verticalScrollbar(
                                                             state = lazyListState,
-                                                            color = MaterialTheme.colorScheme.outlineVariant
+                                                            width = 6.dp,
                                                         ),
                                                     horizontalAlignment = Alignment.Start,
                                                     state = lazyListState,