Skip to content

Conversation

holzgeist
Copy link
Contributor

It looks like the video track is sometimes rotated internally. Without access to rotation or the computed aspectRatio, it's difficult to know the actual size of the video

It looks like the video track is sometimes rotated internally.
Without access to rotation or the computed aspectRatio, it's
difficult to know the actual size of the video
@holzgeist
Copy link
Contributor Author

If RTCVideoValue should not be exposed on this interface, only exposing aspectRatio (and rotation for good measure?) would also work

Copy link
Member

@cloudwebrtc cloudwebrtc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@cloudwebrtc cloudwebrtc merged commit 845fc68 into flutter-webrtc:main Jan 29, 2025
cloudwebrtc pushed a commit to flutter-webrtc/flutter-webrtc that referenced this pull request Jan 29, 2025
daniel-g-favoreto-opl pushed a commit to daniel-g-favoreto-opl/flutter-webrtc that referenced this pull request Mar 24, 2025
cloudwebrtc pushed a commit to livekit/client-sdk-flutter that referenced this pull request Jul 31, 2025
This allows external users to e.g. apply rounded edges. Without this
patch, VideoTrackRenderer will expand to its available space with no way
(known to me..) for parent widgets to know how large the video actually
is.

depends on flutter-webrtc/flutter-webrtc#1754
(and flutter-webrtc/webrtc-interface#30)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants