Don't display hover tip for a target when hovering on one of its (chi…

…ld) task elements. #5
tintoy · Apr 20, 2018 · ce19add · ce19add
1 parent 2c6b4bc
commit ce19add
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/LanguageServer.Engine/Handlers/HoverHandler.cs b/src/LanguageServer.Engine/Handlers/HoverHandler.cs
@@ -189,7 +189,10 @@ async Task<Hover> OnHover(TextDocumentPositionParams parameters, CancellationTok
                         }
                         case MSBuildTarget target:
                         {
-                            hoverContent = contentProvider.Target(target);
+                            // Currently (and this is a bug), an MSBuildTarget is returned by MSBuildLocator when the location being inspected
+                            // is actually on one of its child (task) elements.
+                            if (element.Path == WellKnownElementPaths.Target)
+                                hoverContent = contentProvider.Target(target);
 
                             break;
                         }