Add workaround for https://issuetracker.google.com/180691023

chrisbanes · Feb 22, 2021 · c953d03 · c953d03
1 parent a04b909
commit c953d03
Showing 1 changed file with 22 additions and 14 deletions.
diff --git a/ui-account/src/main/java/app/tivi/account/AccountUiFragment.kt b/ui-account/src/main/java/app/tivi/account/AccountUiFragment.kt
@@ -29,6 +29,7 @@ import androidx.compose.runtime.livedata.observeAsState
 import androidx.compose.ui.platform.ComposeView
 import androidx.fragment.app.DialogFragment
 import androidx.fragment.app.viewModels
+import androidx.lifecycle.ViewTreeLifecycleOwner
 import androidx.navigation.fragment.findNavController
 import app.tivi.common.compose.LocalTiviDateFormatter
 import app.tivi.common.compose.shouldUseDarkColors
@@ -70,22 +71,29 @@ class AccountUiFragment : DialogFragment() {
         inflater: LayoutInflater,
         container: ViewGroup?,
         savedInstanceState: Bundle?
-    ): View = ComposeView(requireContext()).apply {
-        layoutParams = ViewGroup.LayoutParams(MATCH_PARENT, WRAP_CONTENT)
+    ): View {
+        // Workaround for https://issuetracker.google.com/180691023
+        dialog?.window?.decorView?.let { decorView ->
+            ViewTreeLifecycleOwner.set(decorView, viewLifecycleOwner)
+        }
+
+        return ComposeView(requireContext()).apply {
+            layoutParams = ViewGroup.LayoutParams(MATCH_PARENT, WRAP_CONTENT)
 
-        // We use ViewWindowInsetObserver rather than ProvideWindowInsets
-        // See: https://github.com/chrisbanes/accompanist/issues/155
-        val windowInsets = ViewWindowInsetObserver(this).start(consumeWindowInsets = false)
+            // We use ViewWindowInsetObserver rather than ProvideWindowInsets
+            // See: https://github.com/chrisbanes/accompanist/issues/155
+            val windowInsets = ViewWindowInsetObserver(this).start(consumeWindowInsets = false)
 
-        setContent {
-            CompositionLocalProvider(
-                LocalTiviDateFormatter provides tiviDateFormatter,
-                LocalWindowInsets provides windowInsets,
-            ) {
-                TiviTheme(useDarkColors = preferences.shouldUseDarkColors()) {
-                    val viewState by viewModel.liveData.observeAsState()
-                    if (viewState != null) {
-                        AccountUi(viewState!!, ::onAccountUiAction)
+            setContent {
+                CompositionLocalProvider(
+                    LocalTiviDateFormatter provides tiviDateFormatter,
+                    LocalWindowInsets provides windowInsets,
+                ) {
+                    TiviTheme(useDarkColors = preferences.shouldUseDarkColors()) {
+                        val viewState by viewModel.liveData.observeAsState()
+                        if (viewState != null) {
+                            AccountUi(viewState!!, ::onAccountUiAction)
+                        }
                     }
                 }
             }