notedeck

commit 96481a47f336b6643d3b112a558280bca19d7c42
parent 635c9770de36b43666a399e8b331980afb39f5f9
Author: William Casarin <jb55@jb55.com>
Date:   Mon,  3 Feb 2025 19:45:32 -0800

pfp: remove border except for profile

Signed-off-by: William Casarin <jb55@jb55.com>

Diffstat:
M
crates/notedeck_columns/src/ui/column/header.rs
 | 
6
+-----
M
crates/notedeck_columns/src/ui/note/mod.rs
 | 
24
+++++++++---------------
M
crates/notedeck_columns/src/ui/note/post.rs
 | 
12
++----------
M
crates/notedeck_columns/src/ui/profile/preview.rs
 | 
6
+-----
M
crates/notedeck_columns/src/ui/side_panel.rs
 | 
4
+---

5 files changed, 14 insertions(+), 38 deletions(-)
diff --git a/crates/notedeck_columns/src/ui/column/header.rs b/crates/notedeck_columns/src/ui/column/header.rs
@@ -398,11 +398,7 @@ impl<'a> NavTitle<'a> {
             .as_ref()
             .ok()
             .and_then(move |p| {
-                Some(
-                    ui::ProfilePic::from_profile(self.img_cache, p)?
-                        .size(pfp_size)
-                        .border(2.0),
-                )
+                Some(ui::ProfilePic::from_profile(self.img_cache, p)?.size(pfp_size))
             })
     }
 
diff --git a/crates/notedeck_columns/src/ui/note/mod.rs b/crates/notedeck_columns/src/ui/note/mod.rs
@@ -228,19 +228,14 @@ impl<'a> NoteView<'a> {
                     anim_speed,
                 );
 
-                ui.put(
-                    rect,
-                    ui::ProfilePic::new(self.img_cache, pic)
-                        .size(size)
-                        .border(2.0),
-                )
-                .on_hover_ui_at_pointer(|ui| {
-                    ui.set_max_width(300.0);
-                    ui.add(ui::ProfilePreview::new(
-                        profile.as_ref().unwrap(),
-                        self.img_cache,
-                    ));
-                });
+                ui.put(rect, ui::ProfilePic::new(self.img_cache, pic).size(size))
+                    .on_hover_ui_at_pointer(|ui| {
+                        ui.set_max_width(300.0);
+                        ui.add(ui::ProfilePreview::new(
+                            profile.as_ref().unwrap(),
+                            self.img_cache,
+                        ));
+                    });
 
                 if resp.hovered() || resp.clicked() {
                     ui::show_pointer(ui);
@@ -251,8 +246,7 @@ impl<'a> NoteView<'a> {
             None => ui
                 .add(
                     ui::ProfilePic::new(self.img_cache, ui::ProfilePic::no_pfp_url())
-                        .size(pfp_size)
-                        .border(2.0),
+                        .size(pfp_size),
                 )
                 .interact(sense),
         }
diff --git a/crates/notedeck_columns/src/ui/note/post.rs b/crates/notedeck_columns/src/ui/note/post.rs
@@ -113,21 +113,13 @@ impl<'a> PostView<'a> {
             .get_profile_by_pubkey(txn, self.poster.pubkey.bytes())
             .as_ref()
             .ok()
-            .and_then(|p| {
-                Some(
-                    ui::ProfilePic::from_profile(self.img_cache, p)?
-                        .size(pfp_size)
-                        .border(2.0),
-                )
-            });
+            .and_then(|p| Some(ui::ProfilePic::from_profile(self.img_cache, p)?.size(pfp_size)));
 
         if let Some(pfp) = poster_pfp {
             ui.add(pfp);
         } else {
             ui.add(
-                ui::ProfilePic::new(self.img_cache, ui::ProfilePic::no_pfp_url())
-                    .size(pfp_size)
-                    .border(2.0),
+                ui::ProfilePic::new(self.img_cache, ui::ProfilePic::no_pfp_url()).size(pfp_size),
             );
         }
 
diff --git a/crates/notedeck_columns/src/ui/profile/preview.rs b/crates/notedeck_columns/src/ui/profile/preview.rs
@@ -91,11 +91,7 @@ impl egui::Widget for SimpleProfilePreview<'_, '_> {
     fn ui(self, ui: &mut egui::Ui) -> egui::Response {
         Frame::none()
             .show(ui, |ui| {
-                ui.add(
-                    ProfilePic::new(self.cache, get_profile_url(self.profile))
-                        .size(48.0)
-                        .border(2.0),
-                );
+                ui.add(ProfilePic::new(self.cache, get_profile_url(self.profile)).size(48.0));
                 ui.vertical(|ui| {
                     ui.add(display_name_widget(get_display_name(self.profile), true));
                     if !self.is_nsec {
diff --git a/crates/notedeck_columns/src/ui/side_panel.rs b/crates/notedeck_columns/src/ui/side_panel.rs
@@ -266,9 +266,7 @@ impl<'a> DesktopSidePanel<'a> {
         let txn = nostrdb::Transaction::new(self.ndb).expect("should be able to create txn");
         let profile_url = get_account_url(&txn, self.ndb, self.selected_account);
 
-        let widget = ProfilePic::new(self.img_cache, profile_url)
-            .size(cur_pfp_size)
-            .border(2.0);
+        let widget = ProfilePic::new(self.img_cache, profile_url).size(cur_pfp_size);
 
         ui.put(helper.get_animation_rect(), widget);