Add a comment to explain the state of affairs for refs.messagePanel

pull/21833/head
Travis Ralston 2019-03-28 09:50:17 -06:00
parent 4eb9fa6922
commit 6a840a56d6
1 changed files with 6 additions and 0 deletions

View File

@ -429,6 +429,12 @@ module.exports = React.createClass({
roomView.addEventListener('dragend', this.onDragLeaveOrEnd);
}
}
// Note: We check the ref here with a flag because componentDidMount, despite
// documentation, does not define our messagePanel ref. It looks like our spinner
// in render() prevents the ref from being set on first mount, so we try and
// catch the messagePanel when it does mount. Because we only want the ref once,
// we use a boolean flag to avoid duplicate work.
if (this.refs.messagePanel && !this.state.atEndOfLiveTimelineInit) {
this.setState({
atEndOfLiveTimelineInit: true,