hey team how come my LLM setting is grayed out?
# proj-agent-canvas
m
hey team how come my LLM setting is grayed out?
l
I guess it’s because you have the ACP agent selected and it’s not relevant when using the ACP agent. do you see a message when you mouse over?
πŸ™ 1
if not we may want to add a tooltip
b
🐾 Confirming from the code: Graham's right β€” the LLM (and Condenser/Verification) settings are greyed out on purpose while an ACP agent is active, because the ACP sub-agent manages its own LLM/condenser/MCP, so there's nothing for the OpenHands side to configure there. And there is a tooltip already: on the desktop sidebar the disabled item shows "Disabled while {agent} is the active agent" on hover (
SETTINGS$AGENT_DISABLED_TOOLTIP
, rendered via `SidebarNavLink`'s
disabledReason
). So if you switch the active agent away from the ACP one, the LLM settings light back up. @mysterious-iron-46975 β€” if you're not seeing that tooltip on hover (e.g. on mobile/the drawer, or it just isn't showing), say so β€” the desktop path has it, but the mobile drawer may be missing it and we should add it there.
πŸ™ 1
m
yup it's indeed the ACP agent setting. however i don't see a tooltip with mouse over. i'm using macbook
πŸ™ 1
b
🐾 Thanks for checking @mysterious-iron-46975 β€” that's a real bug, and you found it. I dug into the code: The tooltip is wired up on desktop (
SidebarNavLink
wraps the disabled item in a hover
StyledTooltip
), but it can never fire β€” because the same disabled link also gets
pointer-events-none
(`sidebar-nav-link.tsx`:
disabled && "pointer-events-none opacity-50"
). HeroUI's Tooltip is hover-triggered, so if the element ignores pointer events,
onPointerEnter
never fires and the tooltip never opens. Classic
pointer-events-none
vs hover-tooltip conflict β€” so the diagnosis was right, the explanation just isn't reaching you. Fix is small: move
pointer-events-none
off the hover target β€” e.g. keep the link visually disabled (opacity +
aria-disabled
+
preventDefault
, which are already there) but drop
pointer-events-none
, or wrap the tooltip around an enabled element. Want me to open an issue / small PR on agent-canvas for it?
c
yes do both
🐾 1
b
🐾 Fixed it β€” thanks again @mysterious-iron-46975, your hover check is what surfaced it. β€’ Issue: https://github.com/OpenHands/agent-canvas/issues/1498 β€’ PR: https://github.com/OpenHands/agent-canvas/pull/1499 Dropped
pointer-events-none
from the disabled item when it has a tooltip to show (kept it greyed via opacity; clicks are still blocked by the existing handlers), so the HeroUI hover tooltip can finally fire. Added a regression test that fails without the fix. Once it merges, hovering the greyed LLM/Condenser items on desktop will show "Disabled while {agent} is active".
πŸ™Œ 1
πŸ™ 1