From c6314afbf5421ae52aec65023d58c495f2c99718 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Thorbj=C3=B8rn=20Lindeijer?= <bjorn@lindeijer.nl>
Date: Mon, 24 Apr 2023 14:49:27 +0200
Subject: [PATCH] Revert "Re-enabled Space for toggling layer visibility"

This reverts commit 0de0f20e304bbed4d8c52e4cc09586bb12d34c5a.

This shortcut was only provided for backwards compatibility, but it
often conflicts with using Space for panning the map view and since
359dcb93f3a4248e0380f9965ab9dd6b44308eb9 there are global shortcuts
available for toggling layer visility and locked status, making this
shortcut no longer necessary.

See issue #3672.
---
 src/tiled/layerdock.cpp | 14 --------------
 1 file changed, 14 deletions(-)

diff --git a/src/tiled/layerdock.cpp b/src/tiled/layerdock.cpp
index dab729761c8..19d5a43b4f0 100644
--- a/src/tiled/layerdock.cpp
+++ b/src/tiled/layerdock.cpp
@@ -24,7 +24,6 @@
 #include "layerdock.h"
 
 #include "actionmanager.h"
-#include "changelayer.h"
 #include "layer.h"
 #include "layermodel.h"
 #include "map.h"
@@ -478,8 +477,6 @@ void LayerView::contextMenuEvent(QContextMenuEvent *event)
 
 void LayerView::keyPressEvent(QKeyEvent *event)
 {
-    Layer *layer = mMapDocument ? mMapDocument->currentLayer() : nullptr;
-
     switch (event->key()) {
     case Qt::Key_Delete:
     case Qt::Key_Backspace:
@@ -488,17 +485,6 @@ void LayerView::keyPressEvent(QKeyEvent *event)
             return;
         }
         break;
-    case Qt::Key_Space:
-        if (layer) {
-            QUndoCommand *command = nullptr;
-            if (event->modifiers() & Qt::ControlModifier)
-                command = new SetLayerLocked(mMapDocument, { layer }, !layer->isLocked());
-            else
-                command = new SetLayerVisible(mMapDocument, { layer }, !layer->isVisible());
-            mMapDocument->undoStack()->push(command);
-            return;
-        }
-        break;
     }
 
     QTreeView::keyPressEvent(event);