[DYN-7839] Wires & Pins: improve snapping tolerance

src/DynamoCore/Configuration/Configurations.cs

@@ -58,6 +58,16 @@ public class Configurations
         /// </summary>
         public static readonly double ZoomToFitPaddingFactor = 3.5;
 
+        /// <summary>
+        /// Const defining the base stroke thickness for connectors.
+        /// </summary>
+        public static readonly double ConnectorBaseThickness = 3;
+
+        /// <summary>
+        /// Const defining the scaling factor for adjusting connector stroke thickness based on zoom level.
+        /// </summary>
+        public const double ConnectorZoomScalingFactor = 2.0;
+
         #endregion
 
         #region Usage Reporting Error Message

src/DynamoCore/PublicAPI.Unshipped.txt

@@ -3017,6 +3017,8 @@ static readonly Dynamo.Configuration.Configurations.ZoomDirectEditThreshold -> d
 static readonly Dynamo.Configuration.Configurations.ZoomIncrement -> double
 static readonly Dynamo.Configuration.Configurations.ZoomThreshold -> double
 static readonly Dynamo.Configuration.Configurations.ZoomToFitPaddingFactor -> double
+static readonly Dynamo.Configuration.Configurations.ConnectorBaseThickness -> double
+static readonly Dynamo.Configuration.Configurations.ConnectorZoomScalingFactor -> double
 static readonly Dynamo.Configuration.PreferenceSettings.DynamoDefaultTime -> System.DateTime
 virtual Dynamo.Core.CustomNodeManager.OnCustomNodeRemoved(System.Guid functionId) -> void
 virtual Dynamo.Core.CustomNodeManager.OnDefinitionUpdated(Dynamo.CustomNodeDefinition obj) -> void

src/DynamoCoreWpf/UI/Themes/Modern/Connectors.xaml

@@ -5,7 +5,8 @@
                       xmlns:i="clr-namespace:Dynamo.Microsoft.Xaml.Behaviors;assembly=Dynamo.Microsoft.Xaml.Behaviors"
                       xmlns:views="clr-namespace:Dynamo.Controls;assembly=DynamoCoreWpf"
                       xmlns:nodes="clr-namespace:Dynamo.Nodes;assembly=DynamoCoreWpf"
-                      xmlns:mouse="clr-namespace:Dynamo.Wpf.UI;assembly=DynamoCoreWpf">
+                      xmlns:mouse="clr-namespace:Dynamo.Wpf.UI;assembly=DynamoCoreWpf"
+                      xmlns:configuration="clr-namespace:Dynamo.Configuration;assembly=DynamoCore">
 
     <ResourceDictionary.MergedDictionaries>
         <ui:SharedResourceDictionary Source="{x:Static ui:SharedDictionaryManager.DynamoConvertersDictionaryUri}" />
@@ -126,10 +127,15 @@
             </Canvas>
 
             <!--Bezier Path-->
-            <Path Stroke="{DynamicResource BConnectorSelection}" StrokeThickness="3"
+            <Path Stroke="{DynamicResource BConnectorSelection}" StrokeThickness="{Binding Source={x:Static configuration:Configurations.ConnectorBaseThickness}}"
               Name="connector"
               Visibility="{Binding IsCollapsed, Converter={StaticResource InverseBoolToVisibilityCollapsedConverter}, UpdateSourceTrigger=PropertyChanged}" 
-              Style="{StaticResource SConnector}" Canvas.ZIndex="-1" 
+              Style="{StaticResource SConnector}" Canvas.ZIndex="-2" 
+                  Data="{Binding ComputedBezierPathGeometry, UpdateSourceTrigger=PropertyChanged}">
+            </Path>
+            <Path Stroke="Transparent" StrokeThickness="{Binding DynamicStrokeThickness}"
+                  Visibility="{Binding IsCollapsed, Converter={StaticResource InverseBoolToVisibilityCollapsedConverter}, UpdateSourceTrigger=PropertyChanged}"
+                  Style="{StaticResource SConnector}" Canvas.ZIndex="-1"
                   Data="{Binding ComputedBezierPathGeometry, UpdateSourceTrigger=PropertyChanged}">
                 <i:Interaction.Behaviors>
                     <mouse:MouseBehaviour MouseX="{Binding PanelX, Mode=OneWayToSource, UpdateSourceTrigger=PropertyChanged}" MouseY="{Binding PanelY, Mode=OneWayToSource,  UpdateSourceTrigger=PropertyChanged}" />

src/DynamoCoreWpf/ViewModels/Core/ConnectorViewModel.cs

@@ -8,6 +8,7 @@
 using System.Windows.Media;
 using System.Windows.Shapes;
 using System.Windows.Threading;
+using Dynamo.Configuration;
 using Dynamo.Graph;
 using Dynamo.Graph.Connectors;
 using Dynamo.Graph.Nodes;
@@ -48,6 +49,7 @@ public partial class ConnectorViewModel : ViewModelBase
         private double dotLeft;
         private double endDotSize = 6;
         private double zIndex = 3;
+        private double dynamicStrokeThickness;
 
         private Point curvePoint1;
         private Point curvePoint2;
@@ -568,6 +570,16 @@ public PathGeometry ComputedBezierPathGeometry
             }
         }
 
+        public double DynamicStrokeThickness
+        {
+            get => dynamicStrokeThickness;
+            set
+            {
+                dynamicStrokeThickness = value;
+                RaisePropertyChanged(nameof(DynamicStrokeThickness));
+            }
+        }
+
         #endregion
 
         /// <summary>
@@ -959,13 +971,15 @@ public ConnectorViewModel(WorkspaceViewModel workspace, PortModel port)
             this.workspaceViewModel = workspace;
             ConnectorPinViewCollection = new ObservableCollection<ConnectorPinViewModel>();
             ConnectorPinViewCollection.CollectionChanged += HandleCollectionChanged;
+            workspaceViewModel.PropertyChanged += WorkspaceViewModel_PropertyChanged;
 
             IsHidden = !workspaceViewModel.DynamoViewModel.IsShowingConnectors;
             IsConnecting = true;
             MouseHoverOn = false;
             activeStartPort = port;
             ZIndex = SetZIndex();
 
+            UpdateDynamicStrokeThickness();
             Redraw(port.Center);
 
             InitializeCommands();
@@ -1001,6 +1015,7 @@ public ConnectorViewModel(WorkspaceViewModel workspace, ConnectorModel connector
 
             ConnectorPinViewCollection = new ObservableCollection<ConnectorPinViewModel>();
             ConnectorPinViewCollection.CollectionChanged += HandleCollectionChanged;
+            workspaceViewModel.PropertyChanged += WorkspaceViewModel_PropertyChanged;
 
 
             if (connectorModel.ConnectorPinModels != null)
@@ -1028,6 +1043,7 @@ public ConnectorViewModel(WorkspaceViewModel workspace, ConnectorModel connector
                 NodeEnd.PropertyChanged += nodeEndViewModel_PropertyChanged;
             }
 
+            UpdateDynamicStrokeThickness();
             Redraw();
             InitializeCommands();
 
@@ -1164,6 +1180,24 @@ private void HandleCollectionChanged(object sender, NotifyCollectionChangedEvent
             Redraw();
         }
 
+        private void WorkspaceViewModel_PropertyChanged(object sender, PropertyChangedEventArgs e)
+        {
+            if (e.PropertyName == nameof(workspaceViewModel.Zoom))
+            {
+                UpdateDynamicStrokeThickness();
+            }
+        }
+
+        private void UpdateDynamicStrokeThickness()
+        {
+            DynamicStrokeThickness = Math.Max(
+                Configurations.ConnectorBaseThickness,
+                Configurations.ConnectorBaseThickness *
+                (1 / workspaceViewModel.Zoom) *
+                Configurations.ConnectorZoomScalingFactor
+            );
+        }
+
         /// <summary>
         /// Dispose function
         /// </summary>
@@ -1190,6 +1224,8 @@ public override void Dispose()
             }
             ConnectorPinViewCollection.CollectionChanged -= HandleCollectionChanged;
 
+            workspaceViewModel.PropertyChanged -= WorkspaceViewModel_PropertyChanged;
+
             foreach (var pin in ConnectorPinViewCollection.ToList())
             {
                 pin.RequestRedraw -= HandlerRedrawRequest;