Fix all Picker Item cursor to use `Pointer`. (#8877)

Jason Lee created 2 years ago

Before:

https://github.com/zed-industries/zed/assets/5518/84874858-7847-4fa4-b7a3-41ecc65a2f7d

After: 

https://github.com/zed-industries/zed/assets/5518/d395ea96-aa26-4de1-8bfc-73cc43ee75cf


Release Notes:

- Made picker items to use `Pointer` cursor style

Change summary

crates/picker/src/picker.rs                   | 1 +
crates/recent_projects/src/recent_projects.rs | 1 -
crates/vcs_menu/src/lib.rs                    | 1 -
3 files changed, 1 insertion(+), 2 deletions(-)

Detailed changes

crates/picker/src/picker.rs

@@ -325,6 +325,7 @@ impl<D: PickerDelegate> Picker<D> {
     fn render_element(&self, cx: &mut ViewContext<Self>, ix: usize) -> impl IntoElement {
         div()
             .id(("item", ix))
+            .cursor_pointer()
             .on_click(cx.listener(move |this, event: &ClickEvent, cx| {
                 this.handle_click(ix, event.down.modifiers.command, cx)
             }))

crates/recent_projects/src/recent_projects.rs

@@ -133,7 +133,6 @@ impl Render for RecentProjects {
     fn render(&mut self, cx: &mut ViewContext<Self>) -> impl IntoElement {
         v_flex()
             .w(rems(self.rem_width))
-            .cursor_pointer()
             .child(self.picker.clone())
             .on_mouse_down_out(cx.listener(|this, _, cx| {
                 this.picker.update(cx, |this, cx| {

crates/vcs_menu/src/lib.rs

@@ -67,7 +67,6 @@ impl Render for BranchList {
     fn render(&mut self, cx: &mut ViewContext<Self>) -> impl IntoElement {
         v_flex()
             .w(rems(self.rem_width))
-            .cursor_pointer()
             .child(self.picker.clone())
             .on_mouse_down_out(cx.listener(|this, _, cx| {
                 this.picker.update(cx, |this, cx| {