Skip to content

browser-sdk: Add support for render prop for subgrid in video grid#1064

Open
thyal wants to merge 2 commits into
mainfrom
thomas/PAN-2701-render-subgrid-participant
Open

browser-sdk: Add support for render prop for subgrid in video grid#1064
thyal wants to merge 2 commits into
mainfrom
thomas/PAN-2701-render-subgrid-participant

Conversation

@thyal

@thyal thyal commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

Description

Fixes #1047
Adds a new prop renderSubgridParticipant in the <VideoGrid /> component. This is similar to the renderParticipant render prop that exists already.

The change itself is very small, but I've done several changes to the storybook entries for the video grid to allow for easier testing going forward. Added two entries with fake participants, so that the behaviour of the grid can be tested without joining with a bunch of clients.

Summary:

Related Issue:

Testing

  1. pnpm build && pnpm dev
  2. Go to the Video Grid Mocked Custom story and check out the custom subgrid tiles

Screenshots/GIFs (if applicable)

Checklist

  • My code follows the project's coding standards.
  • Prefixed the PR title and commit messages with the service or package name
  • I have written unit tests (if applicable).
  • I have updated the documentation (if applicable).
  • By submitting this pull request, I confirm that my contribution is made
    under the terms of the MIT license.

Additional Information

@changeset-bot

changeset-bot Bot commented Jul 23, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 85b07f2

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@whereby.com/browser-sdk Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@thyal
thyal requested a review from a team July 23, 2026 14:14

@sarahrose-ferris sarahrose-ferris 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.

Looks really good - nice additions with the stories too 👏

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.

Add renderSubgridParticipant prop to VideoGrid component

2 participants