Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Jetcaster] Use contentPadding on the content of the PlayerContentWithBackground #1375

Merged
merged 1 commit into from Apr 29, 2024
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Expand Up @@ -25,6 +25,7 @@ import androidx.compose.foundation.layout.Arrangement
import androidx.compose.foundation.layout.Box
import androidx.compose.foundation.layout.BoxWithConstraints
import androidx.compose.foundation.layout.Column
import androidx.compose.foundation.layout.PaddingValues
import androidx.compose.foundation.layout.Row
import androidx.compose.foundation.layout.Spacer
import androidx.compose.foundation.layout.WindowInsets
Expand Down Expand Up @@ -186,7 +187,7 @@ private fun PlayerScreen(
onAddToQueue()
},
playerControlActions = playerControlActions,
modifier = Modifier.padding(contentPadding)
contentPadding = contentPadding,
)
} else {
FullScreenLoading()
Expand Down Expand Up @@ -214,12 +215,15 @@ fun PlayerContentWithBackground(
onBackPress: () -> Unit,
onAddToQueue: () -> Unit,
playerControlActions: PlayerControlActions,
modifier: Modifier = Modifier
modifier: Modifier = Modifier,
contentPadding: PaddingValues = PaddingValues(0.dp)
) {
Box(modifier = modifier, contentAlignment = Alignment.Center) {
PlayerBackground(
episode = uiState.episodePlayerState.currentEpisode,
modifier = Modifier.fillMaxSize()
modifier = Modifier
.fillMaxSize()
.padding(contentPadding)
)
PlayerContent(
uiState = uiState,
Expand Down