Possible fix for copy text not working

Fix #1060 Auditors: @diracdeltas
brave · Mar 11, 2016 · bf04711 · bf04711 · diracdeltas · Mar 11, 2016
1 parent c6a180b
commit bf04711
Showing 1 changed file with 18 additions and 6 deletions.
diff --git a/app/menu.js b/app/menu.js
@@ -173,25 +173,35 @@ const init = (settingsState, args) => {
   const editSubmenu = [{
     label: 'Undo',
     accelerator: 'CmdOrCtrl+Z',
-    role: 'undo'
+    click: function (item, focusedWindow) {
+      focusedWindow.webContents.undo()
+    }
   }, {
     label: 'Redo',
     accelerator: 'Shift+CmdOrCtrl+Z',
-    role: 'redo'
+    click: function (item, focusedWindow) {
+      focusedWindow.webContents.redo()
+    }
   },
     CommonMenu.separatorMenuItem,
     {
       label: 'Cut',
       accelerator: 'CmdOrCtrl+X',
-      role: 'cut'
+      click: function (item, focusedWindow) {
+        focusedWindow.webContents.cut()
+      }
     }, {
       label: 'Copy',
       accelerator: 'CmdOrCtrl+C',
-      role: 'copy'
+      click: function (item, focusedWindow) {
+        focusedWindow.webContents.copy()
+      }
     }, {
       label: 'Paste',
       accelerator: 'CmdOrCtrl+V',
-      role: 'paste'
+      click: function (item, focusedWindow) {
+        focusedWindow.webContents.paste()
+      }
     }, {
       label: 'Paste without formatting',
       accelerator: 'Shift+CmdOrCtrl+V',
@@ -209,7 +219,9 @@ const init = (settingsState, args) => {
     }, {
       label: 'Select All',
       accelerator: 'CmdOrCtrl+A',
-      role: 'selectall'
+      click: function (item, focusedWindow) {
+        focusedWindow.webContents.selectAll()
+      }
     },
     CommonMenu.separatorMenuItem,
     CommonMenu.findOnPageMenuItem,