Skip to content

[Bugfix] local video tile included in overflow gallery#4136

Merged
dmceachernmsft merged 16 commits intomainfrom
dmceachernmsft/local-video-gallery
Feb 21, 2024
Merged

[Bugfix] local video tile included in overflow gallery#4136
dmceachernmsft merged 16 commits intomainfrom
dmceachernmsft/local-video-gallery

Conversation

@dmceachernmsft
Copy link
Copy Markdown
Member

What

Adds local tile to gallery in default mode

Why

Shows local tile when screenshare is active

  • we were adding it to participant calculations but the render functions don't work on the local participant. and we have the rendered tile to add directly post order organization

https://skype.visualstudio.com/SPOOL/_workitems/edit/3563581

How Tested

image
Validated locally

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 13, 2024

Calling bundle size is decreased✅.

  • Current size: 4958741
  • Base size: 4958759
  • Diff size: -18

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 13, 2024

CallWithChat bundle size is decreased✅.

  • Current size: 5688321
  • Base size: 5688339
  • Diff size: -18

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 13, 2024

Chat bundle size is not changed.

  • Current size: 1451591
  • Base size: 1451591
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

Failed to pass the UI Test. If this PR is for UI change and the error is snapshot mismatch, please add "update_snapshots" label to the PR for updating the snapshot.

@dmceachernmsft dmceachernmsft added the update_snapshots Set this label to request automated update of UI snapshots label Feb 15, 2024
@github-actions github-actions Bot removed the update_snapshots Set this label to request automated update of UI snapshots label Feb 15, 2024
@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@dmceachernmsft dmceachernmsft enabled auto-merge (squash) February 21, 2024 19:05
@github-actions
Copy link
Copy Markdown
Contributor

@dmceachernmsft dmceachernmsft merged commit b018a4c into main Feb 21, 2024
@dmceachernmsft dmceachernmsft deleted the dmceachernmsft/local-video-gallery branch February 21, 2024 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants