Merge pull request nus-cs2103-AY2021S1#90 from Ma-Yueran/branch-improve-task-list-view

Branch improve task list view

Author: Ma-Yueran

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

@@ -24,6 +24,7 @@
 public class MainWindow extends UiPart<Stage> {
 
     private static final String FXML = "MainWindow.fxml";
+    private static MainWindow instance = null;
 
     private final Logger logger = LogsCenter.getLogger(getClass());
 
@@ -67,6 +68,13 @@ public MainWindow(Stage primaryStage, Logic logic) {
         setWindowDefaultSize(logic.getGuiSettings());
 
         // setAccelerators();
+
+        // Set instance
+        instance = this;
+    }
+
+    public static MainWindow getInstance() {
+        return instance;
     }
 
     public Stage getPrimaryStage() {
@@ -173,7 +181,7 @@ private void handleExit() {
      *
      * @see seedu.address.logic.Logic#execute(String)
      */
-    private CommandResult executeCommand(String commandText) throws CommandException, ParseException {
+    public CommandResult executeCommand(String commandText) throws CommandException, ParseException {
         try {
             CommandResult commandResult = logic.execute(commandText);
             logger.info("Result: " + commandResult.getFeedbackToUser());

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

@@ -4,6 +4,8 @@
 import javafx.scene.control.Label;
 import javafx.scene.layout.HBox;
 import javafx.scene.layout.Region;
+import seedu.address.logic.commands.exceptions.CommandException;
+import seedu.address.logic.parser.exceptions.ParseException;
 import seedu.address.model.tag.Tag;
 
 /**
@@ -43,6 +45,21 @@ public TagCard(Tag tag, int displayedIndex) {
         fileAddress.setText(tag.getFileAddress().value);
     }
 
+    /**
+     * Displays the information of the tag in the main window.
+     *
+     * @see seedu.address.logic.Logic#execute(String)
+     */
+    public void showTagInfo() throws CommandException, ParseException {
+        MainWindow mainWindow = MainWindow.getInstance();
+        if (mainWindow == null) {
+            return;
+        }
+
+        String showInfoCommand = "show t/" + tag.getTagName();
+        mainWindow.executeCommand(showInfoCommand);
+    }
+
     @Override
     public boolean equals(Object other) {
         // short circuit if same object

src/main/resources/view/MainWindow.fxml

@@ -5,7 +5,7 @@
 <?import javafx.scene.control.Menu?>
 <?import javafx.scene.control.MenuBar?>
 <?import javafx.scene.control.MenuItem?>
-<?import javafx.scene.layout.HBox?>
+<?import javafx.scene.control.SplitPane?>
 <?import javafx.scene.layout.StackPane?>
 <?import javafx.scene.layout.VBox?>
 <?import javafx.stage.Stage?>
@@ -30,24 +30,21 @@
                       </Menu>
                     </menus>
                   </MenuBar>
-                  <HBox prefHeight="100.0" prefWidth="200.0" spacing="10.0" VBox.vgrow="ALWAYS">
-                     <children>
-                        <VBox prefHeight="314.0" prefWidth="188.0" HBox.hgrow="NEVER">
+                  <SplitPane dividerPositions="0.3224299065420561" prefHeight="160.0" prefWidth="200.0" VBox.vgrow="ALWAYS">
+                    <items>
+                        <VBox prefHeight="200.0" prefWidth="100.0">
                            <children>
-                              <StackPane fx:id="tagListPlaceholder" prefHeight="306.0" prefWidth="139.0" VBox.vgrow="ALWAYS" />
+                              <StackPane fx:id="tagListPlaceholder" prefHeight="294.0" prefWidth="143.0" VBox.vgrow="ALWAYS" />
                            </children>
                         </VBox>
-                        <VBox prefHeight="200.0" prefWidth="100.0" spacing="10.0" HBox.hgrow="ALWAYS">
+                        <VBox prefHeight="200.0" prefWidth="100.0">
                            <children>
                               <StackPane fx:id="resultDisplayPlaceHolder" prefHeight="299.0" prefWidth="359.0" VBox.vgrow="ALWAYS" />
                               <StackPane fx:id="lastInputPlaceHolder" prefHeight="15.0" prefWidth="301.0" VBox.vgrow="NEVER" />
                            </children>
                         </VBox>
-                     </children>
-                     <padding>
-                        <Insets bottom="10.0" left="10.0" right="10.0" top="10.0" />
-                     </padding>
-                  </HBox>
+                    </items>
+                  </SplitPane>
                   <StackPane fx:id="commandBoxPlaceHolder" prefHeight="41.0" prefWidth="501.0">
                      <padding>
                         <Insets left="10.0" right="10.0" />

src/main/resources/view/TagCard.fxml

@@ -10,7 +10,7 @@
 <?import javafx.scene.layout.VBox?>
 <?import javafx.scene.text.Font?>
 
-<HBox id="cardPane" fx:id="cardPane" xmlns="http://javafx.com/javafx/11" xmlns:fx="http://javafx.com/fxml/1">
+<HBox id="cardPane" fx:id="cardPane" onMouseClicked="#showTagInfo" xmlns="http://javafx.com/javafx/11" xmlns:fx="http://javafx.com/fxml/1">
   <GridPane HBox.hgrow="ALWAYS">
     <columnConstraints>
       <ColumnConstraints hgrow="SOMETIMES" minWidth="10" prefWidth="150" />