ThirtyInch icon indicating copy to clipboard operation
ThirtyInch copied to clipboard

[Feature Request] add support for BottomSheetDialogFragment

Open k0shk0sh opened this issue 7 years ago • 5 comments

It'd be cool if BottomSheetDialogFragment is supported out of the box.

Looking forward for the next release 👍

k0shk0sh avatar Apr 15 '17 12:04 k0shk0sh

All you have to do is add this and you have turned your standard DialogFragment into a BottomSheetDialogFragment 😄

iffa avatar Apr 19 '17 18:04 iffa

@iffa well that's genius, I really didn't give it a thought lol. Thanks

k0shk0sh avatar Apr 20 '17 01:04 k0shk0sh

@passsy I guess this could be closed, however it might be good to include this workaround some where.

k0shk0sh avatar Apr 20 '17 01:04 k0shk0sh

We will not provide a TiBottomSheetDialogFragment because it would require a new dependency. But I'll leave it open because we could support composition instead of inheritance in the future without using CompositeAndroid.

ActivityLifecycleCallbacks & FragmentLifecycleCallbacks 🚀

passsy avatar May 04 '17 22:05 passsy

@passsy make sense, but the workaround solution posted here is actually pretty easy to implement and one could use it without writing plenty of code, I guess it would be cool if its documented somewhere for others to notice because now bottomSheets are the new Dialogs .

k0shk0sh avatar May 05 '17 01:05 k0shk0sh