From 5d68e6764272fa74f1c9412f7d1af1fb8a5aab76 Mon Sep 17 00:00:00 2001
From: Martin Aeschlimann <martinae@microsoft.com>
Date: Thu, 12 Oct 2017 11:29:40 +0200
Subject: [PATCH] hide exporer arrows improvements (for #35856)

---
 .../workbench/parts/files/browser/media/explorerviewlet.css | 2 +-
 .../workbench/services/themes/common/fileIconThemeSchema.ts | 4 ++++
 .../services/themes/electron-browser/fileIconThemeData.ts   | 6 +-----
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/vs/workbench/parts/files/browser/media/explorerviewlet.css b/src/vs/workbench/parts/files/browser/media/explorerviewlet.css
index fd26c32d640..4ef2be5ab15 100644
--- a/src/vs/workbench/parts/files/browser/media/explorerviewlet.css
+++ b/src/vs/workbench/parts/files/browser/media/explorerviewlet.css
@@ -107,7 +107,7 @@
 }
 
 .explorer-folders-view.hide-arrows .monaco-tree-row .content::before {
-	background-image: none;
+	display: none;
 }
 
 .explorer-viewlet .explorer-open-editors .monaco-tree .monaco-tree-row:hover > .content .monaco-action-bar,
diff --git a/src/vs/workbench/services/themes/common/fileIconThemeSchema.ts b/src/vs/workbench/services/themes/common/fileIconThemeSchema.ts
index 263f48a9b35..f04fc317d19 100644
--- a/src/vs/workbench/services/themes/common/fileIconThemeSchema.ts
+++ b/src/vs/workbench/services/themes/common/fileIconThemeSchema.ts
@@ -209,6 +209,10 @@ const schema: IJSONSchema = {
 		highContrast: {
 			$ref: '#/definitions/associations',
 			description: nls.localize('schema.highContrast', 'Optional associations for file icons in high contrast color themes.')
+		},
+		hidesExplorerArrows: {
+			type: 'boolean',
+			description: nls.localize('schema.hidesExplorerArrows', 'Configures whether the file explorer\'s arrows should be hidden when this theme is active.')
 		}
 	}
 };
diff --git a/src/vs/workbench/services/themes/electron-browser/fileIconThemeData.ts b/src/vs/workbench/services/themes/electron-browser/fileIconThemeData.ts
index 14bb8048c97..286627526d4 100644
--- a/src/vs/workbench/services/themes/electron-browser/fileIconThemeData.ts
+++ b/src/vs/workbench/services/themes/electron-browser/fileIconThemeData.ts
@@ -129,7 +129,7 @@ function _loadIconThemeDocument(fileSetPath: string): TPromise<IconThemeDocument
 
 function _processIconThemeDocument(id: string, iconThemeDocumentPath: string, iconThemeDocument: IconThemeDocument): { content: string; hasFileIcons: boolean; hasFolderIcons: boolean; hidesExplorerArrows: boolean; } {
 
-	let result = { content: '', hasFileIcons: false, hasFolderIcons: false, hidesExplorerArrows: false };
+	let result = { content: '', hasFileIcons: false, hasFolderIcons: false, hidesExplorerArrows: iconThemeDocument.hidesExplorerArrows };
 
 	if (!iconThemeDocument.iconDefinitions) {
 		return result;
@@ -251,10 +251,6 @@ function _processIconThemeDocument(id: string, iconThemeDocumentPath: string, ic
 		return result;
 	}
 
-	if (iconThemeDocument.hidesExplorerArrows) {
-		result.hidesExplorerArrows = true;
-	}
-
 	let cssRules: string[] = [];
 
 	let fonts = iconThemeDocument.fonts;
-- 
GitLab