Amebis/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Ensure that submenus are also added to the system when their parent

Browse Source 
menu is used in a wxMenuBar::Repalce call.


git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/branches/WX_2_4_BRANCH@20262 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
Robin Dunn
2003-04-17 20:06:17 +00:00
parent 99a5137c2d
commit 4a1a1d848d
2 changed files with 112 additions and 72 deletions
Download Patch File Download Diff File Expand all files Collapse all files

92
src/mac/carbon/menu.cpp
View File

@@ -71,7 +71,7 @@ void wxMenu::Init()
// create the menu // create the menu
Str255 label; Str255 label;
wxMenuItem::MacBuildMenuString( label, NULL , NULL , m_title , false ); wxMenuItem::MacBuildMenuString( label, NULL , NULL , m_title , false );
m_macMenuId = s_macNextMenuId++; m_macMenuId = s_macNextMenuId++;
m_hMenu = ::NewMenu(m_macMenuId, label); m_hMenu = ::NewMenu(m_macMenuId, label);
if ( !m_hMenu ) if ( !m_hMenu )
@@ -172,7 +172,7 @@ bool wxMenu::DoInsertOrAppend(wxMenuItem *pItem, size_t pos)
MacInsertMenuItem(MAC_WXHMENU(m_hMenu), "\p-" , pos); MacInsertMenuItem(MAC_WXHMENU(m_hMenu), "\p-" , pos);
} }
} }
else else
{ {
wxMenu *pSubMenu = pItem->GetSubMenu() ; wxMenu *pSubMenu = pItem->GetSubMenu() ;
if ( pSubMenu != NULL ) if ( pSubMenu != NULL )
@@ -181,12 +181,12 @@ bool wxMenu::DoInsertOrAppend(wxMenuItem *pItem, size_t pos)
wxASSERT_MSG( pSubMenu->m_hMenu != NULL , "invalid submenu added"); wxASSERT_MSG( pSubMenu->m_hMenu != NULL , "invalid submenu added");
pSubMenu->m_menuParent = this ; pSubMenu->m_menuParent = this ;
wxMenuItem::MacBuildMenuString( label , NULL , NULL , pItem->GetText() ,false); wxMenuItem::MacBuildMenuString( label , NULL , NULL , pItem->GetText() ,false);
if (wxMenuBar::MacGetInstalledMenuBar() == m_menuBar) if (wxMenuBar::MacGetInstalledMenuBar() == m_menuBar)
{ {
::InsertMenu( MAC_WXHMENU( pSubMenu->m_hMenu ) , -1 ) ; ::InsertMenu( MAC_WXHMENU( pSubMenu->m_hMenu ) , -1 ) ;
} }
if ( pos == (size_t)-1 ) if ( pos == (size_t)-1 )
{ {
UMAAppendSubMenuItem(MAC_WXHMENU(m_hMenu), label, pSubMenu->m_macMenuId); UMAAppendSubMenuItem(MAC_WXHMENU(m_hMenu), label, pSubMenu->m_macMenuId);
@@ -228,7 +228,7 @@ bool wxMenu::DoInsertOrAppend(wxMenuItem *pItem, size_t pos)
SetMenuItemIconHandle( MAC_WXHMENU(m_hMenu) , CountMenuItems(MAC_WXHMENU(m_hMenu) ) , SetMenuItemIconHandle( MAC_WXHMENU(m_hMenu) , CountMenuItems(MAC_WXHMENU(m_hMenu) ) ,
kMenuColorIconType , (Handle) info.u.cIconHandle ) ; kMenuColorIconType , (Handle) info.u.cIconHandle ) ;
} }
} }
} }
else else
@@ -245,10 +245,10 @@ bool wxMenu::DoInsertOrAppend(wxMenuItem *pItem, size_t pos)
SetMenuItemIconHandle( MAC_WXHMENU(m_hMenu) , pos , SetMenuItemIconHandle( MAC_WXHMENU(m_hMenu) , pos ,
kMenuColorIconType , (Handle) info.u.cIconHandle ) ; kMenuColorIconType , (Handle) info.u.cIconHandle ) ;
} }
} }
} }
if ( pItem->GetId() == idMenuTitle ) if ( pItem->GetId() == idMenuTitle )
{ {
if ( pos == (size_t)-1 ) if ( pos == (size_t)-1 )
{ {
@@ -390,10 +390,10 @@ wxWindow *wxMenu::GetWindow() const
return NULL; return NULL;
} }
// helper functions returning the mac menu position for a certain item, note that this is // helper functions returning the mac menu position for a certain item, note that this is
// mac-wise 1 - based, i.e. the first item has index 1 whereas on MSWin it has pos 0 // mac-wise 1 - based, i.e. the first item has index 1 whereas on MSWin it has pos 0
int wxMenu::MacGetIndexFromId( int id ) int wxMenu::MacGetIndexFromId( int id )
{ {
size_t pos; size_t pos;
wxMenuItemList::Node *node = GetMenuItems().GetFirst(); wxMenuItemList::Node *node = GetMenuItems().GetFirst();
@@ -404,14 +404,14 @@ int wxMenu::MacGetIndexFromId( int id )
node = node->GetNext(); node = node->GetNext();
} }
if (!node) if (!node)
return 0; return 0;
return pos + 1 ; return pos + 1 ;
} }
int wxMenu::MacGetIndexFromItem( wxMenuItem *pItem ) int wxMenu::MacGetIndexFromItem( wxMenuItem *pItem )
{ {
size_t pos; size_t pos;
wxMenuItemList::Node *node = GetMenuItems().GetFirst(); wxMenuItemList::Node *node = GetMenuItems().GetFirst();
@@ -425,23 +425,23 @@ int wxMenu::MacGetIndexFromItem( wxMenuItem *pItem )
if (!node) if (!node)
return 0; return 0;
return pos + 1 ; return pos + 1 ;
} }
void wxMenu::MacEnableMenu( bool bDoEnable ) void wxMenu::MacEnableMenu( bool bDoEnable )
{ {
if ( bDoEnable ) if ( bDoEnable )
UMAEnableMenuItem(MAC_WXHMENU(m_hMenu) , 0 ) ; UMAEnableMenuItem(MAC_WXHMENU(m_hMenu) , 0 ) ;
else else
UMADisableMenuItem(MAC_WXHMENU(m_hMenu) , 0 ) ; UMADisableMenuItem(MAC_WXHMENU(m_hMenu) , 0 ) ;
::DrawMenuBar() ; ::DrawMenuBar() ;
} }
// Menu Bar // Menu Bar
/* /*
Mac Implementation note : Mac Implementation note :
@@ -449,11 +449,11 @@ The Mac has only one global menubar, so we attempt to install the currently
active menubar from a frame, we currently don't take into account mdi-frames active menubar from a frame, we currently don't take into account mdi-frames
which would ask for menu-merging which would ask for menu-merging
Secondly there is no mac api for changing a menubar that is not the current Secondly there is no mac api for changing a menubar that is not the current
menubar, so we have to wait for preparing the actual menubar until the menubar, so we have to wait for preparing the actual menubar until the
wxMenubar is to be used wxMenubar is to be used
We can in subsequent versions use MacInstallMenuBar to provide some sort of We can in subsequent versions use MacInstallMenuBar to provide some sort of
auto-merge for MDI in case this will be necessary auto-merge for MDI in case this will be necessary
*/ */
@@ -540,13 +540,13 @@ void wxMenuBar::RebuildAccelTable()
#endif // wxUSE_ACCEL #endif // wxUSE_ACCEL
void wxMenuBar::MacInstallMenuBar() void wxMenuBar::MacInstallMenuBar()
{ {
if ( s_macInstalledMenuBar == this ) if ( s_macInstalledMenuBar == this )
return ; return ;
wxStAppResource resload ; wxStAppResource resload ;
Handle menubar = ::GetNewMBar( kwxMacMenuBarResource ) ; Handle menubar = ::GetNewMBar( kwxMacMenuBarResource ) ;
wxString message ; wxString message ;
wxCHECK_RET( menubar != NULL, "can't read MBAR resource" ); wxCHECK_RET( menubar != NULL, "can't read MBAR resource" );
@@ -564,7 +564,7 @@ void wxMenuBar::MacInstallMenuBar()
::AppendResMenu(menu, 'DRVR'); ::AppendResMenu(menu, 'DRVR');
} }
#endif #endif
// clean-up the help menu before adding new items // clean-up the help menu before adding new items
MenuHandle mh = NULL ; MenuHandle mh = NULL ;
if ( UMAGetHelpMenu( &mh , &firstUserHelpMenuItem) == noErr ) if ( UMAGetHelpMenu( &mh , &firstUserHelpMenuItem) == noErr )
@@ -594,16 +594,16 @@ void wxMenuBar::MacInstallMenuBar()
{ {
continue ; continue ;
} }
for (pos = 0 , node = menu->GetMenuItems().First(); node; node = node->Next(), pos++) for (pos = 0 , node = menu->GetMenuItems().First(); node; node = node->Next(), pos++)
{ {
item = (wxMenuItem *)node->Data(); item = (wxMenuItem *)node->Data();
subMenu = item->GetSubMenu() ; subMenu = item->GetSubMenu() ;
if (subMenu) if (subMenu)
{ {
// we don't support hierarchical menus in the help menu yet // we don't support hierarchical menus in the help menu yet
} }
else else
{ {
if ( item->IsSeparator() ) if ( item->IsSeparator() )
{ {
@@ -623,7 +623,7 @@ void wxMenuBar::MacInstallMenuBar()
label[1] = ' ' ; label[1] = ' ' ;
} }
if ( item->GetId() == wxApp::s_macAboutMenuItemId ) if ( item->GetId() == wxApp::s_macAboutMenuItemId )
{ {
::SetMenuItemText( GetMenuHandle( kwxMacAppleMenuId ) , 1 , label ); ::SetMenuItemText( GetMenuHandle( kwxMacAppleMenuId ) , 1 , label );
UMAEnableMenuItem( GetMenuHandle( kwxMacAppleMenuId ) , 1 ); UMAEnableMenuItem( GetMenuHandle( kwxMacAppleMenuId ) , 1 );
SetMenuItemCommandID( GetMenuHandle( kwxMacAppleMenuId ) , 1 , item->GetId() ) ; SetMenuItemCommandID( GetMenuHandle( kwxMacAppleMenuId ) , 1 , item->GetId() ) ;
@@ -645,12 +645,12 @@ void wxMenuBar::MacInstallMenuBar()
wxMenuItem::MacBuildMenuString( label, NULL , NULL , m_titles[i] , false ); wxMenuItem::MacBuildMenuString( label, NULL , NULL , m_titles[i] , false );
UMASetMenuTitle( MAC_WXHMENU(menu->GetHMenu()) , label ) ; UMASetMenuTitle( MAC_WXHMENU(menu->GetHMenu()) , label ) ;
wxArrayPtrVoid submenus ; wxArrayPtrVoid submenus ;
for (pos = 0, node = menu->GetMenuItems().First(); node; node = node->Next(), pos++) for (pos = 0, node = menu->GetMenuItems().First(); node; node = node->Next(), pos++)
{ {
item = (wxMenuItem *)node->Data(); item = (wxMenuItem *)node->Data();
subMenu = item->GetSubMenu() ; subMenu = item->GetSubMenu() ;
if (subMenu) if (subMenu)
{ {
submenus.Add(subMenu) ; submenus.Add(subMenu) ;
} }
@@ -662,14 +662,14 @@ void wxMenuBar::MacInstallMenuBar()
wxNode *subnode; wxNode *subnode;
wxMenuItem *subitem; wxMenuItem *subitem;
int subpos ; int subpos ;
for ( subpos = 0 , subnode = submenu->GetMenuItems().First(); subnode; subnode = subnode->Next(), subpos++) for ( subpos = 0 , subnode = submenu->GetMenuItems().First(); subnode; subnode = subnode->Next(), subpos++)
{ {
subitem = (wxMenuItem *)subnode->Data(); subitem = (wxMenuItem *)subnode->Data();
wxMenu* itsSubMenu = subitem->GetSubMenu() ; wxMenu* itsSubMenu = subitem->GetSubMenu() ;
if (itsSubMenu) if (itsSubMenu)
{ {
submenus.Add(itsSubMenu) ; submenus.Add(itsSubMenu) ;
} }
} }
::InsertMenu( MAC_WXHMENU(submenu->GetHMenu()) , -1 ) ; ::InsertMenu( MAC_WXHMENU(submenu->GetHMenu()) , -1 ) ;
} }
@@ -722,7 +722,7 @@ int wxMenuBar::FindMenu(const wxString& title)
{ {
wxString title = wxStripMenuCodes(m_titles[i]); wxString title = wxStripMenuCodes(m_titles[i]);
if ( menuTitle == title ) if ( menuTitle == title )
return i; return i;
} }
return wxNOT_FOUND; return wxNOT_FOUND;
@@ -738,6 +738,25 @@ int wxMenuBar::FindMenu(const wxString& title)
// wxMenuBar construction // wxMenuBar construction
// --------------------------------------------------------------------------- // ---------------------------------------------------------------------------
// This allows Replace to work with menus that have submenus. Is it needed
// anywhere else? (Like in Insert and Append?)
static wxMenuBarInsertSubmenus(wxMenu* menu)
{
int pos ;
wxMenuItemList::Node *node;
wxMenuItem *item;
for (pos = 0, node = menu->GetMenuItems().GetFirst(); node; node = node->GetNext(), pos++)
{
item = (wxMenuItem *)node->GetData();
wxMenu* subMenu = item->GetSubMenu() ;
if (subMenu)
{
::InsertMenu(MAC_WXHMENU( subMenu->GetHMenu()), -1);
wxMenuBarInsertSubmenus(subMenu);
}
}
}
wxMenu *wxMenuBar::Replace(size_t pos, wxMenu *menu, const wxString& title) wxMenu *wxMenuBar::Replace(size_t pos, wxMenu *menu, const wxString& title)
{ {
wxMenu *menuOld = wxMenuBarBase::Replace(pos, menu, title); wxMenu *menuOld = wxMenuBarBase::Replace(pos, menu, title);
@@ -762,6 +781,7 @@ wxMenu *wxMenuBar::Replace(size_t pos, wxMenu *menu, const wxString& title)
{ {
::InsertMenu( MAC_WXHMENU(menu->GetHMenu()) , m_menus[pos+1]->MacGetMenuId() ) ; ::InsertMenu( MAC_WXHMENU(menu->GetHMenu()) , m_menus[pos+1]->MacGetMenuId() ) ;
} }
wxMenuBarInsertSubmenus(menu);
} }
} }
@@ -856,7 +876,7 @@ bool wxMenuBar::Append(wxMenu *menu, const wxString& title)
return FALSE; return FALSE;
m_titles.Add(title); m_titles.Add(title);
Str255 label ; Str255 label ;
wxMenuItem::MacBuildMenuString( label, NULL , NULL , title , false ); wxMenuItem::MacBuildMenuString( label, NULL , NULL , title , false );
UMASetMenuTitle( MAC_WXHMENU(menu->GetHMenu()) , label ) ; UMASetMenuTitle( MAC_WXHMENU(menu->GetHMenu()) , label ) ;

92
src/mac/menu.cpp
View File

@@ -71,7 +71,7 @@ void wxMenu::Init()
// create the menu // create the menu
Str255 label; Str255 label;
wxMenuItem::MacBuildMenuString( label, NULL , NULL , m_title , false ); wxMenuItem::MacBuildMenuString( label, NULL , NULL , m_title , false );
m_macMenuId = s_macNextMenuId++; m_macMenuId = s_macNextMenuId++;
m_hMenu = ::NewMenu(m_macMenuId, label); m_hMenu = ::NewMenu(m_macMenuId, label);
if ( !m_hMenu ) if ( !m_hMenu )
@@ -172,7 +172,7 @@ bool wxMenu::DoInsertOrAppend(wxMenuItem *pItem, size_t pos)
MacInsertMenuItem(MAC_WXHMENU(m_hMenu), "\p-" , pos); MacInsertMenuItem(MAC_WXHMENU(m_hMenu), "\p-" , pos);
} }
} }
else else
{ {
wxMenu *pSubMenu = pItem->GetSubMenu() ; wxMenu *pSubMenu = pItem->GetSubMenu() ;
if ( pSubMenu != NULL ) if ( pSubMenu != NULL )
@@ -181,12 +181,12 @@ bool wxMenu::DoInsertOrAppend(wxMenuItem *pItem, size_t pos)
wxASSERT_MSG( pSubMenu->m_hMenu != NULL , "invalid submenu added"); wxASSERT_MSG( pSubMenu->m_hMenu != NULL , "invalid submenu added");
pSubMenu->m_menuParent = this ; pSubMenu->m_menuParent = this ;
wxMenuItem::MacBuildMenuString( label , NULL , NULL , pItem->GetText() ,false); wxMenuItem::MacBuildMenuString( label , NULL , NULL , pItem->GetText() ,false);
if (wxMenuBar::MacGetInstalledMenuBar() == m_menuBar) if (wxMenuBar::MacGetInstalledMenuBar() == m_menuBar)
{ {
::InsertMenu( MAC_WXHMENU( pSubMenu->m_hMenu ) , -1 ) ; ::InsertMenu( MAC_WXHMENU( pSubMenu->m_hMenu ) , -1 ) ;
} }
if ( pos == (size_t)-1 ) if ( pos == (size_t)-1 )
{ {
UMAAppendSubMenuItem(MAC_WXHMENU(m_hMenu), label, pSubMenu->m_macMenuId); UMAAppendSubMenuItem(MAC_WXHMENU(m_hMenu), label, pSubMenu->m_macMenuId);
@@ -228,7 +228,7 @@ bool wxMenu::DoInsertOrAppend(wxMenuItem *pItem, size_t pos)
SetMenuItemIconHandle( MAC_WXHMENU(m_hMenu) , CountMenuItems(MAC_WXHMENU(m_hMenu) ) , SetMenuItemIconHandle( MAC_WXHMENU(m_hMenu) , CountMenuItems(MAC_WXHMENU(m_hMenu) ) ,
kMenuColorIconType , (Handle) info.u.cIconHandle ) ; kMenuColorIconType , (Handle) info.u.cIconHandle ) ;
} }
} }
} }
else else
@@ -245,10 +245,10 @@ bool wxMenu::DoInsertOrAppend(wxMenuItem *pItem, size_t pos)
SetMenuItemIconHandle( MAC_WXHMENU(m_hMenu) , pos , SetMenuItemIconHandle( MAC_WXHMENU(m_hMenu) , pos ,
kMenuColorIconType , (Handle) info.u.cIconHandle ) ; kMenuColorIconType , (Handle) info.u.cIconHandle ) ;
} }
} }
} }
if ( pItem->GetId() == idMenuTitle ) if ( pItem->GetId() == idMenuTitle )
{ {
if ( pos == (size_t)-1 ) if ( pos == (size_t)-1 )
{ {
@@ -390,10 +390,10 @@ wxWindow *wxMenu::GetWindow() const
return NULL; return NULL;
} }
// helper functions returning the mac menu position for a certain item, note that this is // helper functions returning the mac menu position for a certain item, note that this is
// mac-wise 1 - based, i.e. the first item has index 1 whereas on MSWin it has pos 0 // mac-wise 1 - based, i.e. the first item has index 1 whereas on MSWin it has pos 0
int wxMenu::MacGetIndexFromId( int id ) int wxMenu::MacGetIndexFromId( int id )
{ {
size_t pos; size_t pos;
wxMenuItemList::Node *node = GetMenuItems().GetFirst(); wxMenuItemList::Node *node = GetMenuItems().GetFirst();
@@ -404,14 +404,14 @@ int wxMenu::MacGetIndexFromId( int id )
node = node->GetNext(); node = node->GetNext();
} }
if (!node) if (!node)
return 0; return 0;
return pos + 1 ; return pos + 1 ;
} }
int wxMenu::MacGetIndexFromItem( wxMenuItem *pItem ) int wxMenu::MacGetIndexFromItem( wxMenuItem *pItem )
{ {
size_t pos; size_t pos;
wxMenuItemList::Node *node = GetMenuItems().GetFirst(); wxMenuItemList::Node *node = GetMenuItems().GetFirst();
@@ -425,23 +425,23 @@ int wxMenu::MacGetIndexFromItem( wxMenuItem *pItem )
if (!node) if (!node)
return 0; return 0;
return pos + 1 ; return pos + 1 ;
} }
void wxMenu::MacEnableMenu( bool bDoEnable ) void wxMenu::MacEnableMenu( bool bDoEnable )
{ {
if ( bDoEnable ) if ( bDoEnable )
UMAEnableMenuItem(MAC_WXHMENU(m_hMenu) , 0 ) ; UMAEnableMenuItem(MAC_WXHMENU(m_hMenu) , 0 ) ;
else else
UMADisableMenuItem(MAC_WXHMENU(m_hMenu) , 0 ) ; UMADisableMenuItem(MAC_WXHMENU(m_hMenu) , 0 ) ;
::DrawMenuBar() ; ::DrawMenuBar() ;
} }
// Menu Bar // Menu Bar
/* /*
Mac Implementation note : Mac Implementation note :
@@ -449,11 +449,11 @@ The Mac has only one global menubar, so we attempt to install the currently
active menubar from a frame, we currently don't take into account mdi-frames active menubar from a frame, we currently don't take into account mdi-frames
which would ask for menu-merging which would ask for menu-merging
Secondly there is no mac api for changing a menubar that is not the current Secondly there is no mac api for changing a menubar that is not the current
menubar, so we have to wait for preparing the actual menubar until the menubar, so we have to wait for preparing the actual menubar until the
wxMenubar is to be used wxMenubar is to be used
We can in subsequent versions use MacInstallMenuBar to provide some sort of We can in subsequent versions use MacInstallMenuBar to provide some sort of
auto-merge for MDI in case this will be necessary auto-merge for MDI in case this will be necessary
*/ */
@@ -540,13 +540,13 @@ void wxMenuBar::RebuildAccelTable()
#endif // wxUSE_ACCEL #endif // wxUSE_ACCEL
void wxMenuBar::MacInstallMenuBar() void wxMenuBar::MacInstallMenuBar()
{ {
if ( s_macInstalledMenuBar == this ) if ( s_macInstalledMenuBar == this )
return ; return ;
wxStAppResource resload ; wxStAppResource resload ;
Handle menubar = ::GetNewMBar( kwxMacMenuBarResource ) ; Handle menubar = ::GetNewMBar( kwxMacMenuBarResource ) ;
wxString message ; wxString message ;
wxCHECK_RET( menubar != NULL, "can't read MBAR resource" ); wxCHECK_RET( menubar != NULL, "can't read MBAR resource" );
@@ -564,7 +564,7 @@ void wxMenuBar::MacInstallMenuBar()
::AppendResMenu(menu, 'DRVR'); ::AppendResMenu(menu, 'DRVR');
} }
#endif #endif
// clean-up the help menu before adding new items // clean-up the help menu before adding new items
MenuHandle mh = NULL ; MenuHandle mh = NULL ;
if ( UMAGetHelpMenu( &mh , &firstUserHelpMenuItem) == noErr ) if ( UMAGetHelpMenu( &mh , &firstUserHelpMenuItem) == noErr )
@@ -594,16 +594,16 @@ void wxMenuBar::MacInstallMenuBar()
{ {
continue ; continue ;
} }
for (pos = 0 , node = menu->GetMenuItems().First(); node; node = node->Next(), pos++) for (pos = 0 , node = menu->GetMenuItems().First(); node; node = node->Next(), pos++)
{ {
item = (wxMenuItem *)node->Data(); item = (wxMenuItem *)node->Data();
subMenu = item->GetSubMenu() ; subMenu = item->GetSubMenu() ;
if (subMenu) if (subMenu)
{ {
// we don't support hierarchical menus in the help menu yet // we don't support hierarchical menus in the help menu yet
} }
else else
{ {
if ( item->IsSeparator() ) if ( item->IsSeparator() )
{ {
@@ -623,7 +623,7 @@ void wxMenuBar::MacInstallMenuBar()
label[1] = ' ' ; label[1] = ' ' ;
} }
if ( item->GetId() == wxApp::s_macAboutMenuItemId ) if ( item->GetId() == wxApp::s_macAboutMenuItemId )
{ {
::SetMenuItemText( GetMenuHandle( kwxMacAppleMenuId ) , 1 , label ); ::SetMenuItemText( GetMenuHandle( kwxMacAppleMenuId ) , 1 , label );
UMAEnableMenuItem( GetMenuHandle( kwxMacAppleMenuId ) , 1 ); UMAEnableMenuItem( GetMenuHandle( kwxMacAppleMenuId ) , 1 );
SetMenuItemCommandID( GetMenuHandle( kwxMacAppleMenuId ) , 1 , item->GetId() ) ; SetMenuItemCommandID( GetMenuHandle( kwxMacAppleMenuId ) , 1 , item->GetId() ) ;
@@ -645,12 +645,12 @@ void wxMenuBar::MacInstallMenuBar()
wxMenuItem::MacBuildMenuString( label, NULL , NULL , m_titles[i] , false ); wxMenuItem::MacBuildMenuString( label, NULL , NULL , m_titles[i] , false );
UMASetMenuTitle( MAC_WXHMENU(menu->GetHMenu()) , label ) ; UMASetMenuTitle( MAC_WXHMENU(menu->GetHMenu()) , label ) ;
wxArrayPtrVoid submenus ; wxArrayPtrVoid submenus ;
for (pos = 0, node = menu->GetMenuItems().First(); node; node = node->Next(), pos++) for (pos = 0, node = menu->GetMenuItems().First(); node; node = node->Next(), pos++)
{ {
item = (wxMenuItem *)node->Data(); item = (wxMenuItem *)node->Data();
subMenu = item->GetSubMenu() ; subMenu = item->GetSubMenu() ;
if (subMenu) if (subMenu)
{ {
submenus.Add(subMenu) ; submenus.Add(subMenu) ;
} }
@@ -662,14 +662,14 @@ void wxMenuBar::MacInstallMenuBar()
wxNode *subnode; wxNode *subnode;
wxMenuItem *subitem; wxMenuItem *subitem;
int subpos ; int subpos ;
for ( subpos = 0 , subnode = submenu->GetMenuItems().First(); subnode; subnode = subnode->Next(), subpos++) for ( subpos = 0 , subnode = submenu->GetMenuItems().First(); subnode; subnode = subnode->Next(), subpos++)
{ {
subitem = (wxMenuItem *)subnode->Data(); subitem = (wxMenuItem *)subnode->Data();
wxMenu* itsSubMenu = subitem->GetSubMenu() ; wxMenu* itsSubMenu = subitem->GetSubMenu() ;
if (itsSubMenu) if (itsSubMenu)
{ {
submenus.Add(itsSubMenu) ; submenus.Add(itsSubMenu) ;
} }
} }
::InsertMenu( MAC_WXHMENU(submenu->GetHMenu()) , -1 ) ; ::InsertMenu( MAC_WXHMENU(submenu->GetHMenu()) , -1 ) ;
} }
@@ -722,7 +722,7 @@ int wxMenuBar::FindMenu(const wxString& title)
{ {
wxString title = wxStripMenuCodes(m_titles[i]); wxString title = wxStripMenuCodes(m_titles[i]);
if ( menuTitle == title ) if ( menuTitle == title )
return i; return i;
} }
return wxNOT_FOUND; return wxNOT_FOUND;
@@ -738,6 +738,25 @@ int wxMenuBar::FindMenu(const wxString& title)
// wxMenuBar construction // wxMenuBar construction
// --------------------------------------------------------------------------- // ---------------------------------------------------------------------------
// This allows Replace to work with menus that have submenus. Is it needed
// anywhere else? (Like in Insert and Append?)
static wxMenuBarInsertSubmenus(wxMenu* menu)
{
int pos ;
wxMenuItemList::Node *node;
wxMenuItem *item;
for (pos = 0, node = menu->GetMenuItems().GetFirst(); node; node = node->GetNext(), pos++)
{
item = (wxMenuItem *)node->GetData();
wxMenu* subMenu = item->GetSubMenu() ;
if (subMenu)
{
::InsertMenu(MAC_WXHMENU( subMenu->GetHMenu()), -1);
wxMenuBarInsertSubmenus(subMenu);
}
}
}
wxMenu *wxMenuBar::Replace(size_t pos, wxMenu *menu, const wxString& title) wxMenu *wxMenuBar::Replace(size_t pos, wxMenu *menu, const wxString& title)
{ {
wxMenu *menuOld = wxMenuBarBase::Replace(pos, menu, title); wxMenu *menuOld = wxMenuBarBase::Replace(pos, menu, title);
@@ -762,6 +781,7 @@ wxMenu *wxMenuBar::Replace(size_t pos, wxMenu *menu, const wxString& title)
{ {
::InsertMenu( MAC_WXHMENU(menu->GetHMenu()) , m_menus[pos+1]->MacGetMenuId() ) ; ::InsertMenu( MAC_WXHMENU(menu->GetHMenu()) , m_menus[pos+1]->MacGetMenuId() ) ;
} }
wxMenuBarInsertSubmenus(menu);
} }
} }
@@ -856,7 +876,7 @@ bool wxMenuBar::Append(wxMenu *menu, const wxString& title)
return FALSE; return FALSE;
m_titles.Add(title); m_titles.Add(title);
Str255 label ; Str255 label ;
wxMenuItem::MacBuildMenuString( label, NULL , NULL , title , false ); wxMenuItem::MacBuildMenuString( label, NULL , NULL , title , false );
UMASetMenuTitle( MAC_WXHMENU(menu->GetHMenu()) , label ) ; UMASetMenuTitle( MAC_WXHMENU(menu->GetHMenu()) , label ) ;