Merge pull request #112431 from microsoft/misolori/notebook-color-icons

Miguel Solorio · web-flow · commit 8b5ece7c203f · 2020-12-15T13:11:55.000-08:00
Fix notebook status bar icon colors
diff --git a/src/vs/workbench/contrib/notebook/browser/notebookEditorWidget.ts b/src/vs/workbench/contrib/notebook/browser/notebookEditorWidget.ts
@@ -2191,12 +2191,12 @@ registerThemingParticipant((theme, collector) => {
 
 	const cellStatusSuccessIcon = theme.getColor(cellStatusIconSuccess);
 	if (cellStatusSuccessIcon) {
-		collector.addRule(`.monaco-workbench .notebookOverlay .cell-statusbar-container .cell-run-status .codicon-check { color: ${cellStatusSuccessIcon} }`);
+		collector.addRule(`.monaco-workbench .notebookOverlay .cell-statusbar-container .cell-run-status .codicon-notebook-state-success { color: ${cellStatusSuccessIcon} }`);
 	}
 
 	const cellStatusErrorIcon = theme.getColor(cellStatusIconError);
 	if (cellStatusErrorIcon) {
-		collector.addRule(`.monaco-workbench .notebookOverlay .cell-statusbar-container .cell-run-status .codicon-error { color: ${cellStatusErrorIcon} }`);
+		collector.addRule(`.monaco-workbench .notebookOverlay .cell-statusbar-container .cell-run-status .codicon-notebook-state-error { color: ${cellStatusErrorIcon} }`);
 	}
 
 	const cellStatusRunningIcon = theme.getColor(cellStatusIconRunning);

Original file line number	Diff line number	Diff line change
`@@ -2191,12 +2191,12 @@ registerThemingParticipant((theme, collector) => {`
`2191`	`2191`
`2192`	`2192`	`const cellStatusSuccessIcon = theme.getColor(cellStatusIconSuccess);`
`2193`	`2193`	`if (cellStatusSuccessIcon) {`
`2194`		- collector.addRule(`.monaco-workbench .notebookOverlay .cell-statusbar-container .cell-run-status .codicon-check { color: ${cellStatusSuccessIcon} }`);
	`2194`	+ collector.addRule(`.monaco-workbench .notebookOverlay .cell-statusbar-container .cell-run-status .codicon-notebook-state-success { color: ${cellStatusSuccessIcon} }`);
`2195`	`2195`	`}`
`2196`	`2196`
`2197`	`2197`	`const cellStatusErrorIcon = theme.getColor(cellStatusIconError);`
`2198`	`2198`	`if (cellStatusErrorIcon) {`
`2199`		- collector.addRule(`.monaco-workbench .notebookOverlay .cell-statusbar-container .cell-run-status .codicon-error { color: ${cellStatusErrorIcon} }`);
	`2199`	+ collector.addRule(`.monaco-workbench .notebookOverlay .cell-statusbar-container .cell-run-status .codicon-notebook-state-error { color: ${cellStatusErrorIcon} }`);
`2200`	`2200`	`}`
`2201`	`2201`
`2202`	`2202`	`const cellStatusRunningIcon = theme.getColor(cellStatusIconRunning);`