Sulkta-Coop/element-x-ada
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

MediaViewer: Show divider above the caption only if there is a caption and the media is video (to have a divider with the video controller)

Browse source
This commit is contained in:
Benoit Marty 2024-12-06 10:19:19 +01:00
parent 9c02c4cfde
commit 5a8aabdb4d
1 changed files with 6 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

7
libraries/mediaviewer/impl/src/main/kotlin/io/element/android/libraries/mediaviewer/impl/viewer/MediaViewerView.kt
View file

@ -51,6 +51,7 @@ import io.element.android.compound.theme.ElementTheme
import io.element.android.compound.tokens.generated.CompoundIcons
import io.element.android.libraries.architecture.AsyncData
import io.element.android.libraries.core.mimetype.MimeTypes
import io.element.android.libraries.core.mimetype.MimeTypes.isMimeTypeVideo
import io.element.android.libraries.designsystem.components.button.BackButton
import io.element.android.libraries.designsystem.components.dialogs.RetryDialog
import io.element.android.libraries.designsystem.preview.ElementPreviewDark
@ -125,6 +126,7 @@ fun MediaViewerView(
)
MediaViewerBottomBar(
modifier = Modifier.align(Alignment.BottomCenter),
showDivider = state.mediaInfo.mimeType.isMimeTypeVideo(),
caption = state.mediaInfo.caption,
onHeightChange = { bottomPaddingInPixels = it },
)
@ -360,6 +362,7 @@ private fun MediaViewerTopBar(
@Composable
private fun MediaViewerBottomBar(
caption: String?,
showDivider: Boolean,
onHeightChange: (Int) -> Unit,
modifier: Modifier = Modifier,
) {
@ -371,8 +374,10 @@ private fun MediaViewerBottomBar(
onHeightChange(it.height)
},
) {
HorizontalDivider()
if (caption != null) {
if (showDivider) {
HorizontalDivider()
}
Text(
modifier = Modifier
.fillMaxWidth()