From d3f5e6b361348a630eba0f0b42e3d8afa0bd1ce3 Mon Sep 17 00:00:00 2001
From: Usman Baig <mubaig@pm.me>
Date: Mon, 16 Mar 2026 14:08:08 +0100
Subject: [PATCH] fix: disable sankey labels, reduce margins, dynamic height

Labels were overlapping badly with many nodes. Rely on hover
tooltips instead. Chart height now scales with node count
(400-700px range).
---
 components/journeys/SankeyJourney.tsx | 16 ++++++----------
 1 file changed, 6 insertions(+), 10 deletions(-)

diff --git a/components/journeys/SankeyJourney.tsx b/components/journeys/SankeyJourney.tsx
index aa0ff92..84875a7 100644
--- a/components/journeys/SankeyJourney.tsx
+++ b/components/journeys/SankeyJourney.tsx
@@ -280,17 +280,17 @@ export default function SankeyJourney({
         </div>
       )}
 
-      <div style={{ height: 500 }}>
+      <div style={{ height: Math.max(400, Math.min(700, data.nodes.length * 28)) }}>
         <ResponsiveSankey<SankeyNode, SankeyLink>
           data={data}
-          margin={{ top: 16, right: 140, bottom: 16, left: 140 }}
+          margin={{ top: 8, right: 8, bottom: 8, left: 8 }}
           align="justify"
           sort="descending"
           colors={(node) =>
             COLUMN_COLORS[node.stepIndex % COLUMN_COLORS.length]
           }
-          nodeThickness={14}
-          nodeSpacing={16}
+          nodeThickness={12}
+          nodeSpacing={20}
           nodeInnerPadding={0}
           nodeBorderWidth={0}
           nodeBorderRadius={3}
@@ -300,13 +300,9 @@ export default function SankeyJourney({
           linkOpacity={0.2}
           linkHoverOpacity={0.5}
           linkHoverOthersOpacity={0.05}
-          linkContract={1}
+          linkContract={2}
           enableLinkGradient
-          enableLabels
-          label={(node) => smartLabel(pathFromId(node.id))}
-          labelPosition="outside"
-          labelPadding={12}
-          labelTextColor={labelColor}
+          enableLabels={false}
           isInteractive
           onClick={handleClick}
           nodeTooltip={({ node }) => (