From 409e6fd6b91138619bfb5b3b71cf76ade2bae4cd Mon Sep 17 00:00:00 2001 From: Pouzor Date: Sun, 8 Mar 2026 12:08:10 +0100 Subject: [PATCH] fix: show all explicit edges including proxmox-to-child links The visibleEdges filter was silently dropping any edge between a container-mode Proxmox and its children, even ones the user explicitly drew. Since no edges are auto-generated, the filter only blocked intentional connections. Removed it entirely. --- .../src/components/canvas/CanvasContainer.tsx | 20 ++----------------- 1 file changed, 2 insertions(+), 18 deletions(-) diff --git a/frontend/src/components/canvas/CanvasContainer.tsx b/frontend/src/components/canvas/CanvasContainer.tsx index d8c877b..2a4114e 100644 --- a/frontend/src/components/canvas/CanvasContainer.tsx +++ b/frontend/src/components/canvas/CanvasContainer.tsx @@ -1,4 +1,4 @@ -import { useCallback, useMemo } from 'react' +import { useCallback } from 'react' import { ReactFlow, Background, @@ -39,28 +39,12 @@ export function CanvasContainer({ onConnect: onConnectProp, onEdgeDoubleClick }: onEdgeDoubleClick?.(edge) }, [onEdgeDoubleClick]) - // Hide edges between a container-mode proxmox and its direct children - const visibleEdges = useMemo(() => { - const containerIds = new Set( - nodes - .filter((n) => n.type === 'proxmox' && n.data.container_mode !== false) - .map((n) => n.id) - ) - const childParentMap = new Map( - nodes.filter((n) => n.data.parent_id).map((n) => [n.id, n.data.parent_id as string]) - ) - return (edges as Edge[]).filter((e) => { - if (containerIds.has(e.source) && childParentMap.get(e.target) === e.source) return false - if (containerIds.has(e.target) && childParentMap.get(e.source) === e.target) return false - return true - }) - }, [nodes, edges]) return (