Skip to content

feat: VideoTrackRender.placeholderBuilder - #1109

Merged
hiroshihorie merged 3 commits into
livekit:mainfrom
totem-technologies:placeholder
Jun 30, 2026
Merged

hiroshihorie merged 3 commits into
livekit:mainfrom
totem-technologies:placeholder

Conversation

@bdlukaa

@bdlukaa bdlukaa commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Fixes #1061

Example usage:

VideoTrackRenderer(
  ...,
  placeholderBuilder: (context) {
    return const CircularProgressIndicator.adaptive();
  },
)

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a new placeholderBuilder API to VideoTrackRenderer to allow callers to show a custom widget while the underlying RTCVideoView is still initializing / not yet rendering frames (addressing the “black view during setup” UX gap described in #1061).

Changes:

  • Introduces VideoTrackRenderer.placeholderBuilder (optional WidgetBuilder) as a new public parameter.
  • Wires placeholderBuilder through to flutter_webrtc’s RTCVideoView in both web and native render paths.
  • Adds a .changes entry documenting the new capability.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
lib/src/widgets/video_track_renderer.dart Adds the placeholderBuilder API and forwards it into RTCVideoView.
.changes/track-render-placeholder Adds a changelog fragment for the new placeholder capability.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread lib/src/widgets/video_track_renderer.dart Outdated
Comment thread lib/src/widgets/video_track_renderer.dart
@bdlukaa

bdlukaa commented Jun 21, 2026

Copy link
Copy Markdown
Contributor Author

Related: flutter-webrtc/flutter-webrtc#2102

@hiroshihorie
hiroshihorie merged commit 9e97914 into livekit:main Jun 30, 2026
14 checks passed
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.

[feature] Expose video placeholder

4 participants