Add explanation to the enable video button tooltip about reconnection

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
nextcloud · May 16, 2019 · a66c6a9 · a66c6a9
1 parent 04449b3
commit a66c6a9
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/js/views/mediacontrolsview.js b/js/views/mediacontrolsview.js
@@ -183,7 +183,7 @@
 			} else {
 				this._webrtc.pauseVideo();
 
-				this.getUI('videoButton').attr('data-original-title', t('spreed', 'Enable video (v)'))
+				this.getUI('videoButton').attr('data-original-title', this._getEnableVideoButtonTitle())
 					.addClass('local-video-disabled video-disabled icon-video-off')
 					.removeClass('icon-video');
 				this.getUI('audioButton').addClass('local-video-disabled');
@@ -195,6 +195,14 @@
 			return true;
 		},
 
+		_getEnableVideoButtonTitle: function() {
+			if (!this._app.signaling || this._app.signaling.getSendVideoIfAvailable()) {
+				return t('spreed', 'Enable video (v)');
+			}
+
+			return t('spreed', 'Enable video (v) - Your connection will be briefly interrupted when enabling the video for the first time');
+		},
+
 		/**
 		 * Sets the video as available or not available.
 		 *