From c74e5f5de20b1f0bc88d1ec2765e4293e3ab3aaa Mon Sep 17 00:00:00 2001 From: Danilo Leal <67129314+danilo-leal@users.noreply.github.com> Date: Mon, 6 Jan 2025 15:53:38 -0300 Subject: [PATCH] assistant2: Render placeholder thread title until summary is generated (#22723) This PR ensures we render a "New Thread" placeholder title until a message has been sent, and thus, a summary is generated. https://github.com/user-attachments/assets/1c30e0ff-baaa-44ad-a1a2-42f1ce9fe0b0 Release Notes: - N/A --- crates/assistant2/src/assistant_panel.rs | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/crates/assistant2/src/assistant_panel.rs b/crates/assistant2/src/assistant_panel.rs index 7e229251ef..4c7a1f97e7 100644 --- a/crates/assistant2/src/assistant_panel.rs +++ b/crates/assistant2/src/assistant_panel.rs @@ -300,6 +300,15 @@ impl AssistantPanel { fn render_toolbar(&self, cx: &mut ViewContext) -> impl IntoElement { let focus_handle = self.focus_handle(cx); + let title = if self.thread.read(cx).is_empty() { + SharedString::from("New Thread") + } else { + self.thread + .read(cx) + .summary(cx) + .unwrap_or_else(|| SharedString::from("Loading Summary…")) + }; + h_flex() .id("assistant-toolbar") .justify_between() @@ -309,7 +318,7 @@ impl AssistantPanel { .bg(cx.theme().colors().tab_bar_background) .border_b_1() .border_color(cx.theme().colors().border) - .child(h_flex().children(self.thread.read(cx).summary(cx).map(Label::new))) + .child(h_flex().child(Label::new(title))) .child( h_flex() .h_full()