From 5efdfb6aacfc7e442730890bc85928d9a8ccd641 Mon Sep 17 00:00:00 2001 From: Alexander Capehart Date: Sat, 24 Feb 2024 13:27:52 -0700 Subject: [PATCH] home: make sure to avoid touch overlay crash Only configure the touch overlay when the fragment resumes, and don't do any weird post stuff. --- .../java/org/oxycblt/auxio/home/HomeFragment.kt | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/app/src/main/java/org/oxycblt/auxio/home/HomeFragment.kt b/app/src/main/java/org/oxycblt/auxio/home/HomeFragment.kt index 1d5c5bdde..41a1055a7 100644 --- a/app/src/main/java/org/oxycblt/auxio/home/HomeFragment.kt +++ b/app/src/main/java/org/oxycblt/auxio/home/HomeFragment.kt @@ -222,19 +222,17 @@ class HomeFragment : collect(detailModel.toShow.flow, ::handleShow) } - override fun onStart() { - super.onStart() + override fun onResume() { + super.onResume() // Stock bottom sheet overlay won't work with our nested UI setup, have to replicate // it ourselves. requireBinding().root.rootView.apply { - post { - findViewById(R.id.main_scrim).setOnTouchListener { _, event -> - handleSpeedDialBoundaryTouch(event) - } - findViewById(R.id.sheet_scrim).setOnTouchListener { _, event -> - handleSpeedDialBoundaryTouch(event) - } + findViewById(R.id.main_scrim).setOnTouchListener { _, event -> + handleSpeedDialBoundaryTouch(event) + } + findViewById(R.id.sheet_scrim).setOnTouchListener { _, event -> + handleSpeedDialBoundaryTouch(event) } } }