nikrozman/element-web
1
0
Fork 0
Code Issues Actions 3 Packages Projects Releases Wiki Activity

Don't clear filters on space change (#30903)

Browse Source
This commit is contained in:
David Langley
2025-09-30 10:53:48 +01:00
committed by GitHub
parent a9f4ccaa02
commit 7e48f2b6b6
2 changed files with 3 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/components/viewmodels/roomlist/useFilteredRooms.tsx
View File

@@ -15,8 +15,6 @@ import RoomListStoreV3, {
type RoomsResult,
} from "../../../stores/room-list-v3/RoomListStoreV3";
import { useEventEmitter } from "../../../hooks/useEventEmitter";
import SpaceStore from "../../../stores/spaces/SpaceStore";
import { UPDATE_SELECTED_SPACE } from "../../../stores/spaces";
/**
* Provides information about a primary filter.
@@ -74,9 +72,6 @@ export function useFilteredRooms(): FilteredRooms {
setRoomsResult(newRooms);
}, []);
// Reset filters when active space changes
useEventEmitter(SpaceStore.instance, UPDATE_SELECTED_SPACE, () => setPrimaryFilter(undefined));
const filterUndefined = (array: (FilterKey | undefined)[]): FilterKey[] =>
array.filter((f) => f !== undefined) as FilterKey[];

6
test/unit-tests/components/viewmodels/roomlist/RoomListViewModel-test.tsx
View File

@@ -127,7 +127,7 @@ describe("RoomListViewModel", () => {
expect(vm.current.activePrimaryFilter).toEqual(vm.current.primaryFilters[i]);
});
it("should remove all filters when active space is changed", async () => {
it("should not remove all filters when active space is changed", async () => {
mockAndCreateRooms();
const { result: vm } = renderHook(() => useRoomListViewModel());
@@ -141,8 +141,8 @@ describe("RoomListViewModel", () => {
// Simulate a space change
await act(() => SpaceStore.instance.emit(UPDATE_SELECTED_SPACE));
// Primary filer should have been unapplied
expect(vm.current.activePrimaryFilter).toEqual(undefined);
// Primary filter should remain unchanged
expect(vm.current.activePrimaryFilter?.name).toEqual("People");
});
});