1890796 - Add an abstract ComposeViewBottomSheetFragment

Status: ASSIGNED

(Fenix :: Design System and Theming, task, P3)

Description

[Gabriel Luong [:gl] (ΦωΦ)]

Discussed in https://phabricator.services.mozilla.com/D206791#7108663

The task is to implement a reusable abstract ComposeViewBottomSheetFragment that can be used for all our compose bottom sheet fragment. The benefit here is that we will a common view with the correct composition strategy set so that other developers will not have to worry about which composition strategy to use.

abstract class ComposeViewBottomSheetFragment : BottomSheetDialogFragment() {

    final override fun onCreateView(
        inflater: LayoutInflater,
        container: ViewGroup?,
        savedInstanceState: Bundle?,
    ): View = ComposeView(requireContext()).apply {
        setViewCompositionStrategy(ViewCompositionStrategy.DisposeOnViewTreeLifecycleDestroyed)
        setContent {
            content()
        }
    }

    abstract fun content(): @Composable () -> Unit
}

class MenuBottomSheetFragment : ComposeViewBottomSheetFragment() {
    override fun content(): @Composable () -> Unit = {
        // Composable content
    }
}

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The Bugbug bot thinks this bug should belong to the 'Fenix::Design System and Theming' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.