Bladeren bron

fix(toolbar): exercise conference audio toggling

The redux flows do not account for as many scenarios, such
as config.startWithAudioMuted being true.
master
Leonard Kim 7 jaren geleden
bovenliggende
commit
6916252ce1
1 gewijzigde bestanden met toevoegingen van 14 en 0 verwijderingen
  1. 14
    0
      react/features/toolbox/components/buttons/AudioMuteButton.web.js

+ 14
- 0
react/features/toolbox/components/buttons/AudioMuteButton.web.js Bestand weergeven

@@ -4,6 +4,7 @@ import PropTypes from 'prop-types';
4 4
 import React from 'react';
5 5
 import { connect } from 'react-redux';
6 6
 
7
+import UIEvents from '../../../../../service/UI/UIEvents';
7 8
 import {
8 9
     ACTION_SHORTCUT_TRIGGERED,
9 10
     AUDIO_MUTE,
@@ -122,6 +123,19 @@ export class AudioMuteButton extends AbstractAudioMuteButton {
122 123
 
123 124
     _doToggleAudio: () => void;
124 125
 
126
+    /**
127
+     * Emits an event to signal audio mute should be toggled.
128
+     *
129
+     * @private
130
+     * @returns {void}
131
+     */
132
+    _doToggleAudio() {
133
+        // The old conference logic must be used for now as the redux flows do
134
+        // not handle all cases, such as unmuting when the config
135
+        // startWithAudioMuted is true.
136
+        APP.UI.emitEvent(UIEvents.AUDIO_MUTED, !this.props._audioMuted, true);
137
+    }
138
+
125 139
     _onShortcutToggleAudio: () => void;
126 140
 
127 141
     /**

Laden…
Annuleren
Opslaan