Skip to content

Commit 1fea7d9

Browse files
authored
Merge pull request microsoft#232968 from microsoft/tyriar/232967
Escape to hide terminal chat when terminal is focused
2 parents 231d373 + 8c1dc48 commit 1fea7d9

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

src/vs/workbench/contrib/terminalContrib/chat/browser/terminalChatActions.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,10 @@ registerActiveXtermAction({
6060
keybinding: {
6161
primary: KeyCode.Escape,
6262
secondary: [KeyMod.Shift | KeyCode.Escape],
63-
when: ContextKeyExpr.and(TerminalChatContextKeys.focused, TerminalChatContextKeys.visible),
63+
when: ContextKeyExpr.and(
64+
ContextKeyExpr.or(TerminalContextKeys.focus, TerminalChatContextKeys.focused),
65+
TerminalChatContextKeys.visible
66+
),
6467
weight: KeybindingWeight.WorkbenchContrib,
6568
},
6669
icon: Codicon.close,
@@ -70,9 +73,7 @@ registerActiveXtermAction({
7073
order: 2
7174
},
7275
f1: true,
73-
precondition: ContextKeyExpr.and(
74-
ContextKeyExpr.and(TerminalChatContextKeys.focused, TerminalChatContextKeys.visible)
75-
),
76+
precondition: TerminalChatContextKeys.visible,
7677
run: (_xterm, _accessor, activeInstance) => {
7778
if (isDetachedTerminalInstance(activeInstance)) {
7879
return;

0 commit comments

Comments
 (0)