From f85ac3ef910a17ded55171be5bd5aebe7162ae02 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sa=C3=BAl=20Ibarra=20Corretg=C3=A9?= Date: Wed, 12 Jun 2019 17:25:46 +0200 Subject: [PATCH] rn: fix video unmuting when disabling audio-only When the video unmute button disabled audio-only, also unmute video. This fixes a weird case in which the user need to "unmute twice" if they were muted beofre they enabled audio-only mode. That's ok if the audio-only button was used, but not if the video-unmute button was used, since the expectation is to have video, of course. --- react/features/toolbox/components/VideoMuteButton.js | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/react/features/toolbox/components/VideoMuteButton.js b/react/features/toolbox/components/VideoMuteButton.js index 14a744cfa..8bfaf5a25 100644 --- a/react/features/toolbox/components/VideoMuteButton.js +++ b/react/features/toolbox/components/VideoMuteButton.js @@ -135,14 +135,14 @@ class VideoMuteButton extends AbstractVideoMuteButton { if (this.props._audioOnly) { this.props.dispatch( setAudioOnly(false, /* ensureTrack */ true)); - } else { - this.props.dispatch( - setVideoMuted( - videoMuted, - VIDEO_MUTISM_AUTHORITY.USER, - /* ensureTrack */ true)); } + this.props.dispatch( + setVideoMuted( + videoMuted, + VIDEO_MUTISM_AUTHORITY.USER, + /* ensureTrack */ true)); + // FIXME: The old conference logic still relies on this event being // emitted. typeof APP === 'undefined'