Use Double Left Click to trigger NodeAutoComplete (#11710)

QilongTang · web-flow · commit 24c66bce907b · 2021-05-25T08:48:53.000-07:00
diff --git a/src/DynamoCoreWpf/UI/Themes/Modern/Ports.xaml b/src/DynamoCoreWpf/UI/Themes/Modern/Ports.xaml
@@ -1,15 +1,15 @@
 ﻿<ResourceDictionary   xmlns="https://schemas.microsoft.com/winfx/2006/xaml/presentation"
+                      xmlns:fa="https://schemas.fontawesome.io/icons/"
                       xmlns:x="https://schemas.microsoft.com/winfx/2006/xaml"
-                      xmlns:controls="clr-namespace:Dynamo.Views;assembly=DynamoCore"
-                      xmlns:viewModels="clr-namespace:Dynamo.ViewModels;assembly=DynamoCoreWpf"
                       xmlns:dynui="clr-namespace:Dynamo.UI.Controls;assembly=DynamoCoreWpf"
-                      xmlns:interactivity="clr-namespace:System.Windows.Interactivity;assembly=System.Windows.Interactivity"
                       xmlns:views="clr-namespace:Dynamo.UI.Views;assembly=DynamoCoreWpf"
                       xmlns:ui="clr-namespace:Dynamo.UI;assembly=DynamoCoreWpf"
-                      xmlns:fa="https://schemas.fontawesome.io/icons/"
-                      xmlns:diag="clr-namespace:System.Diagnostics;assembly=WindowsBase"
+                      xmlns:viewModels="clr-namespace:Dynamo.ViewModels;assembly=DynamoCoreWpf"
+                      xmlns:controls="clr-namespace:Dynamo.Views;assembly=DynamoCore"
+                      xmlns:p="clr-namespace:Dynamo.Wpf.Properties;assembly=DynamoCoreWpf"
                       xmlns:command="clr-namespace:GalaSoft.MvvmLight.Command;assembly=GalaSoft.MvvmLight.Platform"
-                      xmlns:p="clr-namespace:Dynamo.Wpf.Properties;assembly=DynamoCoreWpf">
+                      xmlns:diag="clr-namespace:System.Diagnostics;assembly=WindowsBase"
+                      xmlns:interactivity="clr-namespace:System.Windows.Interactivity;assembly=System.Windows.Interactivity">
 
     <!-- Templates
         
@@ -38,17 +38,17 @@
                 <ColumnDefinition Width="*"/>
                 <ColumnDefinition Width="Auto"/>
             </Grid.ColumnDefinitions>
-            <Grid
-                Name="mainGrid"
-                Grid.Column="0"
-                Background="Transparent"
-                IsHitTestVisible="True"
-                Margin="{Binding Path=MarginThickness}"
-                Height="{Binding Path=Height}"
-                ToolTipService.ShowDuration="60000"
-                IsEnabled="{Binding Path=IsEnabled}">
+            <Grid Name="mainGrid"
+                  Grid.Column="0"
+                  Background="Transparent"
+                  IsHitTestVisible="True"
+                  Margin="{Binding Path=MarginThickness}"
+                  Height="{Binding Path=Height}"
+                  ToolTipService.ShowDuration="60000"
+                  IsEnabled="{Binding Path=IsEnabled}">
 
                 <interactivity:Interaction.Triggers>
+                    <!--Bind Connect command to left click-->
                     <views:HandlingEventTrigger EventName="MouseLeftButtonDown">
                         <interactivity:InvokeCommandAction Command="{Binding Path=ConnectCommand}" />
                     </views:HandlingEventTrigger>
@@ -72,6 +72,11 @@
                     </dynui:DynamoToolTip>
                 </Grid.ToolTip>
 
+                <!--Bind NodeAutoComplete to double left click-->
+                <Grid.InputBindings>
+                    <MouseBinding MouseAction="LeftDoubleClick" Command="{Binding Path=NodeAutoCompleteCommand}"></MouseBinding>
+                </Grid.InputBindings>
+
                 <!--
                 We could have changed the Grid opacity directly here but didn't
                 The reason is that if the opacity of the Grid is modified, the 
@@ -228,7 +233,7 @@
                                     Foreground="#555555"/>
                 </StackPanel>
                 <dynui:UseLevelPopup
-                    x:Name="MyPopup"
+                    x:Name="UseLevelPopup"
                     Placement="Right"
                     AllowsTransparency="True"
                     IsOpen="{Binding Path=ShowUseLevelMenu}"
diff --git a/src/DynamoCoreWpf/ViewModels/Core/PortViewModel.cs b/src/DynamoCoreWpf/ViewModels/Core/PortViewModel.cs
@@ -408,9 +408,9 @@ private bool CanConnect(object parameter)
         private void AutoComplete(object parameter)
         {
             var wsViewModel = _node.WorkspaceViewModel;
-            var svm = wsViewModel.NodeAutoCompleteSearchViewModel;
-            svm.PortViewModel = this;
-
+            wsViewModel.NodeAutoCompleteSearchViewModel.PortViewModel = this;
+            // Bail out from connect state
+            wsViewModel.CancelActiveState();
             wsViewModel.OnRequestNodeAutoCompleteSearch(ShowHideFlags.Show);
         }
 
@@ -427,10 +427,7 @@ private bool CanAutoComplete(object parameter)
         /// <param name="parameter">The parameter.</param>
         private void OnRectangleMouseEnter(object parameter)
         {
-            if (MouseEnter != null)
-            {
-                MouseEnter(parameter, null);
-            }
+            MouseEnter?.Invoke(parameter, null);
         }
 
         /// <summary>
@@ -439,8 +436,7 @@ private void OnRectangleMouseEnter(object parameter)
         /// <param name="parameter">The parameter.</param>
         private void OnRectangleMouseLeave(object parameter)
         {
-            if (MouseLeave != null)
-                MouseLeave(parameter, null);
+            MouseLeave?.Invoke(parameter, null);
         }
 
         /// <summary>
@@ -449,8 +445,7 @@ private void OnRectangleMouseLeave(object parameter)
         /// <param name="parameter">The parameter.</param>
         private void OnRectangleMouseLeftButtonDown(object parameter)
         {
-            if (MouseLeftButtonDown != null)
-                MouseLeftButtonDown(parameter, null);
+            MouseLeftButtonDown?.Invoke(parameter, null);
         }
 
         /// <summary>
diff --git a/src/DynamoCoreWpf/ViewModels/Core/StateMachine.cs b/src/DynamoCoreWpf/ViewModels/Core/StateMachine.cs
@@ -802,16 +802,6 @@ internal bool HandlePortClicked(PortViewModel portViewModel)
 
                 if (this.currentState != State.Connection) // Not in a connection attempt...
                 {
-                    if (Keyboard.Modifiers == ModifierKeys.Alt &&
-                        portViewModel.NodeAutoCompleteCommand.CanExecute(portViewModel))
-                    {
-                        portViewModel.NodeAutoCompleteCommand.Execute(portViewModel);
-                        this.currentState = State.Connection;
-                        owningWorkspace.CurrentCursor = CursorLibrary.GetCursor(CursorSet.ArcSelect);
-                        owningWorkspace.IsCursorForced = false;
-                        return true;
-                    }
-
                     Guid nodeId = portModel.Owner.GUID;
                     int portIndex = portModel.Index;
 

Original file line number	Diff line number	Diff line change
`@@ -408,9 +408,9 @@ private bool CanConnect(object parameter)`
`408`	`408`	`private void AutoComplete(object parameter)`
`409`	`409`	`{`
`410`	`410`	`var wsViewModel = _node.WorkspaceViewModel;`
`411`		`- var svm = wsViewModel.NodeAutoCompleteSearchViewModel;`
`412`		`- svm.PortViewModel = this;`
`413`		`-`
	`411`	`+ wsViewModel.NodeAutoCompleteSearchViewModel.PortViewModel = this;`
	`412`	`+ // Bail out from connect state`
	`413`	`+ wsViewModel.CancelActiveState();`
`414`	`414`	`wsViewModel.OnRequestNodeAutoCompleteSearch(ShowHideFlags.Show);`
`415`	`415`	`}`
`416`	`416`
`@@ -427,10 +427,7 @@ private bool CanAutoComplete(object parameter)`
`427`	`427`	`/// <param name="parameter">The parameter.</param>`
`428`	`428`	`private void OnRectangleMouseEnter(object parameter)`
`429`	`429`	`{`
`430`		`- if (MouseEnter != null)`
`431`		`- {`
`432`		`- MouseEnter(parameter, null);`
`433`		`- }`
	`430`	`+ MouseEnter?.Invoke(parameter, null);`
`434`	`431`	`}`
`435`	`432`
`436`	`433`	`/// <summary>`
`@@ -439,8 +436,7 @@ private void OnRectangleMouseEnter(object parameter)`
`439`	`436`	`/// <param name="parameter">The parameter.</param>`
`440`	`437`	`private void OnRectangleMouseLeave(object parameter)`
`441`	`438`	`{`
`442`		`- if (MouseLeave != null)`
`443`		`- MouseLeave(parameter, null);`
	`439`	`+ MouseLeave?.Invoke(parameter, null);`
`444`	`440`	`}`
`445`	`441`
`446`	`442`	`/// <summary>`
`@@ -449,8 +445,7 @@ private void OnRectangleMouseLeave(object parameter)`
`449`	`445`	`/// <param name="parameter">The parameter.</param>`
`450`	`446`	`private void OnRectangleMouseLeftButtonDown(object parameter)`
`451`	`447`	`{`
`452`		`- if (MouseLeftButtonDown != null)`
`453`		`- MouseLeftButtonDown(parameter, null);`
	`448`	`+ MouseLeftButtonDown?.Invoke(parameter, null);`
`454`	`449`	`}`
`455`	`450`
`456`	`451`	`/// <summary>`