feat(config) add ability to hide dominant speaker badge

This commit is contained in:
Shawn Chin 2022-01-28 13:35:01 +00:00 committed by GitHub
parent 5baa2f839f
commit 23b4e93ed9
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 15 additions and 9 deletions

View File

@ -506,7 +506,10 @@ var config = {
// defaultRemoteDisplayName: 'Fellow Jitster',
// Hides the display name from the participant thumbnail
// hideDisplayName: false
// hideDisplayName: false,
// Hides the dominant speaker name badge that hovers above the toolbox
// hideDominantSpeakerBadge: false,
// Default language for the user interface.
// defaultLanguage: 'en',

View File

@ -162,6 +162,7 @@ export default [
'hiddenPremeetingButtons',
'hideConferenceSubject',
'hideDisplayName',
'hideDominantSpeakerBadge',
'hideRecordingLabel',
'hideParticipantsStats',
'hideConferenceTimer',

View File

@ -238,7 +238,7 @@ class Conference extends AbstractConference<Props, *> {
<Filmstrip />
</div>
{ _showPrejoin || _showLobby || <Toolbox showDominantSpeakerName = { true } /> }
{ _showPrejoin || _showLobby || <Toolbox /> }
<Chat />
{_notificationsVisible && (_overflowDrawer

View File

@ -220,6 +220,11 @@ type Props = {
*/
_sharingVideo: boolean,
/**
* Whether or not to show dominant speaker badge.
*/
_showDominantSpeakerBadge: boolean,
/**
* Whether or not the tile view is enabled.
*/
@ -250,11 +255,6 @@ type Props = {
*/
dispatch: Function,
/**
* If the dominant speaker name should be displayed or not.
*/
showDominantSpeakerName?: boolean,
/**
* Invoked to obtain translated strings.
*/
@ -1258,8 +1258,8 @@ class Toolbox extends Component<Props> {
_overflowMenuVisible,
_reactionsEnabled,
_toolbarButtons,
_showDominantSpeakerBadge,
classes,
showDominantSpeakerName,
t
} = this.props;
@ -1278,7 +1278,7 @@ class Toolbox extends Component<Props> {
onMouseOver: this._onMouseOver
}) }>
{ showDominantSpeakerName && <DominantSpeakerName /> }
{ _showDominantSpeakerBadge && <DominantSpeakerName /> }
<div className = 'toolbox-content-items'>
{mainMenuButtons.map(({ Content, key, ...rest }) => Content !== Separator && (
@ -1349,6 +1349,7 @@ function _mapStateToProps(state, ownProps) {
callStatsID,
disableProfile,
enableFeaturesBasedOnToken,
hideDominantSpeakerBadge,
buttonsWithNotifyClick
} = state['features/base/config'];
const {
@ -1404,6 +1405,7 @@ function _mapStateToProps(state, ownProps) {
_raisedHand: hasRaisedHand(localParticipant),
_reactionsEnabled: isReactionsEnabled(state),
_screenSharing: isScreenVideoShared(state),
_showDominantSpeakerBadge: !hideDominantSpeakerBadge,
_tileViewEnabled: shouldDisplayTileView(state),
_toolbarButtons: toolbarButtons,
_virtualSource: state['features/virtual-background'].virtualSource,