Fix bug for list command and select (#201)

* Fix bug for list command and select

* Add documentation

docs/team/t-cheepeng.adoc

@@ -7,7 +7,7 @@
 
 == Overview
 
-My team of 4 computer science students were tasked with changing a basic command line application. Our team
+My team of 5 computer science students were tasked with changing a basic command line application. Our team
 decided to morph the application into ExerHealth. **ExerHealth** is a desktop application used for tracking
 and scheduling the user's exercises. The application has statistical analysis of exercises users have completed
 in the past. Additionally, it also acts as a personal trainer by suggesting different exercises which both

src/main/java/seedu/exercise/ui/ExerciseListPanel.java

@@ -30,7 +30,7 @@ public ExerciseListPanel(ObservableList<Exercise> exerciseList) {
         exerciseListView.getFocusModel().focusedItemProperty().addListener(getDefaultListViewListener());
     }
 
-    public ListView<Exercise> getExerciseListView() {
+    public ListView<Exercise> getResourceListView() {
         return exerciseListView;
     }
 
@@ -54,6 +54,11 @@ protected void selectGivenIndex(int index) {
         }
     }
 
+    @Override
+    protected void resetListSelection() {
+        exerciseListView.getSelectionModel().clearSelection();
+    }
+
     /**
      * Custom {@code ListCell} that displays the graphics of a {@code Exercise} using a {@code ExerciseInfoPanel}.
      */

src/main/java/seedu/exercise/ui/MainWindow.java

@@ -38,10 +38,10 @@ public class MainWindow extends UiPart<Stage> {
     private HelpWindow helpWindow;
     private ResolveWindow resolveWindow;
     private CustomPropertiesWindow customPropertiesWindow;
-    private ExerciseListPanel exerciseListPanel;
-    private RegimeListPanel regimeListPanel;
-    private ScheduleListPanel scheduleListPanel;
-    private SuggestionListPanel suggestionListPanel;
+    private ResourceListPanel exerciseListPanel;
+    private ResourceListPanel regimeListPanel;
+    private ResourceListPanel scheduleListPanel;
+    private ResourceListPanel suggestionListPanel;
     private InfoDisplayPanel infoDisplayPanel;
     private StatsDisplayPanel statsDisplayPanel;
 
@@ -112,19 +112,19 @@ void fillInnerParts() {
 
         exerciseListPanel = new ExerciseListPanel(logic.getSortedExerciseList());
         exerciseListTabPlaceholder = new Tab();
-        exerciseListTabPlaceholder.setContent((exerciseListPanel).getExerciseListView());
+        exerciseListTabPlaceholder.setContent(exerciseListPanel.getResourceListView());
 
         regimeListPanel = new RegimeListPanel(logic.getSortedRegimeList());
         regimeListTabPlaceholder = new Tab();
-        regimeListTabPlaceholder.setContent(regimeListPanel.getRegimeListView());
+        regimeListTabPlaceholder.setContent(regimeListPanel.getResourceListView());
 
         scheduleListPanel = new ScheduleListPanel(logic.getSortedScheduleList());
         scheduleListTabPlaceholder = new Tab();
-        scheduleListTabPlaceholder.setContent(scheduleListPanel.getScheduleListView());
+        scheduleListTabPlaceholder.setContent(scheduleListPanel.getResourceListView());
 
         suggestionListPanel = new SuggestionListPanel(logic.getSuggestedExerciseList());
         suggestionListTabPlaceholder = new Tab();
-        suggestionListTabPlaceholder.setContent(suggestionListPanel.getSuggestionListView());
+        suggestionListTabPlaceholder.setContent(suggestionListPanel.getResourceListView());
 
         resourceListPanelPlaceholder.getTabs().add(exerciseListTabPlaceholder);
         resourceListPanelPlaceholder.getTabs().add(regimeListTabPlaceholder);
@@ -222,6 +222,7 @@ private CommandResult executeCommand(String commandText) throws CommandException
             resultDisplay.setFeedbackToUser(commandResult.getFeedbackToUser());
             setChart();
             setStats();
+            resetResourceListTabs();
 
             shouldShowWindowsBasedOnCommandResult(commandResult);
             shouldExitAppBasedOnCommandResult(commandResult);
@@ -235,6 +236,16 @@ private CommandResult executeCommand(String commandText) throws CommandException
         }
     }
 
+    /**
+     * Resets the selection models of the resource list panels on the left of the GUI.
+     */
+    private void resetResourceListTabs() {
+        exerciseListPanel.resetListSelection();
+        regimeListPanel.resetListSelection();
+        scheduleListPanel.resetListSelection();
+        suggestionListPanel.resetListSelection();
+    }
+
     /**
      * Checks if a secondary window should be shown based on the command results.
      * Method will show the windows if it is to be shown.

src/main/java/seedu/exercise/ui/RegimeListPanel.java

@@ -27,10 +27,6 @@ public RegimeListPanel(ObservableList<Regime> regimeList) {
         regimeListView.getFocusModel().focusedItemProperty().addListener(getDefaultListViewListener());
     }
 
-    public ListView<Regime> getRegimeListView() {
-        return regimeListView;
-    }
-
     @Override
     protected void selectGivenIndex(int index) {
         if (index >= 0) {
@@ -46,6 +42,16 @@ protected void selectGivenIndex(int index) {
         }
     }
 
+    @Override
+    protected void resetListSelection() {
+        regimeListView.getSelectionModel().clearSelection();
+    }
+
+    @Override
+    protected ListView<Regime> getResourceListView() {
+        return regimeListView;
+    }
+
     /**
      * Custom {@code ListCell} that displays the graphics of a {@code Regime} using a {@code RegimeCard}.
      */

src/main/java/seedu/exercise/ui/ResourceListPanel.java

@@ -117,6 +117,10 @@ public void onChanged(Change<? extends Resource> change) {
 
     protected abstract void selectGivenIndex(int index);
 
+    protected abstract void resetListSelection();
+
+    protected abstract ListView<? extends Resource> getResourceListView();
+
     /**
      * Listener for item selection events in this {@code ResourceListPanel}.
      */

src/main/java/seedu/exercise/ui/ScheduleListPanel.java

@@ -27,10 +27,6 @@ public ScheduleListPanel(ObservableList<Schedule> scheduleList) {
         scheduleListView.getFocusModel().focusedItemProperty().addListener(getDefaultListViewListener());
     }
 
-    public ListView<Schedule> getScheduleListView() {
-        return scheduleListView;
-    }
-
     @Override
     protected void selectGivenIndex(int index) {
         if (index >= 0) {
@@ -46,6 +42,16 @@ protected void selectGivenIndex(int index) {
         }
     }
 
+    @Override
+    protected void resetListSelection() {
+        scheduleListView.getSelectionModel().clearSelection();
+    }
+
+    @Override
+    protected ListView<Schedule> getResourceListView() {
+        return scheduleListView;
+    }
+
     /**
      * Custom {@code ListCell} that displays the graphics of a {@code Schedule} using a {@code ScheduleCard}.
      */

src/main/java/seedu/exercise/ui/SuggestionListPanel.java

@@ -46,6 +46,16 @@ protected void selectGivenIndex(int index) {
         }
     }
 
+    @Override
+    protected void resetListSelection() {
+        suggestionListView.getSelectionModel().clearSelection();
+    }
+
+    @Override
+    protected ListView<Exercise> getResourceListView() {
+        return suggestionListView;
+    }
+
     /**
      * Custom {@code ListCell} that displays the graphics of a {@code Exercise} using a {@code ExerciseInfoPanel}.
      */