Fix re-rendering bug

storybookjs · Jun 17, 2024 · 22a699b · 22a699b
1 parent b8af895
commit 22a699b
Showing 1 changed file with 12 additions and 17 deletions.
diff --git a/code/ui/manager/src/components/sidebar/Sidebar.tsx b/code/ui/manager/src/components/sidebar/Sidebar.tsx
@@ -103,6 +103,17 @@ const useCombination = (
   return useMemo(() => ({ hash, entries: Object.entries(hash) }), [hash]);
 };
 
+const updateQueryParams = (params: Record<string, string>) => {
+  const url = new URL(window.location.href);
+  Object.entries(params).forEach(([key, value]) => {
+    if (value) {
+      url.searchParams.set(key, value);
+    } else {
+      url.searchParams.delete(key);
+    }
+  });
+  window.history.pushState({}, '', url);
+};
 export interface SidebarProps extends API_LoadedRefData {
   refs: State['refs'];
   status: State['status'];
@@ -152,23 +163,7 @@ export const Sidebar = React.memo(function Sidebar({
             isLoading={isLoading}
             onMenuClick={onMenuClick}
           />
-          {index && (
-            <TagsFilter
-              api={api}
-              index={index}
-              updateQueryParams={(params) => {
-                const url = new URL(window.location.href);
-                Object.entries(params).forEach(([key, value]) => {
-                  if (value) {
-                    url.searchParams.set(key, value);
-                  } else {
-                    url.searchParams.delete(key);
-                  }
-                });
-                window.history.pushState({}, '', url);
-              }}
-            />
-          )}
+          {index && <TagsFilter api={api} index={index} updateQueryParams={updateQueryParams} />}
           <Search
             dataset={dataset}
             enableShortcuts={enableShortcuts}