feat(client): make menu entries work in all cases

* clicking menu entry will trigger editor action or default behavior in the cases of copy, paste and selectAll depending on an input being focussed or not Related to #1027
camunda · Nov 29, 2018 · d770f51 · d770f51
1 parent 70b66b0
commit d770f51
Show file tree

Hide file tree

Showing 4 changed files with 33 additions and 30 deletions.
diff --git a/app/lib/menu/menu-builder.js b/app/lib/menu/menu-builder.js
@@ -277,12 +277,13 @@ class MenuBuilder {
     }
 
     builder.menu.append(new MenuItem({
-      label: menuItem.label,
       accelerator: menuItem.accelerator,
-      enabled: menuItem.enabled !== undefined ? menuItem.enabled : true,
       click: function() {
         app.emit('menu:action', menuItem.action, menuItem.options);
       },
+      enabled: menuItem.enabled !== undefined ? menuItem.enabled : true,
+      label: menuItem.label,
+      role: menuItem.role,
       submenu
     }));
   }

diff --git a/client/src/app/tabs/bpmn/getBpmnEditMenu.js b/client/src/app/tabs/bpmn/getBpmnEditMenu.js
@@ -8,18 +8,19 @@ import {
 
 function getCopyPasteEntries({
   copy,
-  paste
+  paste,
+  inputActive
 }) {
   return [{
     label: 'Copy',
-    accelerator: 'CommandOrControl+C',
     enabled: copy,
-    action: 'copy'
+    action: 'copy',
+    role: inputActive && 'copy'
   }, {
     label: 'Paste',
-    accelerator: 'CommandOrControl+V',
     enabled: paste,
-    action: 'paste'
+    action: 'paste',
+    role: inputActive && 'paste'
   }];
 }
 

diff --git a/client/src/app/tabs/dmn/getDmnEditMenu.js b/client/src/app/tabs/dmn/getDmnEditMenu.js
@@ -33,28 +33,26 @@ function getDecisionTableEntries({
     ], [
       {
         label: 'Add Clause',
-        submenu: [
-          [
-            {
-              label: 'Input',
-              action: 'addInput'
-            }, {
-              label: 'Output',
-              action: 'addOutput'
-            }
-          ], [
-            {
-              label: 'Left Of Selected',
-              enabled: hasSelection,
-              action: 'addClauseLeft'
-            },
-            {
-              label: 'Right Of Selected',
-              enabled: hasSelection,
-              action: 'addClauseRight'
-            }
-          ]
-        ]
+        submenu: [{
+          label: 'Input',
+          action: 'addInput'
+        }, {
+          label: 'Output',
+          action: 'addOutput'
+        },
+        {
+          type: 'separator'
+        },
+        {
+          label: 'Left Of Selected',
+          enabled: hasSelection,
+          action: 'addClauseLeft'
+        },
+        {
+          label: 'Right Of Selected',
+          enabled: hasSelection,
+          action: 'addClauseRight'
+        }]
       }, {
         label: 'Remove Clause',
         enabled: hasSelection,

diff --git a/client/src/app/tabs/getEditMenu.js b/client/src/app/tabs/getEditMenu.js
@@ -119,13 +119,16 @@ export function getCanvasEntries({
 }
 
 export function getSelectionEntries({
+  inputActive,
   removeSelected,
   selectAll
 }) {
   return [{
     label: 'Select All',
     accelerator: 'CommandOrControl+A',
-    action: 'selectElements'
+    enabled: selectAll,
+    action: 'selectElements',
+    role: inputActive && 'selectAll'
   }, {
     label: 'Remove Selected',
     accelerator: 'Delete',