Fix MessageBoardUI scroll bug and Game logic menhir tile occupant pla…

…cing
Mw3y · May 7, 2024 · 94f2558 · 94f2558
1 parent 281a7f3
commit 94f2558
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/src/ch/epfl/chacun/gui/BoardUI.java b/src/ch/epfl/chacun/gui/BoardUI.java
@@ -138,6 +138,9 @@ public static Node create(int reach, ObservableValue<GameState> gameStateO,
         }
 
         container.setContent(gridPane);
+        // Center board
+        container.setVvalue(.5);
+        container.setHvalue(.5);
         return container;
     }
 

diff --git a/src/ch/epfl/chacun/gui/Main.java b/src/ch/epfl/chacun/gui/Main.java
@@ -96,7 +96,7 @@ public void start(Stage primaryStage) throws Exception {
                     gameStateO.set(stateAction.gameState());
                     tileToPlaceRotationP.set(Rotation.NONE);
 
-                    if (!gameStateO.get().lastTilePotentialOccupants().isEmpty()) {
+                    if (gameStateO.get().nextAction() == GameState.Action.OCCUPY_TILE) {
                         // Display potential occupants
                         Set<Occupant> occupantsToDisplay = new HashSet<>(visibleOccupantsP.get());
                         occupantsToDisplay.addAll(gameStateO.get().lastTilePotentialOccupants());

diff --git a/src/ch/epfl/chacun/gui/MessageBoardUI.java b/src/ch/epfl/chacun/gui/MessageBoardUI.java
@@ -65,7 +65,10 @@ public static Node create(ObservableValue<List<MessageBoard.Message>> messagesO,
                 wrapper.getChildren().add(message);
             });
             // Scroll to the last message
-            Platform.runLater(() -> container.setVvalue(1));
+            Platform.runLater(() -> {
+                container.setVvalue(1);
+                container.layout();
+            });
         });
 
         wrapper.setSpacing(MESSAGES_SPACING);