Merge pull request #248 from hexawyz/connection-focus-outline

miroiu · web-flow · commit 01f67afd54d9 · 2026-01-06T15:43:56.000+02:00
Improve connection focus visual
diff --git a/Nodify/Connections/BaseConnection.cs b/Nodify/Connections/BaseConnection.cs
@@ -1015,7 +1015,9 @@ protected override void OnRender(DrawingContext drawingContext)
                 if (drawPen != null)
                 {
                     var widenPen = new Pen(null, _baseConnection.StrokeThickness + drawPen.Thickness + _baseConnection.FocusVisualPadding * 2d);
-                    drawingContext.DrawGeometry(null, drawPen, _baseConnection.DefiningGeometry.GetWidenedPathGeometry(widenPen));
+                    var geometry = _baseConnection.DefiningGeometry;
+                    var expandedGeometry = Geometry.Combine(geometry, geometry.GetWidenedPathGeometry(widenPen), GeometryCombineMode.Union, Transform.Identity);
+                    drawingContext.DrawGeometry(null, drawPen, expandedGeometry.GetOutlinedPathGeometry());
                 }
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -1015,7 +1015,9 @@ protected override void OnRender(DrawingContext drawingContext)`
`1015`	`1015`	`if (drawPen != null)`
`1016`	`1016`	`{`
`1017`	`1017`	`var widenPen = new Pen(null, _baseConnection.StrokeThickness + drawPen.Thickness + _baseConnection.FocusVisualPadding * 2d);`
`1018`		`- drawingContext.DrawGeometry(null, drawPen, _baseConnection.DefiningGeometry.GetWidenedPathGeometry(widenPen));`
	`1018`	`+ var geometry = _baseConnection.DefiningGeometry;`
	`1019`	`+ var expandedGeometry = Geometry.Combine(geometry, geometry.GetWidenedPathGeometry(widenPen), GeometryCombineMode.Union, Transform.Identity);`
	`1020`	`+ drawingContext.DrawGeometry(null, drawPen, expandedGeometry.GetOutlinedPathGeometry());`
`1019`	`1021`	`}`
`1020`	`1022`	`}`
`1021`	`1023`	`}`