Merge pull request #1620 from penpot/fix_double_click

🐛 Fix problem with double click

CHANGES.md

@@ -36,6 +36,7 @@
 - Fixed handoff shadow type text [Taiga #2717](https://tree.taiga.io/project/penpot/issue/2717)
 - Fixed components get "dirty" marker when moved [Taiga #2764](https://tree.taiga.io/project/penpot/issue/2764)
 - Fixed cannot align objects in a group that is not part of a frame [Taiga #2762](https://tree.taiga.io/project/penpot/issue/2762)
+- Fix problem with double click on exit path editing [Taiga #2906](https://tree.taiga.io/project/penpot/issue/2906)
 
 ### :arrow_up: Deps updates
 

frontend/src/app/main/ui/workspace/viewport/actions.cljs

@@ -186,19 +186,23 @@
 
        (st/emit! (ms/->MouseEvent :double-click ctrl? shift? alt?))
 
-       (when (and (not drawing-path?) shape)
-         (cond frame?
-               (st/emit! (dw/select-shape id shift?))
+       ;; Emit asynchronously so the double click to exit shapes won't break
+       (timers/schedule
+        #(when (and (not drawing-path?) shape)
+           (cond
+             frame?
+             (st/emit! (dw/select-shape id shift?))
 
-               (and group? (> (count @hover-ids) 1))
-               (let [selected (get objects (second @hover-ids))]
-                 (reset! hover selected)
-                 (reset! hover-ids (into [] (rest @hover-ids)))
-                 (st/emit! (dw/select-shape (:id selected))))
+             (and group? (> (count @hover-ids) 1))
+             (let [selected (get objects (second @hover-ids))]
+               (reset! hover selected)
+               (reset! hover-ids (into [] (rest @hover-ids)))
 
-               (not= id edition)
-               (st/emit! (dw/select-shape id)
-                         (dw/start-editing-selected))))))))
+               (st/emit! (dw/select-shape (:id selected))))
+
+             (not= id edition)
+             (st/emit! (dw/select-shape id)
+                       (dw/start-editing-selected)))))))))
 
 (defn on-context-menu
   [hover hover-ids]