First special treatment of space-rooms

src/stores/BreadcrumbsStore.ts

@@ -122,6 +122,7 @@ export class BreadcrumbsStore extends AsyncStoreWithClient<IState> {
     }
 
     private async appendRoom(room: Room) {
+        if (room.isSpaceRoom() && SettingsStore.getValue("feature_spaces")) return; // hide space rooms
         let updated = false;
         const rooms = (this.state.rooms || []).slice(); // cheap clone
 

src/stores/room-list/filters/VisibilityProvider.ts

@@ -18,6 +18,7 @@ import {Room} from "matrix-js-sdk/src/models/room";
 import CallHandler from "../../../CallHandler";
 import { RoomListCustomisations } from "../../../customisations/RoomList";
 import VoipUserMapper from "../../../VoipUserMapper";
+import SettingsStore from "../../../settings/SettingsStore";
 
 export class VisibilityProvider {
     private static internalInstance: VisibilityProvider;
@@ -37,22 +38,23 @@ export class VisibilityProvider {
     }
 
     public isRoomVisible(room: Room): boolean {
-        let isVisible = true; // Returned at the end of this function
-        let forced = false; // When true, this function won't bother calling the customisation points
-
         if (
             CallHandler.sharedInstance().getSupportsVirtualRooms() &&
             VoipUserMapper.sharedInstance().isVirtualRoom(room)
         ) {
-            isVisible = false;
-            forced = true;
+            return false;
+        }
+
+        // hide space rooms as they'll be shown in the SpacePanel
+        if (room.isSpaceRoom() && SettingsStore.getValue("feature_spaces")) {
+            return false;
         }
 
         const isVisibleFn = RoomListCustomisations.isRoomVisible;
-        if (!forced && isVisibleFn) {
-            isVisible = isVisibleFn(room);
+        if (isVisibleFn) {
+            return isVisibleFn(room);
         }
 
-        return isVisible;
+        return true; // default
     }
 }