thread_view: Increase click area of the user rules links (#36549)

Danilo Leal created 3 months ago

Release Notes:

- N/A

Change summary

crates/agent_ui/src/acp/thread_view.rs | 26 ++++++++++++++------------
1 file changed, 14 insertions(+), 12 deletions(-)

Detailed changes

crates/agent_ui/src/acp/thread_view.rs

@@ -2056,6 +2056,7 @@ impl AcpThreadView {
                     parent.child(
                         h_flex()
                             .group("user-rules")
+                            .id("user-rules")
                             .w_full()
                             .child(
                                 Icon::new(IconName::Reader)
@@ -2078,25 +2079,26 @@ impl AcpThreadView {
                                     .icon_color(Color::Ignored)
                                     .visible_on_hover("user-rules")
                                     // TODO: Figure out a way to pass focus handle here so we can display the `OpenRulesLibrary`  keybinding
-                                    .tooltip(Tooltip::text("View User Rules"))
-                                    .on_click(move |_event, window, cx| {
-                                        window.dispatch_action(
-                                            Box::new(OpenRulesLibrary {
-                                                prompt_to_select: first_user_rules_id,
-                                            }),
-                                            cx,
-                                        )
+                                    .tooltip(Tooltip::text("View User Rules")),
+                            )
+                            .on_click(move |_event, window, cx| {
+                                window.dispatch_action(
+                                    Box::new(OpenRulesLibrary {
+                                        prompt_to_select: first_user_rules_id,
                                     }),
-                            ),
+                                    cx,
+                                )
+                            }),
                     )
                 })
                 .when_some(rules_file_text, |parent, rules_file_text| {
                     parent.child(
                         h_flex()
                             .group("project-rules")
+                            .id("project-rules")
                             .w_full()
                             .child(
-                                Icon::new(IconName::File)
+                                Icon::new(IconName::Reader)
                                     .size(IconSize::XSmall)
                                     .color(Color::Disabled),
                             )
@@ -2113,10 +2115,10 @@ impl AcpThreadView {
                                     .shape(ui::IconButtonShape::Square)
                                     .icon_size(IconSize::XSmall)
                                     .icon_color(Color::Ignored)
-                                    .on_click(cx.listener(Self::handle_open_rules))
                                     .visible_on_hover("project-rules")
                                     .tooltip(Tooltip::text("View Project Rules")),
-                            ),
+                            )
+                            .on_click(cx.listener(Self::handle_open_rules)),
                     )
                 })
                 .into_any(),