X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/3857173d3c186214cbc73ce764d122d5cf7b98d6..fb8d7eb7a880f1f2e32d8830f9c5e12b2536e05f:/src/os2/dialog.cpp?ds=sidebyside diff --git a/src/os2/dialog.cpp b/src/os2/dialog.cpp index 12d98447d1..c6520d9698 100644 --- a/src/os2/dialog.cpp +++ b/src/os2/dialog.cpp @@ -25,7 +25,8 @@ #include "wx/os2/private.h" #include "wx/evtloop.h" -#include "wx/ptr_scpd.h" +#include "wx/scopedptr.h" +#include "wx/modalhook.h" #define wxDIALOG_DEFAULT_X 300 #define wxDIALOG_DEFAULT_Y 300 @@ -33,7 +34,6 @@ #define wxDIALOG_DEFAULT_WIDTH 500 #define wxDIALOG_DEFAULT_HEIGHT 500 -IMPLEMENT_DYNAMIC_CLASS(wxDialog, wxTopLevelWindow) // ---------------------------------------------------------------------------- // wxDialogModalData @@ -72,9 +72,9 @@ wxDEFINE_TIED_SCOPED_PTR_TYPE(wxDialogModalData); void wxDialog::Init() { - m_pOldFocus = (wxWindow *)NULL; + m_pOldFocus = NULL; m_isShown = false; - m_pWindowDisabler = (wxWindowDisabler *)NULL; + m_pWindowDisabler = NULL; m_modalData = NULL; SetBackgroundColour(wxSystemSettings::GetColour(wxSYS_COLOUR_3DFACE)); } // end of wxDialog::Init @@ -115,7 +115,7 @@ bool wxDialog::Create( wxWindow* pParent, // // Must defer setting the title until after dialog is created and sized // - if (!rsTitle.IsNull()) + if ( !rsTitle.empty() ) SetTitle(rsTitle); return true; } // end of wxDialog::Create @@ -147,7 +147,7 @@ void wxDialog::SetModal(bool WXUNUSED(bFlag)) wxDialog::~wxDialog() { - m_isBeingDeleted = true; + SendDestroyEvent(); // this will also reenable all the other windows for a modal dialog Show(false); @@ -166,28 +166,6 @@ bool wxDialog::IsModalShowing() const #endif // WXWIN_COMPATIBILITY_2_6 -wxWindow *wxDialog::FindSuitableParent() const -{ - // first try to use the currently active window - HWND hwndFg = ::WinQueryActiveWindow(HWND_DESKTOP); - wxWindow *parent = hwndFg ? wxFindWinFromHandle((WXHWND)hwndFg) - : NULL; - if ( !parent ) - { - // next try the main app window - parent = wxTheApp->GetTopWindow(); - } - - // finally, check if the parent we found is really suitable - if ( !parent || parent == (wxWindow *)this || !parent->IsShown() ) - { - // don't use this one - parent = NULL; - } - - return parent; -} - bool wxDialog::Show( bool bShow ) { if ( bShow == IsShown() ) @@ -202,16 +180,15 @@ bool wxDialog::Show( bool bShow ) // and we will lose activation m_modalData->ExitLoop(); #if 0 - if (m_pWindowDisabler) - { - delete m_pWindowDisabler; - m_pWindowDisabler = NULL; - } + wxDELETE(m_pWindowDisabler); #endif } if (bShow) { + if (CanDoLayoutAdaptation()) + DoLayoutAdaptation(); + // this usually will result in TransferDataToWindow() being called // which will change the controls values so do it before showing as // otherwise we could have some flicker @@ -222,7 +199,7 @@ bool wxDialog::Show( bool bShow ) wxString title = GetTitle(); if (!title.empty()) - ::WinSetWindowText((HWND)GetHwnd(), (PSZ)title.c_str()); + ::WinSetWindowText((HWND)GetHwnd(), title.c_str()); if ( bShow ) { @@ -243,7 +220,9 @@ bool wxDialog::Show( bool bShow ) // int wxDialog::ShowModal() { - wxASSERT_MSG( !IsModal(), _T("wxDialog::ShowModal() reentered?") ); + WX_HOOK_MODAL_DIALOG(); + + wxASSERT_MSG( !IsModal(), wxT("wxDialog::ShowModal() reentered?") ); m_endModalCalled = false; @@ -255,11 +234,7 @@ int wxDialog::ShowModal() if ( !m_endModalCalled ) { // modal dialog needs a parent window, so try to find one - wxWindow *parent = GetParent(); - if ( !parent ) - { - parent = FindSuitableParent(); - } + wxWindow * const parent = GetParentForModalDialog(); // remember where the focus was wxWindow *oldFocus = m_pOldFocus; @@ -311,7 +286,7 @@ void wxDialog::EndModal( int nRetCode ) { - wxASSERT_MSG( IsModal(), _T("EndModal() called for non modal dialog") ); + wxASSERT_MSG( IsModal(), wxT("EndModal() called for non modal dialog") ); m_endModalCalled = true; SetReturnCode(nRetCode);