Proper content padding for VideoPage

Author: Javernaut

app/src/main/java/com/javernaut/whatthecodec/presentation/root/ui/MainScreen.kt

@@ -1,9 +1,9 @@
 package com.javernaut.whatthecodec.presentation.root.ui
 
 import androidx.compose.foundation.ExperimentalFoundationApi
+import androidx.compose.foundation.layout.PaddingValues
 import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.foundation.layout.height
-import androidx.compose.foundation.layout.padding
 import androidx.compose.foundation.pager.HorizontalPager
 import androidx.compose.foundation.pager.PagerState
 import androidx.compose.foundation.pager.rememberPagerState
@@ -62,7 +62,7 @@ fun MainScreen(
             )
         }
     ) {
-        MainScreenContent(Modifier.padding(it), tabsToShow, screenState, pagerState)
+        MainScreenContent(tabsToShow, screenState, pagerState, it, Modifier.fillMaxSize())
     }
 }
 
@@ -137,15 +137,16 @@ private val AvailableTab.title: Int
 @Composable
 @ExperimentalFoundationApi
 private fun MainScreenContent(
-    modifier: Modifier = Modifier,
     tabsToShow: List<AvailableTab>,
     screenState: ScreenState,
     pagerState: PagerState,
+    contentPadding: PaddingValues,
+    modifier: Modifier = Modifier,
 ) {
     HorizontalPager(pagerState, modifier) { page ->
         val pageModifier = Modifier.fillMaxSize()
         when (tabsToShow[page]) {
-            AvailableTab.VIDEO -> VideoPage(screenState.videoPage!!, pageModifier)
+            AvailableTab.VIDEO -> VideoPage(screenState.videoPage!!, contentPadding, pageModifier)
             AvailableTab.AUDIO -> AudioPage(screenState.audioPage!!, pageModifier)
             AvailableTab.SUBTITLES -> SubtitlePage(screenState.subtitlesPage!!, pageModifier)
         }

app/src/main/java/com/javernaut/whatthecodec/presentation/video/VideoPage.kt

@@ -2,6 +2,7 @@ package com.javernaut.whatthecodec.presentation.video
 
 import android.content.res.Resources
 import androidx.annotation.StringRes
+import androidx.compose.foundation.layout.PaddingValues
 import androidx.compose.foundation.layout.Row
 import androidx.compose.foundation.layout.fillMaxWidth
 import androidx.compose.foundation.layout.padding
@@ -27,13 +28,17 @@ import io.github.javernaut.mediafile.displayable.toDisplayable
 @Composable
 fun VideoPage(
     videoInfo: BasicVideoInfo,
+    contentPadding: PaddingValues,
     modifier: Modifier = Modifier
 ) {
-    LazyColumn(modifier) {
+    LazyColumn(
+        modifier = modifier,
+        contentPadding = contentPadding
+    ) {
         item {
             FramesHeader(videoInfo.preview, Modifier.fillMaxWidth())
         }
-        val commonModifier = Modifier.padding(start = 16.dp, end = 16.dp, bottom = 16.dp)
+        val commonModifier = Modifier.padding(horizontal = 16.dp)
         item {
             Container(
                 basicVideoInfo = videoInfo,
@@ -44,7 +49,7 @@ fun VideoPage(
         item {
             VideoStream(
                 videoStream = videoInfo.videoStream,
-                modifier = commonModifier
+                modifier = commonModifier.padding(vertical = 16.dp)
             )
         }
     }