From 39e14d059e360e34ca4b5e20ca32f4d50330e3d2 Mon Sep 17 00:00:00 2001
From: Abdullah Atta <abdullahatta@streetwriters.co>
Date: Tue, 14 Feb 2023 11:43:05 +0500
Subject: [PATCH] desktop: only show `Copy` menu item if there's a selection

---
 apps/web/desktop/menu.js | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/apps/web/desktop/menu.js b/apps/web/desktop/menu.js
index 53291b962..777f8bbb9 100644
--- a/apps/web/desktop/menu.js
+++ b/apps/web/desktop/menu.js
@@ -90,14 +90,15 @@ function setupMenu() {
         })
       );
 
-    menu.append(
-      new MenuItem({
-        label: "Copy",
-        role: "copy",
-        enabled: params.selectionText.length > 0,
-        accelerator: "CommandOrControl+C"
-      })
-    );
+    if (params.selectionText.length) {
+      menu.append(
+        new MenuItem({
+          label: "Copy",
+          role: "copy",
+          accelerator: "CommandOrControl+C"
+        })
+      );
+    }
 
     if (params.isEditable)
       menu.append(