yenru0/zed
1
0 Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

ui: Fix end_slot_on_hover API (#59805)

Browse Source 
The issue here was that `end_slot_on_hover` would currently only work if
`end_slot` was also set. With this, it will now work even if there is no
`end_slot`.

Release Notes:

- N/A
This commit is contained in:
Finn Evers authored and GitHub committed 2026-06-24 09:11:41 +00:00
1 parent 502810d204
commit 25d2aaa9f3
1 file changed
+27 -21
Download Patch File Download Diff File Expand all files Collapse all files
crates/ui/src/components/list/list_item.rs
+27 -21
View File
@@ -404,30 +404,36 @@ impl RenderOnce for ListItem {
.children(self.children),
)
.when(self.end_slot.is_some(), |this| this.justify_between())
.when_some(self.end_slot, |this, end_slot| {
this.child(match self.end_slot_visibility {
EndSlotVisibility::Always => {
h_flex().flex_shrink_1().overflow_hidden().child(end_slot)
}
EndSlotVisibility::OnHover => h_flex()
.flex_shrink_1()
.overflow_hidden()
.visible_on_hover("list_item")
.child(end_slot),
EndSlotVisibility::SwapOnHover(hover_slot) => h_flex()
.map(|this| match self.end_slot_visibility {
EndSlotVisibility::Always if let Some(end_slot) = self.end_slot => {
this.child(h_flex().flex_shrink_1().overflow_hidden().child(end_slot))
}
EndSlotVisibility::OnHover if let Some(end_slot) = self.end_slot => this
.child(
h_flex()
.flex_shrink_1()
.overflow_hidden()
.visible_on_hover("list_item")
.child(end_slot),
),
EndSlotVisibility::SwapOnHover(hover_slot) => this.child(
h_flex()
.relative()
.flex_shrink_1()
.child(h_flex().visible_on_hover("list_item").child(hover_slot))
.child(
h_flex()
.absolute()
.inset_0()
.justify_end()
.overflow_hidden()
.group_hover("list_item", |this| this.invisible())
.child(end_slot),
),
})
.when_some(self.end_slot, |this, end_slot| {
this.child(
h_flex()
.absolute()
.inset_0()
.justify_end()
.overflow_hidden()
.group_hover("list_item", |this| this.invisible())
.child(end_slot),
)
}),
),
_ => this,
}),
)
}