X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/101b4778087e78ba64e7dd700ea4b01b9c272f5b..9c8cfcb9c56798ebd29184e40227451a194c0c49:/src/motif/menu.cpp diff --git a/src/motif/menu.cpp b/src/motif/menu.cpp index 46e9e5a15f..7ea74148a6 100644 --- a/src/motif/menu.cpp +++ b/src/motif/menu.cpp @@ -1,5 +1,5 @@ ///////////////////////////////////////////////////////////////////////////// -// Name: menu.cpp +// Name: src/motif/menu.cpp // Purpose: wxMenu, wxMenuBar, wxMenuItem // Author: Julian Smart // Modified by: @@ -9,15 +9,10 @@ // Licence: wxWindows licence ///////////////////////////////////////////////////////////////////////////// - // ============================================================================ // declarations // ============================================================================ -#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA) - #pragma implementation "menu.h" -#endif - // ---------------------------------------------------------------------------- // headers // ---------------------------------------------------------------------------- @@ -25,13 +20,15 @@ // For compilers that support precompilation, includes "wx.h". #include "wx/wxprec.h" -#include "wx/defs.h" - #include "wx/menu.h" + +#ifndef WX_PRECOMP + #include "wx/log.h" + #include "wx/app.h" + #include "wx/utils.h" +#endif + #include "wx/menuitem.h" -#include "wx/log.h" -#include "wx/utils.h" -#include "wx/app.h" #include "wx/frame.h" #include "wx/settings.h" @@ -119,18 +116,7 @@ void wxMenu::Break() // function appends a new item or submenu to the menu wxMenuItem* wxMenu::DoAppend(wxMenuItem *pItem) { - if (m_menuWidget) - { - // this is a dynamic Append - pItem->CreateItem(m_menuWidget, GetMenuBar(), m_topLevelMenu); - } - - if ( pItem->IsSubMenu() ) - { - pItem->GetSubMenu()->m_topLevelMenu = m_topLevelMenu; - } - - return wxMenuBase::DoAppend(pItem); + return DoInsert(GetMenuItemCount(), pItem); } wxMenuItem *wxMenu::DoRemove(wxMenuItem *item) @@ -142,12 +128,22 @@ wxMenuItem *wxMenu::DoRemove(wxMenuItem *item) wxMenuItem* wxMenu::DoInsert(size_t pos, wxMenuItem *item) { - if ( wxMenuBase::DoInsert(pos, item) ) - return item; + if (m_menuWidget) + { + // this is a dynamic Append +#ifndef XmNpositionIndex + wxCHECK_MSG( pos == GetMenuItemCount(), -1, wxT("insert not implemented")); +#endif + item->CreateItem(m_menuWidget, GetMenuBar(), m_topLevelMenu, pos); + } - wxFAIL_MSG(wxT("DoInsert not implemented; or error in wxMenuBase::DoInsert")); + if ( item->IsSubMenu() ) + { + item->GetSubMenu()->m_topLevelMenu = m_topLevelMenu; + } - return NULL; + return pos == GetMenuItemCount() ? wxMenuBase::DoAppend(item) : + wxMenuBase::DoInsert(pos, item); } void wxMenu::SetTitle(const wxString& label) @@ -173,15 +169,6 @@ bool wxMenu::ProcessCommand(wxCommandEvent & event) { bool processed = false; -#if wxUSE_MENU_CALLBACK - // Try a callback - if (m_callback) - { - (void) (*(m_callback)) (*this, event); - processed = true; - } -#endif // wxUSE_MENU_CALLBACK - // Try the menu's event handler if ( !processed && GetEventHandler()) { @@ -281,30 +268,27 @@ wxString wxMenuBar::GetLabelTop(size_t pos) const bool wxMenuBar::Append(wxMenu * menu, const wxString& title) { + return Insert(GetMenuCount(), menu, title); +} + +bool wxMenuBar::Insert(size_t pos, wxMenu *menu, const wxString& title) +{ + wxCHECK_MSG( pos <= GetMenuCount(), false, wxT("invalid position") ); wxCHECK_MSG( menu, false, wxT("invalid menu") ); wxCHECK_MSG( !menu->GetParent() && !menu->GetButtonWidget(), false, wxT("menu already appended") ); if ( m_menuBarFrame ) { - WXWidget w = menu->CreateMenu(this, GetMainWidget(), menu, title, true); + WXWidget w = menu->CreateMenu(this, GetMainWidget(), menu, + pos, title, true); wxCHECK_MSG( w, false, wxT("failed to create menu") ); menu->SetButtonWidget(w); } - m_titles.Add(title); + m_titles.Insert(title, pos); - return wxMenuBarBase::Append(menu, title); -} - -bool wxMenuBar::Insert(size_t pos, wxMenu *menu, const wxString& title) -{ - if ( !wxMenuBarBase::Insert(pos, menu, title) ) - return false; - - wxFAIL_MSG(wxT("TODO")); - - return false; + return wxMenuBarBase::Insert(pos, menu, title); } wxMenu *wxMenuBar::Replace(size_t pos, wxMenu *menu, const wxString& title) @@ -356,12 +340,14 @@ wxMenuItem *wxMenuBar::FindItem(int id, wxMenu ** itemMenu) const if (itemMenu) *itemMenu = NULL; - wxMenuItem *item = NULL; size_t menuCount = GetMenuCount(); for (size_t i = 0; i < menuCount; i++) - if ((item = m_menus.Item(i)->GetData()->FindItem(id, itemMenu))) - return item; - return NULL; + { + wxMenuItem *item = m_menus.Item(i)->GetData()->FindItem(id, itemMenu); + if (item) return item; + } + + return NULL; } // Create menubar @@ -378,7 +364,8 @@ bool wxMenuBar::CreateMenuBar(wxFrame* parent) return true; } - Widget menuBarW = XmCreateMenuBar ((Widget) parent->GetMainWidget(), "MenuBar", NULL, 0); + Widget menuBarW = XmCreateMenuBar ((Widget) parent->GetMainWidget(), + wxMOTIF_STR("MenuBar"), NULL, 0); m_mainWidget = (WXWidget) menuBarW; size_t menuCount = GetMenuCount(); @@ -386,7 +373,7 @@ bool wxMenuBar::CreateMenuBar(wxFrame* parent) { wxMenu *menu = GetMenu(i); wxString title(m_titles[i]); - menu->SetButtonWidget(menu->CreateMenu (this, menuBarW, menu, title, true)); + menu->SetButtonWidget(menu->CreateMenu (this, menuBarW, menu, i, title, true)); if (strcmp (wxStripMenuCodes(title), "Help") == 0) XtVaSetValues ((Widget) menuBarW, XmNmenuHelpWidget, (Widget) menu->GetButtonWidget(), NULL); @@ -482,7 +469,12 @@ void wxMenu::DestroyWidgetAndDetach() * */ -WXWidget wxMenu::CreateMenu (wxMenuBar * menuBar, WXWidget parent, wxMenu * topMenu, const wxString& title, bool pullDown) +WXWidget wxMenu::CreateMenu (wxMenuBar * menuBar, + WXWidget parent, + wxMenu * topMenu, + size_t WXUNUSED(index), + const wxString& title, + bool pullDown) { Widget menu = (Widget) 0; Widget buttonWidget = (Widget) 0; @@ -492,7 +484,7 @@ WXWidget wxMenu::CreateMenu (wxMenuBar * menuBar, WXWidget parent, wxMenu * topM if (!pullDown) { - menu = XmCreatePopupMenu ((Widget) parent, "popup", args, 2); + menu = XmCreatePopupMenu ((Widget) parent, wxMOTIF_STR("popup"), args, 2); #if 0 XtAddCallback(menu, XmNunmapCallback, @@ -503,7 +495,7 @@ WXWidget wxMenu::CreateMenu (wxMenuBar * menuBar, WXWidget parent, wxMenu * topM else { char mnem = wxFindMnemonic (title); - menu = XmCreatePulldownMenu ((Widget) parent, "pulldown", args, 2); + menu = XmCreatePulldownMenu ((Widget) parent, wxMOTIF_STR("pulldown"), args, 2); wxString title2(wxStripMenuCodes(title)); wxXmString label_str(title2); @@ -525,13 +517,14 @@ WXWidget wxMenu::CreateMenu (wxMenuBar * menuBar, WXWidget parent, wxMenu * topM m_topLevelMenu = topMenu; + size_t i = 0; for ( wxMenuItemList::compatibility_iterator node = GetMenuItems().GetFirst(); node; - node = node->GetNext() ) + node = node->GetNext(), ++i ) { wxMenuItem *item = node->GetData(); - item->CreateItem(menu, menuBar, topMenu); + item->CreateItem(menu, menuBar, topMenu, i); } SetBackgroundColour(m_backgroundColour); @@ -687,6 +680,8 @@ void wxMenu::ChangeFont(bool keepOriginalSize) if (item->GetSubMenu()) item->GetSubMenu()->ChangeFont(keepOriginalSize); } +#else + wxUnusedVar(keepOriginalSize); #endif } @@ -738,4 +733,3 @@ bool wxMenuBar::SetFont(const wxFont& font) return true; } -