Merge pull request #79 from azrael8576/fix/video-screen-issues

Author: azrael8576

core/data/src/main/java/com/wei/picquest/core/data/repository/DefaultSearchImagesRepository.kt

@@ -20,7 +20,7 @@ class DefaultSearchImagesRepository @Inject constructor(
         return Pager(
             config = PagingConfig(
                 pageSize = 20,
-                prefetchDistance = 5,
+                prefetchDistance = 3,
                 enablePlaceholders = false,
             ),
             pagingSourceFactory = { PixabayImagePagingSource(pqNetworkDataSource, query) },

feature/video/src/main/java/com/wei/picquest/feature/video/videolibrary/VideoLibraryScreen.kt

@@ -151,6 +151,8 @@ internal fun VideoLibraryScreen(
                 isPreview = isPreview,
             )
 
+            PagingStateHandling(lazyPagingItems = lazyPagingItems)
+
             if (!isInPiPMode) {
                 TopBarActions(onBackClick = onBackClick)
             }
@@ -182,8 +184,6 @@ fun VideoPager(
                 )
             }
         }
-
-        PagingStateHandling(lazyPagingItems = lazyPagingItems)
     }
 }
 

feature/video/src/main/java/com/wei/picquest/feature/video/videolibrary/VideoLibraryViewModel.kt

@@ -25,7 +25,7 @@ class VideoLibraryViewModel @Inject constructor(
 
     private val videoLibraryArgs: VideoLibraryArgs = VideoLibraryArgs(savedStateHandle)
 
-    val videoSearchQuery = videoLibraryArgs.videoSearchQuery
+    private val videoSearchQuery = videoLibraryArgs.videoSearchQuery
 
     private val _videosState: MutableStateFlow<PagingData<VideoDetail>> =
         MutableStateFlow(value = PagingData.empty())

feature/video/src/main/java/com/wei/picquest/feature/video/videolibrary/lifecycle/MediaPiPLifecycle.kt

@@ -22,7 +22,7 @@ import kotlinx.coroutines.delay
  *
  * - The ExoPlayer will start playback (`play()`) when the lifecycle event is `ON_RESUME`.
  * - The ExoPlayer will pause playback (`pause()`) when the lifecycle event is `ON_STOP`, which
- *   includes scenarios such as when the application goes into the background or enters Picture-In-Picture mode.
+ *   includes scenarios such as when the application goes into the background.
  *
  * This lifecycle-aware management ensures that video playback is handled efficiently, respecting
  * the application's current state and reducing resource usage when the app is not in the foreground.
@@ -51,7 +51,6 @@ internal fun MediaPiPLifecycle(
 
     if (latestLifecycleEvent == Lifecycle.Event.ON_RESUME) {
         LaunchedEffect(latestLifecycleEvent) {
-            delay(pipEnteringDuration)
             exoPlayer.play()
         }
     }