From 6633d0b4fb1900a68210c523eea466830cfc3234 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andr=C3=A9s=20Moya?= <andres.moya@kaleidos.net>
Date: Tue, 11 Jan 2022 09:53:38 +0100
Subject: [PATCH] :sparkles: Right click on frame title to open its menu

---
 CHANGES.md                                            |  1 +
 frontend/src/app/main/ui/workspace/effects.cljs       | 10 ----------
 .../src/app/main/ui/workspace/viewport/widgets.cljs   | 11 +++++++++++
 3 files changed, 12 insertions(+), 10 deletions(-)

diff --git a/CHANGES.md b/CHANGES.md
index 4ed4e6c35..89f5f6ca2 100644
--- a/CHANGES.md
+++ b/CHANGES.md
@@ -41,6 +41,7 @@
 - Allow import to continue from recoverable failures [#1412](https://github.com/penpot/penpot/issues/1412)
 - Improved behaviour on text options when not text is selected [Taiga #2390](https://tree.taiga.io/project/penpot/issue/2390)
 - Fix decimal numbers in export viewbox [Taiga #2290](https://tree.taiga.io/project/penpot/issue/2290)
+- Right click over artboard name to open its menu [Taiga #1679](https://tree.taiga.io/project/penpot/issue/1679)
 
 ### :arrow_up: Deps updates
 
diff --git a/frontend/src/app/main/ui/workspace/effects.cljs b/frontend/src/app/main/ui/workspace/effects.cljs
index 12515dc0d..667bb64b6 100644
--- a/frontend/src/app/main/ui/workspace/effects.cljs
+++ b/frontend/src/app/main/ui/workspace/effects.cljs
@@ -28,16 +28,6 @@
    (fn []
      (st/emit! (dws/change-hover-state id false)))))
 
-(defn use-context-menu
-  [shape]
-  (mf/use-callback
-   (mf/deps shape)
-   (fn [event]
-     (dom/prevent-default event)
-     (dom/stop-propagation event)
-     (let [position (dom/get-client-position event)]
-       (st/emit! (dw/show-shape-context-menu {:position position :shape shape}))))))
-
 (defn use-mouse-down
   [{:keys [id type blocked]}]
   (mf/use-callback
diff --git a/frontend/src/app/main/ui/workspace/viewport/widgets.cljs b/frontend/src/app/main/ui/workspace/viewport/widgets.cljs
index e910e095f..9a1e44087 100644
--- a/frontend/src/app/main/ui/workspace/viewport/widgets.cljs
+++ b/frontend/src/app/main/ui/workspace/viewport/widgets.cljs
@@ -111,6 +111,16 @@
           (st/emitf (dw/go-to-layout :layers)
                     (dw/start-rename-shape (:id frame))))
 
+        on-context-menu
+        (mf/use-callback
+          (mf/deps frame)
+          (fn [bevent]
+            (let [event    (.-nativeEvent bevent)
+                  position (dom/get-client-position event)]
+              (dom/prevent-default event)
+              (dom/stop-propagation event)
+              (st/emit! (dw/show-shape-context-menu {:position position :shape frame})))))
+
         on-pointer-enter
         (mf/use-callback
          (mf/deps (:id frame) on-frame-enter)
@@ -134,6 +144,7 @@
             :style {:fill (when selected? "var(--color-primary-dark)")}
             :on-mouse-down on-mouse-down
             :on-double-click on-double-click
+            :on-context-menu on-context-menu
             :on-pointer-enter on-pointer-enter
             :on-pointer-leave on-pointer-leave}
      (:name frame)]))