Fix mac copy user/pass shortcuts

buttercup · Oct 15, 2021 · 3cb0d3a · 3cb0d3a
1 parent 249244c
commit 3cb0d3a
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 7 deletions.
diff --git a/src/components/vault/EntriesList.js b/src/components/vault/EntriesList.js
@@ -28,8 +28,8 @@ const EntriesList = ({ className }) => {
     arrowUp: 'up',
     arrowDown: 'down',
     enter: 'enter',
-    copyUsername: 'ctrl+b',
-    copyPassword: 'ctrl+c'
+    copyUsername: ['ctrl+b', 'command+b'],
+    copyPassword: ['ctrl+c', 'command+c']
   };
   const handleNavigation = (event, step) => {
     event.preventDefault();
@@ -57,10 +57,14 @@ const EntriesList = ({ className }) => {
         document.activeElement.click();
       }
     },
-    copyUsername: () =>
-      handleCopyField(entries[currentIndex].fields.find(field => field.title === 'Username')),
-    copyPassword: () =>
-      handleCopyField(entries[currentIndex].fields.find(field => field.title === 'Password'))
+    copyUsername: e => {
+      e.preventDefault();
+      handleCopyField(entries[currentIndex].fields.find(field => field.title === 'Username'));
+    },
+    copyPassword: e => {
+      e.preventDefault();
+      handleCopyField(entries[currentIndex].fields.find(field => field.title === 'Password'));
+    }
   };
 
   return (

diff --git a/src/components/vault/EntryDetails.js b/src/components/vault/EntryDetails.js
@@ -713,7 +713,6 @@ const FieldRow = ({
             {field.property === 'url' ? (
               <a
                 onClick={e => {
-                  console.log(e);
                   (e.ctrlKey || e.metaKey) && window.open(field.value, '_blank');
                 }}
                 title="open with Ctrl/Cmd + click"