Skip to content

Commit

Permalink
filmstrip: don't display toolbar for SIP gateways
Browse files Browse the repository at this point in the history
Note that in the usual (vertical) setup, the "toolbar" is just the hide button.
  • Loading branch information
saghul committed Feb 25, 2020
1 parent 9b14181 commit bde2343
Showing 1 changed file with 13 additions and 2 deletions.
15 changes: 13 additions & 2 deletions react/features/filmstrip/components/web/Filmstrip.js
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,11 @@ type Props = {
*/
_hideScrollbar: boolean,

/**
* Whether the filmstrip toolbar should be hidden or not.
*/
_hideToolbar: boolean,

/**
* Whether or not remote videos are currently being hovered over. Hover
* handling is currently being handled detected outside of react.
Expand Down Expand Up @@ -195,12 +200,17 @@ class Filmstrip extends Component <Props> {
remoteVideosWrapperClassName += ' hide-scrollbar';
}

let toolbar = null;

if (!this.props._hideToolbar) {
toolbar = this.props._filmstripOnly ? <Toolbar /> : this._renderToggleButton();
}

return (
<div
className = { `filmstrip ${this.props._className}` }
style = { filmstripStyle }>
{ this.props._filmstripOnly
? <Toolbar /> : this._renderToggleButton() }
{ toolbar }
<div
className = { this.props._videosClassName }
id = 'remoteVideos'>
Expand Down Expand Up @@ -369,6 +379,7 @@ function _mapStateToProps(state) {
_filmstripOnly: isFilmstripOnly,
_filmstripWidth: filmstripWidth,
_hideScrollbar: Boolean(iAmSipGateway),
_hideToolbar: Boolean(iAmSipGateway),
_hovered: hovered,
_rows: gridDimensions.rows,
_videosClassName: videosClassName,
Expand Down

0 comments on commit bde2343

Please sign in to comment.