From dbd07b5a2f548fc59272263c13361614ee526fc2 Mon Sep 17 00:00:00 2001
From: Richard Dominick <34370238+RichDom2185@users.noreply.github.com>
Date: Fri, 28 Oct 2022 01:56:38 +0800
Subject: [PATCH 01/15] Show overflow when items have many tags

---
 src/main/java/seedu/foodrem/ui/ItemCard.java | 10 +++++++++-
 src/main/resources/view/FoodRem.css          |  5 +++++
 src/main/resources/view/ItemListCard.fxml    |  2 +-
 3 files changed, 15 insertions(+), 2 deletions(-)

diff --git a/src/main/java/seedu/foodrem/ui/ItemCard.java b/src/main/java/seedu/foodrem/ui/ItemCard.java
index 079a5242b86..a5cd9ed65f8 100644
--- a/src/main/java/seedu/foodrem/ui/ItemCard.java
+++ b/src/main/java/seedu/foodrem/ui/ItemCard.java
@@ -16,6 +16,8 @@
  * A UI component that displays information of a {@code Item}.
  */
 public class ItemCard extends UiPart<Region> {
+    private static final int TAGS_LIMIT = 5;
+
     /**
      * Note: Certain keywords such as "location" and "resources" are reserved keywords in JavaFX.
      * As a consequence, UI elements' variable names cannot be set to such keywords
@@ -49,8 +51,14 @@ public ItemCard(Item item, int displayedIndex) {
         quantity.setText(ItemView.buildItemQuantityAndUnitStringFrom(item));
         quantity.setTextAlignment(TextAlignment.RIGHT);
 
-        item.getTagSet().stream().sorted(Comparator.comparing(Tag::getName))
+        final int size = item.getTagSet().size();
+        item.getTagSet().stream().sorted(Comparator.comparing(Tag::getName)).limit(TAGS_LIMIT)
                 .forEach(tag -> tags.getChildren().add(buildTagNodeFrom(tag.getName())));
+        if (size > TAGS_LIMIT) {
+            final Label overflowLabel = new Label(String.format("+%d more...", size - TAGS_LIMIT));
+            overflowLabel.getStyleClass().add("tags-overflow-label");
+            tags.getChildren().add(overflowLabel);
+        }
     }
 
     private static Node buildTagNodeFrom(String tagName) {
diff --git a/src/main/resources/view/FoodRem.css b/src/main/resources/view/FoodRem.css
index 0124f2358bc..ebc9b329c50 100644
--- a/src/main/resources/view/FoodRem.css
+++ b/src/main/resources/view/FoodRem.css
@@ -129,3 +129,8 @@
 .list-view .list-cell:odd {
   -fx-background-radius: 8;
 }
+
+.tags-overflow-label {
+  -fx-font-size: 8pt;
+  -fx-font-fill: gray;
+}
diff --git a/src/main/resources/view/ItemListCard.fxml b/src/main/resources/view/ItemListCard.fxml
index 5b652b1aae5..6bbaad7eb3b 100644
--- a/src/main/resources/view/ItemListCard.fxml
+++ b/src/main/resources/view/ItemListCard.fxml
@@ -24,6 +24,6 @@
         <Label fx:id="price" styleClass="item-listview-data"/>
       </VBox>
     </HBox>
-    <HBox spacing="5" fx:id="tags" />
+    <HBox spacing="5" alignment="CENTER_LEFT" fx:id="tags" />
   </VBox>
 </HBox>

From 146af80fdd96213d2f68577c17cb8797cc84fc30 Mon Sep 17 00:00:00 2001
From: Richard Dominick <34370238+RichDom2185@users.noreply.github.com>
Date: Fri, 28 Oct 2022 02:00:08 +0800
Subject: [PATCH 02/15] Overload TagView generator

---
 src/main/java/seedu/foodrem/views/TagView.java | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/src/main/java/seedu/foodrem/views/TagView.java b/src/main/java/seedu/foodrem/views/TagView.java
index b27b804b8b0..6b284fb6849 100644
--- a/src/main/java/seedu/foodrem/views/TagView.java
+++ b/src/main/java/seedu/foodrem/views/TagView.java
@@ -17,8 +17,18 @@ private TagView() {} // Prevents instantiation
      * @return the node to be displayed in the UI.
      */
     public static Node from(Tag tag) {
+        return from(tag, false);
+    }
+
+    /**
+     * Creates a new view of the given string.
+     * @param tag the tag to be displayed.
+     * @param isSmall whether the tag is small.
+     * @return the node to be displayed in the UI.
+     */
+    public static Node from(Tag tag, boolean isSmall) {
         final Label label = new Label(tag.getName());
-        label.getStyleClass().add("item-detail-tag");
+        label.getStyleClass().add(isSmall ? "item-listview-tag" : "item-detail-tag");
         return label;
     }
 }

From c19aeac8fa96836eb1c2f034d94f031743872aa9 Mon Sep 17 00:00:00 2001
From: Richard Dominick <34370238+RichDom2185@users.noreply.github.com>
Date: Fri, 28 Oct 2022 02:01:06 +0800
Subject: [PATCH 03/15] Use TagView in ItemCard

Reduces code duplication.
---
 src/main/java/seedu/foodrem/ui/ItemCard.java | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/src/main/java/seedu/foodrem/ui/ItemCard.java b/src/main/java/seedu/foodrem/ui/ItemCard.java
index a5cd9ed65f8..12688437601 100644
--- a/src/main/java/seedu/foodrem/ui/ItemCard.java
+++ b/src/main/java/seedu/foodrem/ui/ItemCard.java
@@ -11,6 +11,7 @@
 import seedu.foodrem.model.item.Item;
 import seedu.foodrem.model.tag.Tag;
 import seedu.foodrem.views.ItemView;
+import seedu.foodrem.views.TagView;
 
 /**
  * A UI component that displays information of a {@code Item}.
@@ -53,7 +54,7 @@ public ItemCard(Item item, int displayedIndex) {
 
         final int size = item.getTagSet().size();
         item.getTagSet().stream().sorted(Comparator.comparing(Tag::getName)).limit(TAGS_LIMIT)
-                .forEach(tag -> tags.getChildren().add(buildTagNodeFrom(tag.getName())));
+                .forEach(tag -> tags.getChildren().add(TagView.from(tag, true)));
         if (size > TAGS_LIMIT) {
             final Label overflowLabel = new Label(String.format("+%d more...", size - TAGS_LIMIT));
             overflowLabel.getStyleClass().add("tags-overflow-label");
@@ -61,12 +62,6 @@ public ItemCard(Item item, int displayedIndex) {
         }
     }
 
-    private static Node buildTagNodeFrom(String tagName) {
-        final Label label = new Label(tagName);
-        label.getStyleClass().add("item-listview-tag");
-        return label;
-    }
-
     @Override
     public boolean equals(Object other) {
         if (other == this) {

From 909497bd4629d7d55a8ee6544696ac3f15873050 Mon Sep 17 00:00:00 2001
From: Richard Dominick <34370238+RichDom2185@users.noreply.github.com>
Date: Fri, 28 Oct 2022 02:19:13 +0800
Subject: [PATCH 04/15] Show colors of tags based on hashCode

---
 src/main/java/seedu/foodrem/views/TagView.java | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/src/main/java/seedu/foodrem/views/TagView.java b/src/main/java/seedu/foodrem/views/TagView.java
index 6b284fb6849..f235951eaa9 100644
--- a/src/main/java/seedu/foodrem/views/TagView.java
+++ b/src/main/java/seedu/foodrem/views/TagView.java
@@ -9,6 +9,11 @@
  * @author Richard Dominick
  */
 public class TagView {
+    private static final String[] COLORS = new String[] {
+        "f3d3b5", "abdee6", "ffccb6", "ff9aa2", "c7ceea", "a5e3b3", "e2f0cb", "c5d7c0", "f3d1dc", "e6ede5",
+        "c8b4ba", "98afba", "70ae98"
+    };
+
     private TagView() {} // Prevents instantiation
 
     /**
@@ -29,6 +34,12 @@ public static Node from(Tag tag) {
     public static Node from(Tag tag, boolean isSmall) {
         final Label label = new Label(tag.getName());
         label.getStyleClass().add(isSmall ? "item-listview-tag" : "item-detail-tag");
+        label.setStyle(String.format("-fx-background-color: #%s;", getColor(tag)));
         return label;
     }
+
+    private static String getColor(Tag tag) {
+        final int index = tag.hashCode() % COLORS.length;
+        return COLORS[index];
+    }
 }

From 993aab82e476738477cde652d578e237c3786375 Mon Sep 17 00:00:00 2001
From: Richard Dominick <34370238+RichDom2185@users.noreply.github.com>
Date: Fri, 28 Oct 2022 02:52:04 +0800
Subject: [PATCH 05/15] Remove hardcoding for max tag size

---
 src/main/java/seedu/foodrem/ui/ItemCard.java | 26 ++++++++++++++++----
 1 file changed, 21 insertions(+), 5 deletions(-)

diff --git a/src/main/java/seedu/foodrem/ui/ItemCard.java b/src/main/java/seedu/foodrem/ui/ItemCard.java
index 12688437601..0a645f7d7e1 100644
--- a/src/main/java/seedu/foodrem/ui/ItemCard.java
+++ b/src/main/java/seedu/foodrem/ui/ItemCard.java
@@ -1,6 +1,8 @@
 package seedu.foodrem.ui;
 
+import java.util.ArrayList;
 import java.util.Comparator;
+import java.util.List;
 
 import javafx.fxml.FXML;
 import javafx.scene.Node;
@@ -17,7 +19,8 @@
  * A UI component that displays information of a {@code Item}.
  */
 public class ItemCard extends UiPart<Region> {
-    private static final int TAGS_LIMIT = 5;
+    private static final int CHAR_LIMIT = 45;
+    private static final int SPACING_UNIT = 3;
 
     /**
      * Note: Certain keywords such as "location" and "resources" are reserved keywords in JavaFX.
@@ -52,11 +55,24 @@ public ItemCard(Item item, int displayedIndex) {
         quantity.setText(ItemView.buildItemQuantityAndUnitStringFrom(item));
         quantity.setTextAlignment(TextAlignment.RIGHT);
 
+        final List<Tag> tagList = new ArrayList<>(item.getTagSet());
+        tagList.sort(Comparator.comparing(Tag::getName));
+
+        int currentLength = 0;
+        int currentIndex;
+        for (currentIndex = 0; currentIndex < tagList.size(); currentIndex++) {
+            Tag tag = tagList.get(currentIndex);
+            currentLength += tag.getName().length();
+            if (currentLength > CHAR_LIMIT) {
+                break;
+            }
+            currentLength += SPACING_UNIT; // Account for padding between tags
+            tags.getChildren().add(TagView.from(tag, true));
+        }
+
         final int size = item.getTagSet().size();
-        item.getTagSet().stream().sorted(Comparator.comparing(Tag::getName)).limit(TAGS_LIMIT)
-                .forEach(tag -> tags.getChildren().add(TagView.from(tag, true)));
-        if (size > TAGS_LIMIT) {
-            final Label overflowLabel = new Label(String.format("+%d more...", size - TAGS_LIMIT));
+        if (size > currentIndex) {
+            final Label overflowLabel = new Label(String.format("+%d more...", size - currentIndex));
             overflowLabel.getStyleClass().add("tags-overflow-label");
             tags.getChildren().add(overflowLabel);
         }

From 85572a1f16d3a344e5ea1055f340b8595cf40ccb Mon Sep 17 00:00:00 2001
From: Richard Dominick <34370238+RichDom2185@users.noreply.github.com>
Date: Fri, 28 Oct 2022 02:59:38 +0800
Subject: [PATCH 06/15] Make result display scrollable

---
 src/main/java/seedu/foodrem/ui/ResultDisplay.java | 14 ++++++++++----
 src/main/resources/view/ResultDisplay.fxml        |  9 +++++----
 2 files changed, 15 insertions(+), 8 deletions(-)

diff --git a/src/main/java/seedu/foodrem/ui/ResultDisplay.java b/src/main/java/seedu/foodrem/ui/ResultDisplay.java
index 2bf949a785b..676b2d1a1df 100644
--- a/src/main/java/seedu/foodrem/ui/ResultDisplay.java
+++ b/src/main/java/seedu/foodrem/ui/ResultDisplay.java
@@ -2,24 +2,30 @@
 
 import javafx.fxml.FXML;
 import javafx.scene.Node;
+import javafx.scene.control.ScrollPane;
 import javafx.scene.layout.Region;
-import javafx.scene.layout.StackPane;
+import javafx.scene.layout.VBox;
 
 /**
  * A UI for the status bar that is displayed at the header of the application.
  */
 public class ResultDisplay extends UiPart<Region> {
-    @FXML private StackPane placeHolder;
+    @FXML private ScrollPane scrollPane;
+    @FXML private VBox pane;
 
     public ResultDisplay() {
         super("ResultDisplay.fxml");
     }
 
     public void clear() {
-        placeHolder.getChildren().clear();
+        pane.getChildren().clear();
     }
 
+    /**
+     * Places a node into the scrollPane.
+     */
     public void place(Node item) {
-        placeHolder.getChildren().add(item);
+        pane.getChildren().add(item);
+        scrollPane.setContent(pane);
     }
 }
diff --git a/src/main/resources/view/ResultDisplay.fxml b/src/main/resources/view/ResultDisplay.fxml
index 50cfd48f83a..87142ab4a35 100644
--- a/src/main/resources/view/ResultDisplay.fxml
+++ b/src/main/resources/view/ResultDisplay.fxml
@@ -1,6 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 
-<?import javafx.scene.control.TextArea?>
-<?import javafx.scene.layout.StackPane?>
-
-<StackPane fx:id="placeHolder" xmlns="http://javafx.com/javafx/8" xmlns:fx="http://javafx.com/fxml/1" styleClass="result-box"/>
+<?import javafx.scene.control.ScrollPane?>
+<?import javafx.scene.layout.VBox?>
+<ScrollPane fx:id="scrollPane" xmlns="http://javafx.com/javafx/8" xmlns:fx="http://javafx.com/fxml/1" fitToWidth="true" styleClass="scroll-result-box">
+    <VBox fx:id="pane" styleClass="result-box"/>
+</ScrollPane>

From 5bca14b0bf89ada72d2ff90b5bbf634ded3ac9d2 Mon Sep 17 00:00:00 2001
From: Richard Dominick <34370238+RichDom2185@users.noreply.github.com>
Date: Fri, 28 Oct 2022 02:59:58 +0800
Subject: [PATCH 07/15] Make result display full-height

---
 src/main/resources/view/MainWindow.fxml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/resources/view/MainWindow.fxml b/src/main/resources/view/MainWindow.fxml
index 3528804454a..9693b24cd0e 100644
--- a/src/main/resources/view/MainWindow.fxml
+++ b/src/main/resources/view/MainWindow.fxml
@@ -43,7 +43,7 @@
 
           <!-- Right Column -->
           <VBox spacing="12">
-            <StackPane VBox.vgrow="NEVER" fx:id="resultDisplayPlaceholder" styleClass="result-placeholder" />
+            <StackPane VBox.vgrow="ALWAYS" fx:id="resultDisplayPlaceholder" styleClass="result-placeholder" />
           </VBox>
         </HBox>
         <StackPane fx:id="statusbarPlaceholder" VBox.vgrow="NEVER"/>

From 136e799de7f7b4f0b0e351bfd6231142619ab230 Mon Sep 17 00:00:00 2001
From: Richard Dominick <34370238+RichDom2185@users.noreply.github.com>
Date: Fri, 28 Oct 2022 03:00:21 +0800
Subject: [PATCH 08/15] Fix styling for result display

---
 src/main/resources/view/FoodRem.css | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/main/resources/view/FoodRem.css b/src/main/resources/view/FoodRem.css
index ebc9b329c50..eaba2cd9b5f 100644
--- a/src/main/resources/view/FoodRem.css
+++ b/src/main/resources/view/FoodRem.css
@@ -105,6 +105,10 @@
   -fx-font-weight: bold; // TODO
 }
 
+.scroll-result-box {
+  -fx-background-color: transparent;
+}
+
 /* ============================ *
  * ItemCard                     *
  * ============================ */

From ca0c9fc795e166845bfe2241c55626407f5a3f53 Mon Sep 17 00:00:00 2001
From: Richard Dominick <34370238+RichDom2185@users.noreply.github.com>
Date: Fri, 28 Oct 2022 03:01:01 +0800
Subject: [PATCH 09/15] Style scrollbars

Co-authored-by: Tan Yi Xian <>
---
 src/main/resources/view/FoodRem.css | 45 +++++++++++++++++++++++++++++
 1 file changed, 45 insertions(+)

diff --git a/src/main/resources/view/FoodRem.css b/src/main/resources/view/FoodRem.css
index eaba2cd9b5f..985d1f04c0a 100644
--- a/src/main/resources/view/FoodRem.css
+++ b/src/main/resources/view/FoodRem.css
@@ -138,3 +138,48 @@
   -fx-font-size: 8pt;
   -fx-font-fill: gray;
 }
+
+/* ============================ *
+ * ScrollBar                    *
+ * ============================ */
+.scroll-bar {
+  -fx-background-color: #ffffff;
+}
+
+.scroll-bar .thumb {
+  -fx-background-color: #cccccc;
+  -fx-background-insets: 3;
+}
+
+.scroll-bar .increment-button,
+.scroll-bar .decrement-button {
+  -fx-background-color: transparent;
+  -fx-padding: 0 0 0 0;
+}
+
+.scroll-bar .increment-arrow,
+.scroll-bar .decrement-arrow {
+  -fx-shape: " ";
+}
+
+.scroll-bar:vertical .increment-arrow,
+.scroll-bar:vertical .decrement-arrow {
+  -fx-padding: 1 8 1 8;
+}
+
+.scroll-bar:horizontal .increment-arrow,
+.scroll-bar:horizontal .decrement-arrow {
+  -fx-padding: 8 1 8 1;
+}
+
+/* ============================ *
+* ResultDisplay                *
+* ============================ */
+.scroll-pane > .viewport {
+  -fx-background-color: #ffffff;
+  -fx-border-color: #ffffff;
+}
+
+.scroll-pane {
+  -fx-background-color: transparent;
+}

From c7c0f2ff81093d561eef01bac68b46655073fb67 Mon Sep 17 00:00:00 2001
From: Richard Dominick <34370238+RichDom2185@users.noreply.github.com>
Date: Fri, 28 Oct 2022 03:32:11 +0800
Subject: [PATCH 10/15] Set ItemView elements to wrap

---
 src/main/java/seedu/foodrem/views/ItemView.java | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/src/main/java/seedu/foodrem/views/ItemView.java b/src/main/java/seedu/foodrem/views/ItemView.java
index 9c13449b10f..f080315d9ba 100644
--- a/src/main/java/seedu/foodrem/views/ItemView.java
+++ b/src/main/java/seedu/foodrem/views/ItemView.java
@@ -4,10 +4,13 @@
 import javafx.scene.Node;
 import javafx.scene.control.Label;
 import javafx.scene.control.Separator;
+import javafx.scene.layout.FlowPane;
 import javafx.scene.layout.HBox;
 import javafx.scene.layout.Priority;
 import javafx.scene.layout.VBox;
 import javafx.scene.text.TextAlignment;
+import javafx.scene.layout.Region;
+import javafx.scene.layout.VBox;
 import seedu.foodrem.model.item.Item;
 
 /**
@@ -25,15 +28,18 @@ private ItemView() {} // Prevents instantiation
      * @return the node to be displayed in the UI.
      */
     public static Node from(Item item) {
+        // Name
         final Label name = new Label(item.getName().toString());
         name.getStyleClass().add("item-detail-name");
         name.prefWidth(Double.MAX_VALUE);
+        name.setWrapText(true);
 
-        // Name and tags at the top left
-        final HBox tags = new HBox(new Label("Tags: "));
+        // Tags
+        final FlowPane tags = new FlowPane(new Label("Tags: "));
         tags.getChildren().addAll(TagsView.from(item.getTagSet()));
         tags.setAlignment(Pos.CENTER_LEFT);
-        tags.setSpacing(SPACING_UNIT);
+        tags.setHgap(SPACING_UNIT);
+        tags.setVgap(SPACING_UNIT);
 
         // Quantity and unit at the top right
         final Label quantityLabel = new Label("Quantity\nRemaining:");

From 2bf6b8721cffa01f815a95f6cb67ad06f7cf88f3 Mon Sep 17 00:00:00 2001
From: Richard Dominick <34370238+RichDom2185@users.noreply.github.com>
Date: Fri, 28 Oct 2022 03:38:24 +0800
Subject: [PATCH 11/15] Update ItemView layout

---
 .../java/seedu/foodrem/views/ItemView.java    | 30 ++++++++-----------
 1 file changed, 12 insertions(+), 18 deletions(-)

diff --git a/src/main/java/seedu/foodrem/views/ItemView.java b/src/main/java/seedu/foodrem/views/ItemView.java
index f080315d9ba..bd865ec8446 100644
--- a/src/main/java/seedu/foodrem/views/ItemView.java
+++ b/src/main/java/seedu/foodrem/views/ItemView.java
@@ -41,29 +41,23 @@ public static Node from(Item item) {
         tags.setHgap(SPACING_UNIT);
         tags.setVgap(SPACING_UNIT);
 
-        // Quantity and unit at the top right
-        final Label quantityLabel = new Label("Quantity\nRemaining:");
-        quantityLabel.setTextAlignment(TextAlignment.RIGHT);
-        final Label quantityAndUnit = new Label(buildItemQuantityAndUnitStringFrom(item));
-        quantityAndUnit.getStyleClass().add("item-detail-quantity");
-        final VBox quantityBox = new VBox(quantityLabel, quantityAndUnit);
-        quantityBox.setAlignment(Pos.CENTER_RIGHT);
+        // Quantity/unit and price row
+        final Region spacer = new Region();
+        HBox.setHgrow(spacer, Priority.ALWAYS);
+        final HBox quantityAndPrice = new HBox(
+                new Label("Quantity: " + buildItemQuantityAndUnitStringFrom(item)),
+                spacer,
+                new Label("Price: $" + item.getPrice().toString()));
 
-        // Set up top half
-        final VBox col1 = new VBox(name, tags);
-        col1.setSpacing(SPACING_UNIT);
-        final HBox row1 = new HBox(col1, quantityBox);
-        HBox.setHgrow(col1, Priority.ALWAYS);
-        row1.setSpacing(SPACING_UNIT);
-
-        final Label remarks = new Label(
-                String.valueOf(item.getRemarks()).isBlank() ? "-" : item.getRemarks().toString());
+        final Label remarks = new Label(item.getRemarks().toString().isBlank() ? "-" : item.getRemarks().toString());
         remarks.setWrapText(true);
 
         // Combine everything
         final VBox itemView = new VBox(
-                row1,
-                new Label("$" + item.getPrice().toString()),
+                name,
+                quantityAndPrice,
+                new Separator(),
+                tags,
                 new Separator(),
                 new Label("Bought Date: " + buildBoughtDateStringFrom(item)),
                 new Label("Expiry Date: " + buildExpiryDateStringFrom(item)),

From 3c52d12816df77db6e6a0e45fa5ff27edfb5f7e3 Mon Sep 17 00:00:00 2001
From: Richard Dominick <34370238+RichDom2185@users.noreply.github.com>
Date: Fri, 28 Oct 2022 03:47:10 +0800
Subject: [PATCH 12/15] Update UI for ItemView

---
 src/main/java/seedu/foodrem/views/ItemView.java | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/main/java/seedu/foodrem/views/ItemView.java b/src/main/java/seedu/foodrem/views/ItemView.java
index bd865ec8446..4f21299981e 100644
--- a/src/main/java/seedu/foodrem/views/ItemView.java
+++ b/src/main/java/seedu/foodrem/views/ItemView.java
@@ -53,17 +53,19 @@ public static Node from(Item item) {
         remarks.setWrapText(true);
 
         // Combine everything
+        final Separator linedSeparator = new Separator();
+        linedSeparator.getStyleClass().add("lined-separator");
         final VBox itemView = new VBox(
                 name,
                 quantityAndPrice,
                 new Separator(),
-                tags,
-                new Separator(),
                 new Label("Bought Date: " + buildBoughtDateStringFrom(item)),
                 new Label("Expiry Date: " + buildExpiryDateStringFrom(item)),
                 new Separator(),
                 new Label("Remarks:"),
-                remarks);
+                remarks,
+                linedSeparator,
+                tags);
         itemView.setSpacing(SPACING_UNIT);
         return itemView;
     }

From 80e5648aef22514ae24cde2ac3cf3ded4fde3d99 Mon Sep 17 00:00:00 2001
From: Richard Dominick <34370238+RichDom2185@users.noreply.github.com>
Date: Fri, 28 Oct 2022 03:49:37 +0800
Subject: [PATCH 13/15] Update item remarks validation

Co-authored-by: Tan Yi Xian <>
---
 .../model/item/itemvalidators/ItemRemarksValidator.java        | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/main/java/seedu/foodrem/model/item/itemvalidators/ItemRemarksValidator.java b/src/main/java/seedu/foodrem/model/item/itemvalidators/ItemRemarksValidator.java
index afcd5b40c8b..d1eaa175fbd 100644
--- a/src/main/java/seedu/foodrem/model/item/itemvalidators/ItemRemarksValidator.java
+++ b/src/main/java/seedu/foodrem/model/item/itemvalidators/ItemRemarksValidator.java
@@ -9,7 +9,8 @@
  */
 public class ItemRemarksValidator implements Validator {
     // Validation for remarks length
-    private static final int MAX_LENGTH = 1000;
+    // TODO: Ensure UG is standardised
+    private static final int MAX_LENGTH = 10000;
     private static final String MESSAGE_FOR_REMARKS_TOO_LONG =
             String.format("The item remark should not exceed %d characters", MAX_LENGTH);
 

From 84936185914498101fc111406a049943777e9fd2 Mon Sep 17 00:00:00 2001
From: Richard Dominick <34370238+RichDom2185@users.noreply.github.com>
Date: Fri, 28 Oct 2022 03:49:55 +0800
Subject: [PATCH 14/15] Add validations to TagName

Co-authored-by: Tan Yi Xian <>
---
 src/main/java/seedu/foodrem/model/tag/TagName.java | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/src/main/java/seedu/foodrem/model/tag/TagName.java b/src/main/java/seedu/foodrem/model/tag/TagName.java
index d94bfd5d139..5777db12f52 100644
--- a/src/main/java/seedu/foodrem/model/tag/TagName.java
+++ b/src/main/java/seedu/foodrem/model/tag/TagName.java
@@ -10,6 +10,14 @@
  * Guarantees: immutable; is valid as declared in {@link StringUtil#isValidString(String)}
  */
 public class TagName {
+    private static final String MESSAGE_FOR_NAME_IS_BLANK =
+            "The tag name should not be blank.";
+
+    // Validation for name length
+    private static final int MAX_LENGTH = 20;
+    private static final String MESSAGE_FOR_NAME_TOO_LONG =
+            String.format("The tag name should not exceed %d characters", MAX_LENGTH);
+
     private final String fullName;
 
     /**
@@ -19,6 +27,12 @@ public class TagName {
      */
     public TagName(String name) {
         requireNonNull(name);
+
+        boolean isNamePresent = !name.isBlank();
+        checkArgument(isNamePresent, MESSAGE_FOR_NAME_IS_BLANK);
+        boolean isNameLengthLessThanEqualMaxLength = name.length() <= MAX_LENGTH;
+        checkArgument(isNameLengthLessThanEqualMaxLength, MESSAGE_FOR_NAME_TOO_LONG);
+
         checkArgument(StringUtil.isValidString(name), StringUtil.getInvalidCharactersMessage("tag name"));
         fullName = name;
     }

From be2977161a8f153e4774f8089dfb88b6d300c191 Mon Sep 17 00:00:00 2001
From: Richard Dominick <34370238+RichDom2185@users.noreply.github.com>
Date: Fri, 28 Oct 2022 03:56:55 +0800
Subject: [PATCH 15/15] Fix imports

---
 src/main/java/seedu/foodrem/ui/ItemCard.java    | 1 -
 src/main/java/seedu/foodrem/views/ItemView.java | 2 --
 2 files changed, 3 deletions(-)

diff --git a/src/main/java/seedu/foodrem/ui/ItemCard.java b/src/main/java/seedu/foodrem/ui/ItemCard.java
index 0a645f7d7e1..e77da6c2874 100644
--- a/src/main/java/seedu/foodrem/ui/ItemCard.java
+++ b/src/main/java/seedu/foodrem/ui/ItemCard.java
@@ -5,7 +5,6 @@
 import java.util.List;
 
 import javafx.fxml.FXML;
-import javafx.scene.Node;
 import javafx.scene.control.Label;
 import javafx.scene.layout.HBox;
 import javafx.scene.layout.Region;
diff --git a/src/main/java/seedu/foodrem/views/ItemView.java b/src/main/java/seedu/foodrem/views/ItemView.java
index 4f21299981e..c6fea4b0c8b 100644
--- a/src/main/java/seedu/foodrem/views/ItemView.java
+++ b/src/main/java/seedu/foodrem/views/ItemView.java
@@ -7,8 +7,6 @@
 import javafx.scene.layout.FlowPane;
 import javafx.scene.layout.HBox;
 import javafx.scene.layout.Priority;
-import javafx.scene.layout.VBox;
-import javafx.scene.text.TextAlignment;
 import javafx.scene.layout.Region;
 import javafx.scene.layout.VBox;
 import seedu.foodrem.model.item.Item;