From adddb0f0d2518cf37cc608aa04433b14884f1b17 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C5=A0imon=20Brandner?= <simon.bra.ag@gmail.com>
Date: Thu, 13 May 2021 18:11:47 +0200
Subject: [PATCH] Show screensharign button only if connected
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
---
 src/components/views/voip/CallView.tsx | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/src/components/views/voip/CallView.tsx b/src/components/views/voip/CallView.tsx
index dc84cbed60..a9d605c9f1 100644
--- a/src/components/views/voip/CallView.tsx
+++ b/src/components/views/voip/CallView.tsx
@@ -506,10 +506,14 @@ export default class CallView extends React.Component<IProps, IState> {
         /> : null;
 
         let screensharingButton;
-        // Screensharing is possible, if we can send a second stream and identify
-        // it using SDPStreamMetadata or if we can replace the already existing
-        // usermedia track by a screensharing track
-        if (this.props.call.opponentSupportsSDPStreamMetadata() || this.props.call.type === CallType.Video) {
+        // Screensharing is possible, if we can send a second stream and
+        // identify it using SDPStreamMetadata or if we can replace the already
+        // existing usermedia track by a screensharing track. We also need to be
+        // connected to know the state of the other side
+        if (
+            (this.props.call.opponentSupportsSDPStreamMetadata() || this.props.call.type === CallType.Video) &&
+            this.props.call.state === CallState.Connected
+        ) {
             screensharingButton = (
                 <AccessibleButton
                     className={screensharingClasses}