Properly handle keyboard events

2 files changed, 9 insertions, 7 deletions

diff --git a/src/ch/epfl/xblast/client/KeyboardEventHandler.java b/src/ch/epfl/xblast/client/KeyboardEventHandler.java
index a0d3848..2060959 100644
--- a/src/ch/epfl/xblast/client/KeyboardEventHandler.java
+++ b/src/ch/epfl/xblast/client/KeyboardEventHandler.java
@@ -37,7 +37,7 @@ public class KeyboardEventHandler extends KeyAdapter {
      * @param e event (pressed key)
      */
     @Override
-    public void keyTyped(KeyEvent e) {
+    public void keyPressed(KeyEvent e) {
         PlayerAction act = this.playerActionMap.get(e.getKeyCode());
 
         if (Objects.nonNull(act))
diff --git a/test/ch/epfl/xblast/simulation/GraphicalSimulation.java b/test/ch/epfl/xblast/simulation/GraphicalSimulation.java
index 8009d9f..9329cc7 100644
--- a/test/ch/epfl/xblast/simulation/GraphicalSimulation.java
+++ b/test/ch/epfl/xblast/simulation/GraphicalSimulation.java
@@ -11,6 +11,7 @@ import ch.epfl.xblast.server.Level;
 import ch.epfl.xblast.server.painter.BoardPainter;
 
 import javax.swing.*;
+import java.awt.*;
 import java.awt.event.KeyEvent;
 import java.util.Collections;
 import java.util.HashMap;
@@ -48,27 +49,28 @@ public class GraphicalSimulation extends Simulation {
         return GameStateDeserializer.deserialize(serializedGameState);
     }
 
-    private static JFrame buildFrame(XBlastComponent gui) {
+    private static JFrame buildFrame(Container content) {
         JFrame frame = new JFrame();
         frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
         frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
         frame.setVisible(true);
 
-        frame.setContentPane(gui);
+        frame.setContentPane(content);
         frame.pack();
         return frame;
     }
 
-    private void attachKeyboardHandler(JFrame frame) {
-        frame.addKeyListener(new KeyboardEventHandler(buildPlayerActionMap(), PLAYER_ACTION_CONSUMER));
-        frame.requestFocusInWindow();
+    private void attachKeyboardHandler(Component comp) {
+        comp.addKeyListener(new KeyboardEventHandler(buildPlayerActionMap(), PLAYER_ACTION_CONSUMER));
+        comp.requestFocusInWindow();
     }
 
     private final XBlastComponent gui;
 
     private GraphicalSimulation() {
         this.gui = new XBlastComponent();
-        attachKeyboardHandler(buildFrame(this.gui));
+        buildFrame(this.gui);
+        attachKeyboardHandler(this.gui);
     }
 
     @Override