Merge pull request #199 from DropSnorz/fix/welcome-view

Fix various issues on the welcome dialog
DropSnorz · Aug 17, 2023 · a36c1b1 · a36c1b1
2 parents 5daa508 + 17e5f32
commit a36c1b1
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 5 deletions.
diff --git a/owlplug-client/src/main/java/com/owlplug/core/components/DialogManager.java b/owlplug-client/src/main/java/com/owlplug/core/components/DialogManager.java
@@ -28,22 +28,29 @@
 import org.springframework.stereotype.Component;
 import org.springframework.stereotype.Controller;
 
+import java.util.Stack;
+
 @Component
 public class DialogManager {
 
   @Autowired
   private MainController mainController;
 
-  private JFXDialog dialog;
+  private Stack<JFXDialog> dialogStack = new Stack<>();
 
   /**
    * Creates a new dialog.
    * 
    * @return
    */
   public JFXDialog newDialog() {
-    dialog = new JFXDialog();
+    JFXDialog dialog = new JFXDialog();
     dialog.setDialogContainer(mainController.getRootPane());
+    dialogStack.push(dialog);
+
+    dialog.setOnDialogClosed(e -> {
+      dialogStack.pop();
+    });
 
     return dialog;
   }
@@ -124,7 +131,7 @@ public JFXDialog newDialog(double width, double height, Node body, Node heading)
    */
   public JFXDialog newDialog(JFXDialogLayout layout) {
 
-    newDialog();
+    JFXDialog dialog = newDialog();
     dialog.setContent(layout);
     return dialog;
   }
@@ -143,6 +150,7 @@ public JFXDialog newSimpleInfoDialog(String title, String body) {
    */
   public JFXDialog newSimpleInfoDialog(Node title, Node body) {
     JFXDialogLayout layout = new JFXDialogLayout();
+    JFXDialog dialog = newDialog(layout);
 
     layout.setHeading(title);
     layout.setBody(body);
@@ -154,12 +162,12 @@ public JFXDialog newSimpleInfoDialog(Node title, Node body) {
     });
 
     layout.setActions(button);
-    return newDialog(layout);
+    return dialog;
 
   }
 
   public JFXDialog getDialog() {
-    return dialog;
+    return dialogStack.peek();
   }
 
 }
diff --git a/...ent/src/main/java/com/owlplug/core/controllers/dialogs/ListDirectoryDialogController.java b/...ent/src/main/java/com/owlplug/core/controllers/dialogs/ListDirectoryDialogController.java
@@ -38,13 +38,18 @@ public class ListDirectoryDialogController extends AbstractDialogController impl
 
   @Autowired
   private OptionsController optionsController;
+  @Autowired
+  private WelcomeDialogController welcomeDialogController;
+
   @Autowired
   private LazyViewRegistry lazyViewRegistry;
 
   @FXML
   private JFXButton addDirectoryButton;
   @FXML
   private ListView<String> directoryListView;
+  @FXML
+  private JFXButton closeButton;
 
   private final String newDirectoryItem = "[New directory] (double-click to update)";
   private String currentPreferenceKey;
@@ -56,6 +61,9 @@ public void initialize() {
       observableItems.add(newDirectoryItem);
     });
     directoryListView.setCellFactory(TextFieldListCell.forListView());
+    closeButton.setOnAction(e -> {
+      this.close();
+    });
   }
 
   public void configure (String preferenceKey) {
@@ -87,6 +95,7 @@ public void onChanged(Change<? extends String> change) {
     this.getPreferences().putList(currentPreferenceKey, prefList);
 
     optionsController.refreshView();
+    welcomeDialogController.refreshView();
   }
 
 

diff --git a/owlplug-client/src/main/resources/fxml/dialogs/ListDirectoryView.fxml b/owlplug-client/src/main/resources/fxml/dialogs/ListDirectoryView.fxml
@@ -13,5 +13,10 @@
          </children>
       </HBox>
       <ListView fx:id="directoryListView" editable="true" prefHeight="200.0" prefWidth="200.0" />
+      <HBox alignment="CENTER_RIGHT" prefHeight="50.0" spacing="10.0">
+         <children>
+            <JFXButton fx:id="closeButton" mnemonicParsing="false" text="Close" />
+         </children>
+      </HBox>
    </children>
 </VBox>