From 11afc45ee14e750fcde51c9dac63c8d1d4cef3db Mon Sep 17 00:00:00 2001
From: Eric Amodio <eamodio@gmail.com>
Date: Sat, 22 Sep 2018 16:00:14 -0400
Subject: [PATCH] Fixes blame toolbar button to work on the left diff

---
 src/commands/clearFileAnnotations.ts | 13 +++++++------
 src/commands/toggleFileBlame.ts      |  7 +++----
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/src/commands/clearFileAnnotations.ts b/src/commands/clearFileAnnotations.ts
index 3d4b266..6142415 100644
--- a/src/commands/clearFileAnnotations.ts
+++ b/src/commands/clearFileAnnotations.ts
@@ -1,6 +1,7 @@
 'use strict';
 import { TextEditor, TextEditorEdit, Uri, window } from 'vscode';
 import { UriComparer } from '../comparers';
+import { isTextEditor } from '../constants';
 import { Container } from '../container';
 import { Logger } from '../logger';
 import { Messages } from '../messages';
@@ -12,13 +13,13 @@ export class ClearFileAnnotationsCommand extends EditorCommand {
     }
 
     async execute(editor: TextEditor, edit: TextEditorEdit, uri?: Uri): Promise<any> {
-        if (editor == null) return undefined;
-
         // Handle the case where we are focused on a non-editor editor (output, debug console)
-        if (uri != null && !UriComparer.equals(uri, editor.document.uri)) {
-            const e = window.visibleTextEditors.find(e => UriComparer.equals(uri, e.document.uri));
-            if (e !== undefined) {
-                editor = e;
+        if (editor != null && !isTextEditor(editor)) {
+            if (uri != null && !UriComparer.equals(uri, editor.document.uri)) {
+                const e = window.visibleTextEditors.find(e => UriComparer.equals(uri, e.document.uri));
+                if (e !== undefined) {
+                    editor = e;
+                }
             }
         }
 
diff --git a/src/commands/toggleFileBlame.ts b/src/commands/toggleFileBlame.ts
index 1d37615..bdd51aa 100644
--- a/src/commands/toggleFileBlame.ts
+++ b/src/commands/toggleFileBlame.ts
@@ -2,6 +2,7 @@
 import { TextEditor, Uri, window } from 'vscode';
 import { UriComparer } from '../comparers';
 import { FileAnnotationType } from '../configuration';
+import { isTextEditor } from '../constants';
 import { Container } from '../container';
 import { Logger } from '../logger';
 import { ActiveEditorCommand, Commands } from './common';
@@ -17,10 +18,8 @@ export class ToggleFileBlameCommand extends ActiveEditorCommand {
     }
 
     async execute(editor: TextEditor, uri?: Uri, args: ToggleFileBlameCommandArgs = {}): Promise<any> {
-        // if (editor == null) return undefined;
-
-        if (editor != null) {
-            // Handle the case where we are focused on a non-editor editor (output, debug console)
+        // Handle the case where we are focused on a non-editor editor (output, debug console)
+        if (editor != null && !isTextEditor(editor)) {
             if (uri != null && !UriComparer.equals(uri, editor.document.uri)) {
                 const e = window.visibleTextEditors.find(e => UriComparer.equals(uri, e.document.uri));
                 if (e !== undefined) {