From 7f50b56c265bfeff75b23bc2fc3d080efd93e79b Mon Sep 17 00:00:00 2001 From: Siddarth Kumar Date: Tue, 6 Sep 2022 13:27:39 +0530 Subject: [PATCH] fix icon missing regression (#13907) This was a tricky one :P removing un-necessary check of new-ui? this is a better solution The core issue was that when new-ui? was passed instead of the actual value just the key was passed --- src/quo/components/list/item.cljs | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/quo/components/list/item.cljs b/src/quo/components/list/item.cljs index 10e2c60e9f..85ab5759d4 100644 --- a/src/quo/components/list/item.cljs +++ b/src/quo/components/list/item.cljs @@ -12,6 +12,7 @@ [status-im.ui.components.icons.icons :as icons] [quo2.foundations.colors :as quo2.colors] [quo2.components.icon :as quo.icons] + [status-im.utils.config :as config] [quo.components.animated.pressable :as animated])) (defn themes [theme] @@ -77,14 +78,14 @@ children)) (defn icon-column - [{:keys [icon icon-bg-color icon-color size icon-container-style new-ui?]}] + [{:keys [icon icon-bg-color icon-color size icon-container-style]}] (when icon - (let [icon-size (size->icon-size size)] + (let [icon-size (size->icon-size size) + new-ui? @config/new-ui-enabled?] [rn/view {:style (or icon-container-style (:tiny spacing/padding-horizontal))} (cond (vector? icon) icon - (keyword? icon) (if new-ui? [quo.icons/icon icon {:container-style {:align-items :center @@ -270,8 +271,7 @@ :subtitle subtitle :subtitle-max-lines subtitle-max-lines :subtitle-secondary subtitle-secondary - :right-side-present? (or accessory chevron) - :new-ui? :new-ui?}] + :right-side-present? (or accessory chevron)}] [right-side {:chevron chevron :active active :disabled disabled