]> git.saurik.com Git - wxWidgets.git/blame - src/msw/menu.cpp
changed wxDC::GetTextExtent() and related methods to take const wxFont pointer, for...
[wxWidgets.git] / src / msw / menu.cpp
CommitLineData
2bda0e17 1/////////////////////////////////////////////////////////////////////////////
7ec69821 2// Name: src/msw/menu.cpp
2bda0e17
KB
3// Purpose: wxMenu, wxMenuBar, wxMenuItem
4// Author: Julian Smart
5// Modified by: Vadim Zeitlin
6// Created: 04/01/98
7// RCS-ID: $Id$
6c9a19aa 8// Copyright: (c) Julian Smart
65571936 9// Licence: wxWindows licence
2bda0e17
KB
10/////////////////////////////////////////////////////////////////////////////
11
c2dcfdef
VZ
12// ===========================================================================
13// declarations
14// ===========================================================================
15
16// ---------------------------------------------------------------------------
17// headers
18// ---------------------------------------------------------------------------
19
2bda0e17
KB
20// For compilers that support precompilation, includes "wx.h".
21#include "wx/wxprec.h"
22
23#ifdef __BORLANDC__
c626a8b7 24 #pragma hdrstop
2bda0e17
KB
25#endif
26
1e6feb95
VZ
27#if wxUSE_MENUS
28
3b3dc801
WS
29#include "wx/menu.h"
30
2bda0e17 31#ifndef WX_PRECOMP
57bd4c60 32 #include "wx/msw/wrapcctl.h" // include <commctrl.h> "properly"
c626a8b7 33 #include "wx/frame.h"
c626a8b7 34 #include "wx/utils.h"
0c589ad0 35 #include "wx/intl.h"
717a57c2 36 #include "wx/log.h"
2bda0e17
KB
37#endif
38
47d67540 39#if wxUSE_OWNER_DRAWN
c626a8b7 40 #include "wx/ownerdrw.h"
2bda0e17
KB
41#endif
42
43#include "wx/msw/private.h"
2bda0e17 44
39d2f9a7
JS
45#ifdef __WXWINCE__
46#include <windows.h>
47#include <windowsx.h>
48#include <tchar.h>
49#include <ole2.h>
eae4425d 50#include <shellapi.h>
781a24e8 51#if (_WIN32_WCE < 400) && !defined(__HANDHELDPC__)
39d2f9a7 52#include <aygshell.h>
39d2f9a7
JS
53#endif
54
2d36b3d8
JS
55#include "wx/msw/wince/missing.h"
56
39d2f9a7
JS
57#endif
58
2bda0e17 59// other standard headers
2bda0e17
KB
60#include <string.h>
61
a0369cb3
CE
62//VC6 needs these defining, though they are in winuser.h
63#ifndef MIIM_BITMAP
64#define MIIM_STRING 0x00000040
65#define MIIM_BITMAP 0x00000080
66#define MIIM_FTYPE 0x00000100
67#define HBMMENU_CALLBACK ((HBITMAP) -1)
68typedef struct tagMENUINFO
69{
70 DWORD cbSize;
71 DWORD fMask;
72 DWORD dwStyle;
73 UINT cyMax;
74 HBRUSH hbrBack;
75 DWORD dwContextHelpID;
76 DWORD dwMenuData;
77} MENUINFO, FAR *LPMENUINFO;
78#endif
79
f4322df6 80#if wxUSE_OWNER_DRAWN
9f7e1cff
VZ
81 #include "wx/dynlib.h"
82#endif
83
ec4b5290
VZ
84#ifndef MNS_CHECKORBMP
85 #define MNS_CHECKORBMP 0x04000000
86#endif
87#ifndef MIM_STYLE
88 #define MIM_STYLE 0x00000010
89#endif
90
c626a8b7
VZ
91// ----------------------------------------------------------------------------
92// global variables
93// ----------------------------------------------------------------------------
94
b8d3a4f1
VZ
95// ----------------------------------------------------------------------------
96// constants
97// ----------------------------------------------------------------------------
98
99// the (popup) menu title has this special id
c25f373e 100static const int idMenuTitle = -3;
b8d3a4f1
VZ
101
102// ----------------------------------------------------------------------------
0472ece7 103// private functions
b8d3a4f1 104// ----------------------------------------------------------------------------
c626a8b7 105
0472ece7 106// make the given menu item default
7bea7b91
WS
107static void SetDefaultMenuItem(HMENU WXUNUSED_IN_WINCE(hmenu),
108 UINT WXUNUSED_IN_WINCE(id))
0472ece7 109{
4676948b 110#ifndef __WXWINCE__
0472ece7
VZ
111 MENUITEMINFO mii;
112 wxZeroMemory(mii);
113 mii.cbSize = sizeof(MENUITEMINFO);
114 mii.fMask = MIIM_STATE;
115 mii.fState = MFS_DEFAULT;
116
117 if ( !::SetMenuItemInfo(hmenu, id, FALSE, &mii) )
118 {
119 wxLogLastError(wxT("SetMenuItemInfo"));
120 }
4676948b
JS
121#endif
122}
123
124#ifdef __WXWINCE__
125UINT GetMenuState(HMENU hMenu, UINT id, UINT flags)
126{
127 MENUITEMINFO info;
128 wxZeroMemory(info);
129 info.cbSize = sizeof(info);
130 info.fMask = MIIM_STATE;
3519d946
JS
131 // MF_BYCOMMAND is zero so test MF_BYPOSITION
132 if ( !::GetMenuItemInfo(hMenu, id, flags & MF_BYPOSITION ? TRUE : FALSE , & info) )
4676948b
JS
133 wxLogLastError(wxT("GetMenuItemInfo"));
134 return info.fState;
0472ece7 135}
4676948b 136#endif
2bda0e17
KB
137
138// ============================================================================
139// implementation
140// ============================================================================
141
7ec69821 142#include "wx/listimpl.cpp"
e70b4f10 143
259c43f6 144WX_DEFINE_LIST( wxMenuInfoList )
e70b4f10
SC
145
146#if wxUSE_EXTENDED_RTTI
147
148WX_DEFINE_FLAGS( wxMenuStyle )
149
3ff066a4
SC
150wxBEGIN_FLAGS( wxMenuStyle )
151 wxFLAGS_MEMBER(wxMENU_TEAROFF)
152wxEND_FLAGS( wxMenuStyle )
e70b4f10
SC
153
154IMPLEMENT_DYNAMIC_CLASS_XTI(wxMenu, wxEvtHandler,"wx/menu.h")
155
3ff066a4 156wxCOLLECTION_TYPE_INFO( wxMenuItem * , wxMenuItemList ) ;
e70b4f10
SC
157
158template<> void wxCollectionToVariantArray( wxMenuItemList const &theList, wxxVariantArray &value)
159{
160 wxListCollectionToVariantArray<wxMenuItemList::compatibility_iterator>( theList , value ) ;
161}
162
3ff066a4 163wxBEGIN_PROPERTIES_TABLE(wxMenu)
665b71b1 164 wxEVENT_PROPERTY( Select , wxEVT_COMMAND_MENU_SELECTED , wxCommandEvent)
3ff066a4 165 wxPROPERTY( Title, wxString , SetTitle, GetTitle, wxString(), 0 /*flags*/ , wxT("Helpstring") , wxT("group") )
af498247 166 wxREADONLY_PROPERTY_FLAGS( MenuStyle , wxMenuStyle , long , GetStyle , EMPTY_MACROVALUE , 0 /*flags*/ , wxT("Helpstring") , wxT("group")) // style
3ff066a4
SC
167 wxPROPERTY_COLLECTION( MenuItems , wxMenuItemList , wxMenuItem* , Append , GetMenuItems , 0 /*flags*/ , wxT("Helpstring") , wxT("group"))
168wxEND_PROPERTIES_TABLE()
e70b4f10 169
3ff066a4
SC
170wxBEGIN_HANDLERS_TABLE(wxMenu)
171wxEND_HANDLERS_TABLE()
e70b4f10 172
3ff066a4 173wxDIRECT_CONSTRUCTOR_2( wxMenu , wxString , Title , long , MenuStyle )
e70b4f10
SC
174
175WX_DEFINE_FLAGS( wxMenuBarStyle )
176
3ff066a4
SC
177wxBEGIN_FLAGS( wxMenuBarStyle )
178 wxFLAGS_MEMBER(wxMB_DOCKABLE)
179wxEND_FLAGS( wxMenuBarStyle )
e70b4f10
SC
180
181// the negative id would lead the window (its superclass !) to vetoe streaming out otherwise
2341cf5f 182bool wxMenuBarStreamingCallback( const wxObject *WXUNUSED(object), wxWriter * , wxPersister * , wxxVariantArray & )
e70b4f10
SC
183{
184 return true ;
185}
186
187IMPLEMENT_DYNAMIC_CLASS_XTI_CALLBACK(wxMenuBar, wxWindow ,"wx/menu.h",wxMenuBarStreamingCallback)
188
189IMPLEMENT_DYNAMIC_CLASS_XTI(wxMenuInfo, wxObject , "wx/menu.h" )
190
3ff066a4 191wxBEGIN_PROPERTIES_TABLE(wxMenuInfo)
af498247 192 wxREADONLY_PROPERTY( Menu , wxMenu* , GetMenu , EMPTY_MACROVALUE , 0 /*flags*/ , wxT("Helpstring") , wxT("group"))
3ff066a4
SC
193 wxREADONLY_PROPERTY( Title , wxString , GetTitle , wxString() , 0 /*flags*/ , wxT("Helpstring") , wxT("group"))
194wxEND_PROPERTIES_TABLE()
e70b4f10 195
3ff066a4
SC
196wxBEGIN_HANDLERS_TABLE(wxMenuInfo)
197wxEND_HANDLERS_TABLE()
e70b4f10 198
598ddd96 199wxCONSTRUCTOR_2( wxMenuInfo , wxMenu* , Menu , wxString , Title )
e70b4f10 200
3ff066a4 201wxCOLLECTION_TYPE_INFO( wxMenuInfo * , wxMenuInfoList ) ;
e70b4f10
SC
202
203template<> void wxCollectionToVariantArray( wxMenuInfoList const &theList, wxxVariantArray &value)
204{
205 wxListCollectionToVariantArray<wxMenuInfoList::compatibility_iterator>( theList , value ) ;
206}
207
3ff066a4
SC
208wxBEGIN_PROPERTIES_TABLE(wxMenuBar)
209 wxPROPERTY_COLLECTION( MenuInfos , wxMenuInfoList , wxMenuInfo* , Append , GetMenuInfos , 0 /*flags*/ , wxT("Helpstring") , wxT("group"))
210wxEND_PROPERTIES_TABLE()
e70b4f10 211
3ff066a4
SC
212wxBEGIN_HANDLERS_TABLE(wxMenuBar)
213wxEND_HANDLERS_TABLE()
e70b4f10 214
3ff066a4 215wxCONSTRUCTOR_DUMMY( wxMenuBar )
e70b4f10
SC
216
217#else
0472ece7
VZ
218IMPLEMENT_DYNAMIC_CLASS(wxMenu, wxEvtHandler)
219IMPLEMENT_DYNAMIC_CLASS(wxMenuBar, wxWindow)
e70b4f10
SC
220IMPLEMENT_DYNAMIC_CLASS(wxMenuInfo, wxObject)
221#endif
0472ece7 222
e70b4f10
SC
223const wxMenuInfoList& wxMenuBar::GetMenuInfos() const
224{
225 wxMenuInfoList* list = const_cast< wxMenuInfoList* >( &m_menuInfos ) ;
226 WX_CLEAR_LIST( wxMenuInfoList , *list ) ;
227 for( size_t i = 0 ; i < GetMenuCount() ; ++i )
228 {
229 wxMenuInfo* info = new wxMenuInfo() ;
230 info->Create( const_cast<wxMenuBar*>(this)->GetMenu(i) , GetLabelTop(i) ) ;
231 list->Append( info ) ;
232 }
233 return m_menuInfos ;
234}
066f1b7a 235
c2dcfdef
VZ
236// ---------------------------------------------------------------------------
237// wxMenu construction, adding and removing menu items
238// ---------------------------------------------------------------------------
2bda0e17
KB
239
240// Construct a menu with optional title (then use append)
717a57c2 241void wxMenu::Init()
c626a8b7 242{
598ddd96 243 m_doBreak = false;
0472ece7 244 m_startRadioGroup = -1;
c626a8b7 245
717a57c2
VZ
246 // create the menu
247 m_hMenu = (WXHMENU)CreatePopupMenu();
248 if ( !m_hMenu )
249 {
f6bcfd97 250 wxLogLastError(wxT("CreatePopupMenu"));
717a57c2
VZ
251 }
252
253 // if we have a title, insert it in the beginning of the menu
ed7fdb86 254 if ( !m_title.empty() )
c626a8b7 255 {
ad9bb75f
VZ
256 Append(idMenuTitle, m_title);
257 AppendSeparator();
c626a8b7 258 }
2bda0e17
KB
259}
260
261// The wxWindow destructor will take care of deleting the submenus.
b8d3a4f1 262wxMenu::~wxMenu()
2bda0e17 263{
717a57c2
VZ
264 // we should free Windows resources only if Windows doesn't do it for us
265 // which happens if we're attached to a menubar or a submenu of another
266 // menu
267 if ( !IsAttached() && !GetParent() )
c2dcfdef 268 {
ad9bb75f
VZ
269 if ( !::DestroyMenu(GetHmenu()) )
270 {
f6bcfd97 271 wxLogLastError(wxT("DestroyMenu"));
ad9bb75f 272 }
c2dcfdef 273 }
c626a8b7 274
ad9bb75f
VZ
275#if wxUSE_ACCEL
276 // delete accels
277 WX_CLEAR_ARRAY(m_accels);
278#endif // wxUSE_ACCEL
2bda0e17
KB
279}
280
b8d3a4f1 281void wxMenu::Break()
2bda0e17 282{
717a57c2 283 // this will take effect during the next call to Append()
598ddd96 284 m_doBreak = true;
2bda0e17
KB
285}
286
0472ece7
VZ
287void wxMenu::Attach(wxMenuBarBase *menubar)
288{
289 wxMenuBase::Attach(menubar);
290
291 EndRadioGroup();
292}
293
717a57c2
VZ
294#if wxUSE_ACCEL
295
296int wxMenu::FindAccel(int id) const
297{
298 size_t n, count = m_accels.GetCount();
299 for ( n = 0; n < count; n++ )
300 {
301 if ( m_accels[n]->m_command == id )
302 return n;
303 }
304
305 return wxNOT_FOUND;
306}
307
308void wxMenu::UpdateAccel(wxMenuItem *item)
2bda0e17 309{
f6bcfd97 310 if ( item->IsSubMenu() )
717a57c2 311 {
f6bcfd97 312 wxMenu *submenu = item->GetSubMenu();
222ed1d6 313 wxMenuItemList::compatibility_iterator node = submenu->GetMenuItems().GetFirst();
f6bcfd97
BP
314 while ( node )
315 {
316 UpdateAccel(node->GetData());
317
318 node = node->GetNext();
319 }
717a57c2 320 }
f6bcfd97 321 else if ( !item->IsSeparator() )
717a57c2 322 {
99f0dc68
VZ
323 // recurse upwards: we should only modify m_accels of the top level
324 // menus, not of the submenus as wxMenuBar doesn't look at them
325 // (alternative and arguable cleaner solution would be to recurse
326 // downwards in GetAccelCount() and CopyAccels())
327 if ( GetParent() )
328 {
329 GetParent()->UpdateAccel(item);
330 return;
331 }
332
f6bcfd97 333 // find the (new) accel for this item
90527a50 334 wxAcceleratorEntry *accel = wxAcceleratorEntry::Create(item->GetText());
717a57c2 335 if ( accel )
f6bcfd97
BP
336 accel->m_command = item->GetId();
337
338 // find the old one
339 int n = FindAccel(item->GetId());
340 if ( n == wxNOT_FOUND )
341 {
342 // no old, add new if any
343 if ( accel )
344 m_accels.Add(accel);
345 else
346 return; // skipping RebuildAccelTable() below
347 }
717a57c2 348 else
f6bcfd97
BP
349 {
350 // replace old with new or just remove the old one if no new
351 delete m_accels[n];
352 if ( accel )
353 m_accels[n] = accel;
354 else
b54e41c5 355 m_accels.RemoveAt(n);
f6bcfd97 356 }
717a57c2 357
f6bcfd97
BP
358 if ( IsAttached() )
359 {
4224f059 360 GetMenuBar()->RebuildAccelTable();
f6bcfd97 361 }
42e69d6b 362 }
f6bcfd97 363 //else: it is a separator, they can't have accels, nothing to do
717a57c2
VZ
364}
365
366#endif // wxUSE_ACCEL
367
368// append a new item or submenu to the menu
369bool wxMenu::DoInsertOrAppend(wxMenuItem *pItem, size_t pos)
370{
371#if wxUSE_ACCEL
372 UpdateAccel(pItem);
d427503c 373#endif // wxUSE_ACCEL
42e69d6b 374
c626a8b7 375 UINT flags = 0;
2bda0e17 376
c2dcfdef
VZ
377 // if "Break" has just been called, insert a menu break before this item
378 // (and don't forget to reset the flag)
c626a8b7
VZ
379 if ( m_doBreak ) {
380 flags |= MF_MENUBREAK;
598ddd96 381 m_doBreak = false;
c626a8b7
VZ
382 }
383
384 if ( pItem->IsSeparator() ) {
385 flags |= MF_SEPARATOR;
386 }
2bda0e17 387
c2dcfdef
VZ
388 // id is the numeric id for normal menu items and HMENU for submenus as
389 // required by ::AppendMenu() API
c626a8b7 390 UINT id;
c2dcfdef
VZ
391 wxMenu *submenu = pItem->GetSubMenu();
392 if ( submenu != NULL ) {
717a57c2
VZ
393 wxASSERT_MSG( submenu->GetHMenu(), wxT("invalid submenu") );
394
395 submenu->SetParent(this);
2bda0e17 396
c2dcfdef 397 id = (UINT)submenu->GetHMenu();
2bda0e17 398
c626a8b7
VZ
399 flags |= MF_POPUP;
400 }
401 else {
402 id = pItem->GetId();
403 }
2bda0e17 404
39d2f9a7 405
cb9eed05
VZ
406 // prepare to insert the item in the menu
407 wxString itemText = pItem->GetText();
408 LPCTSTR pData = NULL;
409 if ( pos == (size_t)-1 )
410 {
e7084852
VZ
411 // append at the end (note that the item is already appended to
412 // internal data structures)
413 pos = GetMenuItemCount() - 1;
cb9eed05
VZ
414 }
415
5c5871a4
VZ
416 // adjust position to account for the title, if any
417 if ( !m_title.empty() )
418 pos += 2; // for the title itself and its separator
419
cb9eed05 420 BOOL ok = false;
3633deed 421
99a7bebb
JS
422#if wxUSE_OWNER_DRAWN
423 // Currently, mixing owner-drawn and non-owner-drawn items results in
424 // inconsistent margins, so we force this to be owner-drawn if any other
425 // items already are. Later we might want to use a boolean in the wxMenu
426 // to avoid search. Also we might make this fix unnecessary by getting the correct
427 // margin using NONCLIENTMETRICS.
3633deed 428 if ( !pItem->IsOwnerDrawn() && !pItem->IsSeparator() )
99a7bebb
JS
429 {
430 // Check if any other items are ownerdrawn, and make ownerdrawn if so
431 wxMenuItemList::compatibility_iterator node = GetMenuItems().GetFirst();
432 while (node)
433 {
434 if (node->GetData()->IsOwnerDrawn())
435 {
436 pItem->SetOwnerDrawn(true);
437 break;
438 }
439 node = node->GetNext();
440 }
441 }
442#endif
2bda0e17 443
cb9eed05 444 // check if we have something more than a simple text item
47d67540 445#if wxUSE_OWNER_DRAWN
cb9eed05
VZ
446 if ( pItem->IsOwnerDrawn() )
447 {
9c80e160
VZ
448 // is the item owner-drawn just because of the [checked] bitmap?
449 if ( (pItem->GetBitmap(false).Ok() || pItem->GetBitmap(true).Ok()) &&
cb9eed05
VZ
450 !pItem->GetTextColour().Ok() &&
451 !pItem->GetBackgroundColour().Ok() &&
9c80e160 452 !pItem->GetFont().Ok() )
cb9eed05 453 {
f4322df6 454 // try to use InsertMenuItem() as it's guaranteed to look correct
6a17b868 455 // while our owner-drawn code is not
f4322df6 456#ifndef __DMC__
a0369cb3
CE
457 // DMC at march 2007 doesn't have HBITMAP hbmpItem tagMENUITEMINFOA /W
458 // MIIM_BITMAP only works under WinME/2000+
cb9eed05 459 WinStruct<MENUITEMINFO> mii;
783e2cc6 460 if ( wxGetWinVersion() >= wxWinVersion_98 )
f4322df6 461 {
2919a8b5 462 mii.fMask = MIIM_STRING | MIIM_DATA | MIIM_BITMAP;
9c80e160
VZ
463 if ( pItem->IsCheckable() )
464 {
465 // need to set checked/unchecked bitmaps as otherwise our
466 // MSWOnDrawItem() item is not called
467 mii.fMask |= MIIM_CHECKMARKS;
468 }
469
cb9eed05 470 mii.cch = itemText.length();
c9f78968 471 mii.dwTypeData = wx_const_cast(wxChar *, itemText.wx_str());
cb9eed05 472
2919a8b5
JG
473 if (flags & MF_POPUP)
474 {
475 mii.fMask |= MIIM_SUBMENU;
476 mii.hSubMenu = (HMENU)pItem->GetSubMenu()->GetHMenu();
477 }
478 else
479 {
480 mii.fMask |= MIIM_ID;
481 mii.wID = id;
482 }
483
cb9eed05
VZ
484 // we can't pass HBITMAP directly as hbmpItem for 2 reasons:
485 // 1. we can't draw it with transparency then (this is not
486 // very important now but would be with themed menu bg)
6a17b868 487 // 2. worse, Windows inverts the bitmap for the selected
cb9eed05
VZ
488 // item and this looks downright ugly
489 //
490 // so instead draw it ourselves in MSWOnDrawItem()
491 mii.dwItemData = wx_reinterpret_cast(ULONG_PTR, pItem);
9c80e160
VZ
492 if ( pItem->IsCheckable() )
493 {
494 mii.hbmpChecked =
495 mii.hbmpUnchecked = HBMMENU_CALLBACK;
496 }
cb9eed05
VZ
497 mii.hbmpItem = HBMMENU_CALLBACK;
498
499 ok = ::InsertMenuItem(GetHmenu(), pos, TRUE /* by pos */, &mii);
500 if ( !ok )
501 {
502 wxLogLastError(wxT("InsertMenuItem()"));
503 }
504 else // InsertMenuItem() ok
505 {
506 // we need to remove the extra indent which is reserved for
507 // the checkboxes by default as it looks ugly unless check
508 // boxes are used together with bitmaps and this is not the
509 // case in wx API
510 WinStruct<MENUINFO> mi;
511
9f7e1cff
VZ
512 // don't call SetMenuInfo() directly, this would prevent
513 // the app from starting up under Windows 95/NT 4
514 typedef BOOL (WINAPI *SetMenuInfo_t)(HMENU, MENUINFO *);
515
516 wxDynamicLibrary dllUser(_T("user32"));
517 wxDYNLIB_FUNCTION(SetMenuInfo_t, SetMenuInfo, dllUser);
518 if ( pfnSetMenuInfo )
519 {
520 mi.fMask = MIM_STYLE;
521 mi.dwStyle = MNS_CHECKORBMP;
522 if ( !(*pfnSetMenuInfo)(GetHmenu(), &mi) )
523 wxLogLastError(_T("SetMenuInfo(MNS_NOCHECK)"));
524 }
cb9eed05
VZ
525
526 // tell the item that it's not really owner-drawn but only
527 // needs to draw its bitmap, the rest is done by Windows
528 pItem->ResetOwnerDrawn();
529 }
530 }
a0369cb3 531#endif // __DMC__
cb9eed05
VZ
532 }
533
534 if ( !ok )
535 {
536 // item draws itself, pass pointer to it in data parameter
537 flags |= MF_OWNERDRAW;
538 pData = (LPCTSTR)pItem;
539 }
c626a8b7
VZ
540 }
541 else
cb9eed05 542#endif // wxUSE_OWNER_DRAWN
c626a8b7 543 {
6a17b868 544 // item is just a normal string (passed in data parameter)
c626a8b7 545 flags |= MF_STRING;
8fb3a512 546
39d2f9a7 547#ifdef __WXWINCE__
cb9eed05 548 itemText = wxMenuItem::GetLabelFromText(itemText);
39d2f9a7 549#endif
2bda0e17 550
c9f78968 551 pData = (wxChar*)itemText.wx_str();
717a57c2
VZ
552 }
553
6a17b868 554 // item might have already been inserted by InsertMenuItem() above
717a57c2 555 if ( !ok )
c626a8b7 556 {
cb9eed05
VZ
557 if ( !::InsertMenu(GetHmenu(), pos, flags | MF_BYPOSITION, id, pData) )
558 {
559 wxLogLastError(wxT("InsertMenu[Item]()"));
717a57c2 560
cb9eed05
VZ
561 return false;
562 }
c626a8b7 563 }
42e69d6b 564
cb9eed05 565
0472ece7 566 // if we just appended the title, highlight it
0472ece7
VZ
567 if ( (int)id == idMenuTitle )
568 {
569 // visually select the menu title
570 SetDefaultMenuItem(GetHmenu(), id);
571 }
42e69d6b 572
0472ece7 573 // if we're already attached to the menubar, we must update it
4224f059 574 if ( IsAttached() && GetMenuBar()->IsAttached() )
0472ece7 575 {
4224f059 576 GetMenuBar()->Refresh();
0472ece7
VZ
577 }
578
598ddd96 579 return true;
0472ece7
VZ
580}
581
582void wxMenu::EndRadioGroup()
583{
0472ece7
VZ
584 // we're not inside a radio group any longer
585 m_startRadioGroup = -1;
2bda0e17
KB
586}
587
9add9367 588wxMenuItem* wxMenu::DoAppend(wxMenuItem *item)
2bda0e17 589{
9add9367 590 wxCHECK_MSG( item, NULL, _T("NULL item in wxMenu::DoAppend") );
0472ece7 591
598ddd96 592 bool check = false;
be15b995 593
546bfbea 594 if ( item->GetKind() == wxITEM_RADIO )
0472ece7 595 {
be15b995
VZ
596 int count = GetMenuItemCount();
597
0472ece7
VZ
598 if ( m_startRadioGroup == -1 )
599 {
600 // start a new radio group
be15b995
VZ
601 m_startRadioGroup = count;
602
603 // for now it has just one element
604 item->SetAsRadioGroupStart();
605 item->SetRadioGroupEnd(m_startRadioGroup);
606
607 // ensure that we have a checked item in the radio group
598ddd96 608 check = true;
be15b995
VZ
609 }
610 else // extend the current radio group
611 {
612 // we need to update its end item
613 item->SetRadioGroupStart(m_startRadioGroup);
222ed1d6 614 wxMenuItemList::compatibility_iterator node = GetMenuItems().Item(m_startRadioGroup);
be15b995
VZ
615
616 if ( node )
617 {
618 node->GetData()->SetRadioGroupEnd(count);
619 }
620 else
621 {
622 wxFAIL_MSG( _T("where is the radio group start item?") );
623 }
0472ece7
VZ
624 }
625 }
626 else // not a radio item
627 {
628 EndRadioGroup();
629 }
630
be15b995
VZ
631 if ( !wxMenuBase::DoAppend(item) || !DoInsertOrAppend(item) )
632 {
9add9367 633 return NULL;
be15b995
VZ
634 }
635
636 if ( check )
637 {
638 // check the item initially
598ddd96 639 item->Check(true);
be15b995
VZ
640 }
641
9add9367 642 return item;
2bda0e17
KB
643}
644
9add9367 645wxMenuItem* wxMenu::DoInsert(size_t pos, wxMenuItem *item)
2bda0e17 646{
9add9367
RD
647 if (wxMenuBase::DoInsert(pos, item) && DoInsertOrAppend(item, pos))
648 return item;
649 else
650 return NULL;
2bda0e17
KB
651}
652
717a57c2 653wxMenuItem *wxMenu::DoRemove(wxMenuItem *item)
2bda0e17 654{
6a17b868 655 // we need to find the item's position in the child list
717a57c2 656 size_t pos;
222ed1d6 657 wxMenuItemList::compatibility_iterator node = GetMenuItems().GetFirst();
717a57c2 658 for ( pos = 0; node; pos++ )
c626a8b7 659 {
717a57c2 660 if ( node->GetData() == item )
c626a8b7 661 break;
717a57c2
VZ
662
663 node = node->GetNext();
c626a8b7
VZ
664 }
665
6a17b868 666 // DoRemove() (unlike Remove) can only be called for an existing item!
717a57c2 667 wxCHECK_MSG( node, NULL, wxT("bug in wxMenu::Remove logic") );
c626a8b7 668
717a57c2
VZ
669#if wxUSE_ACCEL
670 // remove the corresponding accel from the accel table
671 int n = FindAccel(item->GetId());
672 if ( n != wxNOT_FOUND )
673 {
674 delete m_accels[n];
c626a8b7 675
b54e41c5 676 m_accels.RemoveAt(n);
c626a8b7 677 }
717a57c2
VZ
678 //else: this item doesn't have an accel, nothing to do
679#endif // wxUSE_ACCEL
680
681 // remove the item from the menu
682 if ( !::RemoveMenu(GetHmenu(), (UINT)pos, MF_BYPOSITION) )
683 {
f6bcfd97 684 wxLogLastError(wxT("RemoveMenu"));
c626a8b7
VZ
685 }
686
4224f059 687 if ( IsAttached() && GetMenuBar()->IsAttached() )
717a57c2 688 {
6a17b868 689 // otherwise, the change won't be visible
4224f059 690 GetMenuBar()->Refresh();
717a57c2 691 }
2bda0e17 692
717a57c2
VZ
693 // and from internal data structures
694 return wxMenuBase::DoRemove(item);
695}
d427503c 696
42e69d6b
VZ
697// ---------------------------------------------------------------------------
698// accelerator helpers
699// ---------------------------------------------------------------------------
700
717a57c2
VZ
701#if wxUSE_ACCEL
702
6a17b868 703// create the wxAcceleratorEntries for our accels and put them into the provided
42e69d6b
VZ
704// array - return the number of accels we have
705size_t wxMenu::CopyAccels(wxAcceleratorEntry *accels) const
706{
707 size_t count = GetAccelCount();
708 for ( size_t n = 0; n < count; n++ )
709 {
974e8d94 710 *accels++ = *m_accels[n];
42e69d6b
VZ
711 }
712
713 return count;
714}
715
d427503c
VZ
716#endif // wxUSE_ACCEL
717
c2dcfdef 718// ---------------------------------------------------------------------------
717a57c2 719// set wxMenu title
c2dcfdef
VZ
720// ---------------------------------------------------------------------------
721
2bda0e17
KB
722void wxMenu::SetTitle(const wxString& label)
723{
ed7fdb86 724 bool hasNoTitle = m_title.empty();
c626a8b7 725 m_title = label;
b8d3a4f1 726
c50f1fb9 727 HMENU hMenu = GetHmenu();
b8d3a4f1 728
c626a8b7 729 if ( hasNoTitle )
b8d3a4f1 730 {
ed7fdb86 731 if ( !label.empty() )
c626a8b7 732 {
717a57c2
VZ
733 if ( !::InsertMenu(hMenu, 0u, MF_BYPOSITION | MF_STRING,
734 (unsigned)idMenuTitle, m_title) ||
735 !::InsertMenu(hMenu, 1u, MF_BYPOSITION, (unsigned)-1, NULL) )
c626a8b7 736 {
f6bcfd97 737 wxLogLastError(wxT("InsertMenu"));
c626a8b7
VZ
738 }
739 }
b8d3a4f1
VZ
740 }
741 else
742 {
ed7fdb86 743 if ( label.empty() )
c626a8b7
VZ
744 {
745 // remove the title and the separator after it
746 if ( !RemoveMenu(hMenu, 0, MF_BYPOSITION) ||
747 !RemoveMenu(hMenu, 0, MF_BYPOSITION) )
748 {
f6bcfd97 749 wxLogLastError(wxT("RemoveMenu"));
c626a8b7
VZ
750 }
751 }
752 else
753 {
754 // modify the title
4676948b
JS
755#ifdef __WXWINCE__
756 MENUITEMINFO info;
757 wxZeroMemory(info);
758 info.cbSize = sizeof(info);
759 info.fMask = MIIM_TYPE;
760 info.fType = MFT_STRING;
7ec69821 761 info.cch = m_title.length();
4676948b
JS
762 info.dwTypeData = (LPTSTR) m_title.c_str();
763 if ( !SetMenuItemInfo(hMenu, 0, TRUE, & info) )
764 {
765 wxLogLastError(wxT("SetMenuItemInfo"));
766 }
767#else
c626a8b7 768 if ( !ModifyMenu(hMenu, 0u,
717a57c2
VZ
769 MF_BYPOSITION | MF_STRING,
770 (unsigned)idMenuTitle, m_title) )
c626a8b7 771 {
f6bcfd97 772 wxLogLastError(wxT("ModifyMenu"));
c626a8b7 773 }
4676948b 774#endif
c626a8b7 775 }
b8d3a4f1 776 }
b8d3a4f1 777
42e69d6b 778#ifdef __WIN32__
c626a8b7 779 // put the title string in bold face
ed7fdb86 780 if ( !m_title.empty() )
a3f4e9e8 781 {
0472ece7 782 SetDefaultMenuItem(GetHmenu(), (UINT)idMenuTitle);
a3f4e9e8 783 }
717a57c2 784#endif // Win32
2bda0e17
KB
785}
786
c2dcfdef
VZ
787// ---------------------------------------------------------------------------
788// event processing
789// ---------------------------------------------------------------------------
2bda0e17 790
debe6624 791bool wxMenu::MSWCommand(WXUINT WXUNUSED(param), WXWORD id)
2bda0e17 792{
a3f4e9e8 793 // ignore commands from the menu title
a3f4e9e8
VZ
794 if ( id != (WXWORD)idMenuTitle )
795 {
18138662
VZ
796 // update the check item when it's clicked
797 wxMenuItem * const item = FindItem(id);
798 if ( item && item->IsCheckable() )
799 item->Toggle();
800
191abe25
VZ
801 // get the status of the menu item: note that it has been just changed
802 // by Toggle() above so here we already get the new state of the item
36f1f456 803 UINT menuState = ::GetMenuState(GetHmenu(), id, MF_BYCOMMAND);
191abe25 804 SendEvent(id, menuState & MF_CHECKED);
a3f4e9e8
VZ
805 }
806
598ddd96 807 return true;
2bda0e17
KB
808}
809
c2dcfdef
VZ
810// ---------------------------------------------------------------------------
811// other
812// ---------------------------------------------------------------------------
2bda0e17 813
717a57c2
VZ
814wxWindow *wxMenu::GetWindow() const
815{
816 if ( m_invokingWindow != NULL )
817 return m_invokingWindow;
4224f059
DE
818 else if ( GetMenuBar() != NULL)
819 return GetMenuBar()->GetFrame();
717a57c2
VZ
820
821 return NULL;
c2dcfdef
VZ
822}
823
824// ---------------------------------------------------------------------------
2bda0e17 825// Menu Bar
c2dcfdef
VZ
826// ---------------------------------------------------------------------------
827
828void wxMenuBar::Init()
2bda0e17 829{
c626a8b7 830 m_eventHandler = this;
c626a8b7 831 m_hMenu = 0;
3d487566 832#if wxUSE_TOOLBAR && defined(__WXWINCE__)
39d2f9a7 833 m_toolBar = NULL;
a96b4743
JS
834#endif
835 // Not using a combined wxToolBar/wxMenuBar? then use
836 // a commandbar in WinCE .NET just to implement the
837 // menubar.
3d487566 838#if defined(WINCE_WITH_COMMANDBAR)
a96b4743 839 m_commandBar = NULL;
a9928e9d 840 m_adornmentsAdded = false;
39d2f9a7 841#endif
cba2db0c 842}
2bda0e17 843
c2dcfdef
VZ
844wxMenuBar::wxMenuBar()
845{
846 Init();
847}
848
cba2db0c
JS
849wxMenuBar::wxMenuBar( long WXUNUSED(style) )
850{
c2dcfdef 851 Init();
2bda0e17
KB
852}
853
294ea16d 854wxMenuBar::wxMenuBar(size_t count, wxMenu *menus[], const wxString titles[], long WXUNUSED(style))
2bda0e17 855{
c2dcfdef
VZ
856 Init();
857
a8cfd0cb 858 m_titles.Alloc(count);
c2dcfdef 859
d2103c8c 860 for ( size_t i = 0; i < count; i++ )
a8cfd0cb
VZ
861 {
862 m_menus.Append(menus[i]);
863 m_titles.Add(titles[i]);
2bda0e17 864
a8cfd0cb
VZ
865 menus[i]->Attach(this);
866 }
2bda0e17
KB
867}
868
b8d3a4f1 869wxMenuBar::~wxMenuBar()
2bda0e17 870{
a96b4743 871 // In Windows CE (not .NET), the menubar is always associated
39d2f9a7 872 // with a toolbar, which destroys the menu implicitly.
a9102b36 873#if defined(WINCE_WITHOUT_COMMANDBAR) && defined(__POCKETPC__)
bf95a04f 874 if (GetToolBar())
a9102b36
JS
875 {
876 wxToolMenuBar* toolMenuBar = wxDynamicCast(GetToolBar(), wxToolMenuBar);
877 if (toolMenuBar)
878 toolMenuBar->SetMenuBar(NULL);
879 }
bf95a04f 880#else
7a0363dd
JS
881 // we should free Windows resources only if Windows doesn't do it for us
882 // which happens if we're attached to a frame
883 if (m_hMenu && !IsAttached())
884 {
3d487566 885#if defined(WINCE_WITH_COMMANDBAR)
a96b4743
JS
886 ::DestroyWindow((HWND) m_commandBar);
887 m_commandBar = (WXHWND) NULL;
888#else
7a0363dd 889 ::DestroyMenu((HMENU)m_hMenu);
598ddd96 890#endif
7a0363dd
JS
891 m_hMenu = (WXHMENU)NULL;
892 }
39d2f9a7 893#endif
c2dcfdef 894}
2bda0e17 895
c2dcfdef
VZ
896// ---------------------------------------------------------------------------
897// wxMenuBar helpers
898// ---------------------------------------------------------------------------
899
900void wxMenuBar::Refresh()
901{
c849ecef
VZ
902 if ( IsFrozen() )
903 return;
904
065de612 905 wxCHECK_RET( IsAttached(), wxT("can't refresh unattached menubar") );
c2dcfdef 906
3fd239fa 907#if defined(WINCE_WITHOUT_COMMANDBAR)
39d2f9a7
JS
908 if (GetToolBar())
909 {
910 CommandBar_DrawMenuBar((HWND) GetToolBar()->GetHWND(), 0);
911 }
3fd239fa 912#elif defined(WINCE_WITH_COMMANDBAR)
a96b4743
JS
913 if (m_commandBar)
914 DrawMenuBar((HWND) m_commandBar);
39d2f9a7 915#else
1e6feb95 916 DrawMenuBar(GetHwndOf(GetFrame()));
39d2f9a7 917#endif
c2dcfdef
VZ
918}
919
920WXHMENU wxMenuBar::Create()
921{
6a17b868 922 // Note: this doesn't work at all on Smartphone,
beb471b5
JS
923 // since you have to use resources.
924 // We'll have to find another way to add a menu
925 // by changing/adding menu items to an existing menu.
3d487566 926#if defined(WINCE_WITHOUT_COMMANDBAR)
39d2f9a7
JS
927 if ( m_hMenu != 0 )
928 return m_hMenu;
929
930 if (!GetToolBar())
931 return 0;
932
933 HWND hCommandBar = (HWND) GetToolBar()->GetHWND();
934 HMENU hMenu = (HMENU)::SendMessage(hCommandBar, SHCMBM_GETMENU, (WPARAM)0, (LPARAM)0);
72e7ec5b 935
3b3dc801 936 // hMenu may be zero on Windows Mobile 5. So add the menus anyway.
72e7ec5b 937 if (1) // (hMenu)
39d2f9a7 938 {
598ddd96 939 TBBUTTON tbButton;
39d2f9a7
JS
940 memset(&tbButton, 0, sizeof(TBBUTTON));
941 tbButton.iBitmap = I_IMAGENONE;
942 tbButton.fsState = TBSTATE_ENABLED;
943 tbButton.fsStyle = TBSTYLE_DROPDOWN | TBSTYLE_NO_DROPDOWN_ARROW | TBSTYLE_AUTOSIZE;
598ddd96 944
39d2f9a7
JS
945 size_t i;
946 for (i = 0; i < GetMenuCount(); i++)
947 {
948 HMENU hPopupMenu = (HMENU) GetMenu(i)->GetHMenu() ;
949 tbButton.dwData = (DWORD)hPopupMenu;
950 wxString label = wxStripMenuCodes(GetLabelTop(i));
951 tbButton.iString = (int) label.c_str();
beb471b5
JS
952
953 int position = i;
598ddd96 954
39d2f9a7 955 tbButton.idCommand = NewControlId();
beb471b5 956 if (!::SendMessage(hCommandBar, TB_INSERTBUTTON, position, (LPARAM)&tbButton))
39d2f9a7
JS
957 {
958 wxLogLastError(wxT("TB_INSERTBUTTON"));
959 }
960 }
961 }
962 m_hMenu = (WXHMENU) hMenu;
963 return m_hMenu;
964#else
3ca6a5f0 965 if ( m_hMenu != 0 )
717a57c2 966 return m_hMenu;
1cf27c63 967
c2dcfdef 968 m_hMenu = (WXHMENU)::CreateMenu();
2bda0e17 969
c2dcfdef 970 if ( !m_hMenu )
c626a8b7 971 {
f6bcfd97 972 wxLogLastError(wxT("CreateMenu"));
c626a8b7 973 }
c2dcfdef 974 else
c626a8b7 975 {
222ed1d6
MB
976 size_t count = GetMenuCount(), i;
977 wxMenuList::iterator it;
978 for ( i = 0, it = m_menus.begin(); i < count; i++, it++ )
c2dcfdef
VZ
979 {
980 if ( !::AppendMenu((HMENU)m_hMenu, MF_POPUP | MF_STRING,
222ed1d6 981 (UINT)(*it)->GetHMenu(),
c2dcfdef
VZ
982 m_titles[i]) )
983 {
f6bcfd97 984 wxLogLastError(wxT("AppendMenu"));
c2dcfdef
VZ
985 }
986 }
c626a8b7 987 }
c626a8b7 988
c2dcfdef 989 return m_hMenu;
39d2f9a7 990#endif
2bda0e17
KB
991}
992
b2c5f143
DE
993int wxMenuBar::MSWPositionForWxMenu(wxMenu *menu, int wxpos)
994{
995 wxASSERT(menu);
996 wxASSERT(menu->GetHMenu());
997 wxASSERT(m_hMenu);
8cc4850c
JS
998
999#if defined(__WXWINCE__)
1000 int totalMSWItems = GetMenuCount();
1001#else
b2c5f143 1002 int totalMSWItems = GetMenuItemCount((HMENU)m_hMenu);
8cc4850c
JS
1003#endif
1004
b2c5f143
DE
1005 int i; // For old C++ compatibility
1006 for(i=wxpos; i<totalMSWItems; i++)
1007 {
1008 if(GetSubMenu((HMENU)m_hMenu,i)==(HMENU)menu->GetHMenu())
1009 return i;
1010 }
1011 for(i=0; i<wxpos; i++)
1012 {
1013 if(GetSubMenu((HMENU)m_hMenu,i)==(HMENU)menu->GetHMenu())
1014 return i;
1015 }
1016 wxFAIL;
1017 return -1;
1018}
1019
c2dcfdef 1020// ---------------------------------------------------------------------------
3dfac970 1021// wxMenuBar functions to work with the top level submenus
c2dcfdef
VZ
1022// ---------------------------------------------------------------------------
1023
3dfac970
VZ
1024// NB: we don't support owner drawn top level items for now, if we do these
1025// functions would have to be changed to use wxMenuItem as well
2bda0e17 1026
a8cfd0cb 1027void wxMenuBar::EnableTop(size_t pos, bool enable)
2bda0e17 1028{
717a57c2 1029 wxCHECK_RET( IsAttached(), wxT("doesn't work with unattached menubars") );
b2c5f143 1030 wxCHECK_RET( pos < GetMenuCount(), wxT("invalid menu index") );
717a57c2
VZ
1031
1032 int flag = enable ? MF_ENABLED : MF_GRAYED;
2bda0e17 1033
b2c5f143 1034 EnableMenuItem((HMENU)m_hMenu, MSWPositionForWxMenu(GetMenu(pos),pos), MF_BYPOSITION | flag);
adc6fb16
VZ
1035
1036 Refresh();
2bda0e17
KB
1037}
1038
a8cfd0cb 1039void wxMenuBar::SetLabelTop(size_t pos, const wxString& label)
2bda0e17 1040{
717a57c2
VZ
1041 wxCHECK_RET( pos < GetMenuCount(), wxT("invalid menu index") );
1042
1043 m_titles[pos] = label;
1044
1045 if ( !IsAttached() )
1046 {
1047 return;
1048 }
1049 //else: have to modify the existing menu
1050
b2c5f143
DE
1051 int mswpos = MSWPositionForWxMenu(GetMenu(pos),pos);
1052
8cd85069 1053 UINT id;
b2c5f143 1054 UINT flagsOld = ::GetMenuState((HMENU)m_hMenu, mswpos, MF_BYPOSITION);
c2dcfdef 1055 if ( flagsOld == 0xFFFFFFFF )
c626a8b7 1056 {
223d09f6 1057 wxLogLastError(wxT("GetMenuState"));
c2dcfdef
VZ
1058
1059 return;
1060 }
1061
1062 if ( flagsOld & MF_POPUP )
1063 {
1064 // HIBYTE contains the number of items in the submenu in this case
ad9bb75f 1065 flagsOld &= 0xff;
b2c5f143 1066 id = (UINT)::GetSubMenu((HMENU)m_hMenu, mswpos);
c626a8b7
VZ
1067 }
1068 else
8cd85069
VZ
1069 {
1070 id = pos;
1071 }
1072
4676948b
JS
1073#ifdef __WXWINCE__
1074 MENUITEMINFO info;
1075 wxZeroMemory(info);
1076 info.cbSize = sizeof(info);
1077 info.fMask = MIIM_TYPE;
1078 info.fType = MFT_STRING;
7ec69821 1079 info.cch = label.length();
4676948b
JS
1080 info.dwTypeData = (LPTSTR) label.c_str();
1081 if ( !SetMenuItemInfo(GetHmenu(), id, TRUE, & info) )
1082 {
1083 wxLogLastError(wxT("SetMenuItemInfo"));
1084 }
598ddd96 1085
4676948b 1086#else
b2c5f143 1087 if ( ::ModifyMenu(GetHmenu(), mswpos, MF_BYPOSITION | MF_STRING | flagsOld,
4676948b 1088 id, label) == (int)0xFFFFFFFF )
c2dcfdef 1089 {
f6bcfd97 1090 wxLogLastError(wxT("ModifyMenu"));
c2dcfdef 1091 }
4676948b 1092#endif
717a57c2
VZ
1093
1094 Refresh();
2bda0e17
KB
1095}
1096
a8cfd0cb 1097wxString wxMenuBar::GetLabelTop(size_t pos) const
2bda0e17 1098{
717a57c2
VZ
1099 wxCHECK_MSG( pos < GetMenuCount(), wxEmptyString,
1100 wxT("invalid menu index in wxMenuBar::GetLabelTop") );
8cd85069 1101
4a1c207c 1102 return wxMenuItem::GetLabelFromText(m_titles[pos]);
2bda0e17
KB
1103}
1104
ad9bb75f
VZ
1105// ---------------------------------------------------------------------------
1106// wxMenuBar construction
1107// ---------------------------------------------------------------------------
1108
a8cfd0cb 1109wxMenu *wxMenuBar::Replace(size_t pos, wxMenu *menu, const wxString& title)
1cf27c63 1110{
ad9bb75f
VZ
1111 wxMenu *menuOld = wxMenuBarBase::Replace(pos, menu, title);
1112 if ( !menuOld )
f7f50f49
VZ
1113 return NULL;
1114
ad9bb75f 1115 m_titles[pos] = title;
a8cfd0cb 1116
7e02be85
JS
1117#if defined(WINCE_WITHOUT_COMMANDBAR)
1118 if (IsAttached())
1119#else
1120 if (GetHmenu())
1121#endif
a8cfd0cb 1122 {
b2c5f143
DE
1123 int mswpos = MSWPositionForWxMenu(menuOld,pos);
1124
ad9bb75f 1125 // can't use ModifyMenu() because it deletes the submenu it replaces
b2c5f143 1126 if ( !::RemoveMenu(GetHmenu(), (UINT)mswpos, MF_BYPOSITION) )
a8cfd0cb 1127 {
f6bcfd97 1128 wxLogLastError(wxT("RemoveMenu"));
a8cfd0cb 1129 }
1cf27c63 1130
b2c5f143 1131 if ( !::InsertMenu(GetHmenu(), (UINT)mswpos,
ad9bb75f
VZ
1132 MF_BYPOSITION | MF_POPUP | MF_STRING,
1133 (UINT)GetHmenuOf(menu), title) )
1134 {
f6bcfd97 1135 wxLogLastError(wxT("InsertMenu"));
ad9bb75f
VZ
1136 }
1137
717a57c2
VZ
1138#if wxUSE_ACCEL
1139 if ( menuOld->HasAccels() || menu->HasAccels() )
1140 {
1141 // need to rebuild accell table
1142 RebuildAccelTable();
1143 }
1144#endif // wxUSE_ACCEL
1145
7e02be85
JS
1146 if (IsAttached())
1147 Refresh();
a8cfd0cb 1148 }
ad9bb75f
VZ
1149
1150 return menuOld;
1cf27c63
UM
1151}
1152
a8cfd0cb 1153bool wxMenuBar::Insert(size_t pos, wxMenu *menu, const wxString& title)
1cf27c63 1154{
b2c5f143
DE
1155 // Find out which MSW item before which we'll be inserting before
1156 // wxMenuBarBase::Insert is called and GetMenu(pos) is the new menu.
3d27b574 1157 // If IsAttached() is false this won't be used anyway
7e02be85
JS
1158 bool isAttached =
1159#if defined(WINCE_WITHOUT_COMMANDBAR)
1160 IsAttached();
1161#else
1162 (GetHmenu() != 0);
1163#endif
1164
1165 int mswpos = (!isAttached || (pos == m_menus.GetCount()))
b2c5f143
DE
1166 ? -1 // append the menu
1167 : MSWPositionForWxMenu(GetMenu(pos),pos);
1168
ad9bb75f 1169 if ( !wxMenuBarBase::Insert(pos, menu, title) )
598ddd96 1170 return false;
1cf27c63 1171
ad9bb75f
VZ
1172 m_titles.Insert(title, pos);
1173
7e02be85 1174 if ( isAttached )
ad9bb75f 1175 {
7e02be85 1176#if defined(WINCE_WITHOUT_COMMANDBAR)
39d2f9a7 1177 if (!GetToolBar())
598ddd96
WS
1178 return false;
1179 TBBUTTON tbButton;
39d2f9a7
JS
1180 memset(&tbButton, 0, sizeof(TBBUTTON));
1181 tbButton.iBitmap = I_IMAGENONE;
1182 tbButton.fsState = TBSTATE_ENABLED;
1183 tbButton.fsStyle = TBSTYLE_DROPDOWN | TBSTYLE_NO_DROPDOWN_ARROW | TBSTYLE_AUTOSIZE;
598ddd96 1184
39d2f9a7
JS
1185 HMENU hPopupMenu = (HMENU) menu->GetHMenu() ;
1186 tbButton.dwData = (DWORD)hPopupMenu;
1187 wxString label = wxStripMenuCodes(title);
1188 tbButton.iString = (int) label.c_str();
598ddd96 1189
39d2f9a7
JS
1190 tbButton.idCommand = NewControlId();
1191 if (!::SendMessage((HWND) GetToolBar()->GetHWND(), TB_INSERTBUTTON, pos, (LPARAM)&tbButton))
1192 {
1193 wxLogLastError(wxT("TB_INSERTBUTTON"));
598ddd96 1194 return false;
39d2f9a7 1195 }
b9a958e6 1196 wxUnusedVar(mswpos);
39d2f9a7 1197#else
b2c5f143 1198 if ( !::InsertMenu(GetHmenu(), mswpos,
ad9bb75f
VZ
1199 MF_BYPOSITION | MF_POPUP | MF_STRING,
1200 (UINT)GetHmenuOf(menu), title) )
1201 {
f6bcfd97 1202 wxLogLastError(wxT("InsertMenu"));
ad9bb75f 1203 }
39d2f9a7 1204#endif
717a57c2
VZ
1205#if wxUSE_ACCEL
1206 if ( menu->HasAccels() )
1207 {
1208 // need to rebuild accell table
1209 RebuildAccelTable();
1210 }
1211#endif // wxUSE_ACCEL
1212
7e02be85
JS
1213 if (IsAttached())
1214 Refresh();
a8cfd0cb 1215 }
ad9bb75f 1216
598ddd96 1217 return true;
1cf27c63
UM
1218}
1219
ad9bb75f 1220bool wxMenuBar::Append(wxMenu *menu, const wxString& title)
2bda0e17 1221{
ad9bb75f 1222 WXHMENU submenu = menu ? menu->GetHMenu() : 0;
598ddd96 1223 wxCHECK_MSG( submenu, false, wxT("can't append invalid menu to menubar") );
2bda0e17 1224
717a57c2 1225 if ( !wxMenuBarBase::Append(menu, title) )
598ddd96 1226 return false;
717a57c2 1227
717a57c2
VZ
1228 m_titles.Add(title);
1229
7e02be85
JS
1230#if defined(WINCE_WITHOUT_COMMANDBAR)
1231 if (IsAttached())
1232#else
1233 if (GetHmenu())
1234#endif
ad9bb75f 1235 {
7e02be85 1236#if defined(WINCE_WITHOUT_COMMANDBAR)
39d2f9a7 1237 if (!GetToolBar())
598ddd96
WS
1238 return false;
1239 TBBUTTON tbButton;
39d2f9a7
JS
1240 memset(&tbButton, 0, sizeof(TBBUTTON));
1241 tbButton.iBitmap = I_IMAGENONE;
1242 tbButton.fsState = TBSTATE_ENABLED;
1243 tbButton.fsStyle = TBSTYLE_DROPDOWN | TBSTYLE_NO_DROPDOWN_ARROW | TBSTYLE_AUTOSIZE;
598ddd96 1244
39d2f9a7
JS
1245 size_t pos = GetMenuCount();
1246 HMENU hPopupMenu = (HMENU) menu->GetHMenu() ;
1247 tbButton.dwData = (DWORD)hPopupMenu;
1248 wxString label = wxStripMenuCodes(title);
1249 tbButton.iString = (int) label.c_str();
598ddd96 1250
39d2f9a7
JS
1251 tbButton.idCommand = NewControlId();
1252 if (!::SendMessage((HWND) GetToolBar()->GetHWND(), TB_INSERTBUTTON, pos, (LPARAM)&tbButton))
1253 {
1254 wxLogLastError(wxT("TB_INSERTBUTTON"));
598ddd96 1255 return false;
39d2f9a7
JS
1256 }
1257#else
ad9bb75f
VZ
1258 if ( !::AppendMenu(GetHmenu(), MF_POPUP | MF_STRING,
1259 (UINT)submenu, title) )
1260 {
1261 wxLogLastError(wxT("AppendMenu"));
1262 }
39d2f9a7 1263#endif
ad9bb75f 1264
717a57c2
VZ
1265#if wxUSE_ACCEL
1266 if ( menu->HasAccels() )
1267 {
39d2f9a7 1268 // need to rebuild accelerator table
717a57c2
VZ
1269 RebuildAccelTable();
1270 }
1271#endif // wxUSE_ACCEL
1272
7e02be85
JS
1273 if (IsAttached())
1274 Refresh();
ad9bb75f 1275 }
2bda0e17 1276
598ddd96 1277 return true;
2bda0e17
KB
1278}
1279
a8cfd0cb 1280wxMenu *wxMenuBar::Remove(size_t pos)
2bda0e17 1281{
a8cfd0cb
VZ
1282 wxMenu *menu = wxMenuBarBase::Remove(pos);
1283 if ( !menu )
1284 return NULL;
c626a8b7 1285
7e02be85
JS
1286#if defined(WINCE_WITHOUT_COMMANDBAR)
1287 if (IsAttached())
1288#else
1289 if (GetHmenu())
1290#endif
ad9bb75f 1291 {
7e02be85 1292#if defined(WINCE_WITHOUT_COMMANDBAR)
39d2f9a7
JS
1293 if (GetToolBar())
1294 {
1295 if (!::SendMessage((HWND) GetToolBar()->GetHWND(), TB_DELETEBUTTON, (UINT) pos, (LPARAM) 0))
1296 {
1297 wxLogLastError(wxT("TB_DELETEBUTTON"));
1298 }
1299 }
1300#else
b2c5f143 1301 if ( !::RemoveMenu(GetHmenu(), (UINT)MSWPositionForWxMenu(menu,pos), MF_BYPOSITION) )
ad9bb75f 1302 {
f6bcfd97 1303 wxLogLastError(wxT("RemoveMenu"));
ad9bb75f 1304 }
39d2f9a7 1305#endif
c4053ed3 1306
717a57c2
VZ
1307#if wxUSE_ACCEL
1308 if ( menu->HasAccels() )
1309 {
1310 // need to rebuild accell table
1311 RebuildAccelTable();
1312 }
1313#endif // wxUSE_ACCEL
1314
7e02be85
JS
1315 if (IsAttached())
1316 Refresh();
ad9bb75f 1317 }
2bda0e17 1318
c4053ed3 1319 m_titles.RemoveAt(pos);
2bda0e17 1320
a8cfd0cb 1321 return menu;
2bda0e17
KB
1322}
1323
d427503c 1324#if wxUSE_ACCEL
717a57c2
VZ
1325
1326void wxMenuBar::RebuildAccelTable()
1327{
1328 // merge the accelerators of all menus into one accel table
42e69d6b 1329 size_t nAccelCount = 0;
a8cfd0cb 1330 size_t i, count = GetMenuCount();
222ed1d6
MB
1331 wxMenuList::iterator it;
1332 for ( i = 0, it = m_menus.begin(); i < count; i++, it++ )
42e69d6b 1333 {
222ed1d6 1334 nAccelCount += (*it)->GetAccelCount();
42e69d6b
VZ
1335 }
1336
5df1250b 1337 if ( nAccelCount )
42e69d6b 1338 {
5df1250b 1339 wxAcceleratorEntry *accelEntries = new wxAcceleratorEntry[nAccelCount];
42e69d6b 1340
5df1250b 1341 nAccelCount = 0;
222ed1d6 1342 for ( i = 0, it = m_menus.begin(); i < count; i++, it++ )
5df1250b 1343 {
222ed1d6 1344 nAccelCount += (*it)->CopyAccels(&accelEntries[nAccelCount]);
5df1250b
VZ
1345 }
1346
1347 m_accelTable = wxAcceleratorTable(nAccelCount, accelEntries);
42e69d6b 1348
5df1250b
VZ
1349 delete [] accelEntries;
1350 }
717a57c2
VZ
1351}
1352
1353#endif // wxUSE_ACCEL
1354
1355void wxMenuBar::Attach(wxFrame *frame)
1356{
1e6feb95 1357 wxMenuBarBase::Attach(frame);
717a57c2 1358
3fd239fa 1359#if defined(WINCE_WITH_COMMANDBAR)
a96b4743
JS
1360 if (!m_hMenu)
1361 this->Create();
a96b4743
JS
1362 if (!m_commandBar)
1363 m_commandBar = (WXHWND) CommandBar_Create(wxGetInstance(), (HWND) frame->GetHWND(), NewControlId());
1364 if (m_commandBar)
1365 {
1366 if (m_hMenu)
1367 {
1368 if (!CommandBar_InsertMenubarEx((HWND) m_commandBar, NULL, (LPTSTR) m_hMenu, 0))
1369 {
1370 wxLogLastError(wxT("CommandBar_InsertMenubarEx"));
1371 }
1372 }
1373 }
1374#endif
a96b4743 1375
717a57c2
VZ
1376#if wxUSE_ACCEL
1377 RebuildAccelTable();
d427503c 1378#endif // wxUSE_ACCEL
42e69d6b
VZ
1379}
1380
3fd239fa 1381#if defined(WINCE_WITH_COMMANDBAR)
a9928e9d
JS
1382bool wxMenuBar::AddAdornments(long style)
1383{
1384 if (m_adornmentsAdded || !m_commandBar)
1385 return false;
1386
1387 if (style & wxCLOSE_BOX)
1388 {
1389 if (!CommandBar_AddAdornments((HWND) m_commandBar, 0, 0))
1390 wxLogLastError(wxT("CommandBar_AddAdornments"));
1391 else
1392 return true;
1393 }
1394 return false;
1395}
1396#endif
1397
1cf27c63
UM
1398void wxMenuBar::Detach()
1399{
1e6feb95 1400 wxMenuBarBase::Detach();
2bda0e17
KB
1401}
1402
1e6feb95 1403#endif // wxUSE_MENUS