mirrors/hermes-agent
1
0
Fork 0
mirror of https://github.com/NousResearch/hermes-agent.git synced 2026-06-09 08:21:50 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 10

fix(desktop): switch model on keyboard activation of picker rows

Browse source 
The model row is a Radix sub-trigger (no onSelect), so switching was
pointer-only. Wire Enter/Space alongside onClick so keyboard users can switch
models too.
This commit is contained in:
Brooklyn Nicholson 2026-06-02 19:50:55 -05:00
parent d704df2d6e
commit b6945ce772
1 changed files with 15 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

21
apps/desktop/src/app/shell/model-menu-panel.tsx
View file

@ -164,17 +164,26 @@ export function ModelMenuPanel({ gateway, onSelectModel, requestGateway }: Model
? 'Fast'
: ''
// Every row is a hover-Edit submenu trigger. Clicking switches
// to the family's base model; the Fast toggle inside swaps to
// the -fast sibling (or flips the speed param) as appropriate.
// Every row is a hover-Edit submenu trigger. Activating it
// (pointer or keyboard) switches to the family's base model;
// the Fast toggle inside swaps to the -fast sibling (or flips
// the speed param). The sub-trigger has no `onSelect`, so wire
// both click and Enter/Space for keyboard parity.
const activate = () => {
if (!isCurrent) {
void switchTo(family.id, group.provider.slug)
}
}
return (
<DropdownMenuSub key={`${group.provider.slug}:${family.id}`}>
<DropdownMenuSubTrigger
className={cn(dropdownMenuRow, 'cursor-pointer')}
hideChevron
onClick={() => {
if (!isCurrent) {
void switchTo(family.id, group.provider.slug)
onClick={activate}
onKeyDown={event => {
if (event.key === 'Enter' || event.key === ' ') {
activate()
}
}}
>