Disable clicking Tags and Notes fields while in save confirmation alert

Tags and notes are clickable when the window is showing save confirmation alert, which is causing UI issues. Disable clicking these fields when the message is being shown.
AY2324S1-CS2103T-T13-4 · Nov 9, 2023 · d11fa30 · d11fa30
1 parent 68388fb
commit d11fa30
Show file tree

Hide file tree

Showing 4 changed files with 21 additions and 5 deletions.
diff --git a/src/main/java/seedu/address/ui/PersonProfile.java b/src/main/java/seedu/address/ui/PersonProfile.java
@@ -481,6 +481,8 @@ public boolean isStillEditing() {
     public void setIsInConfirmationDialog(boolean isInConfirmationDialog) {
         uiElements.values().stream()
                 .forEach(field -> field.setIsInConfirmationDialog(isInConfirmationDialog));
+        tagUI.setIsInConfirmationDialog(isInConfirmationDialog);
+        noteUI.setIsInConfirmationDialog(isInConfirmationDialog);
     }
 
     /**

diff --git a/src/main/java/seedu/address/ui/PersonProfileNote.java b/src/main/java/seedu/address/ui/PersonProfileNote.java
@@ -30,6 +30,8 @@ public class PersonProfileNote extends UiPart<SplitPane> {
     private final PersonProfile personProfile;
     private String value;
     private State state;
+    private boolean isInConfirmationDialog;
+
     // endregion
 
     // region Enums
@@ -184,14 +186,20 @@ private void handleLoseFocus() {
 
     @FXML
     void setFocus() {
-        personProfile.triggerEvent(PersonProfile.Event.BEFORE_START_EDIT);
-        updateState(State.EDITING);
+        if (!isInConfirmationDialog) {
+            personProfile.triggerEvent(PersonProfile.Event.BEFORE_START_EDIT);
+            updateState(State.EDITING);
+        }
     }
 
     boolean isEditing() {
         return state == State.EDITING;
     }
 
+    public void setIsInConfirmationDialog(boolean isInConfirmationDialog) {
+        this.isInConfirmationDialog = isInConfirmationDialog;
+    }
+
     // endregion
 
 }
diff --git a/src/main/java/seedu/address/ui/PersonProfileTags.java b/src/main/java/seedu/address/ui/PersonProfileTags.java
@@ -44,6 +44,7 @@ public class PersonProfileTags extends UiPart<SplitPane> {
     private final PersonProfile personProfile;
     private Set<Tag> tags;
     private State state;
+    private boolean isInConfirmationDialog;
     // endregion
 
     // region Enums
@@ -228,14 +229,20 @@ private void handleLoseFocus() {
 
     @FXML
     void setFocus() {
-        personProfile.triggerEvent(PersonProfile.Event.BEFORE_START_EDIT);
-        updateState(State.EDITING);
+        if (!isInConfirmationDialog) {
+            personProfile.triggerEvent(PersonProfile.Event.BEFORE_START_EDIT);
+            updateState(State.EDITING);
+        }
     }
 
     boolean isEditing() {
         return state == State.EDITING;
     }
 
+    public void setIsInConfirmationDialog(boolean isInConfirmationDialog) {
+        this.isInConfirmationDialog = isInConfirmationDialog;
+    }
+
     // endregion
 
 }
diff --git a/src/main/java/seedu/address/ui/UiPart.java b/src/main/java/seedu/address/ui/UiPart.java
@@ -84,5 +84,4 @@ private static URL getFxmlFileUrl(String fxmlFileName) {
         URL fxmlFileUrl = MainApp.class.getResource(fxmlFileNameWithFolder);
         return requireNonNull(fxmlFileUrl);
     }
-
 }