Penpot/penpot
0
Fork 0
mirror of https://github.com/penpot/penpot.git synced 2025-02-13 18:48:37 -05:00
Code Issues Activity

✨ Add proper memoization to the generic shape component.

Browse source
This commit is contained in:
Andrey Antukh 2020-04-10 17:25:53 +02:00 • committed by Alonso Torres
parent 39807186df
commit a14073d170
1 changed files with 19 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

30
frontend/src/uxbox/main/ui/shapes/shape.cljs
View file

@ -21,22 +21,28 @@
[uxbox.main.ui.shapes.frame :as frame]
[uxbox.main.refs :as refs]))
(defn wrap-memo-shape
([component]
(mf/memo'
component
(fn [np op]
(let [n-shape (unchecked-get np "shape")
o-shape (unchecked-get op "shape")]
(= n-shape o-shape))))))
(defn- shape-wrapper-memo-equals?
[np op]
(let [n-shape (unchecked-get np "shape")
o-shape (unchecked-get op "shape")
n-frame (unchecked-get np "frame")
o-frame (unchecked-get op "frame")]
;; (prn "shape-wrapper-memo-equals?" (identical? n-frame o-frame))
(if (= (:type n-shape) :group)
false
(and (identical? n-shape o-shape)
(identical? n-frame o-frame)))))
(declare group-wrapper)
(declare frame-wrapper)
(mf/defc shape-wrapper
{::mf/wrap [wrap-memo-shape]}
[{:keys [shape frame] :as props}]
(let [opts #js {:shape shape :frame frame}]
{::mf/wrap [#(mf/memo' % shape-wrapper-memo-equals?)]
::mf/wrap-props false}
[props]
(let [shape (unchecked-get props "shape")
frame (unchecked-get props "frame")
opts #js {:shape shape :frame frame}]
(when (and shape (not (:hidden shape)))
(case (:type shape)
:group [:> group-wrapper opts]
@ -47,6 +53,8 @@
:path [:> path/path-wrapper opts]
:image [:> image/image-wrapper opts]
:circle [:> circle/circle-wrapper opts]
;; Only used when drawing a new frame.
:frame [:> frame-wrapper opts]
nil))))