From 814cbff0868fc12d3f140fbba96e51d0ef2952d1 Mon Sep 17 00:00:00 2001 From: Danny Avila Date: Thu, 30 Jul 2026 22:20:20 -0400 Subject: [PATCH] fix: advertise the escalation shortcut only while the control is live Codex on the e2e head, one P2: the tooltip appended the chord hint even while the button was disabled, advertising a shortcut that does nothing during an approval pause. The flagged control (InterruptNowButton) was since replaced by the shared EscalateNowButton, which inherited the pattern; the successor now omits the chord whenever the control is disabled, matching the rule the during-run hovercard already follows. --- client/src/components/Chat/Input/SteerMenu.tsx | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/client/src/components/Chat/Input/SteerMenu.tsx b/client/src/components/Chat/Input/SteerMenu.tsx index 75dfdb9079..46c8b6bf8f 100644 --- a/client/src/components/Chat/Input/SteerMenu.tsx +++ b/client/src/components/Chat/Input/SteerMenu.tsx @@ -136,7 +136,9 @@ export function EscalateNowButton({ } /> - {chord ? `${label} · ${chord}` : label} + {/* A disabled control's shortcut does nothing right now, so the + * chord is only advertised while the escalation is live. */} + {chord && !disabled ? `${label} · ${chord}` : label}