Fix formatting. Remove overly verbose selection

Author: corbob

.gitattributes

@@ -1,6 +1,9 @@
 # Set the default behavior, in case people don't have core.autocrlf set.
 * text=auto
 
+# Set svg to binary type, as SVG is unlikely to be editted by hand. Can be treated as checked in blob
+*.svg binary
+
 # .gitattributes in project root
 # npm now seems to be insisting on LF - see https://github.com/npm/npm/issues/17161
 package.json      text eol=lf

package.json

@@ -105,11 +105,11 @@
         "category": "PowerShell"
       },
       {
-				"command": "PowerShell.RefreshCommandsExplorer",
-				"title": "Refresh",
-				"icon": {
-					"light": "resources/light/refresh.svg",
-					"dark": "resources/dark/refresh.svg"
+        "command": "PowerShell.RefreshCommandsExplorer",
+        "title": "Refresh",
+        "icon": {
+          "light": "resources/light/refresh.svg",
+          "dark": "resources/dark/refresh.svg"
         },
         "category": "PowerShell"
       },
@@ -651,4 +651,4 @@
     ]
   },
   "private": true
-}
+}

src/features/GetCommands.ts

@@ -19,18 +19,16 @@ export class GetCommandsFeature implements IFeature {
                 return;
             }
             this.languageClient.sendRequest(GetAllCommandsRequestType, "").then((result) => {
-                this.commandsExplorerProvider.PowerShellCommands = result.map(toCommand);
+                this.commandsExplorerProvider.powerShellCommands = result.map(toCommand);
                 this.commandsExplorerProvider.refresh();
             });
         });
         this.commandsExplorerProvider = new CommandsExplorerProvider();
         vscode.window.registerTreeDataProvider("PowerShellCommands", this.commandsExplorerProvider);
         vscode.commands.registerCommand("PowerShell.InsertCommand", (item) => {
             const editor = vscode.window.activeTextEditor;
-            const document = editor.document;
-            const selection = editor.selection;
-            const sls = selection.start;
-            const sle = selection.end;
+            const sls = editor.selection.start;
+            const sle = editor.selection.end;
             const range = new vscode.Range(sls.line, sls.character, sle.line, sle.character);
             editor.edit((editBuilder) => {
                 editBuilder.replace(range, item.Name);
@@ -50,7 +48,7 @@ export class GetCommandsFeature implements IFeature {
 
 class CommandsExplorerProvider implements vscode.TreeDataProvider<Command> {
     public readonly onDidChangeTreeData: vscode.Event<Command | undefined>;
-    public PowerShellCommands: Command[];
+    public powerShellCommands: Command[];
     private didChangeTreeData: vscode.EventEmitter<Command | undefined> = new vscode.EventEmitter<Command>();
 
     constructor() {
@@ -66,10 +64,11 @@ class CommandsExplorerProvider implements vscode.TreeDataProvider<Command> {
     }
 
     public getChildren(element?: Command): Thenable<Command[]> {
-        return Promise.resolve(this.PowerShellCommands ? this.PowerShellCommands : []);
+        return Promise.resolve(this.powerShellCommands || []);
     }
 
 }
+
 function toCommand(command: any): Command {
     return new Command(
         command.name,

src/features/ShowHelp.ts

@@ -24,11 +24,12 @@ export class ShowHelpFeature implements IFeature {
             }
             if (item === undefined) {
 
-	            const editor = vscode.window.activeTextEditor;
-	            const selection = editor.selection;
-	            const doc = editor.document;
-	            const cwr = doc.getWordRangeAtPosition(selection.active);
-	            const text = doc.getText(cwr);
+                const editor = vscode.window.activeTextEditor;
+
+                const selection = editor.selection;
+                const doc = editor.document;
+                const cwr = doc.getWordRangeAtPosition(selection.active);
+                const text = doc.getText(cwr);
 
 	            this.languageClient.sendRequest(ShowHelpRequestType, text);
             } else {