fixed modal dialogs after event loop change

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@23618 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
Vadim Zeitlin
2003-09-16 12:30:34 +00:00
parent fb761cd53c
commit 6757b5e31c
2 changed files with 58 additions and 49 deletions

View File

@@ -20,7 +20,7 @@
WXDLLEXPORT_DATA(extern const wxChar*) wxDialogNameStr; WXDLLEXPORT_DATA(extern const wxChar*) wxDialogNameStr;
class WXDLLEXPORT wxWindowDisabler; class WXDLLEXPORT wxDialogModalData;
// Dialog boxes // Dialog boxes
class WXDLLEXPORT wxDialog : public wxDialogBase class WXDLLEXPORT wxDialog : public wxDialogBase
@@ -116,9 +116,9 @@ protected:
private: private:
wxWindow *m_oldFocus; wxWindow *m_oldFocus;
// while we are showing a modal dialog we disable the other windows using // this pointer is non-NULL only while the modal event loop is running
// this object wxDialogModalData *m_modalData;
wxWindowDisabler *m_windowDisabler;
DECLARE_DYNAMIC_CLASS(wxDialog) DECLARE_DYNAMIC_CLASS(wxDialog)
DECLARE_EVENT_TABLE() DECLARE_EVENT_TABLE()

View File

@@ -58,13 +58,6 @@
#define wxDIALOG_DEFAULT_WIDTH 500 #define wxDIALOG_DEFAULT_WIDTH 500
#define wxDIALOG_DEFAULT_HEIGHT 500 #define wxDIALOG_DEFAULT_HEIGHT 500
// ----------------------------------------------------------------------------
// globals
// ----------------------------------------------------------------------------
// all modal dialogs currently shown
static wxWindowList wxModalDialogs;
// ---------------------------------------------------------------------------- // ----------------------------------------------------------------------------
// wxWin macros // wxWin macros
// ---------------------------------------------------------------------------- // ----------------------------------------------------------------------------
@@ -138,6 +131,42 @@ BEGIN_EVENT_TABLE(wxDialog, wxDialogBase)
EVT_CLOSE(wxDialog::OnCloseWindow) EVT_CLOSE(wxDialog::OnCloseWindow)
END_EVENT_TABLE() END_EVENT_TABLE()
// ----------------------------------------------------------------------------
// wxDialogModalData
// ----------------------------------------------------------------------------
// this is simply a container for wxEventLoop and wxWindowDisabler which allows
// to have a single opaque pointer in wxDialog itself
class wxDialogModalData
{
public:
wxDialogModalData() { m_windowDisabler = NULL; }
void RunLoop(wxDialog *dialog)
{
m_windowDisabler = new wxWindowDisabler(dialog);
m_evtLoop.Run();
}
void ExitLoop()
{
delete m_windowDisabler;
m_windowDisabler = NULL;
m_evtLoop.Exit();
}
~wxDialogModalData()
{
wxASSERT_MSG( !m_windowDisabler, _T("forgot to call ExitLoop?") );
}
private:
wxEventLoop m_evtLoop;
wxWindowDisabler *m_windowDisabler;
};
// ============================================================================ // ============================================================================
// implementation // implementation
// ============================================================================ // ============================================================================
@@ -152,7 +181,7 @@ void wxDialog::Init()
m_isShown = FALSE; m_isShown = FALSE;
m_windowDisabler = (wxWindowDisabler *)NULL; m_modalData = NULL;
SetBackgroundColour(wxSystemSettings::GetColour(wxSYS_COLOUR_3DFACE)); SetBackgroundColour(wxSystemSettings::GetColour(wxSYS_COLOUR_3DFACE));
} }
@@ -247,7 +276,7 @@ bool wxDialog::IsModal() const
bool wxDialog::IsModalShowing() const bool wxDialog::IsModalShowing() const
{ {
return wxModalDialogs.Find(wxConstCast(this, wxDialog)) != NULL; return m_modalData != NULL;
} }
wxWindow *wxDialog::FindSuitableParent() const wxWindow *wxDialog::FindSuitableParent() const
@@ -277,8 +306,6 @@ void wxDialog::DoShowModal()
wxCHECK_RET( !IsModalShowing(), _T("DoShowModal() called twice") ); wxCHECK_RET( !IsModalShowing(), _T("DoShowModal() called twice") );
wxCHECK_RET( IsModal(), _T("can't DoShowModal() modeless dialog") ); wxCHECK_RET( IsModal(), _T("can't DoShowModal() modeless dialog") );
wxModalDialogs.Append(this);
wxWindow *parent = GetParent(); wxWindow *parent = GetParent();
wxWindow* oldFocus = m_oldFocus; wxWindow* oldFocus = m_oldFocus;
@@ -298,11 +325,6 @@ void wxDialog::DoShowModal()
hwndOldFocus = GetHwndOf(parent); hwndOldFocus = GetHwndOf(parent);
} }
// disable all other app windows
wxASSERT_MSG( !m_windowDisabler, _T("disabling windows twice?") );
m_windowDisabler = new wxWindowDisabler(this);
// before entering the modal loop, reset the "is in OnIdle()" flag (see // before entering the modal loop, reset the "is in OnIdle()" flag (see
// comment in app.cpp) // comment in app.cpp)
extern bool wxIsInOnIdleFlag; extern bool wxIsInOnIdleFlag;
@@ -310,8 +332,10 @@ void wxDialog::DoShowModal()
wxIsInOnIdleFlag = FALSE; wxIsInOnIdleFlag = FALSE;
// enter the modal loop // enter the modal loop
wxEventLoop evtLoop; m_modalData = new wxDialogModalData;
evtLoop.Run(); m_modalData->RunLoop(this);
delete m_modalData;
m_modalData = NULL;
wxIsInOnIdleFlag = wasInOnIdle; wxIsInOnIdleFlag = wasInOnIdle;
@@ -329,16 +353,14 @@ void wxDialog::DoShowModal()
bool wxDialog::Show(bool show) bool wxDialog::Show(bool show)
{ {
if ( !show ) if ( !show && m_modalData )
{ {
// if we had disabled other app windows, reenable them back now because // we need to do this before calling wxDialogBase version because if we
// had disabled other app windows, they must be reenabled right now as
// if they stay disabled Windows will activate another window (one // if they stay disabled Windows will activate another window (one
// which is enabled, anyhow) and we will lose activation // which is enabled, anyhow) when we're hidden in the base class Show()
if ( m_windowDisabler ) // and we will lose activation
{ m_modalData->ExitLoop();
delete m_windowDisabler;
m_windowDisabler = NULL;
}
} }
// ShowModal() may be called for already shown dialog // ShowModal() may be called for already shown dialog
@@ -358,32 +380,19 @@ bool wxDialog::Show(bool show)
// every time is simpler than keeping a flag // every time is simpler than keeping a flag
Layout(); Layout();
// usually will result in TransferDataToWindow() being called // this usually will result in TransferDataToWindow() being called
InitDialog(); InitDialog();
} }
if ( IsModal() ) if ( show && IsModal() )
{ {
if ( show ) // modal dialog needs a parent window, so try to find one
if ( !GetParent() )
{ {
// modal dialog needs a parent window, so try to find one m_parent = FindSuitableParent();
if ( !GetParent() )
{
m_parent = FindSuitableParent();
}
DoShowModal();
} }
else // end of modal dialog
{
// this will cause IsModalShowing() return FALSE and our local
// message loop will terminate
wxModalDialogs.DeleteObject(this);
// ensure that there is another message for this window so the DoShowModal();
// ShowModal loop will exit and won't get stuck in GetMessage().
::PostMessage(GetHwnd(), WM_NULL, 0, 0);
}
} }
return TRUE; return TRUE;