From f900fa80b17e136b3b40508480f954ae8b6a3fc4 Mon Sep 17 00:00:00 2001
From: yggverse <yggverse@project>
Date: Sun, 3 Nov 2024 04:39:27 +0200
Subject: [PATCH] add last focus value holder

---
 .../item/page/navigation/request/widget.rs    | 25 ++++++++++++-------
 1 file changed, 16 insertions(+), 9 deletions(-)

diff --git a/src/app/browser/window/tab/item/page/navigation/request/widget.rs b/src/app/browser/window/tab/item/page/navigation/request/widget.rs
index 39c6cf6d..5eb8213c 100644
--- a/src/app/browser/window/tab/item/page/navigation/request/widget.rs
+++ b/src/app/browser/window/tab/item/page/navigation/request/widget.rs
@@ -54,17 +54,24 @@ impl Widget {
             action_page_reload.activate(None);
         });
 
-        gobject.connect_state_flags_changed(|this, state| {
-            // Select entire text on key release
-            // this behavior implemented in most web-browsers,
-            // to simply overwrite current request with new value
-            // Note:
-            // * Custom GestureClick is not option here, as GTK Entry already has default one
-            if state.contains(StateFlags::ACTIVE | StateFlags::FOCUS_WITHIN) {
-                // Continue on first focus event
-                if this.selection_bounds().is_none() {
+        gobject.connect_state_flags_changed({
+            // Define last focus state container
+            let has_focus = RefCell::new(false);
+            move |this, state| {
+                // Select entire text on first click release
+                // this behavior implemented in most web-browsers,
+                // to simply overwrite current request with new value
+                // Note:
+                // * Custom GestureClick is not an option here, as GTK Entry has default controller
+                // * This is experimental feature as does not follow native GTK behavior @TODO test
+                if !has_focus.take()
+                    && state.contains(StateFlags::ACTIVE | StateFlags::FOCUS_WITHIN)
+                    && this.selection_bounds().is_none()
+                {
                     this.select_region(0, this.text_length().into());
                 }
+                // Update last focus state
+                has_focus.replace(state.contains(StateFlags::FOCUS_WITHIN));
             }
         });