fix(player): hide system bars when closing dialog on API < 30

jarnedemeulemeester · Aug 31, 2024 · ed6c6e8 · ed6c6e8
1 parent 8c148f3
commit ed6c6e8
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 0 deletions.
diff --git a/player/video/src/main/java/dev/jdtech/jellyfin/dialogs/SpeedSelectionDialogFragment.kt b/player/video/src/main/java/dev/jdtech/jellyfin/dialogs/SpeedSelectionDialogFragment.kt
@@ -2,6 +2,9 @@ package dev.jdtech.jellyfin.dialogs
 
 import android.app.Dialog
 import android.os.Bundle
+import androidx.core.view.WindowCompat
+import androidx.core.view.WindowInsetsCompat
+import androidx.core.view.WindowInsetsControllerCompat
 import androidx.fragment.app.DialogFragment
 import com.google.android.material.dialog.MaterialAlertDialogBuilder
 import dev.jdtech.jellyfin.player.video.R
@@ -30,4 +33,15 @@ class SpeedSelectionDialogFragment(
             builder.create()
         } ?: throw IllegalStateException("Activity cannot be null")
     }
+
+    override fun onDestroy() {
+        super.onDestroy()
+        // Fix for hiding the system bars on API < 30
+        activity?.window?.let {
+            WindowCompat.getInsetsController(it, it.decorView).apply {
+                systemBarsBehavior = WindowInsetsControllerCompat.BEHAVIOR_SHOW_TRANSIENT_BARS_BY_SWIPE
+                hide(WindowInsetsCompat.Type.systemBars())
+            }
+        }
+    }
 }
diff --git a/player/video/src/main/java/dev/jdtech/jellyfin/dialogs/TrackSelectionDialogFragment.kt b/player/video/src/main/java/dev/jdtech/jellyfin/dialogs/TrackSelectionDialogFragment.kt
@@ -2,6 +2,9 @@ package dev.jdtech.jellyfin.dialogs
 
 import android.app.Dialog
 import android.os.Bundle
+import androidx.core.view.WindowCompat
+import androidx.core.view.WindowInsetsCompat
+import androidx.core.view.WindowInsetsControllerCompat
 import androidx.fragment.app.DialogFragment
 import androidx.media3.common.C
 import com.google.android.material.dialog.MaterialAlertDialogBuilder
@@ -38,4 +41,15 @@ class TrackSelectionDialogFragment(
             builder.create()
         } ?: throw IllegalStateException("Activity cannot be null")
     }
+
+    override fun onDestroy() {
+        super.onDestroy()
+        // Fix for hiding the system bars on API < 30
+        activity?.window?.let {
+            WindowCompat.getInsetsController(it, it.decorView).apply {
+                systemBarsBehavior = WindowInsetsControllerCompat.BEHAVIOR_SHOW_TRANSIENT_BARS_BY_SWIPE
+                hide(WindowInsetsCompat.Type.systemBars())
+            }
+        }
+    }
 }