X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/c51c4752023d10f398e5480bf3e22e32ba68edac..588c80dea16b3ab32c7f1f8f15b62eb4450a20a9:/src/common/popupcmn.cpp diff --git a/src/common/popupcmn.cpp b/src/common/popupcmn.cpp index 79b3bbf23b..09ca583c04 100644 --- a/src/common/popupcmn.cpp +++ b/src/common/popupcmn.cpp @@ -17,10 +17,6 @@ // headers // ---------------------------------------------------------------------------- -#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA) - #pragma implementation "popupwinbase.h" -#endif - // For compilers that support precompilation, includes "wx.h". #include "wx/wxprec.h" @@ -66,7 +62,7 @@ IMPLEMENT_DYNAMIC_CLASS(wxPopupTransientWindow, wxPopupWindow) class wxPopupWindowHandler : public wxEvtHandler { public: - wxPopupWindowHandler(wxPopupTransientWindow *popup) { m_popup = popup; } + wxPopupWindowHandler(wxPopupTransientWindow *popup) : m_popup(popup) {} protected: // event handlers @@ -82,10 +78,7 @@ private: class wxPopupFocusHandler : public wxEvtHandler { public: - wxPopupFocusHandler(wxPopupTransientWindow *popup) - { - m_popup = popup; - } + wxPopupFocusHandler(wxPopupTransientWindow *popup) : m_popup(popup) {} protected: void OnKillFocus(wxFocusEvent& event); @@ -111,6 +104,12 @@ BEGIN_EVENT_TABLE(wxPopupFocusHandler, wxEvtHandler) EVT_KEY_DOWN(wxPopupFocusHandler::OnKeyDown) END_EVENT_TABLE() +BEGIN_EVENT_TABLE(wxPopupTransientWindow, wxPopupWindow) +#ifdef __WXMSW__ + EVT_IDLE(wxPopupTransientWindow::OnIdle) +#endif +END_EVENT_TABLE() + // ============================================================================ // implementation // ============================================================================ @@ -187,47 +186,41 @@ wxPopupTransientWindow::wxPopupTransientWindow(wxWindow *parent, int style) wxPopupTransientWindow::~wxPopupTransientWindow() { - PopHandlers(); + if (m_handlerPopup && m_handlerPopup->GetNextHandler()) + PopHandlers(); + + wxASSERT(!m_handlerFocus || !m_handlerFocus->GetNextHandler()); + wxASSERT(!m_handlerPopup || !m_handlerPopup->GetNextHandler()); + + delete m_handlerFocus; + delete m_handlerPopup; } void wxPopupTransientWindow::PopHandlers() { if ( m_child ) { - if ( m_handlerPopup && !m_child->RemoveEventHandler(m_handlerPopup) ) + if ( !m_child->RemoveEventHandler(m_handlerPopup) ) { // something is very wrong and someone else probably deleted our // handler - so don't risk deleting it second time m_handlerPopup = NULL; } - - // m_child->ReleaseMouse(); + if (m_child->HasCapture()) + { + m_child->ReleaseMouse(); + } m_child = NULL; } -#ifdef __WXMSW__ if ( m_focus ) { - if ( m_handlerFocus && !m_focus->RemoveEventHandler(m_handlerFocus) ) + if ( !m_focus->RemoveEventHandler(m_handlerFocus) ) { // see above m_handlerFocus = NULL; } } -#else - if ( m_handlerFocus && !RemoveEventHandler(m_handlerFocus) ) - { - // see above - m_handlerFocus = NULL; - } -#endif - - // delete the handlers, they'll be created as necessary in Popup() - delete m_handlerPopup; - m_handlerPopup = NULL; - delete m_handlerFocus; - m_handlerFocus = NULL; - m_focus = NULL; } @@ -245,10 +238,13 @@ void wxPopupTransientWindow::Popup(wxWindow *winFocus) Show(); - delete m_handlerPopup; - m_handlerPopup = new wxPopupWindowHandler(this); + // There is is a problem if these are still in use + wxASSERT(!m_handlerFocus || !m_handlerFocus->GetNextHandler()); + wxASSERT(!m_handlerPopup || !m_handlerPopup->GetNextHandler()); + + if (!m_handlerPopup) + m_handlerPopup = new wxPopupWindowHandler(this); - // m_child->CaptureMouse(); m_child->PushEventHandler(m_handlerPopup); m_focus = winFocus ? winFocus : this; @@ -259,21 +255,19 @@ void wxPopupTransientWindow::Popup(wxWindow *winFocus) // subclass the window which has the focus, and not winFocus passed in or // otherwise everything else breaks down m_focus = FindFocus(); +#elif defined(__WXGTK__) + // GTK+ catches the activate events from the popup + // window, not the focus events from the child window + m_focus = this; +#endif + if ( m_focus ) { - delete m_handlerFocus; - m_handlerFocus = new wxPopupFocusHandler(this); + if (!m_handlerFocus) + m_handlerFocus = new wxPopupFocusHandler(this); m_focus->PushEventHandler(m_handlerFocus); } -#else - // GTK+ catches the activate events from the popup - // window, not the focus events from the child window - delete m_handlerFocus; - m_handlerFocus = new wxPopupFocusHandler(this); - PushEventHandler(m_handlerFocus); -#endif // __WXMSW__ - } bool wxPopupTransientWindow::Show( bool show ) @@ -282,7 +276,7 @@ bool wxPopupTransientWindow::Show( bool show ) if (!show) { gdk_pointer_ungrab( (guint32)GDK_CURRENT_TIME ); - + gtk_grab_remove( m_widget ); } #endif @@ -294,13 +288,20 @@ bool wxPopupTransientWindow::Show( bool show ) } #endif - bool ret = wxPopupWindow::Show( show ); +#ifdef __WXMSW__ + if (!show && m_child && m_child->HasCapture()) + { + m_child->ReleaseMouse(); + } +#endif + bool ret = wxPopupWindow::Show( show ); + #ifdef __WXGTK__ if (show) { gtk_grab_add( m_widget ); - + gdk_pointer_grab( m_widget->window, TRUE, (GdkEventMask) (GDK_BUTTON_PRESS_MASK | @@ -317,7 +318,7 @@ bool wxPopupTransientWindow::Show( bool show ) if (show) { Window xwindow = (Window) m_clientWindow; - + /* int res =*/ XGrabPointer(wxGlobalDisplay(), xwindow, True, ButtonPressMask | ButtonReleaseMask | ButtonMotionMask | EnterWindowMask | LeaveWindowMask | PointerMotionMask, @@ -328,20 +329,27 @@ bool wxPopupTransientWindow::Show( bool show ) CurrentTime ); } #endif + +#ifdef __WXMSW__ + if (show && m_child) + { + // Assume that the mouse is outside the popup to begin with + m_child->CaptureMouse(); + } +#endif + return ret; } void wxPopupTransientWindow::Dismiss() { - PopHandlers(); - Hide(); + PopHandlers(); } void wxPopupTransientWindow::DismissAndNotify() { Dismiss(); - OnDismiss(); } @@ -356,6 +364,35 @@ bool wxPopupTransientWindow::ProcessLeftDown(wxMouseEvent& WXUNUSED(event)) return false; } +#ifdef __WXMSW__ +void wxPopupTransientWindow::OnIdle(wxIdleEvent& event) +{ + event.Skip(); + + if (IsShown() && m_child) + { + wxPoint pos = ScreenToClient(wxGetMousePosition()); + wxRect rect(GetSize()); + + if ( rect.Inside(pos) ) + { + if ( m_child->HasCapture() ) + { + m_child->ReleaseMouse(); + } + } + else + { + if ( !m_child->HasCapture() ) + { + m_child->CaptureMouse(); + } + } + } +} +#endif + + #if wxUSE_COMBOBOX && defined(__WXUNIVERSAL__) // ---------------------------------------------------------------------------- @@ -382,7 +419,7 @@ bool wxPopupComboWindow::Create(wxComboControl *parent) void wxPopupComboWindow::PositionNearCombo() { // the origin point must be in screen coords - wxPoint ptOrigin = m_combo->ClientToScreen(wxPoint(0, 0)); + wxPoint ptOrigin = m_combo->ClientToScreen(wxPoint(0,0)); #if 0 //def __WXUNIVERSAL__ // account for the fact that (0, 0) is not the top left corner of the @@ -425,7 +462,7 @@ void wxPopupWindowHandler::OnLeftDown(wxMouseEvent& event) wxPoint pos = event.GetPosition(); - // scrollbar on which the click occured + // scrollbar on which the click occurred wxWindow *sbar = NULL; wxWindow *win = (wxWindow *)event.GetEventObject();