Merge pull request #195943 from microsoft/merogge/hide-view-2

hide accessible view on input/focusing of the terminal
This commit is contained in:
Megan Rogge 2023-10-18 14:10:50 -07:00 committed by GitHub
commit c7521f9ab4
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -9,6 +9,7 @@ import { DisposableStore } from 'vs/base/common/lifecycle';
import { IModelService } from 'vs/editor/common/services/model';
import { IConfigurationService } from 'vs/platform/configuration/common/configuration';
import { IContextKeyService } from 'vs/platform/contextkey/common/contextkey';
import { IContextViewService } from 'vs/platform/contextview/browser/contextView';
import { IKeybindingService } from 'vs/platform/keybinding/common/keybinding';
import { ResultKind } from 'vs/platform/keybinding/common/keybindingResolver';
import { TerminalCapability, ITerminalCommand } from 'vs/platform/terminal/common/capabilities/capabilities';
@ -34,7 +35,8 @@ export class TerminalAccessibleBufferProvider extends DisposableStore implements
@IConfigurationService configurationService: IConfigurationService,
@IContextKeyService _contextKeyService: IContextKeyService,
@ITerminalService _terminalService: ITerminalService,
@IKeybindingService private readonly _keybindingService: IKeybindingService
@IKeybindingService private readonly _keybindingService: IKeybindingService,
@IContextViewService private readonly _contextViewService: IContextViewService
) {
super();
this.options.customHelp = customHelp;
@ -58,6 +60,7 @@ export class TerminalAccessibleBufferProvider extends DisposableStore implements
if (!shouldFocusTerminal(e.browserEvent, this._keybindingService)) {
return;
}
this._contextViewService.hideContextView();
this._instance.focus();
}