diff --git a/include/wx/msw/dialog.h b/include/wx/msw/dialog.h index f7a07592f8..e8eb98da7d 100644 --- a/include/wx/msw/dialog.h +++ b/include/wx/msw/dialog.h @@ -60,43 +60,38 @@ public: ~wxDialog(); + // override some base class virtuals virtual bool Destroy(); + virtual bool Show(bool show); + virtual void Iconize(bool iconize); + virtual bool IsIconized() const; - virtual void DoSetClientSize(int width, int height); + virtual bool IsTopLevel() const { return TRUE; } - virtual void GetPosition(int *x, int *y) const; + void SetModal(bool flag); + virtual bool IsModal() const; - bool Show(bool show); - bool IsShown() const; - void Iconize(bool iconize); + // For now, same as Show(TRUE) but returns return code + virtual int ShowModal(); + + // may be called to terminate the dialog with the given return code + virtual void EndModal(int retCode); + + // returns TRUE if we're in a modal loop + bool IsModalShowing() const; #if WXWIN_COMPATIBILITY bool Iconized() const { return IsIconized(); }; #endif - virtual bool IsIconized() const; - void Fit(); + // implementation only from now on + // ------------------------------- - virtual bool IsTopLevel() const { return TRUE; } - - void SetTitle(const wxString& title); - wxString GetTitle() const ; - - void OnSize(wxSizeEvent& event); + // event handlers bool OnClose(); void OnCharHook(wxKeyEvent& event); - void OnPaint(wxPaintEvent& event); void OnCloseWindow(wxCloseEvent& event); - void SetModal(bool flag); - - virtual void Centre(int direction = wxBOTH); - virtual bool IsModal() const; - - // For now, same as Show(TRUE) but returns return code - virtual int ShowModal(); - virtual void EndModal(int retCode); - // Standard buttons void OnOK(wxCommandEvent& event); void OnApply(wxCommandEvent& event); @@ -105,30 +100,21 @@ public: // Responds to colour changes void OnSysColourChanged(wxSysColourChangedEvent& event); - // implementation - // -------------- - + // Windows callbacks long MSWWindowProc(WXUINT message, WXWPARAM wParam, WXLPARAM lParam); +#if wxUSE_CTL3D virtual WXHBRUSH OnCtlColor(WXHDC pDC, WXHWND pWnd, WXUINT nCtlColor, WXUINT message, WXWPARAM wParam, WXLPARAM lParam); - - bool IsModalShowing() const { return m_modalShowing; } - - // tooltip management -#if wxUSE_TOOLTIPS - WXHWND GetToolTipCtrl() const { return m_hwndToolTip; } - void SetToolTipCtrl(WXHWND hwndTT) { m_hwndToolTip = hwndTT; } -#endif // tooltips +#endif // wxUSE_CTL3D protected: - bool m_modalShowing; - WXHWND m_hwndOldFocus; // the window which had focus before we were shown + // override more base class virtuals + virtual void DoSetClientSize(int width, int height); + virtual void DoGetPosition(int *x, int *y) const; -private: -#if wxUSE_TOOLTIPS - WXHWND m_hwndToolTip; -#endif // tooltips + // show modal dialog and enter modal loop + void DoShowModal(); private: DECLARE_EVENT_TABLE() diff --git a/src/msw/app.cpp b/src/msw/app.cpp index 1be88f6a6e..44e9132bb9 100644 --- a/src/msw/app.cpp +++ b/src/msw/app.cpp @@ -916,12 +916,10 @@ int wxApp::MainLoop() wxMutexGuiLeaveOrEnter(); #endif // wxUSE_THREADS - while ( !::PeekMessage(&s_currentMsg, 0, 0, 0, PM_NOREMOVE) && - ProcessIdle() ) - { - } - + while ( !Pending() && ProcessIdle() ) + ; + // a message came or no more idle processing to do DoMessage(); } @@ -940,12 +938,13 @@ bool wxApp::ProcessIdle() void wxApp::ExitMainLoop() { + // VZ: why not ::PostQuitMessage()? m_keepGoing = FALSE; } bool wxApp::Pending() { - return (::PeekMessage(&s_currentMsg, 0, 0, 0, PM_NOREMOVE) != 0); + return ::PeekMessage(&s_currentMsg, 0, 0, 0, PM_NOREMOVE) != 0; } void wxApp::Dispatch() diff --git a/src/msw/dialog.cpp b/src/msw/dialog.cpp index 874fecd083..926293e878 100644 --- a/src/msw/dialog.cpp +++ b/src/msw/dialog.cpp @@ -1,5 +1,5 @@ ///////////////////////////////////////////////////////////////////////////// -// Name: dialog.cpp +// Name: src/msw/dialog.cpp // Purpose: wxDialog class // Author: Julian Smart // Modified by: @@ -9,61 +9,98 @@ // Licence: wxWindows licence ///////////////////////////////////////////////////////////////////////////// +// ============================================================================ +// declarations +// ============================================================================ + +// ---------------------------------------------------------------------------- +// headers +// ---------------------------------------------------------------------------- + #ifdef __GNUG__ -#pragma implementation "dialog.h" + #pragma implementation "dialog.h" #endif // For compilers that support precompilation, includes "wx.h". #include "wx/wxprec.h" #ifdef __BORLANDC__ -#pragma hdrstop + #pragma hdrstop #endif #ifndef WX_PRECOMP -#include "wx/dialog.h" -#include "wx/utils.h" -#include "wx/frame.h" -#include "wx/app.h" -#include "wx/settings.h" -#include "wx/intl.h" -#include "wx/log.h" + #include "wx/dialog.h" + #include "wx/utils.h" + #include "wx/frame.h" + #include "wx/app.h" + #include "wx/settings.h" + #include "wx/intl.h" + #include "wx/log.h" #endif #include "wx/msw/private.h" #include "wx/log.h" #if wxUSE_COMMON_DIALOGS -#include + #include #endif +// ---------------------------------------------------------------------------- +// constants +// ---------------------------------------------------------------------------- + +// default dialog pos and size + #define wxDIALOG_DEFAULT_X 300 #define wxDIALOG_DEFAULT_Y 300 -// Lists to keep track of windows, so we can disable/enable them -// for modal dialogs -wxWindowList wxModalDialogs; -wxWindowList wxModelessWindows; // Frames and modeless dialogs +#define wxDIALOG_DEFAULT_WIDTH 500 +#define wxDIALOG_DEFAULT_HEIGHT 500 + +// ---------------------------------------------------------------------------- +// globals +// ---------------------------------------------------------------------------- + +// all objects to be deleted during next idle processing - from window.cpp extern wxList WXDLLEXPORT wxPendingDelete; +// all frames and modeless dialogs - not static, used in frame.cpp, mdi.cpp &c +wxWindowList wxModelessWindows; + +// all modal dialogs currently shown +static wxWindowList wxModalDialogs; + +// ---------------------------------------------------------------------------- +// wxWin macros +// ---------------------------------------------------------------------------- + IMPLEMENT_DYNAMIC_CLASS(wxDialog, wxPanel) BEGIN_EVENT_TABLE(wxDialog, wxPanel) - EVT_SIZE(wxDialog::OnSize) EVT_BUTTON(wxID_OK, wxDialog::OnOK) EVT_BUTTON(wxID_APPLY, wxDialog::OnApply) EVT_BUTTON(wxID_CANCEL, wxDialog::OnCancel) + EVT_CHAR_HOOK(wxDialog::OnCharHook) + EVT_SYS_COLOUR_CHANGED(wxDialog::OnSysColourChanged) + EVT_CLOSE(wxDialog::OnCloseWindow) END_EVENT_TABLE() +// ============================================================================ +// implementation +// ============================================================================ + +// ---------------------------------------------------------------------------- +// wxDialog construction +// ---------------------------------------------------------------------------- + wxDialog::wxDialog() { - m_isShown = FALSE; - m_modalShowing = FALSE; + m_isShown = FALSE; - SetBackgroundColour(wxSystemSettings::GetSystemColour(wxSYS_COLOUR_3DFACE)); + SetBackgroundColour(wxSystemSettings::GetSystemColour(wxSYS_COLOUR_3DFACE)); } bool wxDialog::Create(wxWindow *parent, wxWindowID id, @@ -73,10 +110,6 @@ bool wxDialog::Create(wxWindow *parent, wxWindowID id, long style, const wxString& name) { -#if wxUSE_TOOLTIPS - m_hwndToolTip = 0; -#endif - SetBackgroundColour(wxSystemSettings::GetSystemColour(wxSYS_COLOUR_3DFACE)); SetName(name); @@ -97,18 +130,19 @@ bool wxDialog::Create(wxWindow *parent, wxWindowID id, int width = size.x; int height = size.y; - if (x < 0) x = wxDIALOG_DEFAULT_X; - if (y < 0) y = wxDIALOG_DEFAULT_Y; + if (x < 0) + x = wxDIALOG_DEFAULT_X; + if (y < 0) + y = wxDIALOG_DEFAULT_Y; m_windowStyle = style; m_isShown = FALSE; - m_modalShowing = FALSE; if (width < 0) - width = 500; + width = wxDIALOG_DEFAULT_WIDTH; if (height < 0) - height = 500; + height = wxDIALOG_DEFAULT_HEIGHT; // All dialogs should really have this style m_windowStyle |= wxTAB_TRAVERSAL; @@ -152,58 +186,65 @@ bool wxDialog::Create(wxWindow *parent, wxWindowID id, void wxDialog::SetModal(bool flag) { - if ( flag ) - m_windowStyle |= wxDIALOG_MODAL ; - else - if ( m_windowStyle & wxDIALOG_MODAL ) - m_windowStyle -= wxDIALOG_MODAL ; + if ( flag ) + { + m_windowStyle |= wxDIALOG_MODAL; - wxModelessWindows.DeleteObject(this); - if (!flag) - wxModelessWindows.Append(this); + wxModelessWindows.DeleteObject(this); + } + else + { + m_windowStyle &= ~wxDIALOG_MODAL; + + wxModelessWindows.Append(this); + } } wxDialog::~wxDialog() { - m_isBeingDeleted = TRUE; + m_isBeingDeleted = TRUE; - wxTopLevelWindows.DeleteObject(this); + wxTopLevelWindows.DeleteObject(this); - Show(FALSE); + Show(FALSE); - // VZ: this is bogus and breaks focus handling - it won't be returned to the - // window which had it previosuly if we do this + // VZ: this is bogus and breaks focus handling - it won't be returned to + // the window which had it previosuly if we do this #if 0 - if (m_modalShowing) - { - // For some reason, wxWindows can activate another task altogether - // when a frame is destroyed after a modal dialog has been invoked. - // Try to bring the parent to the top. - // dfgg: I moved this following line from end of function - - // must not call if another window is on top!! - // This can often happen with Close() and delayed deleting - if (GetParent() && GetParent()->GetHWND()) - ::BringWindowToTop((HWND) GetParent()->GetHWND()); - } + if (m_modalShowing) + { + // For some reason, wxWindows can activate another task altogether + // when a frame is destroyed after a modal dialog has been invoked. + // Try to bring the parent to the top. + // dfgg: I moved this following line from end of function - + // must not call if another window is on top!! + // This can often happen with Close() and delayed deleting + if (GetParent() && GetParent()->GetHWND()) + ::BringWindowToTop((HWND) GetParent()->GetHWND()); + } + + m_modalShowing = FALSE; #endif // 0 - m_modalShowing = FALSE; + if ( !IsModal() ) + wxModelessWindows.DeleteObject(this); - if ( (GetWindowStyleFlag() & wxDIALOG_MODAL) != wxDIALOG_MODAL ) - wxModelessWindows.DeleteObject(this); - - // If this is the last top-level window, exit. - if (wxTheApp && (wxTopLevelWindows.Number() == 0)) - { - wxTheApp->SetTopWindow(NULL); - - if (wxTheApp->GetExitOnFrameDelete()) + // If this is the last top-level window, exit. + if ( wxTheApp && (wxTopLevelWindows.Number() == 0) ) { - PostQuitMessage(0); + wxTheApp->SetTopWindow(NULL); + + if ( wxTheApp->GetExitOnFrameDelete() ) + { + ::PostQuitMessage(0); + } } - } } +// ---------------------------------------------------------------------------- +// kbd handling +// ---------------------------------------------------------------------------- + // By default, pressing escape cancels the dialog void wxDialog::OnCharHook(wxKeyEvent& event) { @@ -227,381 +268,402 @@ void wxDialog::OnCharHook(wxKeyEvent& event) event.Skip(); } -void wxDialog::OnPaint(wxPaintEvent& event) -{ - // No: if you call the default procedure, it makes - // the following painting code not work. -// wxWindow::OnPaint(event); -} - -void wxDialog::Fit() -{ - wxWindow::Fit(); -} +// ---------------------------------------------------------------------------- +// Windows dialog boxes can't be iconized +// ---------------------------------------------------------------------------- void wxDialog::Iconize(bool WXUNUSED(iconize)) { - // Windows dialog boxes can't be iconized } bool wxDialog::IsIconized() const { - return FALSE; + return FALSE; } +// ---------------------------------------------------------------------------- +// size/position handling +// ---------------------------------------------------------------------------- + void wxDialog::DoSetClientSize(int width, int height) { - HWND hWnd = (HWND) GetHWND(); - RECT rect; - ::GetClientRect(hWnd, &rect); + HWND hWnd = (HWND) GetHWND(); + RECT rect; + ::GetClientRect(hWnd, &rect); - RECT rect2; - GetWindowRect(hWnd, &rect2); + RECT rect2; + GetWindowRect(hWnd, &rect2); - // Find the difference between the entire window (title bar and all) - // and the client area; add this to the new client size to move the - // window - int actual_width = rect2.right - rect2.left - rect.right + width; - int actual_height = rect2.bottom - rect2.top - rect.bottom + height; + // Find the difference between the entire window (title bar and all) + // and the client area; add this to the new client size to move the + // window + int actual_width = rect2.right - rect2.left - rect.right + width; + int actual_height = rect2.bottom - rect2.top - rect.bottom + height; - MoveWindow(hWnd, rect2.left, rect2.top, actual_width, actual_height, TRUE); + MoveWindow(hWnd, rect2.left, rect2.top, actual_width, actual_height, TRUE); - wxSizeEvent event(wxSize(actual_width, actual_height), m_windowId); - event.SetEventObject( this ); - GetEventHandler()->ProcessEvent(event); + wxSizeEvent event(wxSize(actual_width, actual_height), m_windowId); + event.SetEventObject( this ); + GetEventHandler()->ProcessEvent(event); } -void wxDialog::GetPosition(int *x, int *y) const +void wxDialog::DoGetPosition(int *x, int *y) const { - HWND hWnd = (HWND) GetHWND(); - RECT rect; - GetWindowRect(hWnd, &rect); + RECT rect; + GetWindowRect(GetHwnd(), &rect); - *x = rect.left; - *y = rect.top; + if ( x ) + *x = rect.left; + if ( y ) + *y = rect.top; } -bool wxDialog::IsShown() const -{ - return m_isShown; -} +// ---------------------------------------------------------------------------- +// showing the dialogs +// ---------------------------------------------------------------------------- bool wxDialog::IsModal() const { - return wxModalDialogs.Find((wxDialog *)this) != 0; // const_cast + return (GetWindowStyleFlag() & wxDIALOG_MODAL) != 0; +} + +// VZ: this is the old version unchanged (reindented only), it will be removed +// as soon as we're sure the new one works correctly +#if 0 + +bool wxDialog::Show(bool show) +{ + m_isShown = show; + + if (show) + InitDialog(); + + bool modal = IsModal(); + + if (modal) + { + if (show) + { + // find the top level window which had focus before - we will restore + // focus to it later + m_hwndOldFocus = 0; + for ( HWND hwnd = ::GetFocus(); hwnd; hwnd = ::GetParent(hwnd) ) + { + m_hwndOldFocus = (WXHWND)hwnd; + } + + if (m_modalShowing) + { + BringWindowToTop((HWND) GetHWND()); + return TRUE; + } + + m_modalShowing = TRUE; + wxNode *node = wxModalDialogs.First(); + while (node) + { + wxDialog *box = (wxDialog *)node->Data(); + if (box != this) + ::EnableWindow((HWND) box->GetHWND(), FALSE); + node = node->Next(); + } + + // if we don't do it, some window might be deleted while we have pointers + // to them in our disabledWindows list and the program will crash when it + // will try to reenable them after the modal dialog end + wxTheApp->DeletePendingObjects(); + wxList disabledWindows; + + node = wxModelessWindows.First(); + while (node) + { + wxWindow *win = (wxWindow *)node->Data(); + if (::IsWindowEnabled((HWND) win->GetHWND())) + { + ::EnableWindow((HWND) win->GetHWND(), FALSE); + disabledWindows.Append(win); + } + node = node->Next(); + } + + ShowWindow((HWND) GetHWND(), SW_SHOW); + EnableWindow((HWND) GetHWND(), TRUE); + BringWindowToTop((HWND) GetHWND()); + + if ( !wxModalDialogs.Find(this) ) + wxModalDialogs.Append(this); + + MSG msg; + // Must test whether this dialog still exists: we may not process + // a message before the deletion. + while (wxModalDialogs.Find(this) && m_modalShowing && GetMessage(&msg, NULL, 0, 0)) + { + if ( m_acceleratorTable.Ok() && + ::TranslateAccelerator((HWND)GetHWND(), + (HACCEL)m_acceleratorTable.GetHACCEL(), + &msg) ) + { + // Have processed the message + } + else if ( !wxTheApp->ProcessMessage((WXMSG *)&msg) ) + { + TranslateMessage(&msg); + DispatchMessage(&msg); + } + + // If we get crashes (as per George Tasker's message) with nested modal dialogs, + // we should try removing the m_modalShowing test + + if (m_modalShowing && !::PeekMessage(&msg, 0, 0, 0, PM_NOREMOVE)) + // dfgg: NB MUST test m_modalShowing again as the message loop could have triggered + // a Show(FALSE) in the mean time!!! + // Without the test, we might delete the dialog before the end of modal showing. + { + while (wxTheApp->ProcessIdle() && m_modalShowing) + { + // Keep going until we decide we've done enough + } + } + } + // dfgg: now must specifically re-enable all other app windows that we disabled earlier + node=disabledWindows.First(); + while(node) { + wxWindow* win = (wxWindow*) node->Data(); + if (wxModalDialogs.Find(win) || wxModelessWindows.Find(win)) + { + HWND hWnd = (HWND) win->GetHWND(); + if (::IsWindow(hWnd)) + ::EnableWindow(hWnd,TRUE); + } + node=node->Next(); + } + } + else // !show + { + ::SetFocus((HWND)m_hwndOldFocus); + + wxModalDialogs.DeleteObject(this); + + wxNode *last = wxModalDialogs.Last(); + + // If there's still a modal dialog active, we + // enable it, else we enable all modeless windows + if (last) + { + // VZ: I don't understand what this is supposed to do, so I'll leave + // it out for now and look for horrible consequences + wxDialog *box = (wxDialog *)last->Data(); + HWND hwnd = (HWND) box->GetHWND(); +#if 0 + if (box->IsUserEnabled()) +#endif // 0 + EnableWindow(hwnd, TRUE); + BringWindowToTop(hwnd); + } + else + { + wxNode *node = wxModelessWindows.First(); + while (node) + { + wxWindow *win = (wxWindow *)node->Data(); + HWND hwnd = (HWND) win->GetHWND(); + // Only enable again if not user-disabled. +#if 0 + if (win->IsUserEnabled()) +#endif // 0 + EnableWindow(hwnd, TRUE); + node = node->Next(); + } + } + // Try to highlight the correct window (the parent) + HWND hWndParent = 0; + if (GetParent()) + { + hWndParent = (HWND) GetParent()->GetHWND(); + if (hWndParent) + ::BringWindowToTop(hWndParent); + } + ShowWindow((HWND) GetHWND(), SW_HIDE); + m_modalShowing = FALSE; + } + } + else // !modal + { + if (show) + { + ShowWindow((HWND) GetHWND(), SW_SHOW); + BringWindowToTop((HWND) GetHWND()); + } + else + { + // Try to highlight the correct window (the parent) + HWND hWndParent = 0; + if (GetParent()) + { + hWndParent = (HWND) GetParent()->GetHWND(); + if (hWndParent) + ::BringWindowToTop(hWndParent); + } + + if ( m_hWnd ) + ShowWindow((HWND) GetHWND(), SW_HIDE); + } + } + + return TRUE; +} + +#else // 1 + +bool wxDialog::IsModalShowing() const +{ + return wxModalDialogs.Find((wxDialog *)this) != NULL; // const_cast +} + +void wxDialog::DoShowModal() +{ + wxCHECK_RET( !IsModalShowing(), _T("DoShowModal() called twice") ); + + wxModalDialogs.Append(this); + + wxWindow *parent = GetParent(); + + // remember where the focus was + wxWindow *winFocus = FindFocus(); + if ( !winFocus ) + { + winFocus = parent; + } + if ( !winFocus ) + { + winFocus = wxTheApp->GetTopWindow(); + } + + // disable the parent window first + HWND hwndParent = parent ? GetHwndOf(parent) : (HWND)NULL; + if ( hwndParent ) + { + ::EnableWindow(hwndParent, FALSE); + } + + // enter the modal loop + while ( IsModalShowing() ) + { +#if wxUSE_THREADS + wxMutexGuiLeaveOrEnter(); +#endif // wxUSE_THREADS + + while ( !wxTheApp->Pending() && wxTheApp->ProcessIdle() ) + ; + + // a message came or no more idle processing to do + wxTheApp->DoMessage(); + } + + // reenable the parent window if any + if ( hwndParent ) + { + ::EnableWindow(hwndParent, TRUE); + } + + // and restore focus + if ( winFocus ) + { + winFocus->SetFocus(); + } } bool wxDialog::Show(bool show) { - m_isShown = show; - - if (show) - InitDialog(); - - bool modal = ((GetWindowStyleFlag() & wxDIALOG_MODAL) == wxDIALOG_MODAL) ; - -#if WXGARBAGE_COLLECTION_ON /* MATTHEW: GC */ - if (!modal) { - if (show) { - if (!wxModelessWindows.Find(this)) - wxModelessWindows.Append(this); - } else - wxModelessWindows.DeleteObject(this); - } - if (show) { - if (!wxTopLevelWindows.Find(this)) - wxTopLevelWindows.Append(this); - } else - wxTopLevelWindows.DeleteObject(this); -#endif - - if (modal) - { - if (show) + if ( !wxDialogBase::Show(show) ) { - // find the top level window which had focus before - we will restore - // focus to it later - m_hwndOldFocus = 0; - for ( HWND hwnd = ::GetFocus(); hwnd; hwnd = ::GetParent(hwnd) ) - { - m_hwndOldFocus = (WXHWND)hwnd; - } - - if (m_modalShowing) - { - BringWindowToTop((HWND) GetHWND()); - return TRUE; - } - - m_modalShowing = TRUE; - wxNode *node = wxModalDialogs.First(); - while (node) - { - wxDialog *box = (wxDialog *)node->Data(); - if (box != this) - ::EnableWindow((HWND) box->GetHWND(), FALSE); - node = node->Next(); - } - - // if we don't do it, some window might be deleted while we have pointers - // to them in our disabledWindows list and the program will crash when it - // will try to reenable them after the modal dialog end - wxTheApp->DeletePendingObjects(); - wxList disabledWindows; - - node = wxModelessWindows.First(); - while (node) - { - wxWindow *win = (wxWindow *)node->Data(); - if (::IsWindowEnabled((HWND) win->GetHWND())) - { - ::EnableWindow((HWND) win->GetHWND(), FALSE); - disabledWindows.Append(win); - } - node = node->Next(); - } - - ShowWindow((HWND) GetHWND(), SW_SHOW); - EnableWindow((HWND) GetHWND(), TRUE); - BringWindowToTop((HWND) GetHWND()); - - if ( !wxModalDialogs.Find(this) ) - wxModalDialogs.Append(this); - - MSG msg; - // Must test whether this dialog still exists: we may not process - // a message before the deletion. - while (wxModalDialogs.Find(this) && m_modalShowing && GetMessage(&msg, NULL, 0, 0)) - { - if ( m_acceleratorTable.Ok() && - ::TranslateAccelerator((HWND)GetHWND(), - (HACCEL)m_acceleratorTable.GetHACCEL(), - &msg) ) - { - // Have processed the message - } - else if ( !wxTheApp->ProcessMessage((WXMSG *)&msg) ) - { - TranslateMessage(&msg); - DispatchMessage(&msg); - } - - // If we get crashes (as per George Tasker's message) with nested modal dialogs, - // we should try removing the m_modalShowing test - - if (m_modalShowing && !::PeekMessage(&msg, 0, 0, 0, PM_NOREMOVE)) - // dfgg: NB MUST test m_modalShowing again as the message loop could have triggered - // a Show(FALSE) in the mean time!!! - // Without the test, we might delete the dialog before the end of modal showing. - { - while (wxTheApp->ProcessIdle() && m_modalShowing) - { - // Keep going until we decide we've done enough - } - } - } - // dfgg: now must specifically re-enable all other app windows that we disabled earlier - node=disabledWindows.First(); - while(node) { - wxWindow* win = (wxWindow*) node->Data(); - if (wxModalDialogs.Find(win) || wxModelessWindows.Find(win)) - { - HWND hWnd = (HWND) win->GetHWND(); - if (::IsWindow(hWnd)) - ::EnableWindow(hWnd,TRUE); - } - node=node->Next(); - } + // nothing to do + return FALSE; } - else // !show + + if ( show ) { - ::SetFocus((HWND)m_hwndOldFocus); + // usually will result in TransferDataToWindow() being called + InitDialog(); + } - wxModalDialogs.DeleteObject(this); - - wxNode *last = wxModalDialogs.Last(); - - // If there's still a modal dialog active, we - // enable it, else we enable all modeless windows - if (last) - { - // VZ: I don't understand what this is supposed to do, so I'll leave - // it out for now and look for horrible consequences - wxDialog *box = (wxDialog *)last->Data(); - HWND hwnd = (HWND) box->GetHWND(); -#if 0 - if (box->IsUserEnabled()) -#endif // 0 - EnableWindow(hwnd, TRUE); - BringWindowToTop(hwnd); - } - else - { - wxNode *node = wxModelessWindows.First(); - while (node) + if ( IsModal() ) + { + if ( show ) { - wxWindow *win = (wxWindow *)node->Data(); - HWND hwnd = (HWND) win->GetHWND(); - // Only enable again if not user-disabled. -#if 0 - if (win->IsUserEnabled()) -#endif // 0 - EnableWindow(hwnd, TRUE); - node = node->Next(); + DoShowModal(); + } + else // end of modal dialog + { + // this will cause IsModalShowing() return FALSE and our local + // message loop will terminate + wxModalDialogs.DeleteObject(this); } - } - // Try to highlight the correct window (the parent) - HWND hWndParent = 0; - if (GetParent()) - { - hWndParent = (HWND) GetParent()->GetHWND(); - if (hWndParent) - ::BringWindowToTop(hWndParent); - } - ShowWindow((HWND) GetHWND(), SW_HIDE); - m_modalShowing = FALSE; } - } - else // !modal - { - if (show) - { - ShowWindow((HWND) GetHWND(), SW_SHOW); - BringWindowToTop((HWND) GetHWND()); - } - else - { - // Try to highlight the correct window (the parent) - HWND hWndParent = 0; - if (GetParent()) - { - hWndParent = (HWND) GetParent()->GetHWND(); - if (hWndParent) - ::BringWindowToTop(hWndParent); - } - if ( m_hWnd ) - ShowWindow((HWND) GetHWND(), SW_HIDE); - } - } - return TRUE; + return TRUE; } -void wxDialog::SetTitle(const wxString& title) -{ - SetWindowText((HWND) GetHWND(), title.c_str()); -} - -wxString wxDialog::GetTitle() const -{ - GetWindowText((HWND) GetHWND(), wxBuffer, 1000); - return wxString(wxBuffer); -} - -void wxDialog::Centre(int direction) -{ - int x_offset,y_offset ; - int display_width, display_height; - int width, height, x, y; - wxWindow *parent = GetParent(); - if ((direction & wxCENTER_FRAME) && parent) - { - parent->GetPosition(&x_offset,&y_offset) ; - parent->GetSize(&display_width,&display_height) ; - } - else - { - wxDisplaySize(&display_width, &display_height); - x_offset = 0 ; - y_offset = 0 ; - } - - GetSize(&width, &height); - GetPosition(&x, &y); - - if (direction & wxHORIZONTAL) - x = (int)((display_width - width)/2); - if (direction & wxVERTICAL) - y = (int)((display_height - height)/2); - - SetSize(x+x_offset, y+y_offset, width, height); -} +#endif // 0/1 // Replacement for Show(TRUE) for modal dialogs - returns return code int wxDialog::ShowModal() { - m_windowStyle |= wxDIALOG_MODAL; - Show(TRUE); - return GetReturnCode(); + m_windowStyle |= wxDIALOG_MODAL; + Show(TRUE); + return GetReturnCode(); } +// NB: this function (surprizingly) may be called for both modal and modeless +// dialogs and should work for both of them void wxDialog::EndModal(int retCode) { - SetReturnCode(retCode); - Show(FALSE); + SetReturnCode(retCode); + Show(FALSE); } -// Define for each class of dialog and control -WXHBRUSH wxDialog::OnCtlColor(WXHDC pDC, WXHWND pWnd, WXUINT nCtlColor, - WXUINT message, WXWPARAM wParam, WXLPARAM lParam) -{ -#if wxUSE_CTL3D - HBRUSH hbrush = Ctl3dCtlColorEx(message, wParam, lParam); - return (WXHBRUSH) hbrush; -#else - return 0; -#endif -} +// ---------------------------------------------------------------------------- +// wxWin event handlers +// ---------------------------------------------------------------------------- // Standard buttons void wxDialog::OnOK(wxCommandEvent& event) { if ( Validate() && TransferDataFromWindow() ) { - if ( IsModal() ) - EndModal(wxID_OK); - else - { - SetReturnCode(wxID_OK); - this->Show(FALSE); - } + EndModal(wxID_OK); } } void wxDialog::OnApply(wxCommandEvent& event) { - if (Validate()) - TransferDataFromWindow(); - // TODO probably need to disable the Apply button until things change again + if ( Validate() ) + TransferDataFromWindow(); + + // TODO probably need to disable the Apply button until things change again } void wxDialog::OnCancel(wxCommandEvent& event) { - if ( IsModal() ) - EndModal(wxID_CANCEL); - else - { - SetReturnCode(wxID_CANCEL); - this->Show(FALSE); - } + EndModal(wxID_CANCEL); } void wxDialog::OnCloseWindow(wxCloseEvent& event) { - // We'll send a Cancel message by default, - // which may close the dialog. + // We'll send a Cancel message by default, which may close the dialog. // Check for looping if the Cancel event handler calls Close(). // Note that if a cancel button and handler aren't present in the dialog, // nothing will happen when you close the dialog via the window manager, or - // via Close(). - // We wouldn't want to destroy the dialog by default, since the dialog may have been - // created on the stack. - // However, this does mean that calling dialog->Close() won't delete the dialog - // unless the handler for wxID_CANCEL does so. So use Destroy() if you want to be - // sure to destroy the dialog. - // The default OnCancel (above) simply ends a modal dialog, and hides a modeless dialog. + // via Close(). We wouldn't want to destroy the dialog by default, since + // the dialog may have been created on the stack. However, this does mean + // that calling dialog->Close() won't delete the dialog unless the handler + // for wxID_CANCEL does so. So use Destroy() if you want to be sure to + // destroy the dialog. The default OnCancel (above) simply ends a modal + // dialog, and hides a modeless dialog. + // VZ: this is horrible and MT-unsafe. Can't we reuse some of these global + // lists here? don't dare to change it now, but should be done later! static wxList closing; if ( closing.Member(this) ) @@ -619,29 +681,21 @@ void wxDialog::OnCloseWindow(wxCloseEvent& event) // Destroy the window (delayed, if a managed window) bool wxDialog::Destroy() { - if (!wxPendingDelete.Member(this)) - wxPendingDelete.Append(this); - return TRUE; -} + wxCHECK_MSG( !wxPendingDelete.Member(this), FALSE, + _T("wxDialog destroyed twice") ); -void wxDialog::OnSize(wxSizeEvent& WXUNUSED(event)) -{ - // if we're using constraints - do use them - #if wxUSE_CONSTRAINTS - if ( GetAutoLayout() ) - { - Layout(); - } - #endif + wxPendingDelete.Append(this); + + return TRUE; } void wxDialog::OnSysColourChanged(wxSysColourChangedEvent& event) { #if wxUSE_CTL3D - Ctl3dColorChange(); + Ctl3dColorChange(); #else - SetBackgroundColour(wxSystemSettings::GetSystemColour(wxSYS_COLOUR_3DFACE)); - Refresh(); + SetBackgroundColour(wxSystemSettings::GetSystemColour(wxSYS_COLOUR_3DFACE)); + Refresh(); #endif } @@ -668,3 +722,19 @@ long wxDialog::MSWWindowProc(WXUINT message, WXWPARAM wParam, WXLPARAM lParam) return rc; } + +#if wxUSE_CTL3D + +// Define for each class of dialog and control +WXHBRUSH wxDialog::OnCtlColor(WXHDC WXUNUSED(pDC), + WXHWND WXUNUSED(pWnd), + WXUINT WXUNUSED(nCtlColor), + WXUINT message, + WXWPARAM wParam, + WXLPARAM lParam) +{ + return (WXHBRUSH)Ctl3dCtlColorEx(message, wParam, lParam); +} + +#endif // wxUSE_CTL3D +