// ----------------------------------------------------------------------------
extern wxWindowList wxModelessWindows;
-extern wxList WXDLLEXPORT wxPendingDelete;
extern const wxChar *wxFrameClassName;
#if wxUSE_MENUS_NATIVE
{
SetName(name);
m_windowStyle = style;
-#if wxUSE_MENUS
- m_frameMenuBar = NULL;
-#endif // wxUSE_MENUS
-#if wxUSE_TOOLBAR
- m_frameToolBar = NULL;
-#endif // wxUSE_TOOLBAR
-#if wxUSE_STATUSBAR
- m_frameStatusBar = NULL;
-#endif // wxUSE_STATUSBAR
SetBackgroundColour(wxSystemSettings::GetSystemColour(wxSYS_COLOUR_APPWORKSPACE));
wxTopLevelWindows.Append(this);
- MSWCreate(m_windowId, parent, wxFrameClassName, this, title,
+ // the frame must have NULL parent HWND or it would be always on top of its
+ // parent which is not what we usually want (in fact, we only want it for
+ // frames with the special wxFRAME_TOOL_WINDOW style handled elsewhere)
+ MSWCreate(m_windowId, NULL, wxFrameClassName, this, title,
x, y, width, height, style);
wxModelessWindows.Append(this);
}
#endif // wxUSE_STATUSBAR
-void wxFrameMSW::DetachMenuBar()
-{
-#if wxUSE_MENUS
- if ( m_frameMenuBar )
- {
- m_frameMenuBar->Detach();
- m_frameMenuBar = NULL;
- }
-#endif // wxUSE_MENUS
-}
+#if wxUSE_MENUS_NATIVE
-void wxFrameMSW::SetMenuBar(wxMenuBar *menubar)
+void wxFrameMSW::AttachMenuBar(wxMenuBar *menubar)
{
-#if wxUSE_MENUS
- // detach the old menu bar in any case
- DetachMenuBar();
+ wxFrameBase::AttachMenuBar(menubar);
-#if wxUSE_MENUS_NATIVE
if ( !menubar )
{
// actually remove the menu from the frame
{
m_hMenu = menubar->GetHMenu();
}
- else
+ else // no HMENU yet
{
- if (menubar->IsAttached())
- menubar->Detach();
-
m_hMenu = menubar->Create();
if ( !m_hMenu )
+ {
+ wxFAIL_MSG( _T("failed to create menu bar") );
return;
+ }
}
InternalSetMenuBar();
}
-#endif // wxUSE_MENUS_NATIVE
-
- if ( menubar )
- {
- m_frameMenuBar = menubar;
- menubar->Attach((wxFrame *)this);
- }
-#endif // wxUSE_MENUS
}
-#if wxUSE_MENUS_NATIVE
-
void wxFrameMSW::InternalSetMenuBar()
{
#ifndef __WXMICROWIN__