assistant2: Tweak "Add Context" placeholder (#22596)

Marshall Bowers created 1 year ago

This PR tweaks the "Add Context" placeholder, as the text appeared to be
vertically misaligned.

#### Before

<img width="215" alt="Screenshot 2025-01-02 at 6 03 06 PM"
src="https://github.com/user-attachments/assets/1bac0deb-bd90-4ff3-b681-ee884cbe831d"
/>

#### After

<img width="189" alt="Screenshot 2025-01-02 at 6 03 20 PM"
src="https://github.com/user-attachments/assets/c9673fb0-11d6-42ac-8fec-9af269dfc73c"
/>


Release Notes:

- N/A

Change summary

crates/assistant2/src/context_strip.rs | 25 ++++++++++---------------
1 file changed, 10 insertions(+), 15 deletions(-)

Detailed changes

crates/assistant2/src/context_strip.rs

@@ -4,7 +4,7 @@ use editor::Editor;
 use gpui::{AppContext, FocusHandle, Model, View, WeakModel, WeakView};
 use language::Buffer;
 use project::ProjectEntryId;
-use ui::{prelude::*, PopoverMenu, PopoverMenuHandle, Tooltip};
+use ui::{prelude::*, KeyBinding, PopoverMenu, PopoverMenuHandle, Tooltip};
 use workspace::Workspace;
 
 use crate::context::ContextKind;
@@ -14,7 +14,6 @@ use crate::thread::{Thread, ThreadId};
 use crate::thread_store::ThreadStore;
 use crate::ui::ContextPill;
 use crate::{AssistantPanel, ToggleContextPicker};
-use settings::Settings;
 
 pub struct ContextStrip {
     context_store: Model<ContextStore>,
@@ -153,22 +152,18 @@ impl Render for ContextStrip {
                 |parent| {
                     parent.child(
                         h_flex()
-                            .id("no-content-info")
                             .ml_1p5()
                             .gap_2()
-                            .font(theme::ThemeSettings::get_global(cx).buffer_font.clone())
-                            .text_size(TextSize::Small.rems(cx))
-                            .text_color(cx.theme().colors().text_muted)
-                            .child("Add Context")
-                            .children(
-                                ui::KeyBinding::for_action_in(
-                                    &ToggleContextPicker,
-                                    &focus_handle,
-                                    cx,
-                                )
-                                .map(|binding| binding.into_any_element()),
+                            .child(
+                                Label::new("Add Context")
+                                    .size(LabelSize::Small)
+                                    .color(Color::Muted),
                             )
-                            .opacity(0.5),
+                            .opacity(0.5)
+                            .children(
+                                KeyBinding::for_action_in(&ToggleContextPicker, &focus_handle, cx)
+                                    .map(|binding| binding.into_any_element()),
+                            ),
                     )
                 }
             })