Status Update
Comments
ab...@google.com <ab...@google.com> #2
Connie, do you think we can just remove it? And recommend ModalBottomSheet or BottomSheetScaffold instead?
ab...@google.com <ab...@google.com> #3
ap...@google.com <ap...@google.com> #4
I think it's worth noting in M2 that ModalBottomSheet is a layout while bottomdrawer functions more as a truly modal object, even though it does not have perfect dialog-like functionality. For the usecase recommended (bottom app bar menus), BottomDrawer seems like a better fit than ModalBottomSheetLayout. However in M3 it makes sense to explicitly use ModalBottomSheet once implemented.
Wdyt Matvei?
ap...@google.com <ap...@google.com> #5
I think it's worth noting in M2 that ModalBottomSheet is a layout while bottomdrawer functions more as a truly modal object
Could you comment on that? What's the actual difference in the behaviour?
ap...@google.com <ap...@google.com> #6
Bottom Drawer is an atomic component that can be placed on top of content on the screen while ModalBottomSheet is a layout that wraps around all the screen content and places accordingly. BottomDrawer is much more of a splashable component in M2 and asking users to migrate BottomDrawer uses may be cumbersome.
ap...@google.com <ap...@google.com> #7
Ok, sounds good. I'm happy to keep it if you think it brings value to m2 users.
If we are not removing it, we need to make it stable in this case.
ap...@google.com <ap...@google.com> #8
Found some outstanding bugs I will look into before deciding to promote, especially the swipeable migrations which will likely update the API
ap...@google.com <ap...@google.com> #9
Given the planned work to migrate the swipeable logic of bottomdrawer, this API should remain experimental. Closing for now.
ap...@google.com <ap...@google.com> #10
If these are still experimental, please keep this bug open and just target the next release
ab...@google.com <ab...@google.com>
na...@google.com <na...@google.com> #11
Where can i find the respective hotlist for the next release?
Description
DD: go/wv-process-wide-config