Little heads up not to use androidx.compose.material3.ModalBottomSheet if you’re also doing `android...

# compose

[Stylianos Gakis]

Little heads up not to use androidx.compose.material3.ModalBottomSheet if you’re also doing android:configChanges="orientation|screenSize|screenLayout" and you allow screen rotations.
Not sure exactly which one of those breaks it (maybe the combination of them) but rotating the device makes it look horrible since the Modal does not resize itself on the rotation 😅

[Stylianos Gakis]

Sat Jul 22 2023 21_46_37 GMT+0200 (Central European Summer Time).png,Sat Jul 22 2023 21_46_52 GMT+0200 (Central European Summer Time).png

[Stylianos Gakis]

This one looks extra special 😄

[Marco Pierucci]

probably worth filing a bug?

[Stylianos Gakis]

Yeah of course
https://issuetracker.google.com/issues/292204649

[Dan Nichols]

I believe that’s the intended behaviour of bottom sheets, looking at the material 3 spec.

Although it does say it should be overridable and I can’t see anything in the compose API that allows for that

[Albert Chang]

This looks like an old bug which has been fixed in 1.5.0-beta03.
Maybe not. M3 ModalBottomSheet doesn’t seem to be using Dialog or Popup internally.

[Stylianos Gakis]

Dan, do you mind looking at the screenshots again? The first screenshot is okay for both of them. The second one isn't "intended behavior" at all, not even close 😅

[Dan Nichols]

Ahh yes my apologies, I thought you were simply referring to the behaviour in landscape. Admittedly skimmed and missed your extra special screenshot 😅. Sorry about that