Fix behavior when switching to full-screen mode, hide system bars.

TeamNewPipe · Nov 4, 2022 · 6b87a36 · 6b87a36
1 parent 6c15b14
commit 6b87a36
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 13 deletions.
diff --git a/app/src/main/java/org/schabi/newpipe/fragments/detail/VideoDetailFragment.java b/app/src/main/java/org/schabi/newpipe/fragments/detail/VideoDetailFragment.java
@@ -2009,7 +2009,7 @@ private void showSystemUi() {
         }
 
         final var window = activity.getWindow();
-        WindowCompat.setDecorFitsSystemWindows(window, false);
+        WindowCompat.setDecorFitsSystemWindows(window, true);
         WindowCompat.getInsetsController(window, window.getDecorView())
                 .show(WindowInsetsCompat.Type.systemBars());
         window.setStatusBarColor(ThemeHelper.resolveColorFromAttr(requireContext(),
@@ -2029,20 +2029,12 @@ private void hideSystemUi() {
         final var windowInsetsController = WindowCompat.getInsetsController(window,
                 window.getDecorView());
 
-        WindowCompat.setDecorFitsSystemWindows(window, true);
+        WindowCompat.setDecorFitsSystemWindows(window, false);
         windowInsetsController.setSystemBarsBehavior(WindowInsetsControllerCompat
                 .BEHAVIOR_SHOW_TRANSIENT_BARS_BY_SWIPE);
-        int flags = WindowInsetsCompat.Type.navigationBars();
-
-        // In multiWindow mode status bar is not transparent for devices with cutout
-        // if I include this flag. So without it is better in this case
-        final boolean isInMultiWindow = DeviceUtils.isInMultiWindow(activity);
-        if (!isInMultiWindow) {
-            flags |= WindowInsetsCompat.Type.statusBars();
-        }
-        windowInsetsController.hide(flags);
+        windowInsetsController.hide(WindowInsetsCompat.Type.systemBars());
 
-        if (isInMultiWindow || isFullscreen()) {
+        if (DeviceUtils.isInMultiWindow(activity) || isFullscreen()) {
             window.setStatusBarColor(Color.TRANSPARENT);
             window.setNavigationBarColor(Color.TRANSPARENT);
         }

diff --git a/app/src/main/java/org/schabi/newpipe/player/ui/MainPlayerUi.java b/app/src/main/java/org/schabi/newpipe/player/ui/MainPlayerUi.java
@@ -436,7 +436,7 @@ public void showSystemUIPartially() {
                 window.setNavigationBarColor(Color.TRANSPARENT);
                 WindowCompat.setDecorFitsSystemWindows(window, false);
                 WindowCompat.getInsetsController(window, window.getDecorView())
-                        .hide(WindowInsetsCompat.Type.statusBars());
+                        .show(WindowInsetsCompat.Type.statusBars());
             });
         }
     }