Add url_status

Author: ranfdev

data/resources/ui/window.blp

@@ -80,37 +80,54 @@ template GeopardWindow: Adw.ApplicationWindow {
         text: bind GeopardWindow.progress;
       }
       [overlay]
-      Gtk.Revealer bottom_bar_revealer {
-        transition-type: slide_up;
+      Gtk.Box {
+        orientation: vertical;
         valign: end;
-        [child]
-        Adw.HeaderBar bottom_bar {
-          show-end-title-buttons: false;
-          show-start-title-buttons: false;
-          [start]
-          Gtk.Button {
-            icon-name: "go-previous-symbolic";
-            action-name: "win.previous";
+        Gtk.Box url_status_box {
+          styles ["background"]
+          Gtk.Label url_status {
+            xalign: 0.0;
+            halign: start;
+            wrap: true;
+            wrap-mode: word_char;
+            margin-top: 6;
+            margin-bottom: 6;
+            margin-start: 6;
+            margin-end: 6;
           }
-          [start]
-          Gtk.Button {
-            icon-name: "go-next-symbolic";
-            action-name: "win.next";
-          }
-          [title]
-          Gtk.Button {
-            icon-name: "system-search-symbolic";
-            action-name: "win.focus-url-bar";
-          }
-          [end]
-          Gtk.MenuButton {
-            icon-name: "open-menu";
-            menu-model: primary_menu;
-          }
-          [end]
-          Gtk.Button {
-            icon-name: "tab-new-symbolic";
-            action-name: "win.new-tab";
+        }
+        Gtk.Revealer bottom_bar_revealer {
+          transition-type: slide_up;
+          valign: end;
+          [child]
+          Adw.HeaderBar bottom_bar {
+            show-end-title-buttons: false;
+            show-start-title-buttons: false;
+            [start]
+            Gtk.Button {
+              icon-name: "go-previous-symbolic";
+              action-name: "win.previous";
+            }
+            [start]
+            Gtk.Button {
+              icon-name: "go-next-symbolic";
+              action-name: "win.next";
+            }
+            [title]
+            Gtk.Button {
+              icon-name: "system-search-symbolic";
+              action-name: "win.focus-url-bar";
+            }
+            [end]
+            Gtk.MenuButton {
+              icon-name: "open-menu";
+              menu-model: primary_menu;
+            }
+            [end]
+            Gtk.Button {
+              icon-name: "tab-new-symbolic";
+              action-name: "win.new-tab";
+            }
           }
         }
       }

src/widgets/tab.rs

@@ -72,6 +72,8 @@ pub mod imp {
         pub(crate) title: RefCell<String>,
         #[property(get)]
         pub(crate) url: RefCell<String>,
+        #[property(get)]
+        pub(crate) hover_url: RefCell<String>,
     }
 
     #[glib::object_subclass]
@@ -203,15 +205,23 @@ impl Tab {
         let gemini_text_ext = imp.gemini_text_ext.borrow();
         let gemini_text_ext = gemini_text_ext.as_ref().unwrap();
         let link = Self::extract_linkhandler(gemini_text_ext, x, y);
-        match link {
-            Ok(_) => {
-                gemini_text_ext
-                    .text_view
-                    .set_cursor_from_name(Some("pointer"));
-            }
-            Err(_) => {
-                gemini_text_ext.text_view.set_cursor_from_name(Some("text"));
+
+        let link_ref = link.as_ref().map(|x| x.as_ref()).unwrap_or("");
+
+        // May need optimization. Comparing two strings for each motion event is expensive
+        if *imp.hover_url.borrow() != link_ref {
+            match link_ref {
+                "" => {
+                    gemini_text_ext.text_view.set_cursor_from_name(Some("text"));
+                }
+                _ => {
+                    gemini_text_ext
+                        .text_view
+                        .set_cursor_from_name(Some("pointer"));
+                }
             }
+            imp.hover_url.replace(link.unwrap_or_default());
+            self.notify("hover-url");
         }
 
         Ok(())

src/widgets/window.rs

@@ -43,6 +43,10 @@ pub mod imp {
         #[template_child]
         pub(crate) bottom_bar_revealer: TemplateChild<gtk::Revealer>,
         #[template_child]
+        pub(crate) url_status: TemplateChild<gtk::Label>,
+        #[template_child]
+        pub(crate) url_status_box: TemplateChild<gtk::Box>,
+        #[template_child]
         pub(crate) header_small: TemplateChild<gtk::WindowHandle>,
         #[template_child]
         pub(crate) squeezer: TemplateChild<adw::Squeezer>,
@@ -302,6 +306,13 @@ impl Window {
                 this.set_special_color_from_hash();
             }),
         );
+
+        let ctrl = gtk::EventControllerMotion::new();
+        imp.url_status_box.add_controller(&ctrl);
+        let url_status_box_clone = imp.url_status_box.clone();
+        ctrl.connect_motion(move |_, _, _| {
+            url_status_box_clone.set_visible(false);
+        });
     }
     fn setup_zoom_popover_item(&self) {
         let imp = self.imp();
@@ -390,6 +401,14 @@ impl Window {
                     Some(res.to_value())
                 })
                 .build(),
+                tab.bind_property("hover-url", &*imp.url_status, "label")
+                    .build(),
+                tab.bind_property("hover-url", &*imp.url_status_box, "visible")
+                    .transform_to(|_, v| {
+                        let v: &str = v.get().unwrap();
+                        Some((!v.is_empty()).to_value())
+                    })
+                    .build(),
             ]);
         };
     }