Correctly position popover aside, add list item states

Nate Butler and Marshall Bowers created 2 years ago

Co-Authored-By: Marshall Bowers <[e-mail]>

Change summary

crates/ui2/src/components/list.rs    | 3 ++-
crates/ui2/src/components/popover.rs | 3 ++-
2 files changed, 4 insertions(+), 2 deletions(-)

Detailed changes

crates/ui2/src/components/list.rs

@@ -364,12 +364,13 @@ impl Component for ListItem {
                     }
                 }
             })
-            .bg(cx.theme().colors().surface_background)
             // TODO: Add focus state
             // .when(self.state == InteractionState::Focused, |this| {
             //     this.border()
             //         .border_color(cx.theme().colors().border_focused)
             // })
+            .hover(|style| style.bg(cx.theme().colors().ghost_element_hover))
+            .active(|style| style.bg(cx.theme().colors().ghost_element_active))
             .child(
                 sized_item
                     .when(self.variant == ListItemVariant::Inset, |this| this.px_2())

crates/ui2/src/components/popover.rs

@@ -27,7 +27,8 @@ impl Component for Popover {
                 this.child(
                     v_stack()
                         .top_0()
-                        .neg_right_1()
+                        .left_full()
+                        .ml_1()
                         .absolute()
                         .elevation_2(cx)
                         .p_1()