From: Kevin Hock <hockkn@yahoo.com>
Date: Wed, 6 Oct 2004 16:15:30 +0000 (+0000)
Subject: Added back compile-time check for about/prefs menu separator; OS 9 should not have... 
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/262ce38a2af9de50f6efca91463123c75a237ed4

Added back compile-time check for about/prefs menu separator; OS 9 should not have this separator appended. Can be changed later if needed to run-time check.  This is better than nothing before 2.5.3.


git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@29671 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---

diff --git a/src/mac/carbon/menu.cpp b/src/mac/carbon/menu.cpp
index 8c0a41ab45..37e4141243 100644
--- a/src/mac/carbon/menu.cpp
+++ b/src/mac/carbon/menu.cpp
@@ -551,10 +551,13 @@ void wxMenuBar::MacInstallMenuBar()
     verify_noerr( CreateNewMenu( kwxMacAppleMenuId , 0 , &appleMenu ) ) ;
     verify_noerr( SetMenuTitle( appleMenu , (ConstStr255Param) appleMenuTitle ) );
 
-//RN: under 10.3 it is a standard to include a seperator between About and Preferences
-//    10.2 and less are inconsistant about this, but in the interest of being more
-//    consistant with future OS's, we're choosing to include the seperator.
+    // Add About/Preferences separator only on OS X
+    // KH/RN: Separator is always present on 10.3 but not on 10.2
+    // However, the change from 10.2 to 10.3 suggests it is preferred
+#if TARGET_API_MAC_OSX
     MacInsertMenuItem( appleMenu , "\p-" , 0 ) ;
+#endif
+
     MacInsertMenuItem( appleMenu , "\pAbout..." , 0 ) ;
     MacInsertMenu( appleMenu , 0 ) ;