Fix 'x' not doing anything in About menu

Author: SwiftWinds

src/main/java/com/airsquared/blobsaver/Controller.java

@@ -28,6 +28,12 @@
 import javafx.event.ActionEvent;
 import javafx.event.EventHandler;
 import javafx.fxml.FXML;
+import javafx.scene.control.Button;
+import javafx.scene.control.Label;
+import javafx.scene.control.Menu;
+import javafx.scene.control.MenuBar;
+import javafx.scene.control.MenuItem;
+import javafx.scene.control.TextField;
 import javafx.scene.control.*;
 import javafx.scene.effect.DropShadow;
 import javafx.scene.input.MouseEvent;
@@ -52,12 +58,7 @@
 
 import static com.airsquared.blobsaver.Main.appPrefs;
 import static com.airsquared.blobsaver.Main.primaryStage;
-import static com.airsquared.blobsaver.Shared.githubIssue;
-import static com.airsquared.blobsaver.Shared.newReportableError;
-import static com.airsquared.blobsaver.Shared.newUnreportableError;
-import static com.airsquared.blobsaver.Shared.redditPM;
-import static com.airsquared.blobsaver.Shared.reportError;
-import static com.airsquared.blobsaver.Shared.resizeAlertButtons;
+import static com.airsquared.blobsaver.Shared.*;
 
 public class Controller {
 
@@ -335,7 +336,7 @@ public void initialize() {
                 @Override
                 public void handle(WindowEvent event) {
                     useMacOSMenuBar();
-                    log("using macos menu bar");
+                    log("using macOS menu bar");
                     primaryStage.removeEventHandler(event.getEventType(), this);
                 }
             });
@@ -851,11 +852,23 @@ public void aboutMenuHandler() {
         ButtonType viewLicense = new ButtonType("View License");
         ButtonType librariesUsed = new ButtonType("Libraries Used");
         ButtonType donate = new ButtonType("Donate!");
-        Alert alert = new Alert(Alert.AlertType.INFORMATION, "About text here", librariesUsed, viewLicense, donate, githubRepo, ButtonType.OK);
+        ButtonType customOK = new ButtonType("OK", ButtonBar.ButtonData.CANCEL_CLOSE);
+        Alert alert = new Alert(Alert.AlertType.INFORMATION, "About text here",
+                librariesUsed, viewLicense, donate, githubRepo, customOK);
         alert.setTitle("About");
+
+        //Deactivate default behavior for librariesUsed Button:
+        Button libButton = (Button) alert.getDialogPane().lookupButton(librariesUsed);
+        libButton.setDefaultButton(false);
+
+        //Activate default behavior for OK-Button:
+        Button OkButton = (Button) alert.getDialogPane().lookupButton(customOK);
+        OkButton.setDefaultButton(true);
+
         alert.setHeaderText("blobsaver " + Main.appVersion);
         alert.setContentText("blobsaver Copyright (c) 2018  airsquared\n\n" +
                 "This program is licensed under GNU GPL v3.0-only");
+        
         resizeAlertButtons(alert);
         alert.showAndWait();
         switch (alert.getResult().getText()) {
@@ -885,7 +898,7 @@ public void aboutMenuHandler() {
                     out.close();
                     licenseFile.deleteOnExit();
                     licenseFile.setReadOnly();
-                    java.awt.Desktop.getDesktop().edit(licenseFile);
+                    Desktop.getDesktop().edit(licenseFile);
                 } catch (IOException e) {
                     e.printStackTrace();
                 }
@@ -909,7 +922,7 @@ public void aboutMenuHandler() {
                     out.close();
                     libsUsedFile.deleteOnExit();
                     libsUsedFile.setReadOnly();
-                    java.awt.Desktop.getDesktop().edit(libsUsedFile);
+                    Desktop.getDesktop().edit(libsUsedFile);
                 } catch (IOException e) {
                     e.printStackTrace();
                 }
@@ -956,7 +969,7 @@ private void useMacOSMenuBar() {
         windowMenu.getItems().add(tk.createMinimizeMenuItem());
         windowMenu.getItems().add(tk.createCycleWindowsItem());
 
-        MenuItem debugLogMenuItem = new MenuItem("Open/Close Debug log");
+        MenuItem debugLogMenuItem = new MenuItem("Open/Close Debug Log");
         debugLogMenuItem.setOnAction(event -> {
             debugLogHandler();
             tk.setMenuBar(DebugWindow.getDebugStage(), macOSMenuBar);
@@ -1135,7 +1148,9 @@ public void resetAppHandler() {
         try {
             Alert confirmationAlert = new Alert(Alert.AlertType.CONFIRMATION, "Are you sure you would like to uninstall this application?", ButtonType.NO, ButtonType.YES);
             confirmationAlert.showAndWait();
-            if ((confirmationAlert.getResult() == null) || ButtonType.CANCEL.equals(confirmationAlert.getResult()) || ButtonType.NO.equals(confirmationAlert.getResult())) {
+            if ((confirmationAlert.getResult() == null)
+                    || ButtonType.CANCEL.equals(confirmationAlert.getResult())
+                    || ButtonType.NO.equals(confirmationAlert.getResult())) {
                 return;
             }
             Preferences prefs = Preferences.userRoot().node("airsquared/blobsaver");

src/main/resources/com/airsquared/blobsaver/blobsaver.fxml

@@ -26,7 +26,7 @@
     <VBox maxHeight="Infinity" maxWidth="-Infinity" minHeight="580.0" prefWidth="500.0">
         <MenuBar fx:id="menuBar">
             <Menu mnemonicParsing="false" text="Options">
-                <MenuItem mnemonicParsing="false" onAction="#debugLogHandler" text="Debug log"/>
+                <MenuItem mnemonicParsing="false" onAction="#debugLogHandler" text="Debug Log"/>
                 <MenuItem mnemonicParsing="false" onAction="#checkBlobs" text="Check for valid blobs..."/>
                 <MenuItem mnemonicParsing="false" onAction="#checkForUpdatesHandler"
                           text="Check for Updates..."/>
@@ -48,8 +48,8 @@
                               text="How do I get a crash log?"/>
                 </Menu>
                 <MenuItem mnemonicParsing="false" onAction="#newGithubIssue"
-                          text="Send Feedback(Github Issue)"/>
-                <MenuItem mnemonicParsing="false" onAction="#sendRedditPM" text="Send Feedback(Reddit PM)"/>
+                          text="Send Feedback (Github Issue)"/>
+                <MenuItem mnemonicParsing="false" onAction="#sendRedditPM" text="Send Feedback (Reddit PM)"/>
                 <SeparatorMenuItem mnemonicParsing="false"/>
                 <MenuItem mnemonicParsing="false" onAction="#donate" text="Donate!"/>
                 <SeparatorMenuItem mnemonicParsing="false"/>