Refactored function to get status of menus in MacOSX

madMAx43v3r · Sep 28, 2017 · 369b23a · 369b23a
1 parent 59a57a9
commit 369b23a
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/app/src/processing/app/Base.java b/app/src/processing/app/Base.java
@@ -971,7 +971,7 @@ public boolean handleQuit() {
       // Save out the current prefs state
       PreferencesData.save();
 
-      if (!OSUtils.isMacOS() || System.getProperty("apple.laf.useScreenMenuBar") == "false") {
+      if (!OSUtils.hasMacOSStyleMenus()) {
         // If this was fired from the menu or an AppleEvent (the Finder),
         // then Mac OS X will send the terminate signal itself.
         System.exit(0);

diff --git a/app/src/processing/app/Editor.java b/app/src/processing/app/Editor.java
@@ -619,7 +619,7 @@ public void actionPerformed(ActionEvent e) {
     fileMenu.add(item);
 
     // macosx already has its own preferences and quit menu
-    if (!OSUtils.isMacOS() || System.getProperty("apple.laf.useScreenMenuBar") == "false") {
+    if (!OSUtils.hasMacOSStyleMenus()) {
       fileMenu.addSeparator();
 
       item = newJMenuItem(tr("Preferences"), ',');
@@ -1252,7 +1252,7 @@ public void actionPerformed(ActionEvent e) {
     menu.add(item);
 
     // macosx already has its own about menu
-    if (!OSUtils.isMacOS() || System.getProperty("apple.laf.useScreenMenuBar") == "false") {
+    if (!OSUtils.hasMacOSStyleMenus()) {
       menu.addSeparator();
       item = new JMenuItem(tr("About Arduino"));
       item.addActionListener(new ActionListener() {
@@ -1812,7 +1812,7 @@ protected boolean checkModified() {
     String prompt = I18n.format(tr("Save changes to \"{0}\"?  "),
                                 sketch.getName());
 
-    if (!OSUtils.isMacOS() || System.getProperty("apple.laf.useScreenMenuBar") == "false") {
+    if (!OSUtils.hasMacOSStyleMenus()) {
       int result =
         JOptionPane.showConfirmDialog(this, prompt, tr("Close"),
                                       JOptionPane.YES_NO_CANCEL_OPTION,

diff --git a/arduino-core/src/processing/app/helpers/OSUtils.java b/arduino-core/src/processing/app/helpers/OSUtils.java
@@ -26,4 +26,7 @@ static public boolean isMacOS() {
     return System.getProperty("os.name").contains("Mac");
   }
 
+  static public boolean hasMacOSStyleMenus() {
+    return OSUtils.isMacOS() && System.getProperty("apple.laf.useScreenMenuBar").equals("true");
+  }
 }