[#615] fix missing space in outline view text

fixes #615
eclipse · May 15, 2023 · 0750c3f · 0750c3f
1 parent c6d9e20
commit 0750c3f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/org.eclipse.lsp4e.test/src/org/eclipse/lsp4e/test/outline/SymbolsLabelProviderTest.java b/org.eclipse.lsp4e.test/src/org/eclipse/lsp4e/test/outline/SymbolsLabelProviderTest.java
@@ -72,7 +72,7 @@ public void testDocumentSymbolDetail () {
 		DocumentSymbol info = new DocumentSymbol("Foo", SymbolKind.Class,
 				new Range(new Position(1, 0), new Position(1, 2)),
 				new Range(new Position(1, 0), new Position(1, 2)),
-				" : additional detail");
+				": additional detail");
 		assertEquals("Foo : additional detail", labelProvider.getStyledText(info).getString());		
 	}
 
@@ -82,7 +82,7 @@ public void testDocumentSymbolDetailWithKind () {
 		DocumentSymbol info = new DocumentSymbol("Foo", SymbolKind.Class,
 				new Range(new Position(1, 0), new Position(1, 2)),
 				new Range(new Position(1, 0), new Position(1, 2)),
-				" : additional detail");
+				": additional detail");
 		assertEquals("Foo : additional detail :Class", labelProvider.getStyledText(info).getString());		
 	}
 
@@ -92,7 +92,7 @@ public void testDocumentSymbolWithFileDetail () {
 		DocumentSymbol info = new DocumentSymbol("Foo", SymbolKind.Class,
 				new Range(new Position(1, 0), new Position(1, 2)),
 				new Range(new Position(1, 0), new Position(1, 2)),
-				" : additional detail");
+				": additional detail");
 		SymbolsModel.DocumentSymbolWithFile infoWithFile = new SymbolsModel.DocumentSymbolWithFile(info, null);
 		assertEquals("Foo : additional detail", labelProvider.getStyledText(infoWithFile).getString());		
 	}
@@ -103,7 +103,7 @@ public void testDocumentSymbolDetailWithFileWithKind () {
 		DocumentSymbol info = new DocumentSymbol("Foo", SymbolKind.Class,
 				new Range(new Position(1, 0), new Position(1, 2)),
 				new Range(new Position(1, 0), new Position(1, 2)),
-				" : additional detail");
+				": additional detail");
 		SymbolsModel.DocumentSymbolWithFile infoWithFile = new SymbolsModel.DocumentSymbolWithFile(info, null);
 		assertEquals("Foo : additional detail :Class", labelProvider.getStyledText(infoWithFile).getString());		
 	}

diff --git a/org.eclipse.lsp4e/src/org/eclipse/lsp4e/outline/SymbolsLabelProvider.java b/org.eclipse.lsp4e/src/org/eclipse/lsp4e/outline/SymbolsLabelProvider.java
@@ -301,6 +301,7 @@ public StyledString getStyledText(Object element) {
 		}
 
 		if (detail != null) {
+			res.append(' ');
 			res.append(detail, StyledString.DECORATIONS_STYLER);
 		}