fix #52911

microsoft · Jun 29, 2018 · 1c8a1f5 · 1c8a1f5
1 parent 4991227
commit 1c8a1f5
Showing 1 changed file with 2 additions and 7 deletions.
diff --git a/src/vs/base/browser/ui/actionbar/actionbar.ts b/src/vs/base/browser/ui/actionbar/actionbar.ts
@@ -268,11 +268,6 @@ export class ActionItem extends BaseActionItem {
 		this._updateChecked();
 	}
 
-	public focus(): void {
-		super.focus();
-		this.$e.domFocus();
-	}
-
 	public _updateLabel(): void {
 		if (this.options.label) {
 			this.$e.text(this.getAction().label);
@@ -318,11 +313,11 @@ export class ActionItem extends BaseActionItem {
 		if (this.getAction().enabled) {
 			this.builder.removeClass('disabled');
 			this.$e.removeClass('disabled');
-			this.$e.attr({ tabindex: 0 });
+			this.builder.attr({ tabindex: 0 });
 		} else {
 			this.builder.addClass('disabled');
 			this.$e.addClass('disabled');
-			DOM.removeTabIndexAndUpdateFocus(this.$e.getHTMLElement());
+			DOM.removeTabIndexAndUpdateFocus(this.builder.getHTMLElement());
 		}
 	}