Further refine of #15226: wxRichTextCtrl: Implement setting properties with undo...

[wxWidgets.git] / src / xrc / xh_ribbon.cpp

diff --git a/src/xrc/xh_ribbon.cpp b/src/xrc/xh_ribbon.cpp
index 872803318647c2e583eef215acd3abd87b5b0039..cc352103f61440a957eef8eb554dc4028e77cbd4 100644 (file)
--- a/src/xrc/xh_ribbon.cpp
+++ b/src/xrc/xh_ribbon.cpp
@@ -3,7 +3,6 @@
 // Purpose:     XML resource handler for wxRibbon related classes
 // Author:      Armel Asselin
 // Created:     2010-04-23
-// RCS-ID:      $Id$
 // Copyright:   (c) 2010 Armel Asselin
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
@@ -25,6 +24,10 @@
 
 #include "wx/scopeguard.h"
 
+#ifndef WX_PRECOMP
+    #include "wx/menu.h"
+#endif
+
 // Ribbon bars can contain only pages which are usually panels but may contain
 // any wxWindow.
 //
@@ -142,7 +145,12 @@ wxObject* wxRibbonXmlHandler::Handle_button()
     if (GetBool(wxT("hybrid")))
         kind = wxRIBBON_BUTTON_HYBRID;
 
-#if wxUSE_MENUS
+    // FIXME: The code below uses wxXmlNode directly but this can't be done
+    //        in the ribbon library code as it would force it to always link
+    //        with the xml library. Disable it for now but the real solution
+    //        would be to virtualize GetChildren() and GetNext() methods via
+    //        wxXmlResourceHandler, just as we already do for many others.
+#if 0 // wxUSE_MENUS
     // check whether we have dropdown tag inside
     wxMenu *menu = NULL; // menu for drop down items
     wxXmlNode * const nodeDropdown = GetParamNode("dropdown");