Skip to content

VIDSOL-636: Support displaying multiple screen sharings (up to 2) in ActiveSpeakerLayout#113

Draft
Copilot wants to merge 2 commits intodevelopfrom
copilot/vidsol-636-multi-screen-sharing
Draft

VIDSOL-636: Support displaying multiple screen sharings (up to 2) in ActiveSpeakerLayout#113
Copilot wants to merge 2 commits intodevelopfrom
copilot/vidsol-636-multi-screen-sharing

Conversation

Copy link
Copy Markdown

Copilot AI commented Mar 26, 2026

What is this PR doing?

When two screenshare participants are present, both are now displayed in the main content area instead of sending the second one to the sidebar/overflow.

  • HorizontalActiveSpeakerLayoutView: Main area wraps in a VStack to stack both screenshares vertically within the 70% width region. Sidebar shows remaining non-screenshare participants.
  • VerticalActiveSpeakerLayoutView: Second screenshare renders below the first, above the regular participant tiles.
  • Both views add a secondScreenshare computed property that checks if the first two sorted participants are screenshares, and adjusts restOfParticipants accordingly.

Single screenshare and no-screenshare cases are unchanged.

How should this be manually tested?

  1. Join a room where two participants are sharing their screens simultaneously.
  2. Verify both screenshares appear in the main content area (stacked vertically) on both iPad (horizontal layout) and iPhone (vertical layout).
  3. Verify non-screenshare participants appear in the sidebar/below as before.
  4. Verify single-screenshare and no-screenshare scenarios are unaffected.

What are the relevant tickets?

VIDSOL-636

Justification for skipping ci, if applied?


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

When two screenshare participants are present at the front of the sorted
participants list, both are displayed in the main content area:
- Horizontal layout: stacked vertically in the 70% width main area
- Vertical layout: stacked vertically above the regular participants

Non-screenshare participants continue to appear in the sidebar (horizontal)
or below (vertical) as before.

Agent-Logs-Url: https://github.com/Vonage/vonage-video-ios-app/sessions/8635dfa0-b806-4f1c-8350-131e591cd5da

Co-authored-by: VZaphod <210490533+VZaphod@users.noreply.github.com>
Copilot AI changed the title [WIP] Add support for displaying two screen sharings Support displaying multiple screen sharings (up to 2) in ActiveSpeakerLayout Mar 26, 2026
Copilot AI requested a review from VZaphod March 26, 2026 22:23
@VZaphod VZaphod changed the title Support displaying multiple screen sharings (up to 2) in ActiveSpeakerLayout VIDSOL-636: Support displaying multiple screen sharings (up to 2) in ActiveSpeakerLayout Mar 27, 2026
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