From b79c6a933135d2711679737641905c26335b26c6 Mon Sep 17 00:00:00 2001 From: Jeff Davidson Date: Mon, 14 Oct 2024 19:54:10 -0700 Subject: [PATCH] Properly store deleted puzzle group collapse state. As best as I can tell, this was a typo. Regular puzzle groups have their state stored as long as there is no active search query. This makes sense - as commit 2e09393894024aa7c44a7b2ddac106d656ba6140 notes, it would cause confusion when searching if we paid attention to collapse state when showing search results. However, the deleted group behaves the _opposite_ way - the state is _only_ persisted during searching. I can't imagine why persisting the state in this case would be desirable, but not during the no-search case. So this change makes everything consistent. See #2278 --- imports/client/components/PuzzleListPage.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/imports/client/components/PuzzleListPage.tsx b/imports/client/components/PuzzleListPage.tsx index ae7daab1e..9b2dee970 100644 --- a/imports/client/components/PuzzleListPage.tsx +++ b/imports/client/components/PuzzleListPage.tsx @@ -479,7 +479,7 @@ const PuzzleListView = ({ includeCount={false} canUpdate={canUpdate} suppressedTagIds={[]} - trackPersistentExpand={searchString !== ""} + trackPersistentExpand={searchString === ""} /> )}