explorer service: reorder methods

src/vs/workbench/parts/files/electron-browser/explorerService.ts

@@ -84,6 +84,8 @@ export class ExplorerService implements IExplorerService {
 		return model;
 	}
 
+	// IExplorerService methods
+
 	findClosest(resource: URI): ExplorerItem {
 		return this.model.findClosest(resource);
 	}
@@ -121,14 +123,6 @@ export class ExplorerService implements IExplorerService {
 		}, e => { this.notificationService.error(e); });
 	}
 
-	private onConfigurationUpdated(configuration: IFilesConfiguration, event?: IConfigurationChangeEvent): void {
-		const configSortOrder = configuration && configuration.explorer && configuration.explorer.sortOrder || 'default';
-		if (this.sortOrder !== configSortOrder) {
-			this.sortOrder = configSortOrder;
-			this.roots.forEach(r => this._onDidChangeItem.fire(r));
-		}
-	}
-
 	// File events
 
 	private onFileOperation(e: FileOperationEvent): void {
@@ -297,6 +291,14 @@ export class ExplorerService implements IExplorerService {
 		}));
 	}
 
+	private onConfigurationUpdated(configuration: IFilesConfiguration, event?: IConfigurationChangeEvent): void {
+		const configSortOrder = configuration && configuration.explorer && configuration.explorer.sortOrder || 'default';
+		if (this.sortOrder !== configSortOrder) {
+			this.sortOrder = configSortOrder;
+			this.roots.forEach(r => this._onDidChangeItem.fire(r));
+		}
+	}
+
 	dispose(): void {
 		dispose(this.disposables);
 	}

src/vs/workbench/parts/files/electron-browser/views/explorerView.ts

@@ -198,18 +198,6 @@ export class ExplorerView extends ViewletPanel {
 		}
 	}
 
-	private getActiveFile(): URI {
-		const input = this.editorService.activeEditor;
-
-		// ignore diff editor inputs (helps to get out of diffing when returning to explorer)
-		if (input instanceof DiffEditorInput) {
-			return undefined;
-		}
-
-		// check for files
-		return toResource(input, { supportSideBySide: true });
-	}
-
 	private createTree(container: HTMLElement): void {
 		this.filter = this.instantiationService.createInstance(FilesFilter);
 		this.disposables.push(this.filter);
@@ -276,13 +264,6 @@ export class ExplorerView extends ViewletPanel {
 		this.disposables.push(this.tree.onContextMenu(e => this.onContextMenu(e)));
 	}
 
-	getOptimalWidth(): number {
-		const parentNode = this.tree.getHTMLElement();
-		const childNodes = ([] as HTMLElement[]).slice.call(parentNode.querySelectorAll('.explorer-item .label-name')); // select all file labels
-
-		return DOM.getLargestChildWidth(parentNode, childNodes);
-	}
-
 	// React on events
 
 	private onConfigurationUpdated(configuration: IFilesConfiguration, event?: IConfigurationChangeEvent): void {
@@ -357,6 +338,13 @@ export class ExplorerView extends ViewletPanel {
 		return this.tree.refresh(toRefresh, true);
 	}
 
+	getOptimalWidth(): number {
+		const parentNode = this.tree.getHTMLElement();
+		const childNodes = ([] as HTMLElement[]).slice.call(parentNode.querySelectorAll('.explorer-item .label-name')); // select all file labels
+
+		return DOM.getLargestChildWidth(parentNode, childNodes);
+	}
+
 	private setTreeInput(newRoots?: IWorkspaceFolder[]): Promise<void> {
 		if (!this.isVisible()) {
 			this.shouldRefresh = true;
@@ -392,6 +380,18 @@ export class ExplorerView extends ViewletPanel {
 		return promise;
 	}
 
+	private getActiveFile(): URI {
+		const input = this.editorService.activeEditor;
+
+		// ignore diff editor inputs (helps to get out of diffing when returning to explorer)
+		if (input instanceof DiffEditorInput) {
+			return undefined;
+		}
+
+		// check for files
+		return toResource(input, { supportSideBySide: true });
+	}
+
 	private onSelectItem(fileStat: ExplorerItem, reveal = this.autoReveal): Promise<void> {
 		if (!fileStat || !this.isVisible()) {
 			return Promise.resolve(void 0);

src/vs/workbench/parts/files/test/electron-browser/explorerModel.test.ts

@@ -20,7 +20,6 @@ function toResource(path) {
 	} else {
 		return URI.file(join('/home/john', path));
 	}
-
 }
 
 suite('Files - View Model', () => {