Status bar spinner is white in light themes and hard to see (#46083)

fixes #45191
microsoft · Mar 19, 2018 · 9b25468 · 9b25468
1 parent 3a70cdf
commit 9b25468
Show file tree

Hide file tree

Showing 5 changed files with 24 additions and 39 deletions.
diff --git a/src/vs/workbench/browser/parts/editor/editorPart.ts b/src/vs/workbench/browser/parts/editor/editorPart.ts
@@ -550,7 +550,7 @@ export class EditorPart extends Part implements IEditorPart, IEditorGroupService
 		if (this.partService.isCreated() && !errors.isPromiseCanceledError(error)) {
 			const actions: INotificationActions = { primary: [] };
 			if (errors.isErrorWithActions(error)) {
-				actions.primary = error.actions;
+				actions.primary = (error as errors.IErrorWithActions).actions;
 			}
 
 			const handle = this.notificationService.notify({

diff --git a/src/vs/workbench/services/configuration/node/configuration.ts b/src/vs/workbench/services/configuration/node/configuration.ts
@@ -254,7 +254,7 @@ export class FolderConfiguration extends Disposable {
 		for (let i = 0, len = events.length; i < len; i++) {
 			const resource = events[i].resource;
 			const isJson = paths.extname(resource.fsPath) === '.json';
-			const isDeletedSettingsFolder = (events[i].type === FileChangeType.DELETED && paths.isEqual(paths.basename(resource.fsPath), this.configFolderRelativePath));
+			const isDeletedSettingsFolder = (events[i].type === FileChangeType.DELETED && paths.basename(resource.fsPath) === this.configFolderRelativePath);
 			if (!isJson && !isDeletedSettingsFolder) {
 				continue; // only JSON files or the actual settings folder
 			}

diff --git a/src/vs/workbench/services/progress/browser/media/progress.svg b/src/vs/workbench/services/progress/browser/media/progress.svg
diff --git a/src/vs/workbench/services/progress/browser/media/progressService2.css b/src/vs/workbench/services/progress/browser/media/progressService2.css
@@ -4,11 +4,11 @@
  *--------------------------------------------------------------------------------------------*/
 
 .monaco-workbench > .part.statusbar > .statusbar-item.progress {
-	background-image: url(progress.svg);
-	background-repeat: no-repeat;
-	background-position: left;
-	padding-left: 14px;
-	display: inline;
+	padding-left: 5px;
+}
+
+.monaco-workbench > .part.statusbar > .statusbar-item.progress .spinner-container {
+	padding-right: 5px;
 }
 
 .monaco-workbench .progress-badge > .badge-content {

diff --git a/src/vs/workbench/services/progress/browser/progressService2.ts b/src/vs/workbench/services/progress/browser/progressService2.ts
@@ -30,9 +30,25 @@ class WindowProgressItem implements IStatusbarItem {
 
 	render(element: HTMLElement): IDisposable {
 		this._element = element;
-		this._label = new OcticonLabel(this._element);
 		this._element.classList.add('progress');
+
+		const container = document.createElement('span');
+		this._element.appendChild(container);
+
+		const spinnerContainer = document.createElement('span');
+		spinnerContainer.classList.add('spinner-container');
+		container.appendChild(spinnerContainer);
+
+		const spinner = new OcticonLabel(spinnerContainer);
+		spinner.text = '$(sync~spin)';
+
+		const labelContainer = document.createElement('span');
+		container.appendChild(labelContainer);
+
+		this._label = new OcticonLabel(labelContainer);
+
 		this.hide();
+
 		return null;
 	}