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

Improve zeroing of RoomList notification badges

Browse source 
Use an action and force an update when zeroing the number of notifications in a room. This is better than waiting for a `render` to happen at some point. This will hopefully fix https://github.com/vector-im/riot-web/issues/3257
This commit is contained in:
Luke Barnard 2017-03-28 10:38:57 +01:00
parent aa486b605d
commit f4dc7ae8b1
2 changed files with 10 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/components/structures/TimelinePanel.js
View file

@ -476,9 +476,10 @@ var TimelinePanel = React.createClass({
// if we are scrolled to the bottom, do a quick-reset of our unreadNotificationCount
// to avoid having to wait from the remote echo from the homeserver.
if (this.isAtEndOfLiveTimeline()) {
this.props.timelineSet.room.setUnreadNotificationCount('total', 0);
this.props.timelineSet.room.setUnreadNotificationCount('highlight', 0);
// XXX: i'm a bit surprised we don't have to emit an event or dispatch to get this picked up
dis.dispatch({
action: 'on_room_read',
room: this.props.timelineSet.room,
});
}
var currentReadUpToEventId = this._getCurrentReadReceipt(true);

6
src/components/views/rooms/RoomList.js
View file

@ -96,6 +96,12 @@ module.exports = React.createClass({
});
}
break;
case 'on_room_read':
payload.room.setUnreadNotificationCount('total', 0);
payload.room.setUnreadNotificationCount('highlight', 0);
// Force an update because this state is too deep to cause an update
this.forceUpdate();
break;
}
},