From ab86d5238a72880f7f97b597bb0835904771c97e Mon Sep 17 00:00:00 2001 From: Pablo Alba Date: Fri, 21 Oct 2022 17:00:01 +0200 Subject: [PATCH 01/10] :bug: Fix boards name do not disappear in focus mode --- CHANGES.md | 1 + frontend/src/app/main/ui/workspace/viewport.cljs | 3 ++- frontend/src/app/main/ui/workspace/viewport/widgets.cljs | 8 ++++++-- 3 files changed, 9 insertions(+), 3 deletions(-) diff --git a/CHANGES.md b/CHANGES.md index a498157c7..84eeafa50 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -8,6 +8,7 @@ - Fix justify alignes text left [Taiga #4322](https://tree.taiga.io/project/penpot/issue/4322) - Fix text out of borders with "auto width" and center align [Taiga #4308](https://tree.taiga.io/project/penpot/issue/4308) - Fix auto-width for texts can make text appear stretched [Github #2482](https://github.com/penpot/penpot/issues/2482) +- Fix boards name do not disappear in focus mode [#4272](https://tree.taiga.io/project/penpot/issue/4272) ## 1.16.0-beta diff --git a/frontend/src/app/main/ui/workspace/viewport.cljs b/frontend/src/app/main/ui/workspace/viewport.cljs index ee4110124..3cf07759b 100644 --- a/frontend/src/app/main/ui/workspace/viewport.cljs +++ b/frontend/src/app/main/ui/workspace/viewport.cljs @@ -330,7 +330,8 @@ :show-artboard-names? show-artboard-names? :on-frame-enter on-frame-enter :on-frame-leave on-frame-leave - :on-frame-select on-frame-select}] + :on-frame-select on-frame-select + :focus focus}] (when show-prototypes? [:& widgets/frame-flows diff --git a/frontend/src/app/main/ui/workspace/viewport/widgets.cljs b/frontend/src/app/main/ui/workspace/viewport/widgets.cljs index 1d85295f6..42596402e 100644 --- a/frontend/src/app/main/ui/workspace/viewport/widgets.cljs +++ b/frontend/src/app/main/ui/workspace/viewport/widgets.cljs @@ -163,11 +163,15 @@ on-frame-enter (unchecked-get props "on-frame-enter") on-frame-leave (unchecked-get props "on-frame-leave") on-frame-select (unchecked-get props "on-frame-select") - frames (ctt/get-frames objects)] + frames (ctt/get-frames objects) + focus (unchecked-get props "focus")] [:g.frame-titles (for [frame frames] - (when (= (:parent-id frame) uuid/zero) + (when (and + (= (:parent-id frame) uuid/zero) + (or (empty? focus) + (contains? focus (:id frame)))) [:& frame-title {:key (dm/str "frame-title-" (:id frame)) :frame frame :selected? (contains? selected (:id frame)) From 02382b95f62de4e984bb8e851eefc8e2b7ee0fa7 Mon Sep 17 00:00:00 2001 From: Alejandro Alonso Date: Wed, 19 Oct 2022 14:36:04 +0200 Subject: [PATCH 02/10] :bug: Fix wrong validation text after interaction with 2 and more files --- CHANGES.md | 2 +- frontend/src/app/main/ui/dashboard/file_menu.cljs | 4 ++-- frontend/translations/en.po | 9 ++++++--- frontend/translations/es.po | 8 ++++++-- 4 files changed, 15 insertions(+), 8 deletions(-) diff --git a/CHANGES.md b/CHANGES.md index 84eeafa50..168bf5e1d 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -7,9 +7,9 @@ - Fix firefox changing layer color type is not applied [Taiga #4292](https://tree.taiga.io/project/penpot/issue/4292) - Fix justify alignes text left [Taiga #4322](https://tree.taiga.io/project/penpot/issue/4322) - Fix text out of borders with "auto width" and center align [Taiga #4308](https://tree.taiga.io/project/penpot/issue/4308) +- Fix wrong validation text after interaction with 2 and more files [Taiga #4276](https://tree.taiga.io/project/penpot/issue/4276) - Fix auto-width for texts can make text appear stretched [Github #2482](https://github.com/penpot/penpot/issues/2482) - Fix boards name do not disappear in focus mode [#4272](https://tree.taiga.io/project/penpot/issue/4272) - ## 1.16.0-beta ### :boom: Breaking changes & Deprecations diff --git a/frontend/src/app/main/ui/dashboard/file_menu.cljs b/frontend/src/app/main/ui/dashboard/file_menu.cljs index e7f70c225..ddb3f9dbb 100644 --- a/frontend/src/app/main/ui/dashboard/file_menu.cljs +++ b/frontend/src/app/main/ui/dashboard/file_menu.cljs @@ -81,12 +81,12 @@ on-duplicate (fn [_] (apply st/emit! (map dd/duplicate-file files)) - (st/emit! (dm/success (tr "dashboard.success-duplicate-file")))) + (st/emit! (dm/success (tr "dashboard.success-duplicate-file" (i18n/c (count files)))))) delete-fn (fn [_] (apply st/emit! (map dd/delete-file files)) - (st/emit! (dm/success (tr "dashboard.success-delete-file")))) + (st/emit! (dm/success (tr "dashboard.success-delete-file" (i18n/c (count files)))))) on-delete (fn [event] diff --git a/frontend/translations/en.po b/frontend/translations/en.po index c3422c650..6c669652a 100644 --- a/frontend/translations/en.po +++ b/frontend/translations/en.po @@ -620,7 +620,9 @@ msgstr "Show all files" #: src/app/main/ui/dashboard/file_menu.cljs msgid "dashboard.success-delete-file" -msgstr "Your file has been deleted successfully" +msgid_plural "dashboard.success-delete-file" +msgstr[0] "Your file has been deleted successfully" +msgstr[1] "Your files have been deleted successfully" #: src/app/main/ui/dashboard/project_menu.cljs msgid "dashboard.success-delete-project" @@ -628,7 +630,9 @@ msgstr "Your project has been deleted successfully" #: src/app/main/ui/dashboard/file_menu.cljs msgid "dashboard.success-duplicate-file" -msgstr "Your file has been duplicated successfully" +msgid_plural "dashboard.success-delete-file" +msgstr[0] "Your file has been duplicated successfully" +msgstr[1] "Your files have been duplicated successfully" #: src/app/main/ui/dashboard/project_menu.cljs msgid "dashboard.success-duplicate-project" @@ -4564,4 +4568,3 @@ msgstr "The font %s could not be loaded" msgid "errors.bad-font-plural" msgstr "The fonts %s could not be loaded" - diff --git a/frontend/translations/es.po b/frontend/translations/es.po index 529eb38ca..9258d52c2 100644 --- a/frontend/translations/es.po +++ b/frontend/translations/es.po @@ -635,7 +635,9 @@ msgstr "Ver todos los archivos" #: src/app/main/ui/dashboard/file_menu.cljs msgid "dashboard.success-delete-file" -msgstr "Tu archivo ha sido borrado con éxito" +msgid_plural "dashboard.success-delete-file" +msgstr[0] "Tu archivo ha sido borrado con éxito" +msgstr[1] "Tus archivos han sido borrado con éxito" #: src/app/main/ui/dashboard/project_menu.cljs msgid "dashboard.success-delete-project" @@ -643,7 +645,9 @@ msgstr "Tu proyecto ha sido borrado con éxito" #: src/app/main/ui/dashboard/file_menu.cljs msgid "dashboard.success-duplicate-file" -msgstr "Tu archivo ha sido duplicado con éxito" +msgid_plural "dashboard.success-delete-file" +msgstr[0] "Tu archivo ha sido duplicado con éxito" +msgstr[1] "Tus archivos han sido duplicado con éxito" #: src/app/main/ui/dashboard/project_menu.cljs msgid "dashboard.success-duplicate-project" From 821981e57918cc4585b788605a6efa13ab96538a Mon Sep 17 00:00:00 2001 From: Pablo Alba Date: Tue, 18 Oct 2022 12:30:38 +0200 Subject: [PATCH 03/10] :bug: Fix wrong email in the info message at change email --- CHANGES.md | 2 ++ .../src/app/main/ui/settings/change_email.cljs | 18 +++++++++++------- 2 files changed, 13 insertions(+), 7 deletions(-) diff --git a/CHANGES.md b/CHANGES.md index 168bf5e1d..8f785fa6a 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -10,6 +10,8 @@ - Fix wrong validation text after interaction with 2 and more files [Taiga #4276](https://tree.taiga.io/project/penpot/issue/4276) - Fix auto-width for texts can make text appear stretched [Github #2482](https://github.com/penpot/penpot/issues/2482) - Fix boards name do not disappear in focus mode [#4272](https://tree.taiga.io/project/penpot/issue/4272) +- Fix wrong email in the info message at change email [Taiga #4274](https://tree.taiga.io/project/penpot/issue/4274) + ## 1.16.0-beta ### :boom: Breaking changes & Deprecations diff --git a/frontend/src/app/main/ui/settings/change_email.cljs b/frontend/src/app/main/ui/settings/change_email.cljs index 14d69e1e5..468f18dde 100644 --- a/frontend/src/app/main/ui/settings/change_email.cljs +++ b/frontend/src/app/main/ui/settings/change_email.cljs @@ -52,20 +52,19 @@ (rx/throw error))) (defn- on-success - [form data] + [profile data] (if (:changed data) (st/emit! (du/fetch-profile) (modal/hide)) - (let [email (get-in @form [:clean-data :email-1]) - message (tr "notifications.validation-email-sent" email)] + (let [message (tr "notifications.validation-email-sent" (:email profile))] (st/emit! (dm/info message) (modal/hide))))) (defn- on-submit - [form _event] + [profile form _event] (let [params {:email (get-in @form [:clean-data :email-1])} mdata {:on-error (partial on-error form) - :on-success (partial on-success form)}] + :on-success (partial on-success profile)}] (st/emit! (du/request-email-change (with-meta params mdata))))) (mf/defc change-email-modal @@ -77,7 +76,12 @@ :validators [email-equality] :initial profile) on-close - (mf/use-callback #(st/emit! (modal/hide)))] + (mf/use-callback #(st/emit! (modal/hide))) + + on-submit + (mf/use-callback + (mf/deps profile) + (partial on-submit profile))] [:div.modal-overlay [:div.modal-container.change-email-modal.form-container @@ -86,7 +90,7 @@ [:div.modal-header [:div.modal-header-title - [:h2 {:data-test "change-email-title"} + [:h2 {:data-test "change-email-title"} (tr "modals.change-email.title")]] [:div.modal-close-button {:on-click on-close} i/close]] From 72f69050779a3378fc8fea2fee0750b8a215b4cc Mon Sep 17 00:00:00 2001 From: Andrey Antukh Date: Wed, 19 Oct 2022 11:17:30 +0200 Subject: [PATCH 04/10] :paperclip: Add commented code for easy debug release modal on ui ns --- CHANGES.md | 3 +++ common/src/app/common/pages/helpers.cljc | 4 ++++ .../app/main/ui/workspace/context_menu.cljs | 21 ++++++++++--------- .../ui/workspace/viewport/path_actions.cljs | 15 +++++++++---- frontend/src/app/util/path/tools.cljs | 17 ++++++++++++--- 5 files changed, 43 insertions(+), 17 deletions(-) diff --git a/CHANGES.md b/CHANGES.md index 8f785fa6a..10a3b0319 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -11,6 +11,9 @@ - Fix auto-width for texts can make text appear stretched [Github #2482](https://github.com/penpot/penpot/issues/2482) - Fix boards name do not disappear in focus mode [#4272](https://tree.taiga.io/project/penpot/issue/4272) - Fix wrong email in the info message at change email [Taiga #4274](https://tree.taiga.io/project/penpot/issue/4274) +- Fix transform to path RMB menu item is not relevant if shape is already path [Taiga #4302](https://tree.taiga.io/project/penpot/issue/4302) +- Fix join nodes icon is active when 2 already joined nodes are selected [Taiga #4370](https://tree.taiga.io/project/penpot/issue/4370) +- Fix path nodes panel. "To curve" and "To corner" icons are active if node is already curved/cornered [Taiga #4371](https://tree.taiga.io/project/penpot/issue/4371) ## 1.16.0-beta diff --git a/common/src/app/common/pages/helpers.cljc b/common/src/app/common/pages/helpers.cljc index 02e21f8c2..a07645f30 100644 --- a/common/src/app/common/pages/helpers.cljc +++ b/common/src/app/common/pages/helpers.cljc @@ -51,6 +51,10 @@ [{:keys [type]}] (= type :svg-raw)) +(defn path-shape? + [{:keys [type]}] + (= type :path)) + (defn unframed-shape? "Checks if it's a non-frame shape in the top level." [shape] diff --git a/frontend/src/app/main/ui/workspace/context_menu.cljs b/frontend/src/app/main/ui/workspace/context_menu.cljs index a21ac8451..bdf518466 100644 --- a/frontend/src/app/main/ui/workspace/context_menu.cljs +++ b/frontend/src/app/main/ui/workspace/context_menu.cljs @@ -265,18 +265,19 @@ (mf/defc context-menu-path [{:keys [shapes disable-flatten? disable-booleans?]}] - (let [multiple? (> (count shapes) 1) - single? (= (count shapes) 1) + (let [multiple? (> (count shapes) 1) + single? (= (count shapes) 1) - has-group? (->> shapes (d/seek cph/group-shape?)) - has-bool? (->> shapes (d/seek cph/bool-shape?)) - has-frame? (->> shapes (d/seek cph/frame-shape?)) + has-group? (->> shapes (d/seek cph/group-shape?)) + has-bool? (->> shapes (d/seek cph/bool-shape?)) + has-frame? (->> shapes (d/seek cph/frame-shape?)) + has-path? (->> shapes (d/seek cph/path-shape?)) - is-group? (and single? has-group?) - is-bool? (and single? has-bool?) - is-frame? (and single? has-frame?) + is-group? (and single? has-group?) + is-bool? (and single? has-bool?) + is-frame? (and single? has-frame?) - do-start-editing (fn [] (timers/schedule #(st/emit! (dw/start-editing-selected)))) + do-start-editing (fn [] (timers/schedule #(st/emit! (dw/start-editing-selected)))) do-transform-to-path #(st/emit! (dw/convert-selected-to-path)) make-do-bool @@ -296,7 +297,7 @@ :shortcut (sc/get-tooltip :start-editing) :on-click do-start-editing}]) - (when-not (or disable-flatten? has-frame?) + (when-not (or disable-flatten? has-frame? has-path?) [:& menu-entry {:title (tr "workspace.shape.menu.transform-to-path") :on-click do-transform-to-path}]) diff --git a/frontend/src/app/main/ui/workspace/viewport/path_actions.cljs b/frontend/src/app/main/ui/workspace/viewport/path_actions.cljs index 0cf7db8a8..b66d01848 100644 --- a/frontend/src/app/main/ui/workspace/viewport/path_actions.cljs +++ b/frontend/src/app/main/ui/workspace/viewport/path_actions.cljs @@ -17,15 +17,22 @@ (defn check-enabled [content selected-points] (let [segments (upt/get-segments content selected-points) + num-segments (count segments) num-points (count selected-points) points-selected? (seq selected-points) - segments-selected? (seq segments)] - {:make-corner points-selected? - :make-curve points-selected? + segments-selected? (seq segments) + ;; max segments for n points is (n × (n -1)) / 2 + max-segments (-> num-points + (* (- num-points 1)) + (/ 2)) + is-curve? (some #(upt/is-curve? content %) selected-points)] + + {:make-corner (and points-selected? is-curve?) + :make-curve (and points-selected? (not is-curve?)) :add-node segments-selected? :remove-node points-selected? :merge-nodes segments-selected? - :join-nodes (and points-selected? (>= num-points 2)) + :join-nodes (and points-selected? (>= num-points 2) (< num-segments max-segments)) :separate-nodes segments-selected?})) (mf/defc path-actions [{:keys [shape]}] diff --git a/frontend/src/app/util/path/tools.cljs b/frontend/src/app/util/path/tools.cljs index 2ba942815..a49cd7068 100644 --- a/frontend/src/app/util/path/tools.cljs +++ b/frontend/src/app/util/path/tools.cljs @@ -74,6 +74,13 @@ (assoc-in [:params :c2x] (:x h2)) (assoc-in [:params :c2y] (:y h2))))) +(defn is-curve? + [content point] + (let [handlers (-> (upc/content->handlers content) + (get point)) + handler-points (map #(upc/handler->point content (first %) (second %)) handlers)] + (some #(not= point %) handler-points))) + (defn make-curve-point "Changes the content to make the point a 'curve'. The handlers will be positioned in the same vector that results from te previous->next points but with fixed length." @@ -99,7 +106,6 @@ :next-p (upc/command->point next) :command cmd))))) - points (->> vectors (mapcat #(vector (:next-p %) (:prev-p %))) (remove nil?) (into #{}))] (cond @@ -124,8 +130,7 @@ next-correction (when (some? next-h) (gpt/scale (gpt/to-vec next-h point) (/ 1 3))) prev-h (when (some? prev-h) (gpt/add prev-h prev-correction)) - next-h (when (some? next-h) (gpt/add next-h next-correction)) - ] + next-h (when (some? next-h) (gpt/add next-h next-correction))] (cond-> content (and (= :line-to (:command cur-cmd)) (some? prev-p)) (update index upc/update-curve-to prev-p prev-h) @@ -147,7 +152,13 @@ (= :line-to (:command command)) (update index #(line->curve prev-p %)) + (= :curve-to (:command command)) + (update index #(line->curve prev-p %)) + (= :line-to (:command next-c)) + (update next-i #(line->curve point %)) + + (= :curve-to (:command next-c)) (update next-i #(line->curve point %))))] (->> vectors (reduce add-curve content)))))) From f4a0b304a972374c6795d100a79c4dde75743b83 Mon Sep 17 00:00:00 2001 From: Pablo Alba Date: Mon, 24 Oct 2022 16:26:49 +0200 Subject: [PATCH 05/10] :bug: Fix displaying comments settings are not applied via "Comments" menu drop-down on the top navbar on view mode --- CHANGES.md | 1 + frontend/src/app/main/ui/viewer/comments.cljs | 48 +++++++++---------- 2 files changed, 23 insertions(+), 26 deletions(-) diff --git a/CHANGES.md b/CHANGES.md index 10a3b0319..301fc7d29 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -14,6 +14,7 @@ - Fix transform to path RMB menu item is not relevant if shape is already path [Taiga #4302](https://tree.taiga.io/project/penpot/issue/4302) - Fix join nodes icon is active when 2 already joined nodes are selected [Taiga #4370](https://tree.taiga.io/project/penpot/issue/4370) - Fix path nodes panel. "To curve" and "To corner" icons are active if node is already curved/cornered [Taiga #4371](https://tree.taiga.io/project/penpot/issue/4371) +- Fix displaying comments settings are not applied via "Comments" menu drop-down on the top navbar on view mode [Taiga #4389](https://tree.taiga.io/project/penpot/issue/4389) ## 1.16.0-beta diff --git a/frontend/src/app/main/ui/viewer/comments.cljs b/frontend/src/app/main/ui/viewer/comments.cljs index b0fe50432..116b5dcce 100644 --- a/frontend/src/app/main/ui/viewer/comments.cljs +++ b/frontend/src/app/main/ui/viewer/comments.cljs @@ -6,7 +6,6 @@ (ns app.main.ui.viewer.comments (:require - [app.common.data :as d] [app.common.geom.matrix :as gmt] [app.common.geom.point :as gpt] [app.common.geom.shapes :as gsh] @@ -27,21 +26,26 @@ {::mf/wrap [mf/memo] ::mf/wrap-props false} [] - (let [local (mf/deref refs/comments-local) - owner-filter (:owner-filter local) - status-filter (:status-filter local) - show-sidebar? (:show-sidebar? local) + (let [{cmode :mode cshow :show show-sidebar? :show-sidebar?} (mf/deref refs/comments-local) show-dropdown? (mf/use-state false) toggle-dropdown (mf/use-fn #(swap! show-dropdown? not)) hide-dropdown (mf/use-fn #(reset! show-dropdown? false)) - update-option (mf/use-fn - (fn [event] - (let [target (dom/get-current-target event) - key (d/read-string (dom/get-attribute target "data-key")) - val (d/read-string (dom/get-attribute target "data-val"))] - (st/emit! (dcm/update-options {key val})))))] + update-mode + (mf/use-callback + (fn [mode] + (st/emit! (dcm/update-filters {:mode mode})))) + + update-show + (mf/use-callback + (fn [mode] + (st/emit! (dcm/update-filters {:show mode})))) + + update-options + (mf/use-callback + (fn [mode] + (st/emit! (dcm/update-options {:show-sidebar? mode}))))] [:div.view-options {:on-click toggle-dropdown} [:span.label (tr "labels.comments")] @@ -50,34 +54,26 @@ :on-close hide-dropdown} [:ul.dropdown.with-check - [:li {:class (dom/classnames :selected (= :all owner-filter)) - :data-key ":owner-filter" - :data-val ":all" - :on-click update-option} + [:li {:class (dom/classnames :selected (or (= :all cmode) (nil? cmode))) + :on-click #(update-mode :all)} [:span.icon i/tick] [:span.label (tr "labels.show-all-comments")]] - [:li {:class (dom/classnames :selected (= :yours owner-filter)) - :data-key ":owner-filter" - :data-val ":yours" - :on-click update-option} + [:li {:class (dom/classnames :selected (= :yours cmode)) + :on-click #(update-mode :yours)} [:span.icon i/tick] [:span.label (tr "labels.show-your-comments")]] [:hr] - [:li {:class (dom/classnames :selected (= :pending status-filter)) - :data-key ":status-filter" - :data-val (if (= :pending status-filter) ":all" ":pending") - :on-click update-option} + [:li {:class (dom/classnames :selected (= :pending cshow)) + :on-click #(update-show (if (= :pending cshow) :all :pending))} [:span.icon i/tick] [:span.label (tr "labels.hide-resolved-comments")]] [:hr] [:li {:class (dom/classnames :selected show-sidebar?) - :data-key ":show-sidebar?" - :data-val (if show-sidebar? "false" "true") - :on-click update-option} + :on-click #(update-options (not show-sidebar?))} [:span.icon i/tick] [:span.label (tr "labels.show-comments-list")]]]]])) From 6eb85b2c8cee8610dc234ba27ace75fda3b85f7f Mon Sep 17 00:00:00 2001 From: Eva Date: Wed, 26 Oct 2022 17:53:08 +0200 Subject: [PATCH 06/10] :bug: Fix nudge error --- CHANGES.md | 1 + backend/src/app/rpc/commands/auth.clj | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/CHANGES.md b/CHANGES.md index bb0ad1e42..e3d82334a 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -4,6 +4,7 @@ ### :bug: Bugs fixed +- Fix nudge error - Fix firefox changing layer color type is not applied [Taiga #4292](https://tree.taiga.io/project/penpot/issue/4292) - Fix justify alignes text left [Taiga #4322](https://tree.taiga.io/project/penpot/issue/4322) - Fix text out of borders with "auto width" and center align [Taiga #4308](https://tree.taiga.io/project/penpot/issue/4308) diff --git a/backend/src/app/rpc/commands/auth.clj b/backend/src/app/rpc/commands/auth.clj index 5ce869de5..3fc2527a4 100644 --- a/backend/src/app/rpc/commands/auth.clj +++ b/backend/src/app/rpc/commands/auth.clj @@ -287,7 +287,8 @@ props (-> (audit/extract-utm-params params) (merge (:props params)) (merge {:viewed-tutorial? false - :viewed-walkthrough? false}) + :viewed-walkthrough? false + :nudge {:big 10 :small 1}}) (db/tjson)) password (if-let [password (:password params)] From 21c9c205cb6a2cf4f06c420e3d3873ff24dff163 Mon Sep 17 00:00:00 2001 From: Eva Date: Thu, 27 Oct 2022 09:43:03 +0200 Subject: [PATCH 07/10] :bug: Add nudge by default --- frontend/src/app/main/data/workspace/transforms.cljs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/frontend/src/app/main/data/workspace/transforms.cljs b/frontend/src/app/main/data/workspace/transforms.cljs index 84ba8d689..0842f1885 100644 --- a/frontend/src/app/main/data/workspace/transforms.cljs +++ b/frontend/src/app/main/data/workspace/transforms.cljs @@ -697,7 +697,7 @@ stopper (->> move-events (rx/debounce 100) (rx/take 1)) - scale (if shift? (gpt/point (:big nudge)) (gpt/point (:small nudge))) + scale (if shift? (gpt/point (or (:big nudge) 10)) (gpt/point (or (:small nudge) 1))) mov-vec (gpt/multiply (get-displacement direction) scale)] (rx/concat From f276910ce3f10c8b81cd2c37ae3fbca8de90a47f Mon Sep 17 00:00:00 2001 From: Pablo Alba Date: Thu, 27 Oct 2022 11:58:28 +0200 Subject: [PATCH 08/10] :bug: fix bad behaviour on hovering and click nested artboards --- CHANGES.md | 1 + common/src/app/common/types/shape_tree.cljc | 8 ++++---- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/CHANGES.md b/CHANGES.md index 301fc7d29..0b2a3e964 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -15,6 +15,7 @@ - Fix join nodes icon is active when 2 already joined nodes are selected [Taiga #4370](https://tree.taiga.io/project/penpot/issue/4370) - Fix path nodes panel. "To curve" and "To corner" icons are active if node is already curved/cornered [Taiga #4371](https://tree.taiga.io/project/penpot/issue/4371) - Fix displaying comments settings are not applied via "Comments" menu drop-down on the top navbar on view mode [Taiga #4389](https://tree.taiga.io/project/penpot/issue/4389) +- Fix bad behaviour on hovering and click nested artboards[Taiga #4018](https://tree.taiga.io/project/penpot/issue/4018) and [Taiga #4269](https://tree.taiga.io/project/penpot/us/4269) ## 1.16.0-beta diff --git a/common/src/app/common/types/shape_tree.cljc b/common/src/app/common/types/shape_tree.cljc index 633e757a4..e0bcc9994 100644 --- a/common/src/app/common/types/shape_tree.cljc +++ b/common/src/app/common/types/shape_tree.cljc @@ -183,11 +183,11 @@ (let [type-a (dm/get-in objects [id-a :type]) type-b (dm/get-in objects [id-b :type])] (cond - (and bottom-frames? (= :frame type-a) (not= :frame type-b)) - 1 + (and (= :frame type-a) (not= :frame type-b)) + (if bottom-frames? 1 -1) - (and bottom-frames? (not= :frame type-a) (= :frame type-b)) - -1 + (and (not= :frame type-a) (= :frame type-b)) + (if bottom-frames? -1 1) (= id-a id-b) 0 From b669047f83eef04cac1aca36684f3daf0544b126 Mon Sep 17 00:00:00 2001 From: Andrey Antukh Date: Mon, 31 Oct 2022 10:29:35 +0100 Subject: [PATCH 09/10] :paperclip: Update changelog --- CHANGES.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CHANGES.md b/CHANGES.md index 0552e8501..702ad177a 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -4,7 +4,7 @@ ### :bug: Bugs fixed -- Fix nudge error +- Fix unexpected exception related to default nudge value - Fix firefox changing layer color type is not applied [Taiga #4292](https://tree.taiga.io/project/penpot/issue/4292) - Fix justify alignes text left [Taiga #4322](https://tree.taiga.io/project/penpot/issue/4322) - Fix text out of borders with "auto width" and center align [Taiga #4308](https://tree.taiga.io/project/penpot/issue/4308) From abbdd13b5dbe6fef73ac7032f62120c80f8b94fb Mon Sep 17 00:00:00 2001 From: Andrey Antukh Date: Mon, 31 Oct 2022 10:29:35 +0100 Subject: [PATCH 10/10] :paperclip: Update changelog --- CHANGES.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CHANGES.md b/CHANGES.md index 5b875992b..20a088067 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -4,7 +4,7 @@ ### :bug: Bugs fixed -- Fix nudge error +- Fix unexpected exception related to default nudge value - Fix firefox changing layer color type is not applied [Taiga #4292](https://tree.taiga.io/project/penpot/issue/4292) - Fix justify alignes text left [Taiga #4322](https://tree.taiga.io/project/penpot/issue/4322) - Fix text out of borders with "auto width" and center align [Taiga #4308](https://tree.taiga.io/project/penpot/issue/4308)