From 1511d71dcdb853373935e86e55d1cbb4e8b73321 Mon Sep 17 00:00:00 2001 From: stroblme Date: Tue, 25 Aug 2026 22:13:32 +0200 Subject: [PATCH] Draw the edge sparkline only for a value it can plot The popover showed a non-numeric value twice: only numbers are recorded, so MessageSparkline fell through to a collapsed ValuePreview and the inspector rendered its own expanded one below it. --- frontend/src/components/Flow/EdgeInspector.tsx | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/frontend/src/components/Flow/EdgeInspector.tsx b/frontend/src/components/Flow/EdgeInspector.tsx index f2aa90e..53c2b3e 100644 --- a/frontend/src/components/Flow/EdgeInspector.tsx +++ b/frontend/src/components/Flow/EdgeInspector.tsx @@ -132,15 +132,18 @@ export function EdgeInspector({ {/* The same curve the node panel draws for this message: one value - says little, how it has been moving says the rest. */} - {live === undefined ? null : ( + says little, how it has been moving says the rest. Only for a + number, because only numbers are recorded — for anything else the + sparkline falls through to its own copy of the value, which the + popover already shows below. */} + {typeof live?.value === "number" ? (
- )} + ) : null} {/* Several nodes can publish one message, and so can a dashboard or another flow — so the value alone does not say what caused it. */}