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

Handle cases where the user rapidly clicks between rooms

Browse source 
Once the user has breadcrumbs, there should always be breadcrumbs. Therefore it is safe to ignore any updates which have zero entries.
This commit is contained in:
Travis Ralston 2019-04-04 16:21:57 -06:00
parent 406196e11c
commit 64a22236c3
1 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -88,7 +88,9 @@ export default class RoomBreadcrumbs extends React.Component {
} }
const roomIds = rooms.map((r) => r.room.roomId); const roomIds = rooms.map((r) => r.room.roomId);
SettingsStore.setValue("breadcrumb_rooms", null, SettingLevel.ACCOUNT, roomIds); if (roomIds.length > 0) {
SettingsStore.setValue("breadcrumb_rooms", null, SettingLevel.ACCOUNT, roomIds);
}
} }
onAction(payload) { onAction(payload) {
@ -147,6 +149,8 @@ export default class RoomBreadcrumbs extends React.Component {
}; };
_loadRoomIds(roomIds) { _loadRoomIds(roomIds) {
if (!roomIds || roomIds.length <= 0) return; // Skip updates with no rooms
// If we're here, the list changed. // If we're here, the list changed.
const rooms = roomIds.map((r) => MatrixClientPeg.get().getRoom(r)).filter((r) => r).map((r) => { const rooms = roomIds.map((r) => MatrixClientPeg.get().getRoom(r)).filter((r) => r).map((r) => {
const badges = this._calculateBadgesForRoom(r) || {}; const badges = this._calculateBadgesForRoom(r) || {};