From 818cbb24157334adac4a448c237e0ab35019e41c Mon Sep 17 00:00:00 2001
From: Conrad Irwin <conrad.irwin@gmail.com>
Date: Sat, 13 Jan 2024 22:19:21 -0700
Subject: [PATCH] Show a border when scrolled in chat

---
 crates/collab_ui/src/chat_panel.rs | 37 ++++++++++++++++++------------
 crates/gpui/src/elements/list.rs   |  2 ++
 2 files changed, 24 insertions(+), 15 deletions(-)

diff --git a/crates/collab_ui/src/chat_panel.rs b/crates/collab_ui/src/chat_panel.rs
index dd3d7415225b4aae56bd8927c6884981a6681c74..ee0241eb1209dc6a36150c7d68b68250476c59e5 100644
--- a/crates/collab_ui/src/chat_panel.rs
+++ b/crates/collab_ui/src/chat_panel.rs
@@ -112,7 +112,7 @@ impl ChatPanel {
                 if event.visible_range.start < MESSAGE_LOADING_THRESHOLD {
                     this.load_more_messages(cx);
                 }
-                this.is_scrolled_to_bottom = event.visible_range.end == event.count;
+                this.is_scrolled_to_bottom = !event.is_scrolled;
             }));
 
             let mut this = Self {
@@ -567,7 +567,7 @@ impl Render for ChatPanel {
                     ),
                 ),
             )
-            .child(div().flex_grow().px_2().py_1().map(|this| {
+            .child(div().flex_grow().px_2().pt_1().map(|this| {
                 if self.active_chat.is_some() {
                     this.child(list(self.message_list.clone()).full())
                 } else {
@@ -597,19 +597,26 @@ impl Render for ChatPanel {
                     )
                 }
             }))
-            .child(h_stack().p_2().map(|el| {
-                if self.active_chat.is_some() {
-                    el.child(self.message_editor.clone())
-                } else {
-                    el.child(
-                        div()
-                            .rounded_md()
-                            .h_7()
-                            .w_full()
-                            .bg(cx.theme().colors().editor_background),
-                    )
-                }
-            }))
+            .child(
+                h_stack()
+                    .when(!self.is_scrolled_to_bottom, |el| {
+                        el.border_t_1().border_color(cx.theme().colors().border)
+                    })
+                    .p_2()
+                    .map(|el| {
+                        if self.active_chat.is_some() {
+                            el.child(self.message_editor.clone())
+                        } else {
+                            el.child(
+                                div()
+                                    .rounded_md()
+                                    .h_7()
+                                    .w_full()
+                                    .bg(cx.theme().colors().editor_background),
+                            )
+                        }
+                    }),
+            )
             .into_any()
     }
 }
diff --git a/crates/gpui/src/elements/list.rs b/crates/gpui/src/elements/list.rs
index 2a47a16741cf67c0cefb8a094d2f9e506cacbdf4..50e7af5138480562212c8d182352bfebdc88db36 100644
--- a/crates/gpui/src/elements/list.rs
+++ b/crates/gpui/src/elements/list.rs
@@ -43,6 +43,7 @@ pub enum ListAlignment {
 pub struct ListScrollEvent {
     pub visible_range: Range<usize>,
     pub count: usize,
+    pub is_scrolled: bool,
 }
 
 #[derive(Clone)]
@@ -253,6 +254,7 @@ impl StateInner {
                 &ListScrollEvent {
                     visible_range,
                     count: self.items.summary().count,
+                    is_scrolled: self.logical_scroll_top.is_some(),
                 },
                 cx,
             );