[#18453] fix: design feedbacks

status-im · Feb 7, 2024 · 9d45810 · 9d45810
1 parent 5754998
commit 9d45810
Show file tree

Hide file tree

Showing 5 changed files with 19 additions and 11 deletions.
diff --git a/src/quo/components/colors/color_picker/view.cljs b/src/quo/components/colors/color_picker/view.cljs
@@ -35,7 +35,7 @@
             (when (and @ref (>= index 0))
               (some-> ^js @ref
                       (.scrollToIndex #js
-                                       {:animated     true
+                                       {:animated     false
                                         :index        index
                                         :viewPosition 0.5})))))
         50)))

diff --git a/src/status_im/contexts/profile/edit/accent_colour/style.cljs b/src/status_im/contexts/profile/edit/accent_colour/style.cljs
@@ -11,20 +11,19 @@
 (def screen-container
   {:flex            1
    :padding-top     14
-   :padding-left    20
    :justify-content :space-between})
 
-(def padding-right
-  {:padding-right 20})
+(def padding-horizontal
+  {:padding-horizontal 20})
 
 (def color-title
   {:color         colors/white-70-blur
    :margin-top    20
    :margin-bottom 16})
 
 (def button-wrapper
-  {:margin-vertical 12
-   :padding-right   20})
+  {:margin-vertical    12
+   :padding-horizontal 20})
 
 (def profile-card
   {:margin-top    22

diff --git a/src/status_im/contexts/profile/edit/accent_colour/view.cljs b/src/status_im/contexts/profile/edit/accent_colour/view.cljs
@@ -34,7 +34,7 @@
           {:key   :content
            :style style/screen-container}
           [rn/view {:style {:flex 1}}
-           [rn/view {:style style/padding-right}
+           [rn/view {:style style/padding-horizontal}
             [quo/text-combinations {:title (i18n/label :t/accent-colour)}]
             [quo/profile-card
              {:profile-picture     profile-picture
@@ -50,6 +50,7 @@
             {:blur?            true
              :default-selected @unsaved-custom-color
              :on-change        on-change
+             :container-style  style/padding-horizontal
              :window-width     window-width}]]
           [rn/view {:style style/button-wrapper}
            [quo/button

diff --git a/src/status_im/navigation/options.cljs b/src/status_im/navigation/options.cljs
@@ -2,7 +2,8 @@
   (:require
     [quo.foundations.colors :as colors]
     [quo.theme :as quo.theme]
-    [react-native.platform :as platform]))
+    [react-native.platform :as platform]
+    [status-im.navigation.transitions :as transitions]))
 
 (defn default-options
   []
@@ -88,6 +89,13 @@
                              :orientation              ["portrait"]
                              :backgroundColor          :transparent}}))
 
+(def transparent-modal-screen-options
+  (merge
+   transparent-screen-options
+   {:animations (merge
+                 transitions/new-to-status-modal-animations
+                 transitions/push-animations-for-transparent-background)}))
+
 (def sheet-options
   {:layout                 {:componentBackgroundColor :transparent
                             :orientation              ["portrait"]

diff --git a/src/status_im/navigation/screens.cljs b/src/status_im/navigation/screens.cljs
@@ -172,15 +172,15 @@
      :component profiles/view}
 
     {:name      :edit-profile
-     :options   options/transparent-screen-options
+     :options   options/transparent-modal-screen-options
      :component edit-profile/view}
 
     {:name      :edit-accent-colour
-     :options   options/transparent-screen-options
+     :options   options/transparent-modal-screen-options
      :component edit-accent-colour/view}
 
     {:name      :edit-name
-     :options   options/transparent-screen-options
+     :options   options/transparent-modal-screen-options
      :component edit-name/view}
 
     {:name      :new-to-status