fix UI bugs

src/main/java/seedu/address/logic/relationship/AnySearchCommand.java

@@ -48,6 +48,9 @@ public CommandResult execute(Model model) throws CommandException {
         }
         UUID fullOriginUuid = model.getFullUuid(originUuid);
         UUID fullTargetUuid = model.getFullUuid(targetUuid);
+        if (fullOriginUuid == null || originUuid == null) {
+            throw new CommandException(("you have not added the contacts of the people you are looking for!"));
+        }
         if (fullOriginUuid == fullTargetUuid) {
             throw new CommandException("anySearch must be performed between two different persons.");
         }

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

@@ -27,21 +27,30 @@
      */
     public PersonList(ObservableList<Person> personList, ObservableList<Relationship> relationships) {
         super(FXML);
-        setPersonListCardItems(personList, relationships);
+        setPersonListCardItems(personList, relationships, "No Contacts Found :< !"
+                + "Start Adding Contacts with AddPerson !!!");
     }
 
     /**
      * Constructor To intialise an empty Section
      */
-    public PersonList() {
+    public PersonList(String text) {
         super(FXML);
-        displayPlaceholderText("You Have To Use AnySearch First!");
+        displayPlaceholderText(text);
     }
-    public void setPersonListCardItems(ObservableList<Person> personList, ObservableList<Relationship> relationships) {
+
+    /**
+     * set items in personList Section
+     * @param personList
+     * @param relationships
+     * @param text
+     */
+    public void setPersonListCardItems(ObservableList<Person> personList, ObservableList<Relationship> relationships,
+                                       String text) {
         personListCardVbox.getChildren().clear();
         personListScrollPane.setVvalue(0);
         if (personList.size() == 0) {
-            displayPlaceholderText("You Have No Contacts, Start Adding Them!");
+            displayPlaceholderText(text);
             return;
         }
         for (Person p: personList) {
@@ -56,7 +65,7 @@
     }
     private void displayPlaceholderText(String text) {
         CustomPlaceholder customPlaceholder = new CustomPlaceholder(
-                text, 20);
+                text, 30);
         personListCardVbox.getChildren().add(customPlaceholder.getRoot());
     }
 }

src/main/java/seedu/address/ui/personlistsection/AllContactsSection.java

@@ -31,7 +31,8 @@
      *
      */
     public void update(ObservableList<Person> personList, ObservableList<Relationship> relationships) {
-        this.personList.setPersonListCardItems(personList, relationships);
+        this.personList.setPersonListCardItems(personList, relationships, "No Contacts Found :< ! "
+                 + "Start Adding Contacts with AddPerson !!!");
     }
 }
 

src/main/java/seedu/address/ui/personlistsection/AnyListSection.java

@@ -26,17 +26,23 @@
      */
     public AnyListSection() {
         super(FXML);
-        this.personList = new PersonList();
+        this.personList = new PersonList("You Have To Use AnySearch First");
         this.treeMapFlowPane = new TreeMapFlowPane();
         personListPlaceHolder.getChildren().add(this.personList.getRoot());
         treeMapPlaceHolder.getChildren().add(this.treeMapFlowPane.getRoot());
+        treeMapPlaceHolder.setVisible(false);
+        treeMapPlaceHolder.setManaged(false);
     }
     /**
      * Updates the sorted modules in the ModuleListSection.
      *
      */
     public void update(ObservableList<Person> personList, ObservableList<Relationship> relationships) {
-        this.personList.setPersonListCardItems(personList, relationships);
+        this.personList.setPersonListCardItems(personList, relationships, "No Contacts Related "
+                 + "to this Person Found !!\n"
+                 + "Start Adding Relations With AddRelation");
+        treeMapPlaceHolder.setVisible(true);
+        treeMapPlaceHolder.setManaged(true);
         this.treeMapFlowPane.updateFlowPane(personList, relationships);
     }
 }

src/main/java/seedu/address/ui/personlistsection/TreeMapFlowPane.java

@@ -34,7 +34,6 @@ public class TreeMapFlowPane extends UiPart<Region> {
      */
     public TreeMapFlowPane() {
         super(FXML);
-        addEmptyPlaceholderText("Use Any Search First!");
     }
 
     /**

src/main/resources/view/CustomPlaceHolder.fxml

@@ -4,18 +4,26 @@
 
 <?import javafx.scene.control.Label?>
 <?import javafx.scene.layout.HBox?>
-<?import javafx.scene.layout.Priority?>
+<?import javafx.scene.layout.VBox?>
 
-<HBox fx:id="placeholderContainer" alignment="CENTER" fillHeight="true" xmlns="http://javafx.com/javafx/11" xmlns:fx="http://javafx.com/fxml/1" styleClass="placeholder_container">
+<?import javafx.scene.image.ImageView?>
+<?import javafx.scene.image.Image?>
+<VBox fx:id="placeholderVbox" alignment="CENTER" maxHeight="Infinity" xmlns="http://javafx.com/javafx/11" xmlns:fx="http://javafx.com/fxml/1">
     <stylesheets>
         <URL value="@custom-placeholder.css" />
     </stylesheets>
-    <!-- Left Decorative Label -->
-    <HBox fx:id="leftDecoration" HBox.hgrow="ALWAYS" maxWidth="Infinity" alignment="CENTER" styleClass="placeholder_deco"/>
-
-    <!-- Middle Label with Placeholder -->
-    <Label fx:id="placeholderText" HBox.hgrow="NEVER" styleClass="placeholder_text"/>
-
-    <!-- Right Decorative Label -->
-    <HBox fx:id="rightDecoration" HBox.hgrow="ALWAYS" maxWidth="Infinity" alignment="CENTER" styleClass="placeholder_deco"/>
-</HBox>
+    <ImageView fitHeight="200" fitWidth="200" preserveRatio="true" styleClass="main-window_navbar_image">
+        <image>
+            <Image url="@/images/no-results.png"/> <!-- Replace "your_image.png" with your image path -->
+        </image>
+    </ImageView>
+    <HBox fx:id="placeholderContainer" alignment="CENTER" maxWidth="550" prefWidth="550" maxHeight="Infinity" styleClass="placeholder_container">
+        <children>
+            <HBox fx:id="leftDecoration" HBox.hgrow="ALWAYS" maxWidth="Infinity" alignment="CENTER" styleClass="placeholder_deco"/>
+            <!-- Middle Label with Placeholder -->
+            <Label fx:id="placeholderText" HBox.hgrow="NEVER" styleClass="placeholder_text" maxHeight="Infinity" maxWidth="550" alignment="CENTER" wrapText="true"/>
+            <!-- Right Decorative Label -->
+            <HBox fx:id="rightDecoration" HBox.hgrow="ALWAYS" maxWidth="Infinity" alignment="CENTER" styleClass="placeholder_deco"/>
+        </children>
+    </HBox>
+</VBox>

src/main/resources/view/MainWindow.fxml

@@ -12,7 +12,7 @@
 <?import javafx.scene.image.ImageView?>
 
 <fx:root type="javafx.stage.Stage" xmlns="http://javafx.com/javafx/17" xmlns:fx="http://javafx.com/fxml/1"
-         title="GENE-nie" minWidth="400" minHeight="750" onCloseRequest="#handleExit">
+         title="GENE-nie" minWidth="400" minHeight="780" onCloseRequest="#handleExit">
   <icons>
     <Image url="@/images/genie_32x32.png" />
   </icons>
@@ -33,15 +33,15 @@
             <MenuItem fx:id="helpMenuItem" mnemonicParsing="false" onAction="#handleHelp" text="Help" styleClass="menu-bar_item"/>
           </Menu>
         </MenuBar>
-        <HBox styleClass="main-window_navbar" fx:id="mainWindowNavBar">
+        <HBox styleClass="main-window_navbar" fx:id="mainWindowNavBar" alignment="CENTER">
           <HBox styleClass="main-window_navbar_image-container">
             <ImageView fitHeight="150" fitWidth="150" preserveRatio="true" styleClass="main-window_navbar_image">
               <image>
-                <Image url="@/images/genie.png"/> <!-- Replace "your_image.png" with your image path -->
+                <Image url="@/images/gene-ie.png"/> <!-- Replace "your_image.png" with your image path -->
               </image>
             </ImageView>
           </HBox>
-          <HBox styleClass="main-window_navbar-panel" alignment="CENTER" HBox.hgrow="ALWAYS" fx:id="mainWindowNavBarButtonPlaceholder"/>
+          <HBox styleClass="main-window_navbar-panel" alignment="CENTER" fx:id="mainWindowNavBarButtonPlaceholder"/>
         </HBox>
         <HBox alignment="CENTER" spacing="20" fx:id="mainWindowBody" VBox.vgrow="ALWAYS">
           <VBox fx:id="displaySectionPlaceholder" alignment="CENTER"/>

src/main/resources/view/display-section/NavBarButton.fxml

@@ -11,6 +11,6 @@
         <URL value="@nav-bar-button.css" />
     </stylesheets>
 
-    <Button fx:id="footerButton" styleClass="footer-button, h1" />
+    <Button fx:id="footerButton" styleClass="footer-button" />
 </VBox>
 

src/main/resources/view/display-section/nav-bar-button.css

@@ -1,28 +1,29 @@
 .footer-button-container {
     -fx-cursor: hand;
-    -fx-border-radius: 5;
-    -fx-background-radius: 5;
-    -fx-max-height: 100;
+    -fx-border-radius: 20;
+    -fx-background-radius: 20;
+    -fx-max-height: 60;
+    -fx-padding: 0 5 0 5;
 }
 .footer-button-container-selected {
     -fx-scale-x: 1.1;
     -fx-scale-y: 1.1;
     -fx-background-color: -rose-red;
 }
 .footer-button {
-    -fx-font-family: 'Raleway SemiBold Italic';
+    -fx-font-family: 'Raleway SemiBold';
     -fx-text-fill: -rose-red;
     -fx-background-color: transparent;
     -font-weight: 700;
+    -fx-font-size: 20;
 }
 .footer-button-selected {
     -fx-text-fill: -white;
 }
 
 .footer-button-container:hover,
-.footer-button-selected-deco,
-.footer-button:hover {
-    -fx-opacity: 0.75;
+.footer-button-selected-deco {
+    -fx-opacity: 0.9;
     -fx-translate-y: -3;
     -fx-effect: dropshadow(gaussian, rgba(0,0,0,0.3), 15, 0.0, 5, 10);
 }

src/main/resources/view/person-list-section/PersonList.fxml

@@ -7,7 +7,7 @@
 
 <VBox fx:id="personListVbox" maxHeight="-Infinity" maxWidth="-Infinity" minHeight="400.0" minWidth="400.0" stylesheets="@person-list.css" VBox.vgrow="ALWAYS" xmlns="http://javafx.com/javafx/11" xmlns:fx="http://javafx.com/fxml/1" alignment="CENTER">
    <children>
-      <ScrollPane id="personListContainerId" fx:id="personListScrollPane" hbarPolicy="NEVER" minHeight="500.0" minViewportHeight="500.0" styleClass="personList_container" stylesheets="@person-list.css" vbarPolicy="ALWAYS">
+      <ScrollPane id="personListContainerId" fx:id="personListScrollPane" hbarPolicy="NEVER" minHeight="500.0" minViewportHeight="500.0" styleClass="personList_container" stylesheets="@person-list.css" vbarPolicy="ALWAYS" fitToWidth="true">
          <content>
             <VBox fillWidth="true" fx:id="personListCardVbox" alignment="CENTER" maxHeight="1.7976931348623157E308" minHeight="500.0" minWidth="550.0" styleClass="personList_vbox_container" stylesheets="@person-list.css" spacing="10" VBox.vgrow="ALWAYS">
                <effect>

src/main/resources/view/theme.css

@@ -86,8 +86,8 @@
     -fx-text-fill: -black;
 }
 .main-window_navbar {
-    -fx-padding: 40 150 0 100;
-    -fx-spacing: 120;
+    -fx-padding: 20;
+    -fx-spacing: 80;
 }
 .main-window_navbar_image-container {
 

src/test/java/seedu/address/logic/relationship/AnySearchCommandTest.java

@@ -51,6 +51,12 @@ public void testListFilteredCorrectlyForInvalidAnySearch() throws CommandExcepti
         assertEquals("No Relationship pathway found", result.getFeedbackToUser());
     }
     @Test
+    public void testInputsNotInAddressBook() {
+        AnySearchCommand anySearchCommand1 = new AnySearchCommand("0020", "0020");
+        assertCommandFailure(anySearchCommand1, model, "you have not added the contacts "
+                + "of the people you are looking for!");
+    }
+    @Test
     void testEqualsSameObject() {
         AnySearchCommand command = new AnySearchCommand("uuid1", "uuid2");
         assertTrue(command.equals(command));