notedeck

commit f27b1fe957ce5a986d92cd1957990dbb88e70fdd
parent 45803b6bb05b88e9e37c4e74abd34c1ed3257b24
Author: William Casarin <jb55@jb55.com>
Date:   Wed, 16 Jul 2025 09:01:48 -0700

Merge chrome sidebar features from fernando

Fernando López Guevara (2):
      fix(compose-button): apply icon_color to compose button edge circles & add hover text
      fix(chrome): add hover text and pointer cursor to sidebar elements

Diffstat:
M
crates/notedeck_columns/src/ui/post.rs
 | 
14
++++++++------
M
crates/notedeck_columns/src/ui/side_panel.rs
 | 
23
++++++++++++++++++-----

2 files changed, 26 insertions(+), 11 deletions(-)
diff --git a/crates/notedeck_columns/src/ui/post.rs b/crates/notedeck_columns/src/ui/post.rs
@@ -45,11 +45,13 @@ pub fn compose_note_button(dark_mode: bool) -> impl Widget {
             [west_edge, east_edge],
             Stroke::new(use_line_width, icon_color),
         );
-        painter.circle_filled(north_edge, use_edge_circle_radius, Color32::WHITE);
-        painter.circle_filled(south_edge, use_edge_circle_radius, Color32::WHITE);
-        painter.circle_filled(west_edge, use_edge_circle_radius, Color32::WHITE);
-        painter.circle_filled(east_edge, use_edge_circle_radius, Color32::WHITE);
-
-        helper.take_animation_response()
+        painter.circle_filled(north_edge, use_edge_circle_radius, icon_color);
+        painter.circle_filled(south_edge, use_edge_circle_radius, icon_color);
+        painter.circle_filled(west_edge, use_edge_circle_radius, icon_color);
+        painter.circle_filled(east_edge, use_edge_circle_radius, icon_color);
+
+        helper
+            .take_animation_response()
+            .on_hover_text("Compose new note")
     }
 }
diff --git a/crates/notedeck_columns/src/ui/side_panel.rs b/crates/notedeck_columns/src/ui/side_panel.rs
@@ -1,4 +1,7 @@
-use egui::{vec2, InnerResponse, Layout, Margin, RichText, ScrollArea, Separator, Stroke, Widget};
+use egui::{
+    vec2, CursorIcon, InnerResponse, Layout, Margin, RichText, ScrollArea, Separator, Stroke,
+    Widget,
+};
 use tracing::{error, info};
 
 use crate::{
@@ -312,7 +315,10 @@ fn add_column_button() -> impl Widget {
                 .shrink((max_size - cur_img_size) / 2.0),
         );
 
-        helper.take_animation_response()
+        helper
+            .take_animation_response()
+            .on_hover_cursor(CursorIcon::PointingHand)
+            .on_hover_text("Add new column")
     }
 }
 
@@ -354,7 +360,10 @@ pub fn search_button() -> impl Widget {
             circle_stroke,
         );
 
-        helper.take_animation_response()
+        helper
+            .take_animation_response()
+            .on_hover_cursor(CursorIcon::PointingHand)
+            .on_hover_text("Open search")
     }
 }
 
@@ -377,7 +386,10 @@ fn add_deck_button() -> impl Widget {
                 .shrink((max_size - cur_img_size) / 2.0),
         );
 
-        helper.take_animation_response()
+        helper
+            .take_animation_response()
+            .on_hover_cursor(CursorIcon::PointingHand)
+            .on_hover_text("Add new deck")
     }
 }
 
@@ -406,7 +418,8 @@ fn show_decks<'a>(
                 40.0,
                 highlight,
             ))
-            .on_hover_text_at_pointer(&deck.name);
+            .on_hover_text_at_pointer(&deck.name)
+            .on_hover_cursor(CursorIcon::PointingHand);
         if deck_icon_resp.clicked() || deck_icon_resp.secondary_clicked() {
             clicked_index = Some(index);
         }