From 2660e914fc9bd845fea74e19cb90c88a48f5606e Mon Sep 17 00:00:00 2001 From: "alonso.torres" Date: Wed, 16 Sep 2020 12:13:49 +0200 Subject: [PATCH] :bug: Fixes problem with picker --- frontend/src/app/main/data/colors.cljs | 2 +- frontend/src/app/main/ui/workspace/colorpicker.cljs | 3 ++- frontend/src/app/main/ui/workspace/viewport.cljs | 2 +- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/frontend/src/app/main/data/colors.cljs b/frontend/src/app/main/data/colors.cljs index f946b1848..e06ac52a9 100644 --- a/frontend/src/app/main/data/colors.cljs +++ b/frontend/src/app/main/data/colors.cljs @@ -229,5 +229,5 @@ (assoc-in [:workspace-local :picking-color?] true) (assoc ::md/modal {:id (random-uuid) :type :colorpicker - :props {:on-close handle-change-color} + :props {:on-change handle-change-color} :allow-click-outside true})))))) diff --git a/frontend/src/app/main/ui/workspace/colorpicker.cljs b/frontend/src/app/main/ui/workspace/colorpicker.cljs index ff5da85a6..052f9bc29 100644 --- a/frontend/src/app/main/ui/workspace/colorpicker.cljs +++ b/frontend/src/app/main/ui/workspace/colorpicker.cljs @@ -401,7 +401,8 @@ (when (or (not= new-value value) (not= new-opacity opacity)) (reset! dirty? true)) (reset! last-change [new-value new-opacity op1 op2]) - (on-change new-value new-opacity op1 op2))] + (when on-change + (on-change new-value new-opacity op1 op2)))] (mf/use-effect (fn [] diff --git a/frontend/src/app/main/ui/workspace/viewport.cljs b/frontend/src/app/main/ui/workspace/viewport.cljs index 167ba7cdf..2fec8d0f2 100644 --- a/frontend/src/app/main/ui/workspace/viewport.cljs +++ b/frontend/src/app/main/ui/workspace/viewport.cljs @@ -224,7 +224,7 @@ ;; Everytime we finish retrieving a new URL we redraw the canvas ;; so even if we're not finished the user can start to pick basic ;; shapes - (mf/deps props fetch-pending) + (mf/deps fetch-pending) (fn [] (try (let [canvas-node (mf/ref-val canvas-ref)