From bbf06ab97b2a5726cc456d8647b882ddf95ffd1f Mon Sep 17 00:00:00 2001
From: pawelsa <pawelsa@stud.ntnu.no>
Date: Tue, 26 May 2026 17:08:56 +0200
Subject: [PATCH 1/8] chore: Cleanup JavaDocs bad syntaxing.

---
 .../java/edu/ntnu/idi/idatt/model/enums/NewspaperEnum.java  | 2 +-
 src/main/java/edu/ntnu/idi/idatt/model/portfolio/Share.java | 3 ++-
 .../idi/idatt/model/transaction/TransactionArchive.java     | 3 ++-
 .../ntnu/idi/idatt/service/transaction/SaleCalculator.java  | 2 +-
 src/main/java/edu/ntnu/idi/idatt/session/UserSession.java   | 6 +++---
 src/main/java/edu/ntnu/idi/idatt/view/SceneFactory.java     | 2 +-
 .../java/edu/ntnu/idi/idatt/view/components/ui/UIAlert.java | 6 +++---
 .../edu/ntnu/idi/idatt/view/components/ui/UIFactory.java    | 3 ++-
 src/main/java/edu/ntnu/idi/idatt/view/entry/StartView.java  | 2 +-
 .../ntnu/idi/idatt/view/primary/exchange/ExchangeModel.java | 2 +-
 .../idi/idatt/view/primary/newspaper/NewspaperModel.java    | 2 +-
 .../idi/idatt/view/primary/newspaper/NewspaperView.java     | 2 +-
 .../idi/idatt/view/primary/portfolio/PortfolioModel.java    | 6 +++---
 .../view/primary/portfolio/sections/PlayerInfoSection.java  | 2 +-
 .../view/primary/portfolio/viewmodel/PlayerInfoModel.java   | 2 +-
 .../edu/ntnu/idi/idatt/view/primary/stock/StockModel.java   | 6 +++---
 .../view/primary/transactions/TransactionController.java    | 3 +--
 .../idatt/view/primary/transactions/TransactionModel.java   | 2 +-
 18 files changed, 29 insertions(+), 27 deletions(-)

diff --git a/src/main/java/edu/ntnu/idi/idatt/model/enums/NewspaperEnum.java b/src/main/java/edu/ntnu/idi/idatt/model/enums/NewspaperEnum.java
index 11b8c22..1c09db1 100644
--- a/src/main/java/edu/ntnu/idi/idatt/model/enums/NewspaperEnum.java
+++ b/src/main/java/edu/ntnu/idi/idatt/model/enums/NewspaperEnum.java
@@ -7,7 +7,7 @@
  * Creates static events that can occur within Newspaper.
  * </p>
  * 
- * @see Newspaper
+ * {@link Newspaper}
  */
 public enum NewspaperEnum {
   NEW_PRODUCT(
diff --git a/src/main/java/edu/ntnu/idi/idatt/model/portfolio/Share.java b/src/main/java/edu/ntnu/idi/idatt/model/portfolio/Share.java
index c828dd6..4be3d3a 100644
--- a/src/main/java/edu/ntnu/idi/idatt/model/portfolio/Share.java
+++ b/src/main/java/edu/ntnu/idi/idatt/model/portfolio/Share.java
@@ -77,7 +77,8 @@ public BigDecimal getTotalPurchasePrice() {
    * SaleCalculator.calculateGross() - getTotalPurchasePrice()
    * </p>
    * 
-   * @see Player (Net worth calculation)
+   * {@link Player}
+   * 
    * @return BigDecimal current profit.
    */
   public BigDecimal getProfit() {
diff --git a/src/main/java/edu/ntnu/idi/idatt/model/transaction/TransactionArchive.java b/src/main/java/edu/ntnu/idi/idatt/model/transaction/TransactionArchive.java
index a861539..b766616 100644
--- a/src/main/java/edu/ntnu/idi/idatt/model/transaction/TransactionArchive.java
+++ b/src/main/java/edu/ntnu/idi/idatt/model/transaction/TransactionArchive.java
@@ -106,7 +106,8 @@ public List<Sale> getSales() {
    * Used to calculate player statuses.
    * </p>
    * 
-   * @see Player
+   * {@link Player}
+   * 
    * @return int amount of distinct weeks.
    */
   public int countDistinctWeeks() {
diff --git a/src/main/java/edu/ntnu/idi/idatt/service/transaction/SaleCalculator.java b/src/main/java/edu/ntnu/idi/idatt/service/transaction/SaleCalculator.java
index bcbc17d..aba30e7 100644
--- a/src/main/java/edu/ntnu/idi/idatt/service/transaction/SaleCalculator.java
+++ b/src/main/java/edu/ntnu/idi/idatt/service/transaction/SaleCalculator.java
@@ -84,7 +84,7 @@ public BigDecimal calculateTotal() {
    * to the net worth calculation method.
    * </p>
    * 
-   * @see Player
+   * {@link Player}
    */
   public BigDecimal calculateProfit() {
     return calculateGross().subtract(purchasePrice.multiply(quantity));
diff --git a/src/main/java/edu/ntnu/idi/idatt/session/UserSession.java b/src/main/java/edu/ntnu/idi/idatt/session/UserSession.java
index 05bacf6..d68ebde 100644
--- a/src/main/java/edu/ntnu/idi/idatt/session/UserSession.java
+++ b/src/main/java/edu/ntnu/idi/idatt/session/UserSession.java
@@ -89,7 +89,7 @@ public void setExchange(Exchange exchange) {
   /**
    * Getter for moneyProperty.
    * 
-   * @return SimpleObjectProperty<BigDecimal>;
+   * @return {@code SimpleObjectProperty<BigDecimal>}
    */
   public SimpleObjectProperty<BigDecimal> moneyProperty() {
     return moneyProperty;
@@ -98,7 +98,7 @@ public SimpleObjectProperty<BigDecimal> moneyProperty() {
   /**
    * Getter for netWorthProperty.
    * 
-   * @return SimpleObjectProperty<BigDecimal>
+   * @return {@code SimpleObjectProperty<BigDecimal>}
    */
   public SimpleObjectProperty<BigDecimal> netWorthProperty() {
     return netWorthProperty;
@@ -126,7 +126,7 @@ public SimpleStringProperty statusProperty() {
    * Method used for updating the game state.
    * <p>
    * Updates the user interface model from diverse
-   * places through the code to synchronize model <-> UI.
+   * places through the code to synchronize model with UI.
    * Saves current session.
    * </p>
    */
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 154e11b..e04ded5 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/SceneFactory.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/SceneFactory.java
@@ -220,7 +220,7 @@ public static Parent createExchangeView() {
    * </p>
    * 
    * @param clean - should it reset the page stack.
-   * @rteturn View's root
+   * @return View's root
    */
   public static Parent createExchangeView(boolean clean) {
     return createDefaultExchangeView(clean);
diff --git a/src/main/java/edu/ntnu/idi/idatt/view/components/ui/UIAlert.java b/src/main/java/edu/ntnu/idi/idatt/view/components/ui/UIAlert.java
index a6d9556..f96b490 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/components/ui/UIAlert.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/components/ui/UIAlert.java
@@ -16,9 +16,9 @@ public class UIAlert {
   /**
    * Constructor for UIAlert.
    * 
-   * @param title  - Window name
-   * @param header - context
-   * @content - description of context
+   * @param title   - Window name
+   * @param header  - context
+   * @param content - description of context
    */
   public UIAlert(String title, String header, String content) {
     alert.setTitle(title);
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 06ba4c8..97e9120 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
@@ -7,6 +7,7 @@
 
 import edu.ntnu.idi.idatt.session.UserSession;
 import edu.ntnu.idi.idatt.view.SceneFactory;
+import edu.ntnu.idi.idatt.view.components.AbstractViewUI;
 import edu.ntnu.idi.idatt.view.components.elements.IconComponent;
 import edu.ntnu.idi.idatt.view.components.elements.SearchBarComponent;
 import edu.ntnu.idi.idatt.view.components.primitives.ActionEventHandler;
@@ -33,7 +34,7 @@
  * equal layouts across multiple views.
  * </p>
  * 
- * @see AbstractViewUI
+ * {@link AbstractViewUI}
  */
 public class UIFactory {
 
diff --git a/src/main/java/edu/ntnu/idi/idatt/view/entry/StartView.java b/src/main/java/edu/ntnu/idi/idatt/view/entry/StartView.java
index ece4e5a..3f05ecf 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/entry/StartView.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/entry/StartView.java
@@ -45,7 +45,7 @@ public StartView() {
   /**
    * Overriden method for createContent()
    * 
-   * @see AbstractView;
+   * @see AbstractView
    * @return root node.
    */
   @Override
diff --git a/src/main/java/edu/ntnu/idi/idatt/view/primary/exchange/ExchangeModel.java b/src/main/java/edu/ntnu/idi/idatt/view/primary/exchange/ExchangeModel.java
index f7bf431..ac4d1c4 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/primary/exchange/ExchangeModel.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/primary/exchange/ExchangeModel.java
@@ -17,7 +17,7 @@ public class ExchangeModel implements Model {
   /**
    * Getter for the stock list.
    * 
-   * @return ObservableList<StockComponent>;
+   * @return {@code ObservableList<StockComponent>}
    */
   public ObservableList<StockComponent> getStockList() {
     return stockList;
diff --git a/src/main/java/edu/ntnu/idi/idatt/view/primary/newspaper/NewspaperModel.java b/src/main/java/edu/ntnu/idi/idatt/view/primary/newspaper/NewspaperModel.java
index e5499be..0df8b7d 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/primary/newspaper/NewspaperModel.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/primary/newspaper/NewspaperModel.java
@@ -14,7 +14,7 @@ public class NewspaperModel implements Model {
   /**
    * Getter for newsList.
    * 
-   * @return ObservableList<Label>
+   * @return {@code ObservableList<Label>}
    */
   public ObservableList<Label> getNewsList() {
     return newsList;
diff --git a/src/main/java/edu/ntnu/idi/idatt/view/primary/newspaper/NewspaperView.java b/src/main/java/edu/ntnu/idi/idatt/view/primary/newspaper/NewspaperView.java
index b5ae773..d192119 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/primary/newspaper/NewspaperView.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/primary/newspaper/NewspaperView.java
@@ -75,7 +75,7 @@ public Parent createNavigation() {
    * Leaving empty, since it's not used.
    * </p>
    * 
-   * @see AbstractViewUI.
+   * @see AbstractViewUI
    */
   @Override
   public Parent createHeader() {
diff --git a/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/PortfolioModel.java b/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/PortfolioModel.java
index 1ea8088..0281fa4 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/PortfolioModel.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/PortfolioModel.java
@@ -22,7 +22,7 @@ public class PortfolioModel implements Model {
   /**
    * Getter for the shareList property.
    * 
-   * @return ObservableList<ShareComponent>;
+   * @return {@code ObservableList<ShareComponent>}
    */
   public ObservableList<ShareComponent> getShareList() {
     return shareList;
@@ -52,7 +52,7 @@ public PlayerInfoModel playerInfoModel() {
   /**
    * Getter for the view's root instance effects.
    * 
-   * @return SimpleObjectProperty<Effect>;
+   * @return {@code SimpleObjectProperty<Effect>}
    */
   public SimpleObjectProperty<Effect> instanceEffects() {
     return instanceEffects;
@@ -61,7 +61,7 @@ public SimpleObjectProperty<Effect> instanceEffects() {
   /**
    * Getter for the view's root instance children.
    * 
-   * @return SimpleObjectProperty<Node>;
+   * @return {@code SimpleObjectProperty<Node>}
    */
   public ObservableList<Node> instanceChildren() {
     return instanceChildren;
diff --git a/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/sections/PlayerInfoSection.java b/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/sections/PlayerInfoSection.java
index 2968661..c3f0c7e 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/sections/PlayerInfoSection.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/sections/PlayerInfoSection.java
@@ -19,7 +19,7 @@
  * This is a component of the PortfolioView.
  * </p>
  * 
- * @see PortfolioView
+ * {@link PortfolioView}
  */
 public class PlayerInfoSection extends VBox {
 
diff --git a/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/viewmodel/PlayerInfoModel.java b/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/viewmodel/PlayerInfoModel.java
index fd5202d..bcefad6 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/viewmodel/PlayerInfoModel.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/viewmodel/PlayerInfoModel.java
@@ -8,7 +8,7 @@
  * This is a component of the PortfolioModel.
  * </p>
  * 
- * @see PortfolioModel
+ * {@link PortfolioModel}
  */
 public class PlayerInfoModel {
 
diff --git a/src/main/java/edu/ntnu/idi/idatt/view/primary/stock/StockModel.java b/src/main/java/edu/ntnu/idi/idatt/view/primary/stock/StockModel.java
index 40b1330..864c8c3 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/primary/stock/StockModel.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/primary/stock/StockModel.java
@@ -26,7 +26,7 @@ public class StockModel implements Model {
   /**
    * Getter for graphNodes observable array list.
    * 
-   * @return ObservableList<Node>;
+   * @return {@code ObservableList<Node>}
    */
   public ObservableList<Node> getGraphNodes() {
     return this.graphNodes;
@@ -53,7 +53,7 @@ public TradeModel tradeModel() {
   /**
    * Getter for view's root instance effect property.
    * 
-   * @return SimpleObjectProperty<Effect>;
+   * @return {@code SimpleObjectProperty<Effect>}
    */
   public SimpleObjectProperty<Effect> instanceEffects() {
     return instanceEffects;
@@ -62,7 +62,7 @@ public SimpleObjectProperty<Effect> instanceEffects() {
   /**
    * Getter for view's root instance children property.
    * 
-   * @return ObservableList<Node>;
+   * @return {@code ObservableList<Node>}
    */
   public ObservableList<Node> instanceChildren() {
     return instanceChildren;
diff --git a/src/main/java/edu/ntnu/idi/idatt/view/primary/transactions/TransactionController.java b/src/main/java/edu/ntnu/idi/idatt/view/primary/transactions/TransactionController.java
index d9bbee1..1fa4867 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/primary/transactions/TransactionController.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/primary/transactions/TransactionController.java
@@ -81,10 +81,9 @@ public void setTransactionModel(List<Transaction> list) {
   /**
    * Method for handeling the searchbar.
    * <p>
-   * Sets transaction in the model based on query.
+   * Sets transaction in the model based on query property.
    * </p>
    * 
-   * @param query - Searchbar input.
    */
   public void handleSearchQuery() {
     if (model.searchQueryProperty().get() == null || model.searchQueryProperty().get().isBlank()) {
diff --git a/src/main/java/edu/ntnu/idi/idatt/view/primary/transactions/TransactionModel.java b/src/main/java/edu/ntnu/idi/idatt/view/primary/transactions/TransactionModel.java
index 8780c28..f699240 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/primary/transactions/TransactionModel.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/primary/transactions/TransactionModel.java
@@ -17,7 +17,7 @@ public class TransactionModel implements Model {
   /**
    * Getter for the transactionList.
    * 
-   * @return ObservableList<TransactionComponent>;
+   * @return {@code ObservableList<TransactionComponent>}
    */
   public ObservableList<TransactionComponent> getTransactionList() {
     return transactionList;

From 76b90c653add2913c54450bdcb190aea32d0451d Mon Sep 17 00:00:00 2001
From: pawelsa <pawelsa@stud.ntnu.no>
Date: Tue, 26 May 2026 17:27:26 +0200
Subject: [PATCH 2/8] feat: Add cursor hand to StockComponent.

---
 .../idi/idatt/view/components/elements/StockComponent.java    | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/main/java/edu/ntnu/idi/idatt/view/components/elements/StockComponent.java b/src/main/java/edu/ntnu/idi/idatt/view/components/elements/StockComponent.java
index 4bc55a9..6e956f1 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/components/elements/StockComponent.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/components/elements/StockComponent.java
@@ -8,6 +8,7 @@
 import edu.ntnu.idi.idatt.view.util.CssUtils;
 import javafx.geometry.Insets;
 import javafx.geometry.Pos;
+import javafx.scene.Cursor;
 import javafx.scene.control.Label;
 import javafx.scene.layout.HBox;
 import javafx.scene.layout.VBox;
@@ -74,6 +75,9 @@ public StockComponent(Stock stock) {
         .unwrap()
         .build();
 
+    // Add click cursor
+    this.setCursor(Cursor.HAND);
+
     this.getChildren().addAll(stockComponent.makeUI());
   }
 

From 5120ce083b4cebce85764cb093530596d51bc811 Mon Sep 17 00:00:00 2001
From: pawelsa <pawelsa@stud.ntnu.no>
Date: Tue, 26 May 2026 17:27:40 +0200
Subject: [PATCH 3/8] fix: Calculation bug of finish screen ROI.

---
 .../ntnu/idi/idatt/view/primary/finish/FinishController.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/main/java/edu/ntnu/idi/idatt/view/primary/finish/FinishController.java b/src/main/java/edu/ntnu/idi/idatt/view/primary/finish/FinishController.java
index 4ad3014..eec90c9 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/primary/finish/FinishController.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/primary/finish/FinishController.java
@@ -116,9 +116,9 @@ private BigDecimal calculateProfits() {
   private BigDecimal calculateROI() {
     BigDecimal profit = session.getPlayer().getMoney().subtract(
         session.getPlayer().getStartingMoney());
-    BigDecimal profitPercent = profit.divide(session.getPlayer().getStartingMoney()).setScale(2, RoundingMode.HALF_UP);
+    BigDecimal profitPercent = profit.divide(session.getPlayer().getStartingMoney());
 
-    BigDecimal returnOfInvestement = profitPercent.multiply(new BigDecimal("100"));
+    BigDecimal returnOfInvestement = profitPercent.multiply(new BigDecimal("100")).setScale(2, RoundingMode.HALF_UP);
 
     return returnOfInvestement;
   }

From 117ce6c4b1922d6509a49633689b97068d14add4 Mon Sep 17 00:00:00 2001
From: pawelsa <pawelsa@stud.ntnu.no>
Date: Tue, 26 May 2026 17:27:57 +0200
Subject: [PATCH 4/8] feat: Add newspaper glow and sort by news in portfolio
 MVC.

---
 .../portfolio/PortfolioController.java        | 41 +++++++++++++++++++
 .../view/primary/portfolio/PortfolioView.java |  2 +
 2 files changed, 43 insertions(+)

diff --git a/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/PortfolioController.java b/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/PortfolioController.java
index f72cc55..fe39f99 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/PortfolioController.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/PortfolioController.java
@@ -12,7 +12,14 @@
 import edu.ntnu.idi.idatt.view.components.elements.ShareComponent;
 import edu.ntnu.idi.idatt.view.components.ui.UIAlert;
 import edu.ntnu.idi.idatt.view.util.CssUtils;
+import javafx.animation.Animation;
+import javafx.animation.KeyFrame;
+import javafx.animation.KeyValue;
+import javafx.animation.Timeline;
+import javafx.scene.effect.DropShadow;
 import javafx.scene.effect.GaussianBlur;
+import javafx.scene.paint.Color;
+import javafx.util.Duration;
 
 import java.math.BigDecimal;
 import java.util.ArrayList;
@@ -31,6 +38,7 @@ public class PortfolioController extends AbstractController<PortfolioModel> {
    * </p>
    */
   public enum SortAction {
+    NEWS,
     OLDEST,
     NEWEST,
     PROFIT,
@@ -71,12 +79,38 @@ public void setShareModel(List<Share> shareList) {
     model.getShareList().clear();
     for (Share share : shareList) {
       ShareComponent shareComponent = new ShareComponent(share);
+
+      if (share.getStock().getNewspaper().hasNewNews()) {
+        applyNewspaperGlow(shareComponent);
+      }
+
       shareComponent.onShareSellButton((sellShare) -> holdingSale(sellShare));
       shareComponent.onShareClick((stockSymbol) -> SceneManager.switchTo(SceneFactory.createStockView(stockSymbol)));
       model.getShareList().add(shareComponent);
     }
   }
 
+  /**
+   * Method that applies glow effect to ShareComponent with new stock news.
+   * 
+   * @param component - The chosen ShareComponent.
+   */
+  public void applyNewspaperGlow(ShareComponent component) {
+    DropShadow glow = new DropShadow(7, Color.WHITE);
+
+    Timeline timeline = new Timeline(
+        new KeyFrame(Duration.ZERO,
+            new KeyValue(glow.radiusProperty(), 10)),
+        new KeyFrame(Duration.seconds(0.5),
+            new KeyValue(glow.radiusProperty(), 50)));
+
+    timeline.setAutoReverse(true);
+    timeline.setCycleCount(Animation.INDEFINITE);
+
+    component.setEffect(glow);
+    timeline.play();
+  }
+
   /**
    * Method for initializing controller dependencies.
    * <p>
@@ -249,6 +283,13 @@ public void sortSharesBy(SortAction action) {
 
     switch (action) {
 
+      case NEWS: {
+        sharesSorted.clear();
+        sharesSorted.addAll(session.getPlayer().getPortfolio().getShares()
+            .stream().filter(share -> share.getStock().getNewspaper().hasNewNews()).toList());
+        break;
+      }
+
       case OLDEST: {
         break;
       }
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 132dc30..b10d3a7 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
@@ -73,10 +73,12 @@ public Parent createHeader() {
     return UIFactory.createHeader("Search after holdings..",
         queryProperty -> searchQueryProperty = queryProperty,
         List.of(
+            "New News",
             "Oldest first",
             "Newest first",
             "Highest profit",
             "Highest loss"),
+        () -> sortHandle.accept(SortAction.NEWS),
         () -> sortHandle.accept(SortAction.OLDEST),
         () -> sortHandle.accept(SortAction.NEWEST),
         () -> sortHandle.accept(SortAction.PROFIT),

From 1ae529659575eb7097a1d97b4c1d99fb52fdb2b8 Mon Sep 17 00:00:00 2001
From: pawelsa <pawelsa@stud.ntnu.no>
Date: Tue, 26 May 2026 18:26:37 +0200
Subject: [PATCH 5/8] feat: Add animation to menu

---
 .../idatt/view/components/AbstractViewUI.java | 29 +++++++++++++++++--
 1 file changed, 27 insertions(+), 2 deletions(-)

diff --git a/src/main/java/edu/ntnu/idi/idatt/view/components/AbstractViewUI.java b/src/main/java/edu/ntnu/idi/idatt/view/components/AbstractViewUI.java
index 93a9574..028b77c 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/components/AbstractViewUI.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/components/AbstractViewUI.java
@@ -1,5 +1,8 @@
 package edu.ntnu.idi.idatt.view.components;
 
+import javafx.animation.KeyFrame;
+import javafx.animation.KeyValue;
+import javafx.animation.Timeline;
 import javafx.beans.property.SimpleBooleanProperty;
 import javafx.geometry.Pos;
 import javafx.scene.Parent;
@@ -9,6 +12,7 @@
 import javafx.scene.layout.Region;
 import javafx.scene.layout.StackPane;
 import javafx.scene.layout.VBox;
+import javafx.util.Duration;
 
 /**
  * Abstract View UI class.
@@ -59,12 +63,12 @@ public AbstractViewUI() {
     disableMenu.setVisible(false);
 
     disableMenu.setOnMouseClicked(e -> {
-      menu.setVisible(false);
+      animationMenu(false);
       disableMenu.setVisible(false);
     });
 
     isMenuVisible.addListener((observer, oldVal, newVal) -> {
-      menu.setVisible(true);
+      animationMenu(true);
       disableMenu.setVisible(true);
     });
 
@@ -165,4 +169,25 @@ public void toggleMenu() {
     isMenuVisible.set(!isMenuVisible.get());
   }
 
+  /**
+   * Method for animating the menu.
+   * 
+   * @param val - in = true, out = false
+   */
+  private void animationMenu(boolean val) {
+    Timeline animation;
+    if (val) {
+      menu.setVisible(true);
+      menu.setTranslateX(menu.getWidth()); // Fix issue from menu initialization
+      // (hide before starting to appear no matter what)
+      animation = new Timeline(new KeyFrame(Duration.millis(200),
+          new KeyValue(menu.translateXProperty(), 0)));
+    } else {
+      animation = new Timeline(new KeyFrame(Duration.millis(200),
+          new KeyValue(menu.translateXProperty(), menu.getWidth())));
+      animation.setOnFinished((e) -> menu.setVisible(false));
+    }
+    animation.play();
+  }
+
 }

From 2314fa1152dd5ac62c28e26623dc9109a1ab4a5b Mon Sep 17 00:00:00 2001
From: pawelsa <pawelsa@stud.ntnu.no>
Date: Tue, 26 May 2026 18:26:56 +0200
Subject: [PATCH 6/8] feat: Add animation methods to CssUtils

---
 .../ntnu/idi/idatt/view/util/CssUtils.java    | 48 +++++++++++++++++++
 1 file changed, 48 insertions(+)

diff --git a/src/main/java/edu/ntnu/idi/idatt/view/util/CssUtils.java b/src/main/java/edu/ntnu/idi/idatt/view/util/CssUtils.java
index 737d34f..c718c4c 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/util/CssUtils.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/util/CssUtils.java
@@ -2,7 +2,16 @@
 
 import java.math.BigDecimal;
 
+import javafx.animation.Animation;
+import javafx.animation.FadeTransition;
+import javafx.animation.KeyFrame;
+import javafx.animation.KeyValue;
+import javafx.animation.Timeline;
+import javafx.scene.Node;
 import javafx.scene.Parent;
+import javafx.scene.effect.DropShadow;
+import javafx.scene.paint.Color;
+import javafx.util.Duration;
 
 /**
  * Utility CSS class.
@@ -69,4 +78,43 @@ public static String generateValueColors(double value) {
     return value >= 0 ? GREEN : RED;
   }
 
+  /**
+   * Method that applies glow effect of new news in newspaper.
+   * 
+   * @param node - The node to animate.
+   */
+  public static void applyNewspaperGlow(Node node) {
+    DropShadow glow = new DropShadow(7, Color.WHITE);
+
+    Timeline timeline = new Timeline(
+        new KeyFrame(Duration.ZERO,
+            new KeyValue(glow.radiusProperty(), 10)),
+        new KeyFrame(Duration.seconds(0.5),
+            new KeyValue(glow.radiusProperty(), 50)));
+
+    timeline.setAutoReverse(true);
+    timeline.setCycleCount(Animation.INDEFINITE);
+
+    node.setEffect(glow);
+    timeline.play();
+  }
+
+  /**
+   * Method for creating a fade in animation.
+   *
+   * @param node - The node to animate
+   *
+   */
+  public static void fadeInAnimation(Node node) {
+    node.setOpacity(0.0);
+
+    FadeTransition fadeIn = new FadeTransition(Duration.seconds(0.5), node);
+    fadeIn.setFromValue(0.0);
+    fadeIn.setToValue(1.0);
+    fadeIn.setCycleCount(1);
+    fadeIn.setAutoReverse(false);
+
+    fadeIn.play();
+  }
+
 }

From 00fe9109d2c9ceece2603cd0527143fb64c329e9 Mon Sep 17 00:00:00 2001
From: pawelsa <pawelsa@stud.ntnu.no>
Date: Tue, 26 May 2026 18:27:22 +0200
Subject: [PATCH 7/8] chore: Unbind navigation to itself from menu

---
 .../idi/idatt/view/primary/transactions/TransactionView.java   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/main/java/edu/ntnu/idi/idatt/view/primary/transactions/TransactionView.java b/src/main/java/edu/ntnu/idi/idatt/view/primary/transactions/TransactionView.java
index 07a7d8c..0aef86f 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/primary/transactions/TransactionView.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/primary/transactions/TransactionView.java
@@ -108,7 +108,8 @@ public Parent createMenu() {
     return UIFactory.createMenu("Account",
         List.of("Portfolio", "Transactions"),
         () -> SceneManager.switchTo(SceneFactory.createPortfolioView()),
-        () -> SceneManager.switchTo(SceneFactory.createTransactionView()));
+        () -> {
+        });
 
   }
 

From ed191643435835bb82763d9de51e0f0ca20225e2 Mon Sep 17 00:00:00 2001
From: pawelsa <pawelsa@stud.ntnu.no>
Date: Tue, 26 May 2026 18:27:56 +0200
Subject: [PATCH 8/8] feat: Add fade in animations for asthetics

---
 .../idatt/view/components/ui/UIFactory.java   |  5 +++-
 .../ntnu/idi/idatt/view/entry/StartView.java  |  3 +++
 .../primary/exchange/ExchangeController.java  | 24 ++---------------
 .../portfolio/PortfolioController.java        | 26 +++----------------
 .../view/primary/stock/StockController.java   |  1 +
 .../transactions/TransactionController.java   |  7 ++++-
 6 files changed, 20 insertions(+), 46 deletions(-)

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 97e9120..25ed060 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
@@ -192,7 +192,10 @@ private static Parent createDefaultNavigation(Label titleLabel, List<String> but
         .properties(nav -> nav.setPadding(new Insets(40)))
         .addContent(titleLabel);
 
-    buttons.forEach(b -> navigationBuilder.addContent(b));
+    buttons.forEach(b -> {
+      navigationBuilder.addContent(b);
+      CssUtils.fadeInAnimation(b);
+    });
 
     navigationBuilder
         .filler();
diff --git a/src/main/java/edu/ntnu/idi/idatt/view/entry/StartView.java b/src/main/java/edu/ntnu/idi/idatt/view/entry/StartView.java
index 3f05ecf..7b51f10 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/entry/StartView.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/entry/StartView.java
@@ -3,6 +3,7 @@
 import java.util.List;
 
 import edu.ntnu.idi.idatt.view.components.AbstractView;
+import edu.ntnu.idi.idatt.view.util.CssUtils;
 import javafx.geometry.Insets;
 import javafx.geometry.Pos;
 import javafx.scene.Parent;
@@ -145,6 +146,8 @@ public void setModel(StartModel model) {
     this.errorLabel.textProperty().bind(model.getError());
     this.fileLabel.textProperty().bind(model.getFileName());
     this.newGameWrapper.visibleProperty().bind(model.isNewGame());
+    this.newGameWrapper.visibleProperty()
+        .addListener((obs) -> this.newGameWrapper.getChildren().forEach(c -> CssUtils.fadeInAnimation(c)));
     this.csvPredefinedCheckBox.selectedProperty().bindBidirectional(model.isPredefinedCSV());
 
     // Binding importCsvWrapper visibleProperty here for convenience
diff --git a/src/main/java/edu/ntnu/idi/idatt/view/primary/exchange/ExchangeController.java b/src/main/java/edu/ntnu/idi/idatt/view/primary/exchange/ExchangeController.java
index cd317d1..af9df1b 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/primary/exchange/ExchangeController.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/primary/exchange/ExchangeController.java
@@ -9,6 +9,7 @@
 import edu.ntnu.idi.idatt.view.SceneManager;
 import edu.ntnu.idi.idatt.view.components.AbstractController;
 import edu.ntnu.idi.idatt.view.components.elements.StockComponent;
+import edu.ntnu.idi.idatt.view.util.CssUtils;
 import javafx.animation.Animation;
 import javafx.animation.KeyFrame;
 import javafx.animation.KeyValue;
@@ -80,7 +81,7 @@ public void setStocksModel(List<Stock> stockList) {
       StockComponent stockComponent = new StockComponent(stock);
 
       if (stock.getNewspaper().hasNewNews()) {
-        applyNewspaperGlow(stockComponent);
+        CssUtils.applyNewspaperGlow(stockComponent);
       }
 
       stockComponent.onStockClick((symbol) -> redirectView(symbol));
@@ -88,27 +89,6 @@ public void setStocksModel(List<Stock> stockList) {
     }
   }
 
-  /**
-   * Method that applies glow effect to StockComponent with new news.
-   * 
-   * @param component - The chosen StockComponent.
-   */
-  public void applyNewspaperGlow(StockComponent component) {
-    DropShadow glow = new DropShadow(7, Color.WHITE);
-
-    Timeline timeline = new Timeline(
-        new KeyFrame(Duration.ZERO,
-            new KeyValue(glow.radiusProperty(), 10)),
-        new KeyFrame(Duration.seconds(0.5),
-            new KeyValue(glow.radiusProperty(), 50)));
-
-    timeline.setAutoReverse(true);
-    timeline.setCycleCount(Animation.INDEFINITE);
-
-    component.setEffect(glow);
-    timeline.play();
-  }
-
   /**
    * Method to handle searchbar query.
    * 
diff --git a/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/PortfolioController.java b/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/PortfolioController.java
index fe39f99..3279774 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/PortfolioController.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/primary/portfolio/PortfolioController.java
@@ -81,36 +81,17 @@ public void setShareModel(List<Share> shareList) {
       ShareComponent shareComponent = new ShareComponent(share);
 
       if (share.getStock().getNewspaper().hasNewNews()) {
-        applyNewspaperGlow(shareComponent);
+        CssUtils.applyNewspaperGlow(shareComponent);
       }
 
+      CssUtils.fadeInAnimation(shareComponent);
+
       shareComponent.onShareSellButton((sellShare) -> holdingSale(sellShare));
       shareComponent.onShareClick((stockSymbol) -> SceneManager.switchTo(SceneFactory.createStockView(stockSymbol)));
       model.getShareList().add(shareComponent);
     }
   }
 
-  /**
-   * Method that applies glow effect to ShareComponent with new stock news.
-   * 
-   * @param component - The chosen ShareComponent.
-   */
-  public void applyNewspaperGlow(ShareComponent component) {
-    DropShadow glow = new DropShadow(7, Color.WHITE);
-
-    Timeline timeline = new Timeline(
-        new KeyFrame(Duration.ZERO,
-            new KeyValue(glow.radiusProperty(), 10)),
-        new KeyFrame(Duration.seconds(0.5),
-            new KeyValue(glow.radiusProperty(), 50)));
-
-    timeline.setAutoReverse(true);
-    timeline.setCycleCount(Animation.INDEFINITE);
-
-    component.setEffect(glow);
-    timeline.play();
-  }
-
   /**
    * Method for initializing controller dependencies.
    * <p>
@@ -169,6 +150,7 @@ public void showReciept(Sale sale) {
     });
 
     model.instanceChildren().add(reciept);
+    CssUtils.fadeInAnimation(reciept);
   }
 
   /**
diff --git a/src/main/java/edu/ntnu/idi/idatt/view/primary/stock/StockController.java b/src/main/java/edu/ntnu/idi/idatt/view/primary/stock/StockController.java
index cfd7b42..fbb5a22 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/primary/stock/StockController.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/primary/stock/StockController.java
@@ -197,6 +197,7 @@ public void showReciept(Purchase purchase) {
     });
 
     model.instanceChildren().add(reciept);
+    CssUtils.fadeInAnimation(reciept);
   }
 
   /**
diff --git a/src/main/java/edu/ntnu/idi/idatt/view/primary/transactions/TransactionController.java b/src/main/java/edu/ntnu/idi/idatt/view/primary/transactions/TransactionController.java
index 1fa4867..4e23523 100644
--- a/src/main/java/edu/ntnu/idi/idatt/view/primary/transactions/TransactionController.java
+++ b/src/main/java/edu/ntnu/idi/idatt/view/primary/transactions/TransactionController.java
@@ -11,6 +11,7 @@
 import edu.ntnu.idi.idatt.session.UserSession;
 import edu.ntnu.idi.idatt.view.components.AbstractController;
 import edu.ntnu.idi.idatt.view.components.elements.TransactionComponent;
+import edu.ntnu.idi.idatt.view.util.CssUtils;
 
 /**
  * Controller class of the Transaction MVC.
@@ -74,7 +75,11 @@ public void initController() {
   public void setTransactionModel(List<Transaction> list) {
     model.getTransactionList().clear();
     for (Transaction transaction : list) {
-      model.getTransactionList().add(new TransactionComponent(transaction));
+      TransactionComponent transactionComponent = new TransactionComponent(transaction);
+
+      CssUtils.fadeInAnimation(transactionComponent);
+
+      model.getTransactionList().add(transactionComponent);
     }
   }