Skip to content
Snippets Groups Projects
Unverified Commit 91ce8e0e authored by Oswaldo Acauan's avatar Oswaldo Acauan Committed by GitHub
Browse files

Merge pull request #5541 from Tainan404/issue-5514

Rename function to be consistent with state name.
parents ddf95664 54a0f16c
No related branches found
No related tags found
No related merge requests found
......@@ -9,7 +9,7 @@ export default class ScreenshareComponent extends React.Component {
loaded: false,
};
this.hideConnectingIcon = this.hideConnectingIcon.bind(this);
this.onVideoLoad = this.onVideoLoad.bind(this);
}
componentDidMount() {
this.props.presenterScreenshareHasStarted();
......@@ -23,14 +23,14 @@ export default class ScreenshareComponent extends React.Component {
this.props.presenterScreenshareHasEnded();
this.props.unshareScreen();
}
hideConnectingIcon() {
onVideoLoad() {
this.setState({ loaded: true });
}
render() {
return (
[!this.state.loaded ? (<div className={styles.connecting} />) : null,
(<video id="screenshareVideo" style={{ maxHeight: '100%', width: '100%' }} autoPlay playsInline onLoadedData={this.hideConnectingIcon} />)]
(<video id="screenshareVideo" style={{ maxHeight: '100%', width: '100%' }} autoPlay playsInline onLoadedData={this.onVideoLoad} />)]
);
}
}
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment