From 4829b86352469039540d49d11e47a31be3e2c8e4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sa=C3=BAl=20Ibarra=20Corretg=C3=A9?= Date: Tue, 28 Nov 2017 12:02:51 +0100 Subject: [PATCH] feat(conference): hide user media permission overlay when tracks are created Don't wait for the connection to be made, since in some cases, when auth is rquired, for instance, that won't happen until the user inputs their credentials, but the dialog would be overshadowed by the overlay. --- conference.js | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/conference.js b/conference.js index 1657f3332..f64a43076 100644 --- a/conference.js +++ b/conference.js @@ -632,11 +632,14 @@ export default { }); } + // Hide permissions overlay when tracks are created + tryCreateLocalTracks.then(() => { + APP.store.dispatch( + mediaPermissionPromptVisibilityChanged(false)); + }); + return Promise.all([ tryCreateLocalTracks, connect(roomName) ]) .then(([ tracks, con ]) => { - APP.store.dispatch( - mediaPermissionPromptVisibilityChanged(false)); - // FIXME If there will be microphone error it will cover any // screensharing dialog, but it's still better than in // the reverse order where the screensharing dialog will