fix DND issue with groups

microsoft · Sep 20, 2016 · c49fe7f · c49fe7f
1 parent d66b0fd
commit c49fe7f
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 2 deletions.
diff --git a/src/vs/workbench/browser/parts/editor/noTabsTitleControl.ts b/src/vs/workbench/browser/parts/editor/noTabsTitleControl.ts
@@ -27,7 +27,6 @@ export class NoTabsTitleControl extends TitleControl {
 		super.create(parent);
 
 		this.titleContainer = parent;
-		DOM.addClass(this.titleContainer, 'show-file-icons');
 
 		// Pin on double click
 		this.toDispose.push(DOM.addDisposableListener(this.titleContainer, DOM.EventType.DBLCLICK, (e: MouseEvent) => this.onTitleDoubleClick(e)));

diff --git a/src/vs/workbench/browser/parts/editor/sideBySideEditorControl.ts b/src/vs/workbench/browser/parts/editor/sideBySideEditorControl.ts
@@ -783,6 +783,7 @@ export class SideBySideEditorControl implements ISideBySideEditorControl, IVerti
 			if (useTabs) {
 				titleContainer.addClass('tabs');
 			}
+			titleContainer.addClass('show-file-icons');
 			this.hookTitleDragListener(titleContainer);
 
 			// Title Control

diff --git a/src/vs/workbench/browser/parts/editor/tabsTitleControl.ts b/src/vs/workbench/browser/parts/editor/tabsTitleControl.ts
@@ -88,7 +88,6 @@ export class TabsTitleControl extends TitleControl {
 		this.tabsContainer = document.createElement('div');
 		this.tabsContainer.setAttribute('role', 'tablist');
 		DOM.addClass(this.tabsContainer, 'tabs-container');
-		DOM.addClass(this.tabsContainer, 'show-file-icons');
 
 		// Forward scrolling inside the container to our custom scrollbar
 		this.toDispose.push(DOM.addDisposableListener(this.tabsContainer, DOM.EventType.SCROLL, e => {