From 43362bfb98f184888c8b389228e0e7825fb756b0 Mon Sep 17 00:00:00 2001 From: Travis Ralston Date: Tue, 9 Apr 2019 09:40:49 -0600 Subject: [PATCH] Explain why the breadcrumbs don't need persisting elsewhere --- src/components/views/rooms/RoomBreadcrumbs.js | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/components/views/rooms/RoomBreadcrumbs.js b/src/components/views/rooms/RoomBreadcrumbs.js index 989e0a1305..273ade0e2a 100644 --- a/src/components/views/rooms/RoomBreadcrumbs.js +++ b/src/components/views/rooms/RoomBreadcrumbs.js @@ -242,6 +242,9 @@ export default class RoomBreadcrumbs extends React.Component { this.refs.scroller.moveToOrigin(); } + // We don't track room aesthetics (badges, membership, etc) over the wire so we + // don't need to do this elsewhere in the file. Just where we alter the room IDs + // and their order. const roomIds = rooms.map((r) => r.room.roomId); if (roomIds.length > 0) { SettingsStore.setValue("breadcrumb_rooms", null, SettingLevel.ACCOUNT, roomIds);