Skip to content

Commit bbe0dbe

Browse files
author
Justin Chen
committed
code clean up on pr
1 parent 2d48d41 commit bbe0dbe

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

src/vs/editor/contrib/codeAction/browser/codeActionMenu.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ export class CodeActionMenu extends Disposable implements IEditorContribution {
146146
private focusedEnabledItem: number | undefined;
147147
private currSelectedItem: number = 0;
148148
private hasSeperator: boolean = false;
149-
private block: HTMLElement | null = null;
149+
private block?: HTMLElement;
150150

151151
public static readonly ID: string = 'editor.contrib.codeActionMenu';
152152

@@ -216,7 +216,6 @@ export class CodeActionMenu extends Disposable implements IEditorContribution {
216216
this.block.style.height = '100%';
217217
this.block.style.zIndex = '-1';
218218

219-
// TODO@Steven: this is never getting disposed
220219
renderDisposables.add(dom.addDisposableListener(this.block, dom.EventType.MOUSE_DOWN, e => e.stopPropagation()));
221220

222221
renderMenu.id = 'codeActionMenuWidget';
@@ -290,8 +289,6 @@ export class CodeActionMenu extends Disposable implements IEditorContribution {
290289
// List Focus
291290
this.codeActionList.value.domFocus();
292291
const focusTracker = dom.trackFocus(element);
293-
294-
element.focus();
295292
const blurListener = focusTracker.onDidBlur(() => {
296293
this.hideCodeActionWidget();
297294
// this._contextViewService.hideContextView({ source: this });

0 commit comments

Comments
 (0)