Change theme
Help
Press space for more information.
Show links for this issue (Shortcut: i, l)
Copy issue ID
Previous Issue (Shortcut: k)
Next Issue (Shortcut: j)
Sign in to use full features.
Vote: I am impacted
Notification menu
Refresh (Shortcut: Shift+r)
Go home (Shortcut: u)
Pending code changes (auto-populated)
View issue level access limits(Press Alt + Right arrow for more information)
Unintended behavior
View staffing
Description
androidx.compose.material.ModalBottomSheetLayout
hassheetGesturesEnabled
, butandroidx.compose.material.navigation.ModalBottomSheetLayout
doesn't. We need this to disable pull-to-dismiss on a destination basis.Using accompanist, it was possible to work around this because
BottomSheetNavigator.sheetContent
was public. Inmaterial.navigation
,BottomSheetNavigator.sheetContent
is internal, so there's currently no way to migrate from accompanist navigation material tocompose.material.navigation
if we need to rely onsheetGesturesEnabled
.If I'm not missing something, it would mean adding two lines of code tohttps://github.com/androidx/androidx/blob/androidx-main/compose/material/material-navigation/src/main/java/androidx/compose/material/navigation/BottomSheet.kt , so a pretty easy change.
I believe this issue would also benefit from this change https://issuetracker.google.com/issues/346717094
Jetpack Compose version:
Material Library Version (M2, M3 or Both?):
Material Compose component used:
Android Studio Build:
Kotlin version: