#2007 fix: volume up/down action bottom sheet is cut off on some devices

Author: sds100

CHANGELOG.md

@@ -1,3 +1,10 @@
+## [4.0.1](https://github.com/sds100/KeyMapper/releases/tag/v4.0.1)
+
+#### TO BE RELEASED
+
+## Fixed
+- #2007 Volume up/down action bottom sheet is cut off on some devices
+
 ## [4.0.0](https://github.com/sds100/KeyMapper/releases/tag/v4.0.0)
 
 #### 25 January 2026

base/src/main/java/io/github/sds100/keymapper/base/actions/VolumeActionBottomSheet.kt

@@ -105,7 +105,9 @@ private fun VolumeActionBottomSheet(
         dragHandle = null,
     ) {
         Column(
-            modifier = Modifier.verticalScroll(scrollState),
+            modifier = Modifier
+                .verticalScroll(scrollState)
+                .weight(1f),
         ) {
             Spacer(modifier = Modifier.height(16.dp))
 
@@ -162,45 +164,45 @@ private fun VolumeActionBottomSheet(
                 isChecked = state.showVolumeUi,
                 onCheckedChange = onToggleShowVolumeUi,
             )
-        }
 
-        Spacer(modifier = Modifier.height(16.dp))
+            Spacer(modifier = Modifier.height(16.dp))
 
-        Row(
-            modifier = Modifier
-                .fillMaxWidth()
-                .padding(horizontal = 16.dp),
-            horizontalArrangement = Arrangement.SpaceBetween,
-            verticalAlignment = Alignment.CenterVertically,
-        ) {
-            OutlinedButton(
-                modifier = Modifier.weight(1f),
-                onClick = {
-                    scope.launch {
-                        sheetState.hide()
-                        onDismissRequest()
-                    }
-                },
+            Row(
+                modifier = Modifier
+                    .fillMaxWidth()
+                    .padding(horizontal = 16.dp),
+                horizontalArrangement = Arrangement.SpaceBetween,
+                verticalAlignment = Alignment.CenterVertically,
             ) {
-                Text(stringResource(R.string.neg_cancel))
-            }
+                OutlinedButton(
+                    modifier = Modifier.weight(1f),
+                    onClick = {
+                        scope.launch {
+                            sheetState.hide()
+                            onDismissRequest()
+                        }
+                    },
+                ) {
+                    Text(stringResource(R.string.neg_cancel))
+                }
 
-            Spacer(modifier = Modifier.width(16.dp))
+                Spacer(modifier = Modifier.width(16.dp))
 
-            Button(
-                modifier = Modifier.weight(1f),
-                onClick = onDoneClick,
-            ) {
-                Text(stringResource(R.string.pos_done))
+                Button(
+                    modifier = Modifier.weight(1f),
+                    onClick = onDoneClick,
+                ) {
+                    Text(stringResource(R.string.pos_done))
+                }
             }
-        }
 
-        Spacer(Modifier.height(16.dp))
+            Spacer(Modifier.height(16.dp))
+        }
     }
 }
 
 @OptIn(ExperimentalMaterial3Api::class)
-@Preview
+@Preview(heightDp = 400)
 @Composable
 private fun PreviewVolumeActionBottomSheet() {
     KeyMapperTheme {