From 9bd358a1195d1a7491769ac23660c51dcd383f44 Mon Sep 17 00:00:00 2001 From: Andrey Antukh Date: Sun, 10 Apr 2016 18:05:04 +0300 Subject: [PATCH] Properly persist as int the grid axis on workspace settings lightbox. --- src/uxbox/ui/workspace/settings.cljs | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/src/uxbox/ui/workspace/settings.cljs b/src/uxbox/ui/workspace/settings.cljs index 87a725954..489d314cb 100644 --- a/src/uxbox/ui/workspace/settings.cljs +++ b/src/uxbox/ui/workspace/settings.cljs @@ -16,7 +16,8 @@ [uxbox.ui.lightbox :as lightbox] [uxbox.ui.colorpicker :as uucp] [uxbox.ui.workspace.base :as wb] - [uxbox.util.dom :as dom])) + [uxbox.util.dom :as dom] + [uxbox.util.data :refer (parse-int)])) ;; --- Lentes @@ -33,7 +34,8 @@ opts (merge (:options page) (deref local))] (letfn [(on-field-change [field event] - (let [value (dom/event->value event)] + (let [value (dom/event->value event) + value (parse-int value)] (swap! local assoc field value))) (on-color-change [color] (swap! local assoc :grid/color color)) @@ -42,6 +44,7 @@ (dom/checked?))] (swap! local assoc :grid/align checked?))) (on-submit [event] + (dom/prevent-default event) (let [page (assoc page :options opts)] (rs/emit! (udp/update-page-metadata page)) (lightbox/close!)))] @@ -52,14 +55,15 @@ [:input#grid-x.input-text {:placeholder "X px" :type "number" - :value (:grid/x-axis opts "2") + ;; TODO: put here the default from constants + :value (:grid/x-axis opts "10") :on-change (partial on-field-change :grid/x-axis) :min 1 ;;TODO check this value :max 100}] [:input#grid-y.input-text {:placeholder "Y px" :type "number" - :value (:grid/y-axis opts "2") + :value (:grid/y-axis opts "10") :on-change (partial on-field-change :grid/y-axis) :min 1 :max 100}]]