From 1fd61d2af3c775f11ae84b3dfeea798e40675f94 Mon Sep 17 00:00:00 2001
From: Karol Baraniecki <karol@baraniecki.eu>
Date: Wed, 8 Mar 2023 17:59:17 +0100
Subject: [PATCH] BrickGame: Disallow manual piece movement while paused

This make the pause feature feel much better.
---
 Userland/Games/BrickGame/BrickGame.cpp | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/Userland/Games/BrickGame/BrickGame.cpp b/Userland/Games/BrickGame/BrickGame.cpp
index d84d5c8588..d05d5942a2 100644
--- a/Userland/Games/BrickGame/BrickGame.cpp
+++ b/Userland/Games/BrickGame/BrickGame.cpp
@@ -463,6 +463,18 @@ void BrickGame::timer_event(Core::TimerEvent&)
 
 void BrickGame::keydown_event(GUI::KeyEvent& event)
 {
+    switch (event.key()) {
+    case KeyCode::Key_Escape:
+    case KeyCode::Key_P:
+        m_brick_game->toggle_pause();
+        return;
+    default:
+        break;
+    }
+
+    if (m_brick_game->state() == Bricks::GameState::Paused)
+        return;
+
     Bricks::RenderRequest render_request { Bricks::RenderRequest::SkipRender };
     switch (event.key()) {
     case KeyCode::Key_A:
@@ -490,10 +502,6 @@ void BrickGame::keydown_event(GUI::KeyEvent& event)
     case KeyCode::Key_Space:
         render_request = m_brick_game->move_down_fast();
         break;
-    case KeyCode::Key_Escape:
-    case KeyCode::Key_P:
-        m_brick_game->toggle_pause();
-        break;
     default:
         event.ignore();
         break;