From 1375cfc877336401c9f12e40f8004da93ba25459 Mon Sep 17 00:00:00 2001
From: Christian Schabesberger <chris.schabesberger@mailbox.org>
Date: Wed, 24 Jul 2024 15:15:45 +0200
Subject: [PATCH] get same orientation fullscreen fixed

---
 .../net/newpipe/newplayer/VideoPlayerView.kt  | 14 ++++++--
 .../newpipe/newplayer/testapp/MainActivity.kt | 36 +++++++++----------
 2 files changed, 30 insertions(+), 20 deletions(-)

diff --git a/new-player/src/main/java/net/newpipe/newplayer/VideoPlayerView.kt b/new-player/src/main/java/net/newpipe/newplayer/VideoPlayerView.kt
index 3e739cd..df5e22b 100644
--- a/new-player/src/main/java/net/newpipe/newplayer/VideoPlayerView.kt
+++ b/new-player/src/main/java/net/newpipe/newplayer/VideoPlayerView.kt
@@ -35,6 +35,12 @@ import net.newpipe.newplayer.ui.theme.VideoPlayerTheme
 class VideoPlayerView : FrameLayout {
 
     var viewModel: VideoPlayerViewModel? = null
+        set(value) {
+            field = value
+            applyViewModel()
+        }
+
+    val composeView:ComposeView
 
     @JvmOverloads
     constructor(
@@ -43,8 +49,12 @@ class VideoPlayerView : FrameLayout {
         defStyleAttr: Int = 0
     ) : super(context, attrs, defStyleAttr) {
         val view = LayoutInflater.from(context).inflate(R.layout.video_player_view, this)
-        val composeView = view.findViewById<ComposeView>(R.id.video_player_compose_view)
+        composeView = view.findViewById<ComposeView>(R.id.video_player_compose_view)
 
+        applyViewModel()
+    }
+
+    private fun applyViewModel() {
         composeView.apply {
             setViewCompositionStrategy(ViewCompositionStrategy.DisposeOnViewTreeLifecycleDestroyed)
             setContent {
@@ -53,6 +63,6 @@ class VideoPlayerView : FrameLayout {
                 }
             }
         }
-
     }
+
 }
\ No newline at end of file
diff --git a/test-app/src/main/java/net/newpipe/newplayer/testapp/MainActivity.kt b/test-app/src/main/java/net/newpipe/newplayer/testapp/MainActivity.kt
index ac4c417..df22b4f 100644
--- a/test-app/src/main/java/net/newpipe/newplayer/testapp/MainActivity.kt
+++ b/test-app/src/main/java/net/newpipe/newplayer/testapp/MainActivity.kt
@@ -22,7 +22,6 @@ package net.newpipe.newplayer.testapp
 
 import android.os.Bundle
 import android.view.View
-import android.webkit.RenderProcessGoneDetail
 import android.widget.Button
 import androidx.activity.enableEdgeToEdge
 import androidx.activity.viewModels
@@ -52,15 +51,15 @@ class MainActivity : AppCompatActivity() {
         enableEdgeToEdge()
         setContentView(R.layout.activity_main)
 
-        val video_view = findViewById<VideoPlayerView>(R.id.new_player_video_view)
-        val start_stream_button = findViewById<Button>(R.id.start_stream_button)
-        val buttons_layout = findViewById<View>(R.id.buttons_layout)
-        val embedded_player_layout = findViewById<View>(R.id.player_column)
-        val fullscreen_player = findViewById<VideoPlayerView>(R.id.fullscreen_player)
+        val embeddedPlayer = findViewById<VideoPlayerView>(R.id.new_player_video_view)
+        val startStreamButton = findViewById<Button>(R.id.start_stream_button)
+        val buttonsLayout = findViewById<View>(R.id.buttons_layout)
+        val embeddedPlayerLayout = findViewById<View>(R.id.player_column)
+        val fullscreenPlayer = findViewById<VideoPlayerView>(R.id.fullscreen_player)
 
-        start_stream_button.setOnClickListener {
+        startStreamButton.setOnClickListener {
             newPlayer.playWhenReady = true
-            newPlayer.setStream(getString(R.string.ccc_6502_video))
+            newPlayer.setStream(getString(R.string.portrait_video_example))
         }
 
 
@@ -80,16 +79,17 @@ class MainActivity : AppCompatActivity() {
                 v.setPadding(0, 0, 0, 0)
                 insets
             }
-            buttons_layout.visibility = View.GONE
-            embedded_player_layout.visibility = View.GONE
-            fullscreen_player.visibility = View.VISIBLE
-            fullscreen_player.viewModel = videoPlayerViewModel
+            buttonsLayout.visibility = View.GONE
+            embeddedPlayerLayout.visibility = View.GONE
+            fullscreenPlayer.visibility = View.VISIBLE
+            embeddedPlayer.viewModel = null
+            fullscreenPlayer.viewModel = videoPlayerViewModel
 
             windowInsetsController.hide(WindowInsetsCompat.Type.systemBars())
         }
 
         val setupEmbeddedView = {
-            buttons_layout.visibility = View.VISIBLE
+            buttonsLayout.visibility = View.VISIBLE
 
             ViewCompat.setOnApplyWindowInsetsListener(findViewById(R.id.main)) { v, insets ->
                 val systemBars = insets.getInsets(WindowInsetsCompat.Type.systemBars())
@@ -102,11 +102,11 @@ class MainActivity : AppCompatActivity() {
                 insets
             }
 
-            buttons_layout.visibility = View.VISIBLE
-            embedded_player_layout.visibility = View.VISIBLE
-            fullscreen_player.visibility = View.GONE
-            video_view.viewModel = videoPlayerViewModel
-
+            buttonsLayout.visibility = View.VISIBLE
+            embeddedPlayerLayout.visibility = View.VISIBLE
+            fullscreenPlayer.visibility = View.GONE
+            fullscreenPlayer.viewModel = null
+            embeddedPlayer.viewModel = videoPlayerViewModel
 
             windowInsetsController.show(WindowInsetsCompat.Type.systemBars())
         }