Skip to content

Commit 5d9b7ef

Browse files
authored
Agents Web: touch for ahs picker (#312651)
1 parent 7326d67 commit 5d9b7ef

2 files changed

Lines changed: 12 additions & 4 deletions

File tree

src/vs/sessions/contrib/chat/browser/agentHost/agentHostSessionConfigPicker.ts

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55

66
import '../media/agentHostSessionConfigPicker.css';
77
import * as dom from '../../../../../base/browser/dom.js';
8+
import { Gesture, EventType as TouchEventType } from '../../../../../base/browser/touch.js';
89
import { renderIcon } from '../../../../../base/browser/ui/iconLabel/iconLabels.js';
910
import { ActionListItemKind, IActionListDelegate, IActionListItem } from '../../../../../platform/actionWidget/browser/actionList.js';
1011
import { IActionWidgetService } from '../../../../../platform/actionWidget/browser/actionWidget.js';
@@ -109,10 +110,13 @@ function renderPickerTrigger(slot: HTMLElement, disabled: boolean, disposables:
109110
trigger.role = 'button';
110111
trigger.tabIndex = 0;
111112
trigger.setAttribute('aria-haspopup', 'listbox');
112-
disposables.add(dom.addDisposableListener(trigger, dom.EventType.CLICK, e => {
113-
dom.EventHelper.stop(e, true);
114-
onOpen();
115-
}));
113+
disposables.add(Gesture.addTarget(trigger));
114+
for (const eventType of [dom.EventType.CLICK, TouchEventType.Tap]) {
115+
disposables.add(dom.addDisposableListener(trigger, eventType, e => {
116+
dom.EventHelper.stop(e, true);
117+
onOpen();
118+
}));
119+
}
116120
disposables.add(dom.addDisposableListener(trigger, dom.EventType.KEY_DOWN, e => {
117121
if (e.key === 'Enter' || e.key === ' ') {
118122
dom.EventHelper.stop(e, true);

src/vs/sessions/contrib/chat/browser/media/agentHostSessionConfigPicker.css

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,3 +12,7 @@
1212
.sessions-chat-agent-host-config:empty {
1313
display: none;
1414
}
15+
16+
.sessions-chat-agent-host-config a.action-label {
17+
touch-action: manipulation;
18+
}

0 commit comments

Comments
 (0)