From 4203b91e84fd6667ebad93a94e6280954447c6a1 Mon Sep 17 00:00:00 2001
From: yggverse <yggverse@project>
Date: Tue, 28 Jan 2025 15:13:11 +0200
Subject: [PATCH] separate update implementation

---
 .../window/tab/item/page/navigation.rs        | 11 ++-
 .../tab/item/page/navigation/request.rs       | 68 ++++++++++---------
 2 files changed, 46 insertions(+), 33 deletions(-)

diff --git a/src/app/browser/window/tab/item/page/navigation.rs b/src/app/browser/window/tab/item/page/navigation.rs
index 7ff5cdd5..3ae50976 100644
--- a/src/app/browser/window/tab/item/page/navigation.rs
+++ b/src/app/browser/window/tab/item/page/navigation.rs
@@ -23,9 +23,10 @@ const MARGIN: i32 = 6;
 const SPACING: i32 = 6;
 
 pub struct Navigation {
+    profile: Rc<Profile>,
     //home: Button,
     //reload: Button,
-    //bookmark: Button,
+    bookmark: Button,
     request: Entry,
     pub g_box: Box,
 }
@@ -62,10 +63,11 @@ impl Navigation {
         g_box.append(&bookmark);
 
         Self {
+            profile: profile.clone(),
             //home,
             request,
             //reload,
-            //bookmark,
+            bookmark,
             g_box,
         }
     }
@@ -135,6 +137,11 @@ impl Navigation {
         self.request.grab_focus()
     }
 
+    pub fn update(&self) {
+        self.bookmark.update(&self.profile, &self.request);
+        self.request.update(&self.profile);
+    }
+
     // Setters
 
     pub fn set_request(&self, value: &str) {
diff --git a/src/app/browser/window/tab/item/page/navigation/request.rs b/src/app/browser/window/tab/item/page/navigation/request.rs
index d7362c7b..41f5dd2b 100644
--- a/src/app/browser/window/tab/item/page/navigation/request.rs
+++ b/src/app/browser/window/tab/item/page/navigation/request.rs
@@ -40,6 +40,8 @@ pub trait Request {
         app_browser_window_tab_item_page_navigation_id: &i64,
     ) -> Result<(), String>;
 
+    fn update(&self, profile: &Profile);
+
     // Setters
 
     fn to_download(&self);
@@ -96,37 +98,7 @@ impl Request for Entry {
                     .set_enabled(uri(&this.text()).is_some_and(|uri| uri.path().len() > 1));
 
                 // Update primary icon
-                this.first_child().unwrap().remove_css_class("success"); // @TODO handle
-
-                this.set_primary_icon_activatable(false);
-                this.set_primary_icon_sensitive(false);
-
-                match primary_icon::from(&this.text()) {
-                    PrimaryIcon::Download { name, tooltip } => {
-                        this.set_primary_icon_name(Some(name));
-                        this.set_primary_icon_tooltip_text(Some(tooltip));
-                    }
-                    PrimaryIcon::Gemini { name, tooltip }
-                    | PrimaryIcon::Titan { name, tooltip } => {
-                        this.set_primary_icon_activatable(true);
-                        this.set_primary_icon_sensitive(true);
-                        this.set_primary_icon_name(Some(name));
-                        if profile.identity.get(&strip_prefix(this.text())).is_some() {
-                            this.first_child().unwrap().add_css_class("success"); // @TODO handle
-                            this.set_primary_icon_tooltip_text(Some(tooltip.1));
-                        } else {
-                            this.set_primary_icon_tooltip_text(Some(tooltip.0));
-                        }
-                    }
-                    PrimaryIcon::Search { name, tooltip } => {
-                        this.set_primary_icon_name(Some(name));
-                        this.set_primary_icon_tooltip_text(Some(tooltip));
-                    }
-                    PrimaryIcon::Source { name, tooltip } => {
-                        this.set_primary_icon_name(Some(name));
-                        this.set_primary_icon_tooltip_text(Some(tooltip));
-                    }
-                }
+                this.update(&profile)
             }
         });
 
@@ -235,6 +207,40 @@ impl Request for Entry {
         Ok(())
     }
 
+    fn update(&self, profile: &Profile) {
+        // Update primary icon
+        self.first_child().unwrap().remove_css_class("success"); // @TODO handle
+
+        self.set_primary_icon_activatable(false);
+        self.set_primary_icon_sensitive(false);
+
+        match primary_icon::from(&self.text()) {
+            PrimaryIcon::Download { name, tooltip } => {
+                self.set_primary_icon_name(Some(name));
+                self.set_primary_icon_tooltip_text(Some(tooltip));
+            }
+            PrimaryIcon::Gemini { name, tooltip } | PrimaryIcon::Titan { name, tooltip } => {
+                self.set_primary_icon_activatable(true);
+                self.set_primary_icon_sensitive(true);
+                self.set_primary_icon_name(Some(name));
+                if profile.identity.get(&strip_prefix(self.text())).is_some() {
+                    self.first_child().unwrap().add_css_class("success"); // @TODO handle
+                    self.set_primary_icon_tooltip_text(Some(tooltip.1));
+                } else {
+                    self.set_primary_icon_tooltip_text(Some(tooltip.0));
+                }
+            }
+            PrimaryIcon::Search { name, tooltip } => {
+                self.set_primary_icon_name(Some(name));
+                self.set_primary_icon_tooltip_text(Some(tooltip));
+            }
+            PrimaryIcon::Source { name, tooltip } => {
+                self.set_primary_icon_name(Some(name));
+                self.set_primary_icon_tooltip_text(Some(tooltip));
+            }
+        }
+    }
+
     // Setters
 
     fn to_download(&self) {