Show tooltip for comparison operators (#1294)

TylerLeonhardt · web-flow · commit f81c92ad76ef · 2020-05-19T11:32:06.000-07:00
diff --git a/src/PowerShellEditorServices/Services/TextDocument/Handlers/CompletionHandler.cs b/src/PowerShellEditorServices/Services/TextDocument/Handlers/CompletionHandler.cs
@@ -266,6 +266,12 @@ private static CompletionItem CreateCompletionItem(
                     {
                         detailString = matches[0].Groups[1].Value;
                     }
+                    // The comparison operators (-eq, -not, -gt, etc) are unfortunately fall into ParameterName
+                    // but they don't have a type associated to them. This allows those tooltips to show up.
+                    else if (!string.IsNullOrEmpty(completionDetails.ToolTipText))
+                    {
+                        detailString = completionDetails.ToolTipText;
+                    }
                     break;
                 case CompletionType.Command:
                     // For Commands, let's extract the resolved command or the path for an exe

Original file line number	Diff line number	Diff line change
`@@ -266,6 +266,12 @@ private static CompletionItem CreateCompletionItem(`
`266`	`266`	`{`
`267`	`267`	`detailString = matches[0].Groups[1].Value;`
`268`	`268`	`}`
	`269`	`+ // The comparison operators (-eq, -not, -gt, etc) are unfortunately fall into ParameterName`
	`270`	`+ // but they don't have a type associated to them. This allows those tooltips to show up.`
	`271`	`+ else if (!string.IsNullOrEmpty(completionDetails.ToolTipText))`
	`272`	`+ {`
	`273`	`+ detailString = completionDetails.ToolTipText;`
	`274`	`+ }`
`269`	`275`	`break;`
`270`	`276`	`case CompletionType.Command:`
`271`	`277`	`// For Commands, let's extract the resolved command or the path for an exe`