fix: Add styling to the Snyk Language Server html

acke · acke · commit 9d6f8e8f60bc · 2025-01-30T11:26:12.000+01:00
diff --git a/plugin/src/main/java/io/snyk/eclipse/plugin/html/StaticPageHtmlProvider.java b/plugin/src/main/java/io/snyk/eclipse/plugin/html/StaticPageHtmlProvider.java
@@ -156,8 +156,8 @@ public String getSummaryInitHtml() {
 				<body>
 				    <div class="container">
 				        <h3 class="summary-header">SUMMARY</h3>
-		                    <button type="button" onclick="window.showAllIssuesTab()">All issues</button>
-		                    <button type="button" onclick="window.showDeltaIssuesTab()">Delta issues</button>
+				                  <button type="button" onclick="window.showAllIssuesTab()">All issues</button>
+				                  <button type="button" onclick="window.showDeltaIssuesTab()">Delta issues</button>
 				        <div class="summary-row">
 				            <span class="icon">⚠️</span>
 				            <span class="text">312 issues found in your project</span>
@@ -170,9 +170,17 @@ public String getSummaryInitHtml() {
 				    </div>
 				</body>
 				</html>
+				""".formatted(head);
+		return replaceCssVariables(html);
+	}
 				"""
 				.formatted(head);
 		return replaceCssVariables(html);
 	}
 
+	public String getFormattedSummaryHtml(String summary) {
+		var html = summary
+				.formatted(head);
+		return replaceCssVariables(html);
+	}
 }
diff --git a/plugin/src/main/java/io/snyk/eclipse/plugin/views/snyktoolview/SummaryBrowserHandler.java b/plugin/src/main/java/io/snyk/eclipse/plugin/views/snyktoolview/SummaryBrowserHandler.java
@@ -64,7 +64,7 @@ public void setDefaultBrowserText() {
 	}
 
 	public void setBrowserText(String summary) {
-		browser.setText(summary);
+		browser.setText(StaticPageHtmlProvider.getInstance().getFormattedSummaryHtml(summary));
 	}
 
 }

Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,7 @@ public void setDefaultBrowserText() {`
`64`	`64`	`}`
`65`	`65`
`66`	`66`	`public void setBrowserText(String summary) {`
`67`		`- browser.setText(summary);`
	`67`	`+ browser.setText(StaticPageHtmlProvider.getInstance().getFormattedSummaryHtml(summary));`
`68`	`68`	`}`
`69`	`69`
`70`	`70`	`}`