From 064f200774de80e307bd1c2c3191ab5b596157a6 Mon Sep 17 00:00:00 2001 From: Eva Marco Date: Fri, 4 Sep 2026 10:21:31 +0200 Subject: [PATCH] :bug: Fix crash pressing Ctrl+D with no shape selected (#11491) AI-assisted-by: claude-sonnet-5 --- frontend/src/app/main/data/workspace/variants.cljs | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/frontend/src/app/main/data/workspace/variants.cljs b/frontend/src/app/main/data/workspace/variants.cljs index 2d224facc2..a8d013012b 100644 --- a/frontend/src/app/main/data/workspace/variants.cljs +++ b/frontend/src/app/main/data/workspace/variants.cljs @@ -539,9 +539,10 @@ (let [objects (dsh/lookup-page-objects state) selected-ids (dsh/lookup-selected state) selected-shapes (map (d/getf objects) selected-ids) - add-new-variant? (every? ctc/is-variant? selected-shapes) + add-new-variant? (and (seq selected-shapes) (every? ctc/is-variant? selected-shapes)) undo-id (js/Symbol)] - (if add-new-variant? + (cond + add-new-variant? (rx/concat (rx/of (ev/event {::ev/name "add-new-variant" ::ev/origin "workspace:shortcut-duplicate"}) @@ -549,7 +550,12 @@ (add-new-variant (first selected-ids) false)) (rx/from (map #(add-new-variant % true) (rest selected-ids))) (rx/of (dwu/commit-undo-transaction undo-id))) - (rx/of (dws/duplicate-selected true))))))) + + (seq selected-ids) + (rx/of (dws/duplicate-selected true)) + + :else + (rx/empty)))))) (defn rename-variant "Rename the variant container and all components belonging to this variant"