From b218ab47550f8b1dee1fbcef8f051ae2c4b16a16 Mon Sep 17 00:00:00 2001 From: Nate Butler Date: Sat, 18 Nov 2023 01:02:21 -0500 Subject: [PATCH] Remove default hover state for icon buttons --- crates/ui2/src/components/icon_button.rs | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/crates/ui2/src/components/icon_button.rs b/crates/ui2/src/components/icon_button.rs index 9b8548e3f9c6cf092f08f41bbda3bfc25293bc98..23d920835e63a04a437e97f9d708264f767d5747 100644 --- a/crates/ui2/src/components/icon_button.rs +++ b/crates/ui2/src/components/icon_button.rs @@ -111,7 +111,10 @@ impl IconButton { .p_1() .bg(bg_color) .cursor_pointer() - .hover(|style| style.bg(bg_hover_color)) + // Nate: Trying to figure out the right places we want to show a + // hover state here. I think it is a bit heavy to have it on every + // place we use an icon button. + // .hover(|style| style.bg(bg_hover_color)) .active(|style| style.bg(bg_active_color)) .child(IconElement::new(self.icon).color(icon_color));