From e588abd4e7414faa8ee0a02e895f404ff5f32e41 Mon Sep 17 00:00:00 2001
From: Stypox <stypox@pm.me>
Date: Thu, 3 Mar 2022 10:14:58 +0100
Subject: [PATCH] Restore handling SPACE as play-pause only in fullscreen

When not in fullscreen SPACE should be not handled by the player, and hence result in a scroll down
---
 app/src/main/java/org/schabi/newpipe/player/Player.java  | 9 ++++++---
 .../newpipe/player/playback/PlayerMediaSession.java      | 1 +
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/app/src/main/java/org/schabi/newpipe/player/Player.java b/app/src/main/java/org/schabi/newpipe/player/Player.java
index 2a88445bb..759c817d5 100644
--- a/app/src/main/java/org/schabi/newpipe/player/Player.java
+++ b/app/src/main/java/org/schabi/newpipe/player/Player.java
@@ -3752,9 +3752,12 @@ public final class Player implements
             default:
                 break;
             case KeyEvent.KEYCODE_SPACE:
-                playPause();
-                if (isPlaying()) {
-                    hideControls(0, 0);
+                if (isFullscreen) {
+                    playPause();
+                    if (isPlaying()) {
+                        hideControls(0, 0);
+                    }
+                    return true;
                 }
                 break;
             case KeyEvent.KEYCODE_BACK:
diff --git a/app/src/main/java/org/schabi/newpipe/player/playback/PlayerMediaSession.java b/app/src/main/java/org/schabi/newpipe/player/playback/PlayerMediaSession.java
index fbe12efb1..ee0a6f118 100644
--- a/app/src/main/java/org/schabi/newpipe/player/playback/PlayerMediaSession.java
+++ b/app/src/main/java/org/schabi/newpipe/player/playback/PlayerMediaSession.java
@@ -88,6 +88,7 @@ public class PlayerMediaSession implements MediaSessionCallback {
     @Override
     public void play() {
         player.play();
+        // hide the player controls even if the play command came from the media session
         player.hideControls(0, 0);
     }