Julia

Potential duplicate of https://github.com/androidx/media/issues/1237. Investigation of Compose interaction with SurfaceView and AspectRatioFrameLayout.

According to: https://github.com/androidx/media/blob/d13a0f4ec62ca092b79746a5725b62a3244cc5b4/libraries/ui/src/main/java/androidx/media3/ui/AspectRatioFrameLayout.java#L174-L209 `RESIZE_MODE_FILL` is actually not doing any resizing ("Ignore target aspect ratio" branch) `RESIZE_MODE_ZOOM` in the case of your media will increase the width (since AspectDeformation>0) And you...

@jdelga Could you try setting `app:surface_type="texture_view"` to see if we can isolate this issue as a `SurfaceView in Compose` issue instead?

Sorry for a delay in replying. Could you explain why setTheme doesn't work? From what I can tell, building for platform with `android.app.AlertDialog.Builder`: https://github.com/androidx/media/blob/d13a0f4ec62ca092b79746a5725b62a3244cc5b4/libraries/ui/src/main/java/androidx/media3/ui/TrackSelectionDialogBuilder.java#L251-L252 and building with AndroidX with `androidx.appcompat.app.AlertDialog.Builder`...

Stumbled upon https://developer.android.com/reference/com/google/android/material/dialog/MaterialAlertDialogBuilder and I quote: > An extension of [AlertDialog.Builder](https://developer.android.com/reference/androidx/appcompat/app/AlertDialog.Builder) for use with a Material theme (e.g., Theme.MaterialComponents). > >This Builder must be used in order for AlertDialog objects...

Hi @jekopena, Thank you for this high-quality PR, your input is very appreciated. Without delving much further into it, the immediate problem I see is that this looks like a...

Please correct me if I'm wrong. Do I understand that you want: ``` playResourceId = a.getResourceId(R.styleable.PlayerView_play_resource_id, /* defValue= */ R.drawable.exo_styled_controls_play); ``` and then you can define various xml views `app:play_resource_id="R.drawable.personal_controls_play"`...

The renaming was as follows: | exo2 | media3 | |---|---| | PlayerControlView | LegacyPlayerControlView | | StyledPlayerControlView | PlayerControlView | | StyledPlayerView | PlayerView | Which means `PlayerControlView` are...

Hi Daniel, Would it be possibly for you to provide a minimal MainActivity.kt that shows `LazyRow` initialization, `Player` intialisation and setting of the `MediaItem`s. I tried to reproduce the issue,...

Hi @ziem, apologies, I have been away for a month. I discussed this internally within the team and I don't see any issue with the proposed solution above. I will...