From 9336f0d71710304403a64c8361c0f4b8099bbac8 Mon Sep 17 00:00:00 2001
From: danieskj <danieskj@stud.ntnu.no>
Date: Thu, 14 May 2026 17:37:57 +0200
Subject: [PATCH] feat(PlayerPortfolioComponent): Change labels to show the
 right info

---
 .../edu/ntnu/idi/idatt/view/SceneFactory.java |  3 +++
 .../elements/PlayerPortfolioComponent.java    | 20 +++++++++----------
 .../idatt/view/components/ui/UIFactory.java   |  3 +--
 .../view/primary/portfolio/PortfolioView.java |  2 +-
 4 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/src/main/java/edu/ntnu/idi/idatt/view/SceneFactory.java b/src/main/java/edu/ntnu/idi/idatt/view/SceneFactory.java
index 02d4d91..cd14152 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/SceneFactory.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/SceneFactory.java
@@ -73,6 +73,9 @@ public static Parent createStartView() {
 
   }
   public static Parent createPortfolioView(){
+
+    mark(() -> createPortfolioView());
+    
     PortfolioModel model = new PortfolioModel();
     PortfolioView view = new PortfolioView();
     PortfolioController controller =  new PortfolioController(model);
diff --git a/src/main/java/edu/ntnu/idi/idatt/view/components/elements/PlayerPortfolioComponent.java b/src/main/java/edu/ntnu/idi/idatt/view/components/elements/PlayerPortfolioComponent.java
index e4cfa9d..7457d36 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/components/elements/PlayerPortfolioComponent.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/components/elements/PlayerPortfolioComponent.java
@@ -8,6 +8,8 @@
 import javafx.scene.layout.HBox;
 import javafx.scene.layout.VBox;
 
+import java.math.BigDecimal;
+import java.math.RoundingMode;
 import java.util.ArrayList;
 import java.util.Collections;
 
@@ -17,32 +19,28 @@ public PlayerPortfolioComponent(Portfolio portfolio){
         this.setMaxSize(Double.MAX_VALUE, 500);
         this.getStyleClass().add("light");
 
-        Label userTitle = new Label("Name Title");
-        Label invisibleTitle = new Label(" ");
+        Label userTitle = new Label(String.format("%s's Portfolio", UserSession.getInstance().getPlayer().getName()));
         Label netWorth = new Label();
         netWorth.textProperty().bind(
                 UserSession.getInstance().netWorthProperty().asString("Net Worth: %.2f $"));
-        Label percentageChange = new Label("Percentage change: ");
+        Label percentageChange = new Label("Percentage change: "+ UserSession.getInstance().netWorthProperty().doubleValue());
         Label playerStatus = new Label("Player status: "+ UserSession.getInstance().getPlayer().getStatus());
-        Label portfolioWorth = new Label("Portfolio net worth: ");
-        Label totalShares = new Label("Total shares owned: ");
-
+        Label totalShares = new Label("Total shares owned: "+ UserSession.getInstance().getPlayer().getPortfolio().getShares().size());
         ArrayList<Label> labels = new ArrayList<>();
-        Collections.addAll(labels, netWorth, percentageChange, playerStatus, portfolioWorth, totalShares);
+        Collections.addAll(labels, netWorth, percentageChange, playerStatus, totalShares);
         labels.forEach(e -> e.getStyleClass().add("portfolio-box-text"));
         userTitle.getStyleClass().add("portfolio-box-title");
-        invisibleTitle.getStyleClass().add("portfolio-box-title");
 
         UICompositor playerPortfolioComponent = new UICompositor.Builder()
                 .parent(new HBox())
                 .growWithAlignment(Pos.CENTER)
                 .wrap(new VBox())
-                .addAllContent(userTitle, netWorth, percentageChange, playerStatus)
+                .addAllContent(userTitle, netWorth, percentageChange)
                 .unwrap()
                 .filler()
                 .wrap(new VBox())
-                .growWithAlignment(Pos.CENTER)
-                .addAllContent(portfolioWorth, totalShares)
+                .growWithAlignment(Pos.BOTTOM_CENTER)
+                .addAllContent(playerStatus, totalShares)
                 .unwrap()
                 .build();
 
diff --git a/src/main/java/edu/ntnu/idi/idatt/view/components/ui/UIFactory.java b/src/main/java/edu/ntnu/idi/idatt/view/components/ui/UIFactory.java
index 6b1b3c7..ed6aeab 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/components/ui/UIFactory.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/components/ui/UIFactory.java
@@ -134,8 +134,7 @@ public static Parent createMenu(String title, List<String> buttonLables, ActionE
 
     UICompositor.Builder menuBuilder = new UICompositor.Builder()
         .parent(new VBox())
-        .growWithAlignment(Pos.TOP_CENTER)
-        .properties((menu) -> menu.setPrefSize(300, Double.MAX_VALUE))
+        .growWithAlignment(Pos.CENTER)
         .addContent(menuIcon);
 
     buttons.forEach(b -> menuBuilder.addContent(b));
diff --git a/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/PortfolioView.java b/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/PortfolioView.java
index 744fd96..184dd5b 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/PortfolioView.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/PortfolioView.java
@@ -26,7 +26,7 @@ public Parent createContent() {
         VBox.setVgrow(userBox, Priority.ALWAYS);
         userBox.setPadding(new Insets(20));
         ScrollPane stockBox = new ScrollPane();
-        stockBox.setPadding(new Insets(10, 10, 20, 20));
+        stockBox.setPadding(new Insets(0, 10, 0, 20));
         shareBox = new VBox(20);
 
         stockBox.setFitToWidth(true);