Skip to content

Conversation

@EnricoSchw
Copy link
Contributor

@EnricoSchw EnricoSchw commented May 9, 2025

BugWPB-17503 [Web/AVS] See black remote video tile when activating camera.

Description

We have a customized process when activating the camera. Since we can't adjust it without significant effort, we're minimizing the transition with this PR. AVS is now waiting for the encryption key until the new media pipeline is released.

Additionally, we have adapted the frontend to avoid black tiles.

Checklist

  • mentions the JIRA issue in the PR name (Ex. [WPB-XXXX])
  • PR has been self reviewed by the author;
  • Hard-to-understand areas of the code have been commented;
  • If it is a core feature, unit tests have been added;

EnricoSchw added 2 commits May 9, 2025 09:29
   - minimize the gap when reinitializing the media pipeline when the camera is turned on
Comment on lines 254 to 270
{doVideoReconnecting && (
<div className="group-video-grid__pause-overlay">
<div className="background">
<div className="background-image"></div>
<div className="background-darken"></div>
</div>

<div
className="group-video-grid__pause-overlay__label"
css={{fontsize: minimized ? '0.6875rem' : '0.875rem'}}
data-uie-name="status-video-paused"
>
{t('videoCallParticipantConnecting')}
</div>
{nameContainer}
</div>
)}
Copy link
Collaborator

Choose a reason for hiding this comment

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

this sounds a bit redundant code to me, maybe we could merge the common parts with what we already have

image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok i will adjust the template

@sonarqubecloud
Copy link

@EnricoSchw EnricoSchw merged commit 69bf60e into dev May 23, 2025
11 of 14 checks passed
@EnricoSchw EnricoSchw deleted the fix/black-tile-on-toggle-camera branch May 23, 2025 11:29
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.

4 participants