Fixes Confusing Listing index issue and Updates UserGuide

nus-cs2103-AY2223S2 · Apr 4, 2023 · 332a8cf · 332a8cf
1 parent 44dddcc
commit 332a8cf
Show file tree

Hide file tree

Showing 11 changed files with 99 additions and 55 deletions.
diff --git a/docs/UserGuide.md b/docs/UserGuide.md
diff --git a/docs/images/delete-homework.jpg b/docs/images/delete-homework.jpg
diff --git a/docs/images/mark-homework.jpg b/docs/images/mark-homework.jpg
diff --git a/docs/images/new-homework.jpg b/docs/images/new-homework.jpg
diff --git a/docs/images/unmark-homework.jpg b/docs/images/unmark-homework.jpg
diff --git a/docs/images/update-homework.jpg b/docs/images/update-homework.jpg
diff --git a/docs/images/view-homework.jpg b/docs/images/view-homework.jpg
diff --git a/src/main/java/seedu/address/ui/exam/ExamCard.java b/src/main/java/seedu/address/ui/exam/ExamCard.java
@@ -19,8 +19,6 @@ public class ExamCard extends UiPart<Region> {
     @FXML
     private HBox cardPane;
     @FXML
-    private Label id;
-    @FXML
     private Label title;
     @FXML
     private Label startTime;
@@ -35,7 +33,6 @@ public class ExamCard extends UiPart<Region> {
      */
     public ExamCard(Exam exam, int id) {
         super(FXML);
-        this.id.setText(id + ". ");
         title.setText(exam.getDescription());
         startTime.setText(exam.getStartTime().format(PRINT_FORMATTER));
         endTime.setText(exam.getEndTime().format(PRINT_FORMATTER));
@@ -53,8 +50,7 @@ public boolean equals(Object other) {
         }
         // state check
         ExamCard card = (ExamCard) other;
-        return id.getText().equals(card.id.getText())
-                && title.getText().equals(card.title.getText())
+        return title.getText().equals(card.title.getText())
                 && startTime.getText().equals(card.startTime.getText())
                 && endTime.getText().equals(card.endTime.getText());
     }

diff --git a/src/main/java/seedu/address/ui/lesson/LessonCard.java b/src/main/java/seedu/address/ui/lesson/LessonCard.java
@@ -20,8 +20,6 @@ public class LessonCard extends UiPart<Region> {
     @FXML
     private HBox cardPane;
     @FXML
-    private Label id;
-    @FXML
     private Label title;
     @FXML
     private Label startTime;
@@ -36,7 +34,6 @@ public class LessonCard extends UiPart<Region> {
      */
     public LessonCard(Lesson lesson, int id) {
         super(FXML);
-        this.id.setText(id + DOT);
         title.setText(lesson.getTitle());
         startTime.setText(lesson.getStartTime().format(PRINT_FORMATTER));
         endTime.setText(lesson.getEndTime().format(PRINT_FORMATTER));
@@ -54,8 +51,7 @@ public boolean equals(Object other) {
         }
         // state check
         LessonCard card = (LessonCard) other;
-        return id.getText().equals(card.id.getText())
-                && title.getText().equals(card.title.getText())
+        return title.getText().equals(card.title.getText())
                 && startTime.getText().equals(card.startTime.getText())
                 && endTime.getText().equals(card.endTime.getText());
     }

diff --git a/src/main/resources/view/ExamListCard.fxml b/src/main/resources/view/ExamListCard.fxml
@@ -11,7 +11,6 @@
     </padding>
     <VBox>
         <HBox spacing="3">
-            <Label fx:id="id" styleClass="cell_big_label" />
             <Label fx:id="title" VBox.vgrow="NEVER" />
         </HBox>
         <HBox>

diff --git a/src/main/resources/view/LessonListCard.fxml b/src/main/resources/view/LessonListCard.fxml
@@ -11,7 +11,6 @@
     </padding>
     <VBox>
         <HBox spacing="3">
-            <Label fx:id="id" styleClass="cell_big_label" />
             <Label fx:id="title" VBox.vgrow="NEVER" />
         </HBox>
         <HBox>