Merge pull request #5093 from xyflow/refactor/minimap-hidden

refactor(minimap): do not show hidden nodes

.changeset/twenty-sloths-reflect.md

@@ -0,0 +1,6 @@
+---
+'@xyflow/react': patch
+'@xyflow/svelte': patch
+---
+
+Hidden nodes are not displayed in the mini map anymore

packages/react/src/additional-components/MiniMap/MiniMap.tsx

@@ -15,6 +15,8 @@ import type { MiniMapProps } from './types';
 const defaultWidth = 200;
 const defaultHeight = 150;
 
+const filterHidden = (node: Node) => !node.hidden;
+
 const selector = (s: ReactFlowState) => {
   const viewBB: Rect = {
     x: -s.transform[0] / s.transform[2],
@@ -25,7 +27,10 @@ const selector = (s: ReactFlowState) => {
 
   return {
     viewBB,
-    boundingRect: s.nodeLookup.size > 0 ? getBoundsOfRects(getInternalNodesBounds(s.nodeLookup), viewBB) : viewBB,
+    boundingRect:
+      s.nodeLookup.size > 0
+        ? getBoundsOfRects(getInternalNodesBounds(s.nodeLookup, { filter: filterHidden }), viewBB)
+        : viewBB,
     rfId: s.rfId,
     panZoom: s.panZoom,
     translateExtent: s.translateExtent,
@@ -113,16 +118,16 @@ function MiniMapComponent<NodeType extends Node = Node>({
 
   const onSvgClick = onClick
     ? (event: MouseEvent) => {
-      const [x, y] = minimapInstance.current?.pointer(event) || [0, 0];
+        const [x, y] = minimapInstance.current?.pointer(event) || [0, 0];
-      onClick(event, { x, y });
+        onClick(event, { x, y });
-    }
+      }
     : undefined;
 
   const onSvgNodeClick = onNodeClick
     ? useCallback((event: MouseEvent, nodeId: string) => {
-      const node = store.getState().nodeLookup.get(nodeId)!;
+        const node = store.getState().nodeLookup.get(nodeId)!;
-      onNodeClick(event, node);
+        onNodeClick(event, node);
-    }, [])
+      }, [])
     : undefined;
 
   return (

packages/svelte/src/lib/plugins/Minimap/Minimap.svelte

@@ -76,7 +76,12 @@
 
   $: {
     boundingRect =
-      $nodeLookup.size > 0 ? getBoundsOfRects(getInternalNodesBounds($nodeLookup), viewBB) : viewBB;
+      $nodeLookup.size > 0
+        ? getBoundsOfRects(
+            getInternalNodesBounds($nodeLookup, { filter: (n) => !n.hidden }),
+            viewBB
+          )
+        : viewBB;
     $nodes;
   }