From c2f5afe9c23ca28aa2feefc3f2047eaa2535eab4 Mon Sep 17 00:00:00 2001 From: Leonard Kim Date: Tue, 14 Nov 2017 11:09:42 -0800 Subject: [PATCH] fix(large-video): switch to dominant speaker with no video On DOMINANT_SPEAKER_CHANGED events, switch to the new dominant speaker regardless of if the speaker has video; it's okay to show the speaker's avatar. --- modules/UI/videolayout/VideoLayout.js | 4 ---- 1 file changed, 4 deletions(-) diff --git a/modules/UI/videolayout/VideoLayout.js b/modules/UI/videolayout/VideoLayout.js index 2a53071b2..3407dea77 100644 --- a/modules/UI/videolayout/VideoLayout.js +++ b/modules/UI/videolayout/VideoLayout.js @@ -721,11 +721,7 @@ const VideoLayout = { // Local video will not have container found, but that's ok // since we don't want to switch to local video. - // Update the large video if the video source is already available, - // otherwise wait for the "videoactive.jingle" event. - // FIXME: there is no "videoactive.jingle" event. if (!interfaceConfig.filmStripOnly && !this.getPinnedId() - && remoteVideo.hasVideoStarted() && !this.getCurrentlyOnLargeContainer().stayOnStage()) { this.updateLargeVideo(id); }