Make display string optional in ts/js hover

microsoft · Sep 21, 2017 · b86108e · b86108e
1 parent 7a3c668
commit b86108e
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/extensions/typescript/src/features/hoverProvider.ts b/extensions/typescript/src/features/hoverProvider.ts
@@ -36,10 +36,14 @@ export default class TypeScriptHoverProvider implements HoverProvider {
 	}
 
 	private static getContents(data: Proto.QuickInfoResponseBody) {
+		const parts = [];
+
+		if (data.displayString) {
+			parts.push({ language: 'typescript', value: data.displayString });
+		}
+
 		const tags = tagsMarkdownPreview(data.tags);
-		return [
-			{ language: 'typescript', value: data.displayString },
-			data.documentation + (tags ? '\n\n' + tags : '')
-		];
+		parts.push(data.documentation + (tags ? '\n\n' + tags : ''));
+		return parts;
 	}
 }