twilio / video-quickstart-android

Twilio Video Quickstart for Android
MIT License
213 stars 160 forks source link

Add VideoView programatically overlay UI after addSink() step #737

Closed ObranS closed 1 year ago

ObranS commented 1 year ago

Code:

val primaryVideoView = VideoView(context)
binding.primaryVideoContainer.addView(primaryVideoView)

...
// after this step primaryVideoView overlay all UI components
remote?.addSink(primaryVideoView!!)

Any suggestions how to fix this?

ObranS commented 1 year ago

In case I use primaryVideoView from layout/video_view_layouy.xml it works perfectly

...
            <com.twilio.video.VideoView
                android:id="@+id/primary_video_view"
                android:layout_width="match_parent"
                android:layout_height="match_parent" />
...
afalls-twilio commented 1 year ago

@ObranS Thank you for the bug report, this issue is reproducible with the quickstart I presume?

ObranS commented 1 year ago

@afalls-twilio unfortunatelly I use VideoView in context of Flutter Platform View where are all components are placed inside CoordinatorLayout.

I'm thinking about, if rewrite quickstart with programatically add VideoView this issue will be reproduced.

I'll make time and try it.

afalls-twilio commented 1 year ago

@ObranS How are you, any update?, how did it work when you modified the quickstart?

afalls-twilio commented 1 year ago

No response, closing