fix review

Author: flexsurfer

src/status_im/ui/components/fast_image.cljs

@@ -7,14 +7,20 @@
   [react/view {:style (merge style {:flex 1 :justify-content :center :align-items :center})}
    child])
 
-(defn fast-image [props]
+(defn fast-image [_]
   (let [loaded? (reagent/atom false)
         error? (reagent/atom false)]
-    (fn []
+    (fn [props]
       [react/fast-image-class (merge
-                               {:on-error #(reset! error? true)
-                                :on-load #(reset! loaded? true)}
-                               props)
+                               props
+                               {:on-error (fn [e]
+                                            (when-let [on-error (:on-error props)]
+                                              (on-error e))
+                                            (reset! error? true))
+                                :on-load (fn [e]
+                                           (when-let [on-load (:on-load props)]
+                                             (on-load e))
+                                           (reset! loaded? true))})
        (when (or @error? (not @loaded?))
          [placeholder (:style props)
           (if @error?

src/status_im/ui/screens/chat/message/message.cljs

@@ -375,8 +375,16 @@
                                      :disabled      in-popover?}
           [react/view {:style               (style/image-message style-opts)
                        :accessibility-label :image-message}
+           (when (or (:error @dimensions) (not (:loaded @dimensions)))
+             [react/view
+              (merge (dissoc style-opts :opacity)
+                     {:flex 1 :align-items :center :justify-content :center :position :absolute})
+              (if (:error @dimensions)
+                [icons/icon :main-icons/cancel]
+                [react/activity-indicator {:animating true}])])
            [fast-image/fast-image {:style       (dissoc style-opts :outgoing)
                                    :on-load     (image-set-size dimensions)
+                                   :on-error    #(swap! dimensions assoc :error true)
                                    :source      {:uri uri}}]
            [react/view {:style (style/image-message-border style-opts)}]]]]))))
 

src/status_im/ui/screens/chat/styles/message/message.cljs

@@ -400,7 +400,8 @@
 
 (defn image-message-border [opts]
   (merge (image-message opts)
-         {:border-width     1
+         {:opacity          (:opacity opts)
+          :border-width     1
           :top              0
           :left             0
           :position         :absolute