Rename utils.navigation to utils.user-actions and add docstring to clarify namespace use case

Author: Parveshdhull

Diff for: src/status_im/common/data_confirmation_sheet/view.cljs

@@ -4,13 +4,13 @@
     [react-native.core :as rn]
     [status-im.common.data-confirmation-sheet.style :as style]
     [utils.i18n :as i18n]
-    [utils.navigation :as navigation]
-    [utils.re-frame :as rf]))
+    [utils.re-frame :as rf]
+    [utils.user-actions :as user-actions]))
 
 (defn on-choice-callback
   [syncing-on-mobile-network?]
   (rf/dispatch [:network/set-syncing-on-mobile-network syncing-on-mobile-network?])
-  (navigation/hide-bottom-sheet))
+  (user-actions/hide-bottom-sheet))
 
 (def on-wifi-only (partial on-choice-callback false))
 (def on-mobile-and-wifi (partial on-choice-callback true))
@@ -19,7 +19,7 @@
   []
   (let [settings-drawer?           (= (rf/sub [:view-id]) :screen/settings.syncing)
         syncing-on-mobile-network? (rf/sub [:profile/syncing-on-mobile-network?])]
-    (rn/use-mount navigation/dismiss-keyboard)
+    (rn/use-mount user-actions/dismiss-keyboard)
     [:<>
      [quo/text
       {:weight              :semi-bold

Diff for: src/status_im/contexts/profile/settings/screens/messages/view.cljs

@@ -3,8 +3,8 @@
     [quo.core :as quo]
     [react-native.core :as rn]
     [utils.i18n :as i18n]
-    [utils.navigation :as navigation]
-    [utils.re-frame :as rf]))
+    [utils.re-frame :as rf]
+    [utils.user-actions :as user-actions]))
 
 (defn- open-blocked-users
   []
@@ -21,7 +21,7 @@
      [quo/page-nav
       {:background :blur
        :icon-name  :i/arrow-left
-       :on-press   navigation/navigate-back}]
+       :on-press   user-actions/navigate-back}]
      [quo/page-top {:title (i18n/label :t/messages)}]
      [quo/category
       {:label     (i18n/label :t/contacts)

Diff for: src/status_im/contexts/profile/settings/screens/syncing/view.cljs

@@ -3,8 +3,8 @@
     [quo.core :as quo]
     [status-im.common.data-confirmation-sheet.view :as data-confirmation-sheet]
     [utils.i18n :as i18n]
-    [utils.navigation :as navigation]
-    [utils.re-frame :as rf]))
+    [utils.re-frame :as rf]
+    [utils.user-actions :as user-actions]))
 
 (defn- open-paired-devices-list
   []
@@ -26,7 +26,7 @@
      [quo/page-nav
       {:background :blur
        :icon-name  :i/arrow-left
-       :on-press   navigation/navigate-back}]
+       :on-press   user-actions/navigate-back}]
      [quo/page-top {:title (i18n/label :t/syncing)}]
      [quo/category
       {:data      [{:title             (i18n/label :t/sync-and-backup)

Diff for: src/status_im/contexts/settings/privacy_and_security/view.cljs

@@ -9,8 +9,8 @@
     [status-im.contexts.settings.privacy-and-security.style :as style]
     [status-im.feature-flags :as ff]
     [utils.i18n :as i18n]
-    [utils.navigation :as navigation]
-    [utils.re-frame :as rf]))
+    [utils.re-frame :as rf]
+    [utils.user-actions :as user-actions]))
 
 (defn- setting-preview-privacy
   [preview-privacy? customization-color on-change]
@@ -68,7 +68,7 @@
       {:key        :header
        :background :blur
        :icon-name  :i/arrow-left
-       :on-press   navigation/navigate-back}]
+       :on-press   user-actions/navigate-back}]
      [quo/standard-title
       {:title               (i18n/label :t/privacy-and-security)
        :container-style     style/title-container

Diff for: src/status_im/contexts/wallet/swap/set_spending_cap/view.cljs

@@ -10,8 +10,8 @@
     [status-im.contexts.wallet.swap.set-spending-cap.style :as style]
     [utils.address :as address-utils]
     [utils.i18n :as i18n]
-    [utils.navigation :as navigation]
-    [utils.re-frame :as rf]))
+    [utils.re-frame :as rf]
+    [utils.user-actions :as user-actions]))
 
 (defn- swap-title
   [{:keys [pay-token-symbol pay-amount account provider]}]
@@ -209,7 +209,7 @@
       {:footer-container-padding 0
        :header                   [quo/page-nav
                                   {:icon-name           :i/close
-                                   :on-press            navigation/navigate-back
+                                   :on-press            user-actions/navigate-back
                                    :margin-top          8
                                    :background          :blur
                                    :accessibility-label :top-bar}]

Diff for: src/status_im/contexts/wallet/swap/setup_swap/view.cljs

@@ -6,8 +6,8 @@
             [status-im.contexts.wallet.common.utils :as utils]
             [status-im.contexts.wallet.swap.setup-swap.style :as style]
             [utils.i18n :as i18n]
-            [utils.navigation :as navigation]
-            [utils.re-frame :as rf]))
+            [utils.re-frame :as rf]
+            [utils.user-actions :as user-actions]))
 
 (defn- data-item
   [{:keys [title subtitle size subtitle-icon loading?]}]
@@ -65,7 +65,7 @@
                                            (set-pay-value #(subs % 0 (dec (count %)))))]
     [rn/view {:style style/container}
      [account-switcher/view
-      {:on-press      navigation/navigate-back
+      {:on-press      user-actions/navigate-back
        :icon-name     :i/arrow-left
        :margin-top    (safe-area/get-top)
        :switcher-type :select-account}]

Diff for: src/utils/navigation.cljs

@@ -0,0 +1,18 @@
+(ns utils.user-actions
+  "Provides a collection of highly reusable dispatch functions for user actions, including
+  navigation and other user-driven events. This namespace focuses on actions that directly
+  impact user interaction with the application, ensuring consistency and efficiency in handling
+  these interactions."
+  (:require [utils.re-frame :as rf]))
+
+(defn navigate-back
+  []
+  (rf/dispatch [:navigate-back]))
+
+(defn hide-bottom-sheet
+  []
+  (rf/dispatch [:hide-bottom-sheet]))
+
+(defn dismiss-keyboard
+  []
+  (rf/dispatch [:dismiss-keyboard]))