From 3dbccc828e4589ab9ea31299d7117e81ec251727 Mon Sep 17 00:00:00 2001
From: Alvaro Parker <64918109+AlvaroParker@users.noreply.github.com>
Date: Mon, 15 Sep 2025 19:48:04 -0300
Subject: [PATCH] Fix hover element on ACP thread mode selector (#38204)
Closes #38197
This will render `^ click to also ...` on MacOS and `Ctrl + click to
also ...` on Windows and Linux.
|Before|After|
|-|-|
| 
| 
|
On Mac:
Release Notes:
- Fixed keymap hint when hovering over mode selector
---------
Co-authored-by: Danilo Leal
---
crates/agent_ui/src/acp/mode_selector.rs | 16 +++++++++-------
1 file changed, 9 insertions(+), 7 deletions(-)
diff --git a/crates/agent_ui/src/acp/mode_selector.rs b/crates/agent_ui/src/acp/mode_selector.rs
index d4d424f41a..4108741266 100644
--- a/crates/agent_ui/src/acp/mode_selector.rs
+++ b/crates/agent_ui/src/acp/mode_selector.rs
@@ -107,13 +107,15 @@ impl ModeSelector {
.text_sm()
.text_color(Color::Muted.color(cx))
.child("Hold")
- .child(div().pt_0p5().children(ui::render_modifiers(
- &gpui::Modifiers::secondary_key(),
- PlatformStyle::platform(),
- None,
- Some(ui::TextSize::Default.rems(cx).into()),
- true,
- )))
+ .child(h_flex().flex_shrink_0().children(
+ ui::render_modifiers(
+ &gpui::Modifiers::secondary_key(),
+ PlatformStyle::platform(),
+ None,
+ Some(ui::TextSize::Default.rems(cx).into()),
+ true,
+ ),
+ ))
.child(div().map(|this| {
if is_default {
this.child("to also unset as default")