Merge pull request nus-cs2103-AY2122S1#87 from rohit0718/update/summa…

…ry-numbering Add summary numbering
bentanjunrong · Oct 14, 2021 · 6165000 · 6165000
2 parents cf8e745 + cece6db
commit 6165000
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/main/java/seedu/address/ui/ModuleListPanel.java b/src/main/java/seedu/address/ui/ModuleListPanel.java
@@ -58,7 +58,7 @@ protected void updateItem(Module module, boolean empty) {
                 setGraphic(null);
                 setText(null);
             } else {
-                setGraphic(new ModuleSummaryCard(module).getRoot());
+                setGraphic(new ModuleSummaryCard(module, getIndex() + 1).getRoot());
             }
         }
     }

diff --git a/src/main/java/seedu/address/ui/ModuleSummaryCard.java b/src/main/java/seedu/address/ui/ModuleSummaryCard.java
@@ -41,11 +41,11 @@ public class ModuleSummaryCard extends UiPart<Region> {
     /**
      * Creates a {@code ModuleCode} with the given {@code Module} and index to display.
      */
-    public ModuleSummaryCard(Module module) {
+    public ModuleSummaryCard(Module module, int displayedIndex) {
         super(FXML);
         this.module = module;
+        id.setText(displayedIndex + ". ");
         String moduleHeader = module.getCode().toString();
-
         if (module.getName().isPresent()) {
             moduleHeader += String.format(" %s", module.getName().get());
         }