Merge 33c95b4 into 5d4d8ac

src/main/java/seedu/address/ui/LoanCard.java

@@ -13,6 +13,8 @@
 public class LoanCard extends UiPart<Region> {
 
     private static final String FXML = "LoanListCard.fxml";
+    private static final String[] TAG_COLOR_STYLES =
+        {"teal", "red", "yellow", "blue", "orange", "brown", "green", "pink", "black", "grey"};
 
     /**
      * Note: Certain keywords such as "location" and "resources" are reserved keywords in JavaFX.
@@ -47,7 +49,27 @@ public LoanCard(Loan loan, int displayedIndex) {
         phone.setText(loan.getPhone().value);
         address.setText(loan.getAddress().value);
         email.setText(loan.getEmail().value);
-        loan.getTags().forEach(tag -> tags.getChildren().add(new Label(tag.value)));
+        initTags(loan);
+    }
+
+    /**
+     * Returns the color style for {@code tagName}'s label.
+     */
+    private String getTagColorStyleFor(String tagName) {
+        // we use the hash code of the tag name to generate a random color, so that the color remain consistent
+        // between different runs of the program while still making it random enough between tags.
+        return TAG_COLOR_STYLES[Math.abs(tagName.hashCode()) % TAG_COLOR_STYLES.length];
+    }
+
+    /**
+     * Creates the tag labels for {@code person}.
+     */
+    private void initTags(Loan loan) {
+        loan.getTags().forEach(tag -> {
+            Label tagLabel = new Label(tag.value);
+            tagLabel.getStyleClass().add(getTagColorStyleFor(tag.value));
+            tags.getChildren().add(tagLabel);
+        });
     }
 
     @Override

src/main/resources/view/DarkTheme.css

@@ -349,3 +349,53 @@
     -fx-background-radius: 2;
     -fx-font-size: 11;
 }
+
+#tags .teal {
+    -fx-text-fill: white;
+    -fx-background-color: #3e7b91;
+}
+
+#tags .red {
+    -fx-text-fill: black;
+    -fx-background-color: red;
+}
+
+#tags .yellow {
+    -fx-background-color: yellow;
+    -fx-text-fill: black;
+}
+
+#tags .blue {
+    -fx-text-fill: white;
+    -fx-background-color: blue;
+}
+
+#tags .orange {
+    -fx-text-fill: black;
+    -fx-background-color: orange;
+}
+
+#tags .brown {
+    -fx-text-fill: white;
+    -fx-background-color: brown;
+}
+
+#tags .green {
+    -fx-text-fill: black;
+    -fx-background-color: green;
+}
+
+#tags .pink {
+    -fx-text-fill: black;
+    -fx-background-color: pink;
+}
+
+#tags .black {
+    -fx-text-fill: white;
+    -fx-background-color: black;
+}
+
+#tags .grey {
+    -fx-text-fill: black;
+    -fx-background-color: grey;
+}

src/test/java/guitests/guihandles/LoanCardHandle.java

@@ -72,6 +72,15 @@ public List<String> getTags() {
                 .collect(Collectors.toList());
     }
 
+    public List<String> getTagStyleClasses(String tag) {
+        return tagLabels
+                .stream()
+                .filter(label -> label.getText().equals(tag))
+                .map(Label::getStyleClass)
+                .findFirst()
+                .orElseThrow(() -> new IllegalArgumentException("No such tag."));
+    }
+
     /**
      * Returns true if this handle contains {@code loan}.
      */

src/test/java/seedu/address/ui/testutil/GuiTestAssert.java

@@ -2,18 +2,21 @@
 
 import static org.junit.Assert.assertEquals;
 
+import java.util.Arrays;
 import java.util.List;
 import java.util.stream.Collectors;
 
 import guitests.guihandles.LoanCardHandle;
 import guitests.guihandles.LoanListPanelHandle;
 import guitests.guihandles.ResultDisplayHandle;
 import seedu.address.model.loan.Loan;
+import seedu.address.ui.LoanCard;
 
 /**
  * A set of assertion methods useful for writing GUI tests.
  */
 public class GuiTestAssert {
+    private static final String LABEL_DEFAULT_STYLE = "label";
     /**
      * Asserts that {@code actualCard} displays the same values as {@code expectedCard}.
      */
@@ -24,6 +27,11 @@ public static void assertCardEquals(LoanCardHandle expectedCard, LoanCardHandle
         assertEquals(expectedCard.getName(), actualCard.getName());
         assertEquals(expectedCard.getPhone(), actualCard.getPhone());
         assertEquals(expectedCard.getTags(), actualCard.getTags());
+
+        expectedCard
+                .getTags()
+                .forEach(tag ->
+                        assertEquals(expectedCard.getTagStyleClasses(tag), actualCard.getTagStyleClasses(tag)));
     }
 
     /**
@@ -34,8 +42,48 @@ public static void assertCardDisplaysLoan(Loan expectedLoan, LoanCardHandle actu
         assertEquals(expectedLoan.getPhone().value, actualCard.getPhone());
         assertEquals(expectedLoan.getEmail().value, actualCard.getEmail());
         assertEquals(expectedLoan.getAddress().value, actualCard.getAddress());
-        assertEquals(expectedLoan.getTags().stream().map(tag -> tag.value).collect(Collectors.toList()),
-                actualCard.getTags());
+        assertTagsEqual(expectedLoan, actualCard);
+    }
+
+    /**
+     * Returns the color style for {@code tagName}'s label. The tag's color is determined by looking up the color
+     * in {@code LoanCard#TAG_COLOR_STYLES}, using an index generated by the hash code of the tag's content.
+     *
+     * @see LoanCard#getTagColorStyleFor(String)
+     */
+    private static String getTagColorStyleFor(String tagName) {
+        switch (tagName) {
+        case "classmates":
+        case "owesMoney":
+            return "teal";
+        case "colleagues":
+        case "neighbours":
+            return "yellow";
+        case "family":
+        case "friend":
+            return "orange";
+        case "friends":
+            return "brown";
+        case "husband":
+            return "grey";
+        default:
+            throw new AssertionError(tagName + " does not have a color assigned.");
+        }
+    }
+
+    /**
+     * Asserts that the tags in {@code actualCard} matches all the tags in {@code expectedPerson} with the correct
+     * color.
+     */
+    private static void assertTagsEqual(Loan expectedLoan, LoanCardHandle actualCard) {
+        List<String> expectedTags = expectedLoan.getTags().stream()
+                .map(tag -> tag.value).collect(Collectors.toList());
+        assertEquals(expectedTags, actualCard.getTags());
+        expectedTags.forEach(tag ->
+                                     assertEquals(
+                                             Arrays.asList(LABEL_DEFAULT_STYLE, getTagColorStyleFor(tag)),
+                                             actualCard.getTagStyleClasses(tag)
+                                     ));
     }
 
     /**