From fe49e7766ad0efc4df999303a4158b7e2d8f1d75 Mon Sep 17 00:00:00 2001
From: Maksim Sukharev <antreesy.web@gmail.com>
Date: Wed, 2 Oct 2024 18:59:43 +0200
Subject: [PATCH] fix: do not show two promoted videos

- PresenterOverlay broke v-else-if flow in previous place
- also add distinct keys to prevent rendering issues

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
---
 src/components/CallView/CallView.vue | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/src/components/CallView/CallView.vue b/src/components/CallView/CallView.vue
index 8bb37a9dac5..f6366974542 100644
--- a/src/components/CallView/CallView.vue
+++ b/src/components/CallView/CallView.vue
@@ -21,7 +21,7 @@
 					:class="{'full-page': showFullPage}">
 					<!-- Selected video override mode -->
 					<VideoVue v-if="showSelectedVideo && selectedCallParticipantModel"
-						:key="selectedVideoPeerId"
+						:key="`promoted-${selectedVideoPeerId}`"
 						:token="token"
 						:model="selectedCallParticipantModel"
 						:shared-data="sharedDatas[selectedVideoPeerId]"
@@ -58,18 +58,9 @@
 						:call-participant-model="shownRemoteScreenCallParticipantModel"
 						:shared-data="sharedDatas[shownRemoteScreenPeerId]"
 						is-big />
-					<!-- presenter overlay -->
-					<PresenterOverlay v-if="shouldShowPresenterOverlay"
-						:token="token"
-						:model="presenterModel"
-						:shared-data="presenterSharedData"
-						:is-local-presenter="showLocalScreen"
-						:local-media-model="localMediaModel"
-						:is-collapsed="!showPresenterOverlay"
-						@click="toggleShowPresenterOverlay" />
 					<!-- Promoted "autopilot" mode -->
 					<VideoVue v-else-if="promotedParticipantModel"
-						:key="promotedParticipantModel.attributes.peerId"
+						:key="`autopilot-${promotedParticipantModel.attributes.peerId}`"
 						:token="token"
 						:model="promotedParticipantModel"
 						:shared-data="sharedDatas[promotedParticipantModel.attributes.peerId]"
@@ -80,6 +71,15 @@
 						:is-one-to-one="isOneToOne"
 						:is-sidebar="isSidebar"
 						@force-promote-video="forcePromotedModel = $event" />
+					<!-- presenter overlay -->
+					<PresenterOverlay v-if="shouldShowPresenterOverlay"
+						:token="token"
+						:model="presenterModel"
+						:shared-data="presenterSharedData"
+						:is-local-presenter="showLocalScreen"
+						:local-media-model="localMediaModel"
+						:is-collapsed="!showPresenterOverlay"
+						@click="toggleShowPresenterOverlay" />
 
 					<div v-else-if="devMode && !isGrid"
 						class="dev-mode-video--promoted">