filmstrip: don't display toolbar for SIP gateways

Note that in the usual (vertical) setup, the "toolbar" is just the hide button.
This commit is contained in:
Saúl Ibarra Corretgé 2020-02-25 13:08:04 +01:00 committed by Saúl Ibarra Corretgé
parent 9b141816d6
commit bde2343951
1 changed files with 13 additions and 2 deletions

View File

@ -58,6 +58,11 @@ type Props = {
*/ */
_hideScrollbar: boolean, _hideScrollbar: boolean,
/**
* Whether the filmstrip toolbar should be hidden or not.
*/
_hideToolbar: boolean,
/** /**
* Whether or not remote videos are currently being hovered over. Hover * Whether or not remote videos are currently being hovered over. Hover
* handling is currently being handled detected outside of react. * handling is currently being handled detected outside of react.
@ -195,12 +200,17 @@ class Filmstrip extends Component <Props> {
remoteVideosWrapperClassName += ' hide-scrollbar'; remoteVideosWrapperClassName += ' hide-scrollbar';
} }
let toolbar = null;
if (!this.props._hideToolbar) {
toolbar = this.props._filmstripOnly ? <Toolbar /> : this._renderToggleButton();
}
return ( return (
<div <div
className = { `filmstrip ${this.props._className}` } className = { `filmstrip ${this.props._className}` }
style = { filmstripStyle }> style = { filmstripStyle }>
{ this.props._filmstripOnly { toolbar }
? <Toolbar /> : this._renderToggleButton() }
<div <div
className = { this.props._videosClassName } className = { this.props._videosClassName }
id = 'remoteVideos'> id = 'remoteVideos'>
@ -369,6 +379,7 @@ function _mapStateToProps(state) {
_filmstripOnly: isFilmstripOnly, _filmstripOnly: isFilmstripOnly,
_filmstripWidth: filmstripWidth, _filmstripWidth: filmstripWidth,
_hideScrollbar: Boolean(iAmSipGateway), _hideScrollbar: Boolean(iAmSipGateway),
_hideToolbar: Boolean(iAmSipGateway),
_hovered: hovered, _hovered: hovered,
_rows: gridDimensions.rows, _rows: gridDimensions.rows,
_videosClassName: videosClassName, _videosClassName: videosClassName,