Hide and disable Keyboard Shortcuts, because they make using Ctrl-C to copy etc impossible.

Thanks to xet7 !

Fixes #5603,
fixes #2423,
fixes #1878,
fixes #931

client/components/sidebar/sidebar.jade

@@ -5,10 +5,10 @@ template(name="sidebar")
     //  title="{{showTongueTitle}}")
     //  i.fa.fa-navicon
     .sidebar-actions
-      .sidebar-shortcuts
-        a.board-header-btn.js-shortcuts(title="{{_ 'keyboard-shortcuts' }}")
-          i.fa.fa-keyboard-o
-          span {{_ 'keyboard-shortcuts' }}
+      //.sidebar-shortcuts
+      //  a.board-header-btn.js-shortcuts(title="{{_ 'keyboard-shortcuts' }}")
+      //    i.fa.fa-keyboard-o
+      //    span {{_ 'keyboard-shortcuts' }}
       a.sidebar-xmark.js-close-sidebar ✕
     .sidebar-content.js-board-sidebar-content
       //a.hide-btn.js-hide-sidebar

client/components/sidebar/sidebar.js

@@ -123,9 +123,11 @@ BlazeComponent.extendComponent({
             location.reload();
           }
         },
+/*
         'click .js-shortcuts'() {
           FlowRouter.go('shortcuts');
         },
+*/
         'click .js-close-sidebar'() {
           Sidebar.toggle()
         },

client/lib/keyboard.js

@@ -1,5 +1,6 @@
 import { ReactiveCache } from '/imports/reactiveCache';
 
+/*
 // XXX There is no reason to define these shortcuts globally, they should be
 // attached to a template (most of them will go in the `board` template).
 
@@ -255,8 +256,11 @@ Mousetrap.bind('n', evt => {
     evt.preventDefault();
   }
 });
+*/
+
 
 Template.keyboardShortcuts.helpers({
+/*
   mapping: [
     {
       keys: ['w'],
@@ -319,4 +323,5 @@ Template.keyboardShortcuts.helpers({
       action: 'remove-labels-multiselect'
     },
   ],
+*/
 });