fixed usertested bugs

src/main/java/edu/ntnu/idi/idatt2003/gruppe42/controller/PopupsController.java

@@ -6,6 +6,7 @@
 import edu.ntnu.idi.idatt2003.gruppe42.view.Popup;
 import java.util.List;
 import java.util.Optional;
+import javafx.application.Platform;
 import javafx.geometry.Point2D;
 import javafx.scene.control.Button;
 import javafx.scene.layout.Pane;
@@ -37,11 +38,14 @@ private void initPopup(Popup popup) {
     makeDraggable(popup);
     popup.getCloseButton().setOnAction(event -> hide(popup.getType()));
 
-    Pane root = popup.getRoot();
-    root.parentProperty().addListener((obs, oldP, newP) -> {
+    popup.getRoot().parentProperty().addListener((obs, oldP, newP) -> {
       if (newP instanceof Pane parent) {
-        parent.widthProperty().addListener((o, ov, nv) -> keepInBounds(popup));
-        parent.heightProperty().addListener((o, ov, nv) -> keepInBounds(popup));
+        parent.widthProperty().addListener((
+            o, ov, nv) ->
+            Platform.runLater(() -> keepInBounds(popup)));
+        parent.heightProperty().addListener((
+            o, ov, nv) ->
+            Platform.runLater(() -> keepInBounds(popup)));
       }
     });
   }
@@ -61,7 +65,7 @@ public boolean show(App type) {
         .findFirst()
         .ifPresent(popup -> {
           popup.show();
-          keepInBounds(popup);
+          Platform.runLater(() -> keepInBounds(popup));
         });
     audioManager.playSfx(Audio.OPEN_APP);
     return true;
@@ -105,7 +109,6 @@ private void makeDraggable(Popup popup) {
     popup.getHeader().setOnMousePressed(event -> {
       root.layoutXProperty().unbind();
       root.layoutYProperty().unbind();
-
       if (root.getParent() instanceof Pane parent) {
         Point2D local = parent.sceneToLocal(event.getSceneX(), event.getSceneY());
         offset[0] = local.getX() - root.getLayoutX();
@@ -119,15 +122,9 @@ private void makeDraggable(Popup popup) {
         Point2D local = parent.sceneToLocal(event.getSceneX(), event.getSceneY());
         double newX = local.getX() - offset[0];
         double newY = local.getY() - offset[1];
-
-        double width = root.getWidth() > 0 ? root.getWidth() : root.prefWidth(-1);
-        double height = root.getHeight() > 0 ? root.getHeight() : root.prefHeight(-1);
-
-        double maxX = Math.max(0, parent.getWidth() - width);
-        double maxY = Math.max(0, parent.getHeight() - height);
-
-        root.setLayoutX(Math.max(0, Math.min(newX, maxX)));
-        root.setLayoutY(Math.max(0, Math.min(newY, maxY)));
+        double[] clamped = clamp(root, parent, newX, newY);
+        root.setLayoutX(clamped[0]);
+        root.setLayoutY(clamped[1]);
       }
       event.consume();
     });
@@ -144,17 +141,29 @@ private void keepInBounds(Popup popup) {
       return;
     }
     if (root.getParent() instanceof Pane parent) {
-      double width = root.getWidth() > 0 ? root.getWidth() : root.prefWidth(-1);
-      double height = root.getHeight() > 0 ? root.getHeight() : root.prefHeight(-1);
-
-      double maxX = Math.max(0, parent.getWidth() - width);
-      double maxY = Math.max(0, parent.getHeight() - height);
-
-      root.setLayoutX(Math.max(0, Math.min(root.getLayoutX(), maxX)));
-      root.setLayoutY(Math.max(0, Math.min(root.getLayoutY(), maxY)));
+      double[] clamped = clamp(root, parent, root.getLayoutX(), root.getLayoutY());
+      root.setLayoutX(clamped[0]);
+      root.setLayoutY(clamped[1]);
     }
   }
 
+  /**
+   * Clamps the given x/y coordinates so the popup stays within its parent bounds.
+   *
+   * @param root the popup root pane
+   * @param parent the parent pane
+   * @param x the desired x position
+   * @param y the desired y position
+   * @return a two-element array containing the clamped x and y values
+   */
+  private double[] clamp(Pane root, Pane parent, double x, double y) {
+    double width = root.getWidth() > 0 ? root.getWidth() : root.prefWidth(-1);
+    double height = root.getHeight() > 0 ? root.getHeight() : root.prefHeight(-1);
+    double maxX = Math.max(0, parent.getWidth() - width);
+    double maxY = Math.max(0, parent.getHeight() - height);
+    return new double[]{Math.max(0, Math.min(x, maxX)), Math.max(0, Math.min(y, maxY))};
+  }
+
   /**
    * Adds multiple popups to the controller.
    *
@@ -181,8 +190,8 @@ public void addPopup(Popup popup) {
    * @return the matching popup, or null if none exists
    */
   public Popup getPopup(App type) {
-    Optional<Popup> match = popups.stream().filter(popup -> popup.getType()
-            .equals(type)).findFirst();
+    Optional<Popup> match = popups.stream().filter(
+        popup -> popup.getType().equals(type)).findFirst();
     return match.orElse(null);
   }
 

src/main/java/edu/ntnu/idi/idatt2003/gruppe42/controller/appcontrollers/MailController.java

@@ -35,7 +35,7 @@ public class MailController implements AppController {
   public MailController(TimeAndWeatherController timeAndWeatherController) {
     this.timeAndWeatherController = timeAndWeatherController;
     unreadCount = Bindings.createIntegerBinding(() ->
-            (int) messages.stream().filter(Message::isSeen).count(),
+            (int) messages.stream().filter(m -> !m.isSeen()).count(),
         messages
     );
 

src/main/java/edu/ntnu/idi/idatt2003/gruppe42/controller/appcontrollers/StockAppController.java

@@ -97,6 +97,9 @@ public StockAppController(
 
     stockApp.getQuantitySpinner().valueProperty().addListener(
         (obs, old, newValue) -> {
+          if (isWeekend) {
+            onMarketClosed.run();
+            }
           if (currentStock != null) {
             updateReceipt();
           }
@@ -199,7 +202,6 @@ private void updateTradeUi() {
     var spinner = stockApp.getQuantitySpinner();
 
     if (isWeekend) {
-      tradeButton.setDisable(true);
       tradeButton.getStyleClass().add("trade-button-weekend");
       tradeButton.setOpacity(0.45);
       spinner.setDisable(true);
@@ -253,7 +255,7 @@ private void handleTransaction() {
     }
 
     if (value > 0 && currentStock.getSalesPrice().compareTo(BigDecimal.ZERO) <= 0) {
-      return; // Buying bankrupt stock disabled
+      return;
     }
 
     BigDecimal quantity = new BigDecimal(Math.abs(value));
@@ -268,18 +270,16 @@ private void handleTransaction() {
     }
   }
 
-  /** Buys given stock. */
   private void handleBuy(final Stock stock, final BigDecimal quantity) throws Exception {
     Transaction transaction = marketController.getExchange()
-          .buy(stock.getSymbol(), quantity);
+        .buy(stock.getSymbol(), quantity);
     if (transaction != null) {
       transaction.commit(player);
       player.getPortfolio().addShare(transaction.getShare());
       player.updateStatus();
     }
   }
 
-  /** Sells given stock. */
   private void handleSell(final Stock stock, final BigDecimal quantity) {
     player.getPortfolio().getShares().stream()
         .filter(share -> share.getStock().getSymbol().equals(stock.getSymbol()))

src/main/java/edu/ntnu/idi/idatt2003/gruppe42/controller/viewcontrollers/DesktopComponentFactory.java

@@ -4,7 +4,6 @@
 import edu.ntnu.idi.idatt2003.gruppe42.controller.appcontrollers.MailController;
 import edu.ntnu.idi.idatt2003.gruppe42.model.App;
 import javafx.beans.binding.Bindings;
-import javafx.geometry.Insets;
 import javafx.geometry.Pos;
 import javafx.scene.Node;
 import javafx.scene.SnapshotParameters;
@@ -13,6 +12,7 @@
 import javafx.scene.input.ClipboardContent;
 import javafx.scene.input.Dragboard;
 import javafx.scene.input.TransferMode;
+import javafx.scene.layout.Pane;
 import javafx.scene.layout.Region;
 import javafx.scene.layout.StackPane;
 import javafx.scene.paint.Color;
@@ -23,20 +23,18 @@
 /**
  * Factory class for creating desktop user interface components.
  */
-public final class DesktopComponentFactory {
+public record DesktopComponentFactory(
+    PopupsController popupsController, MailController mailController) {
 
-  private final PopupsController popupsController;
-  private final MailController mailController;
+  private static final double BADGE_RADIUS = 10;
 
   /**
    * Constructs a DesktopComponentFactory.
    *
    * @param popupsController the popups controller
    * @param mailController the mail controller
    */
-  public DesktopComponentFactory(PopupsController popupsController, MailController mailController) {
-    this.popupsController = popupsController;
-    this.mailController = mailController;
+  public DesktopComponentFactory {
   }
 
   /**
@@ -50,53 +48,30 @@ public Node createAppButton(final App type) {
     button.getStyleClass().add("app-button");
     button.setMinSize(0, 0);
 
-    Node appNode = button;
+    button.parentProperty().addListener((obs, oldParent, newParent) ->
+        bindButtonSize(button, newParent));
 
-    if (type == App.MAIL) {
-      final StackPane container = new StackPane();
-
-      final Circle badge = new Circle(10, Color.RED);
-      Label countLabel = new Label();
-      countLabel.setTextFill(Color.WHITE);
-      countLabel.setFont(Font.font("System", FontWeight.BOLD, 10));
-      countLabel.textProperty().bind(mailController.unreadCountProperty().asString());
-
-      StackPane badgeUi = new StackPane(badge, countLabel);
-      badgeUi.setMouseTransparent(true);
-      badgeUi.setAlignment(Pos.CENTER);
-      badgeUi.visibleProperty().bind(mailController.unreadCountProperty().greaterThan(0));
+    popupsController.bindOpenButton(button, type);
+    setupDrag(button);
+    return button;
+  }
 
-      container.getChildren().addAll(button, badgeUi);
-      StackPane.setAlignment(badgeUi, Pos.TOP_RIGHT);
-      StackPane.setMargin(badgeUi, new Insets(-10, -10, 0, 0));
-
-      // Bind container size to button size so badge is aligned to button corners
-      container.prefWidthProperty().bind(button.prefWidthProperty());
-      container.prefHeightProperty().bind(button.prefHeightProperty());
-      container.maxWidthProperty().bind(button.prefWidthProperty());
-      container.maxHeightProperty().bind(button.prefHeightProperty());
-
-      appNode = container;
+  private void bindButtonSize(final Button button, final Object parent) {
+    if (parent instanceof Region region) {
+      button.prefWidthProperty().bind(
+          Bindings.min(
+              region.widthProperty().multiply(0.8),
+              region.heightProperty()
+          ).multiply(0.8)
+      );
+      button.prefHeightProperty().bind(button.prefWidthProperty());
+    } else {
+      button.prefWidthProperty().unbind();
+      button.prefHeightProperty().unbind();
     }
+  }
 
-    final Node finalNode = appNode;
-    finalNode.parentProperty().addListener((observable, oldParent, newParent) -> {
-      if (newParent instanceof Region region) {
-        button.prefWidthProperty().bind(
-            Bindings.min(
-                region.widthProperty().multiply(0.8),
-                region.heightProperty()
-            ).multiply(0.8)
-        );
-        button.prefHeightProperty().bind(button.prefWidthProperty());
-      } else {
-        button.prefWidthProperty().unbind();
-        button.prefHeightProperty().unbind();
-      }
-    });
-
-    popupsController.bindOpenButton(button, type);
-
+  private void setupDrag(final Button button) {
     button.setOnDragDetected(event -> {
       Dragboard dragboard = button.startDragAndDrop(TransferMode.MOVE);
       SnapshotParameters params = new SnapshotParameters();
@@ -109,12 +84,6 @@ public Node createAppButton(final App type) {
       dragboard.setContent(content);
       event.consume();
     });
-
-    if (type == App.MAIL) {
-      return appNode;
-    }
-
-    return button;
   }
 
   /**
@@ -141,4 +110,4 @@ public void configureCellAsDropTarget(final StackPane cell) {
       event.consume();
     });
   }
-}
+}

src/main/java/edu/ntnu/idi/idatt2003/gruppe42/controller/viewcontrollers/DesktopDialogController.java

@@ -114,7 +114,6 @@ private void wireWeekendRapport() {
    */
   public void showDebtWarning() {
     warningApp.configure(
-        "💔",
         "Starting Week in Debt",
         "Your balance is negative. Proceeding to next week will cost you a life.",
         "Stay on Sunday",
@@ -138,9 +137,8 @@ public void showDebtWarning() {
    */
   public void showMarketClosedWarning() {
     warningApp.configure(
-        "🔒",
+        "It's the WEEKEND!",
         "Market Closed",
-        "Trading is suspended on weekends.\nThe market reopens on Monday.",
         "Got it"
     );
     warningApp.getPrimaryButton().setOnAction(event -> dismissWarning());
@@ -152,7 +150,7 @@ public void showMarketClosedWarning() {
    */
   public void showInsufficientFundsWarning() {
     warningApp.configure(
-            "🥺",
+            "NO MORE MONEY!",
         "Insufficient Funds",
         "You don't have enough cash to complete this purchase.",
         "I'll be back"

src/main/java/edu/ntnu/idi/idatt2003/gruppe42/controller/viewcontrollers/StartViewController.java

@@ -97,9 +97,8 @@ public StartViewController(final Millions application, final StartView startView
     startView.getLoginButton().setOnAction(event -> {
       if (startView.getSelectedMode() == null) {
         warningApp.configure(
-            "⚠",
+            "OBS",
             "Difficulty Required",
-            "Choose how much starting money you want before jumping in.",
             "Got it"
         );
         warningApp.getPrimaryButton().setOnAction(e -> popupsController.hide(App.WARNING));
@@ -172,7 +171,6 @@ private boolean isValidName(final String value) {
    */
   private void showFileWarning() {
     warningApp.configure(
-        "⚠",
         "Invalid File",
         settingsAppController.getErrorMessage(),
         "Revert to default file"

src/main/java/edu/ntnu/idi/idatt2003/gruppe42/model/GameFactory.java

@@ -26,9 +26,9 @@ public static Player createPlayer(
       final String username,
       final Difficulty difficulty
   ) {
-    final BigDecimal easyMoney = new BigDecimal("1000");
-    final BigDecimal mediumMoney = new BigDecimal("100");
-    final BigDecimal hardMoney = new BigDecimal("0");
+    final BigDecimal easyMoney = new BigDecimal("10000");
+    final BigDecimal mediumMoney = new BigDecimal("1000");
+    final BigDecimal hardMoney = new BigDecimal("100");
 
     BigDecimal money = switch (difficulty) {
       case EASY -> easyMoney;