PrivateCoffee/element-web
4
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 3

Remove redundant setState call, always focus composer after sync

Browse source
This commit is contained in:
Luke Barnard 2017-04-06 11:44:15 +01:00
parent 95b40a976c
commit 5f8a7b46a8
1 changed files with 3 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/components/structures/MatrixChat.js
View file

@ -644,7 +644,7 @@ module.exports = React.createClass({
// Wait for the first sync to complete so that if a room does have an alias,
// it would have been retrieved.
let waitFor = q(null);
if (!firstSyncComplete) {
if (!this.firstSyncComplete) {
if (!this.firstSyncPromise) {
console.warn('Cannot view a room before first sync. room_id:', room_info.room_id);
return;
@ -851,13 +851,8 @@ module.exports = React.createClass({
self.firstSyncComplete = true;
self.firstSyncPromise.resolve();
if (!self.state.page_type) {
// Switch to room view but allow _onLoggedIn to specify a room (if any)
self.setState({ready: true});
dis.dispatch({action: 'focus_composer'});
} else {
self.setState({ready: true});
}
dis.dispatch({action: 'focus_composer'});
self.setState({ready: true});
});
cli.on('Call.incoming', function(call) {
dis.dispatch({