Adjust list item & pickers spacing (#14250)

Danilo Leal created 2 years ago

Change summary

crates/assistant/src/prompt_library.rs     | 3 ++-
crates/picker/src/picker.rs                | 4 ++--
crates/ui/src/components/list/list_item.rs | 2 +-
crates/vcs_menu/src/lib.rs                 | 1 +
4 files changed, 6 insertions(+), 4 deletions(-)

Detailed changes

crates/assistant/src/prompt_library.rs

@@ -268,7 +268,7 @@ impl PickerDelegate for PromptPickerDelegate {
             .flex_none()
             .py_1()
             .px_2()
-            .mx_2()
+            .mx_1()
             .child(editor.clone())
     }
 }
@@ -766,6 +766,7 @@ impl PromptLibrary {
             .capture_action(cx.listener(Self::focus_active_prompt))
             .bg(cx.theme().colors().panel_background)
             .h_full()
+            .px_1()
             .w_1_3()
             .overflow_x_hidden()
             .child(

crates/picker/src/picker.rs

@@ -110,7 +110,7 @@ pub trait PickerDelegate: Sized + 'static {
                     .overflow_hidden()
                     .flex_none()
                     .h_9()
-                    .px_4()
+                    .px_3()
                     .child(editor.clone()),
             )
             .child(Divider::horizontal())
@@ -527,7 +527,7 @@ impl<D: PickerDelegate> Picker<D> {
             )
             .with_sizing_behavior(sizing_behavior)
             .flex_grow()
-            .py_2()
+            .py_1()
             .track_scroll(scroll_handle.clone())
             .into_any_element(),
             ElementContainer::List(state) => list(state.clone())

crates/ui/src/components/list/list_item.rs

@@ -162,7 +162,7 @@ impl RenderOnce for ListItem {
             // When an item is inset draw the indent spacing outside of the item
             .when(self.inset, |this| {
                 this.ml(self.indent_level as f32 * self.indent_step_size)
-                    .px_2()
+                    .px_1()
             })
             .when(!self.inset && !self.disabled, |this| {
                 this

crates/vcs_menu/src/lib.rs

@@ -273,6 +273,7 @@ impl PickerDelegate for BranchListDelegate {
         let label = if self.last_query.is_empty() {
             Label::new("Recent Branches")
                 .size(LabelSize::Small)
+                .mt_1()
                 .ml_3()
                 .into_any_element()
         } else {