From ceb7198c47e59a3256af7a957c20a55a3329a7cb Mon Sep 17 00:00:00 2001
From: LimZiJia <zijialim@gmail.com>
Date: Mon, 26 Feb 2024 05:39:47 +0800
Subject: [PATCH] Improve JavaFX

---
 data/duke.txt                          |  8 ++---
 src/main/duke/ui/DialogBox.java        | 14 +++++++--
 src/main/resources/view/DialogBox.fxml | 41 +++++++++++---------------
 3 files changed, 32 insertions(+), 31 deletions(-)

diff --git a/data/duke.txt b/data/duke.txt
index ab2ec37b2a..0287644654 100644
--- a/data/duke.txt
+++ b/data/duke.txt
@@ -1,5 +1,3 @@
-T,0,1
-T,0,2
-T,0,3
-T,0,4
-T,0,5
+T,0,2106_lab_2
+E,0,mid_terms,2024-03-01,2024-03-14
+D,0,2102_assignment_1,2024-03-07
diff --git a/src/main/duke/ui/DialogBox.java b/src/main/duke/ui/DialogBox.java
index 5eb43e78e7..58461de7ba 100644
--- a/src/main/duke/ui/DialogBox.java
+++ b/src/main/duke/ui/DialogBox.java
@@ -18,6 +18,7 @@
 import javafx.scene.layout.CornerRadii;
 import javafx.scene.layout.HBox;
 import javafx.scene.paint.Color;
+import javafx.scene.shape.Circle;
 
 /**
  * An example of a custom control using FXML.
@@ -41,9 +42,16 @@ private DialogBox(String text, Image img, Color c) {
             e.printStackTrace();
         }
 
+        Circle clip = new Circle();
+        clip.setCenterX(90);
+        clip.setCenterY(90);
+        clip.setRadius(82);
         dialog.setText(text);
         displayPicture.setImage(img);
-        setBackground(new Background(new BackgroundFill(c, CornerRadii.EMPTY, Insets.EMPTY)));
+        displayPicture.setClip(clip);
+        setBackground(new Background(new BackgroundFill(c,
+                new CornerRadii(30),
+                new Insets(5, 5, 10, 10))));
     }
 
     /**
@@ -57,11 +65,11 @@ private void flip() {
     }
 
     public static DialogBox getUserDialog(String text, Image img) {
-        return new DialogBox(text, img, Color.AQUA);
+        return new DialogBox(text, img, Color.LIGHTSTEELBLUE);
     }
 
     public static DialogBox getDukeDialog(String text, Image img) {
-        var db = new DialogBox(text, img, Color.AQUAMARINE);
+        var db = new DialogBox(text, img, Color.LIGHTGRAY);
         db.flip();
         return db;
     }
diff --git a/src/main/resources/view/DialogBox.fxml b/src/main/resources/view/DialogBox.fxml
index d06e206165..dd7fad9025 100644
--- a/src/main/resources/view/DialogBox.fxml
+++ b/src/main/resources/view/DialogBox.fxml
@@ -2,35 +2,30 @@
 
 <?import javafx.geometry.Insets?>
 <?import javafx.scene.control.Label?>
-<?import javafx.scene.control.SplitPane?>
 <?import javafx.scene.image.ImageView?>
 <?import javafx.scene.layout.AnchorPane?>
 <?import javafx.scene.layout.HBox?>
 <?import javafx.scene.text.Font?>
 
-<fx:root alignment="TOP_RIGHT" fillHeight="false" maxHeight="1.7976931348623157E308" maxWidth="1.7976931348623157E308" type="javafx.scene.layout.HBox" xmlns="http://javafx.com/javafx/21" xmlns:fx="http://javafx.com/fxml/1">
+<fx:root alignment="CENTER" fillHeight="false" maxHeight="1.7976931348623157E308" maxWidth="1.7976931348623157E308" nodeOrientation="LEFT_TO_RIGHT" prefHeight="240.0" prefWidth="498.0" type="javafx.scene.layout.HBox" xmlns="http://javafx.com/javafx/21" xmlns:fx="http://javafx.com/fxml/1">
     <children>
-      <SplitPane dividerPositions="0.7108" prefWidth="500.0">
-        <items>
-          <AnchorPane minHeight="0.0" minWidth="0.0" prefHeight="198.0" prefWidth="307.0">
-               <children>
-                    <Label fx:id="dialog" prefHeight="198.0" prefWidth="296.0" text="Label" AnchorPane.bottomAnchor="0.0" AnchorPane.leftAnchor="5.0" AnchorPane.rightAnchor="5.0" AnchorPane.topAnchor="0.0">
-                     <padding>
-                        <Insets bottom="10.0" left="10.0" right="10.0" top="10.0" />
-                     </padding>
-                     <font>
-                        <Font name="Consolas" size="12.0" />
-                     </font>
-                  </Label>
-               </children>
-            </AnchorPane>
-          <AnchorPane minHeight="0.0" minWidth="0.0" prefHeight="160.0" prefWidth="100.0">
-               <children>
-                    <ImageView fx:id="displayPicture" fitHeight="209.0" fitWidth="148.0" layoutX="-7.0" layoutY="-8.0" pickOnBounds="true" />
-               </children>
-            </AnchorPane>
-        </items>
-      </SplitPane>
+    <AnchorPane minHeight="0.0" minWidth="0.0" prefHeight="240.0" prefWidth="300.0">
+         <children>
+              <Label fx:id="dialog" layoutX="6.0" prefHeight="240.0" prefWidth="300.0" text="Label" AnchorPane.bottomAnchor="0.0" AnchorPane.leftAnchor="6.0" AnchorPane.rightAnchor="-20.0" AnchorPane.topAnchor="0.0">
+               <padding>
+                  <Insets bottom="10.0" left="10.0" right="10.0" top="10.0" />
+               </padding>
+               <font>
+                  <Font name="Consolas" size="9.0" />
+               </font>
+            </Label>
+         </children>
+      </AnchorPane>
+    <AnchorPane minHeight="0.0" minWidth="0.0" nodeOrientation="LEFT_TO_RIGHT" prefHeight="240.0" prefWidth="188.0">
+         <children>
+              <ImageView fx:id="displayPicture" fitHeight="171.0" fitWidth="170.0" layoutX="9.0" layoutY="35.0" pickOnBounds="true" />
+         </children>
+      </AnchorPane>
     </children>
    <opaqueInsets>
       <Insets />