From 68c7a41906198df7e18917644a16b102c4507a93 Mon Sep 17 00:00:00 2001
From: Nadav Tasher <tashernadav@gmail.com>
Date: Tue, 24 Dec 2024 21:41:25 +0200
Subject: [PATCH] Revert "Hide and disable Keyboard Shortcuts, because they
 make using Ctrl-C to copy etc impossible."

This reverts commit 5606414f8975fa0f75642d2e3a6b48c7559186f9.
---
 client/components/sidebar/sidebar.jade | 8 ++++----
 client/components/sidebar/sidebar.js   | 2 --
 client/lib/keyboard.js                 | 5 -----
 3 files changed, 4 insertions(+), 11 deletions(-)

diff --git a/client/components/sidebar/sidebar.jade b/client/components/sidebar/sidebar.jade
index 31206299e..65c002a47 100644
--- a/client/components/sidebar/sidebar.jade
+++ b/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 &#10005;
     .sidebar-content.js-board-sidebar-content
       //a.hide-btn.js-hide-sidebar
diff --git a/client/components/sidebar/sidebar.js b/client/components/sidebar/sidebar.js
index 25dd1774f..f7e8f2b2b 100644
--- a/client/components/sidebar/sidebar.js
+++ b/client/components/sidebar/sidebar.js
@@ -123,11 +123,9 @@ BlazeComponent.extendComponent({
             location.reload();
           }
         },
-/*
         'click .js-shortcuts'() {
           FlowRouter.go('shortcuts');
         },
-*/
         'click .js-close-sidebar'() {
           Sidebar.toggle()
         },
diff --git a/client/lib/keyboard.js b/client/lib/keyboard.js
index c5fd98d7e..b84c1e494 100644
--- a/client/lib/keyboard.js
+++ b/client/lib/keyboard.js
@@ -1,6 +1,5 @@
 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).
 
@@ -256,11 +255,8 @@ Mousetrap.bind('n', evt => {
     evt.preventDefault();
   }
 });
-*/
-
 
 Template.keyboardShortcuts.helpers({
-/*
   mapping: [
     {
       keys: ['w'],
@@ -323,5 +319,4 @@ Template.keyboardShortcuts.helpers({
       action: 'remove-labels-multiselect'
     },
   ],
-*/
 });