diff --git a/crates/agent/src/active_thread.rs b/crates/agent/src/active_thread.rs index 20310ce464e5cdef65ff8afb3061bb97836e7d39..802826ed1456699583743dc650af398bd7df22af 100644 --- a/crates/agent/src/active_thread.rs +++ b/crates/agent/src/active_thread.rs @@ -1696,7 +1696,7 @@ impl ActiveThread { let message_content = has_content.then(|| { v_flex() - .gap_1p5() + .gap_1() .when(!message_is_empty, |parent| { parent.child( if let Some(edit_message_editor) = edit_message_editor.clone() { @@ -1719,7 +1719,6 @@ impl ActiveThread { .on_action(cx.listener(Self::cancel_editing_message)) .on_action(cx.listener(Self::confirm_editing_message)) .min_h_6() - .pt_1() .child(EditorElement::new( &edit_message_editor, EditorStyle { @@ -1797,7 +1796,7 @@ impl ActiveThread { .border_1() .border_color(colors.border) .shadow_md() - .child(div().py_2().px_2p5().children(message_content)) + .child(div().p_2().children(message_content)) .child( h_flex() .p_1() @@ -2529,7 +2528,7 @@ impl ActiveThread { .upgrade() .map(|workspace| workspace.read(cx).app_state().fs.clone()); let needs_confirmation = matches!(&tool_use.status, ToolUseStatus::NeedsConfirmation); - let edit_tools = tool_use.needs_confirmation; + let needs_confirmation_tools = tool_use.needs_confirmation; let status_icons = div().child(match &tool_use.status { ToolUseStatus::NeedsConfirmation => { @@ -2627,33 +2626,33 @@ impl ActiveThread { )), ), ToolUseStatus::InputStillStreaming | ToolUseStatus::Running => container.child( - results_content_container().child( - h_flex() - .gap_1() - .pb_1() - .border_t_1() - .border_color(self.tool_card_border_color(cx)) - .child( - Icon::new(IconName::ArrowCircle) - .size(IconSize::Small) - .color(Color::Accent) - .with_animation( - "arrow-circle", - Animation::new(Duration::from_secs(2)).repeat(), - |icon, delta| { - icon.transform(Transformation::rotate(percentage( - delta, - ))) - }, - ), - ) - .child( - Label::new("Running…") - .size(LabelSize::XSmall) - .color(Color::Muted) - .buffer_font(cx), - ), - ), + results_content_container() + .border_t_1() + .border_color(self.tool_card_border_color(cx)) + .child( + h_flex() + .gap_1() + .child( + Icon::new(IconName::ArrowCircle) + .size(IconSize::Small) + .color(Color::Accent) + .with_animation( + "arrow-circle", + Animation::new(Duration::from_secs(2)).repeat(), + |icon, delta| { + icon.transform(Transformation::rotate(percentage( + delta, + ))) + }, + ), + ) + .child( + Label::new("Running…") + .size(LabelSize::XSmall) + .color(Color::Muted) + .buffer_font(cx), + ), + ), ), ToolUseStatus::Error(_) => container.child( results_content_container() @@ -2717,8 +2716,8 @@ impl ActiveThread { )) }; - v_flex().gap_1().mb_3().map(|element| { - if !edit_tools { + v_flex().gap_1().mb_2().map(|element| { + if !needs_confirmation_tools { element.child( v_flex() .child( diff --git a/crates/assistant_tools/src/edit_file_tool.rs b/crates/assistant_tools/src/edit_file_tool.rs index c66e2ffb6963e8b4e3625dea8cfe19fb11ec0db0..d2588f08183f0a2f7a290558941f91ba67058566 100644 --- a/crates/assistant_tools/src/edit_file_tool.rs +++ b/crates/assistant_tools/src/edit_file_tool.rs @@ -61,7 +61,7 @@ struct PartialInput { pub struct EditFileTool; -const DEFAULT_UI_TEXT: &str = "Edit file"; +const DEFAULT_UI_TEXT: &str = "Editing file"; impl Tool for EditFileTool { fn name(&self) -> String { @@ -87,7 +87,7 @@ impl Tool for EditFileTool { fn ui_text(&self, input: &serde_json::Value) -> String { match serde_json::from_value::(input.clone()) { Ok(input) => input.display_description, - Err(_) => "Edit file".to_string(), + Err(_) => "Editing file".to_string(), } }