Hi , Is there a way to make ModalBottomSheetLayout...

# compose

[wintersoldier]

Hi , Is there a way to make ModalBottomSheetLayout non dismissable.

[wintersoldier]

Doing this is not recommended,

val bottomsheetState =
  rememberModalBottomSheetState(
    initialValue = ModalBottomSheetValue.Hidden,
    confirmStateChange = {false}
  )

Is there a workaround.

[Tin Tran]

You can override the behavior of the back button with your own logic in this case. Does not fix the case when user click on the outside of the content though

[wintersoldier]

Back button handling is fine, only the main issue is when touched outside the content

[Tin Tran]

Is the bottom sheet general for all screens or just 1 specific screen?

[Tin Tran]

If it’s specific then I think the only way to workaround this is to control the bottom sheet yourself and ditch the bottom sheet navigation

[wintersoldier]

I see my usecase is basically create a wrapper around ModalBottomSheetLayout and expose it for all bottomsheet usecases.

[Tin Tran]

U can still use it in other screens that do not require special logic to dismiss the bottom sheet though. It’d save some effort

[wintersoldier]

Okay, so are you suggesting this approach ,
https://kotlinlang.slack.com/archives/CJLTWPH7S/p1676462242129839?thread_ts=1676462146.245549&channel=CJLTWPH7S&message_ts=1676462242.129839 or something else

[wintersoldier]

With the above approach i was facing unintended behaviours.

[Tin Tran]

Let’s say u have screen A with bottom sheet B
Instead of doing this

composable("/a){}
bottomSheet("/b"){}

You can do this

composable("/a)
{
 val bottomsheetState =
  rememberModalBottomSheetState(
    initialValue =   ModalBottomSheetValue.Hidden,
    confirmStateChange = {
       // your logic
   }
  BottomSheetModalLayout{


  }

}

[wintersoldier]

Thanks Tin,I will check for the confirmStatChange logic much more deeper

[Tin Tran]

It’s basically just this

onDismiss = {
                    if (sheetState.confirmStateChange(Hidden)) {
                        scope.launch { sheetState.hide() }
                    }
                },

But you can’t override it in the case of bottom sheet navigator because it has to do some synchronization to control the state of the back stack