EVT_KEY_DOWN(wxPopupFocusHandler::OnKeyDown)
END_EVENT_TABLE()
+BEGIN_EVENT_TABLE(wxPopupTransientWindow, wxPopupWindow)
+#ifdef __WXMSW__
+ EVT_IDLE(wxPopupTransientWindow::OnIdle)
+#endif
+END_EVENT_TABLE()
+
// ============================================================================
// implementation
// ============================================================================
// wxPopupTransientWindow
// ----------------------------------------------------------------------------
-BEGIN_EVENT_TABLE(wxPopupTransientWindow, wxPopupWindow)
-#ifdef __WXMSW__
- EVT_ENTER_WINDOW(wxPopupTransientWindow::OnEnter)
- EVT_LEAVE_WINDOW(wxPopupTransientWindow::OnLeave)
- EVT_LEFT_DOWN(wxPopupTransientWindow::OnLeftDown)
-#endif
-END_EVENT_TABLE()
-
void wxPopupTransientWindow::Init()
{
m_child =
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;
}
-
+ if (m_child->HasCapture())
+ {
+ m_child->ReleaseMouse();
+ }
m_child = NULL;
}
-#ifdef __WXMSW__
- if ( HasCapture() )
- {
- ReleaseCapture();
- }
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;
}
Show();
- // There is is a problem if these are still valid
- wxASSERT_MSG(!m_handlerPopup, wxT("Popup handler not deleted"));
- wxASSERT_MSG(!m_handlerFocus, wxT("Focus handler not deleted"));
+ // There is is a problem if these are still in use
+ wxASSERT(!m_handlerFocus || !m_handlerFocus->GetNextHandler());
+ wxASSERT(!m_handlerPopup || !m_handlerPopup->GetNextHandler());
- delete m_handlerPopup;
- m_handlerPopup = new wxPopupWindowHandler(this);
+ if (!m_handlerPopup)
+ m_handlerPopup = new wxPopupWindowHandler(this);
m_child->PushEventHandler(m_handlerPopup);
// subclass the window which has the focus, and not winFocus passed in or
// otherwise everything else breaks down
m_focus = FindFocus();
+#elif __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__
// catch destroy events, if you close a program with a popup shown in MSW
// you get a segfault if m_child or m_focus are deleted before this is
m_child->Connect(wxEVT_DESTROY,
wxWindowDestroyEventHandler(wxPopupTransientWindow::OnDestroy),
NULL, this);
- m_focus->Connect(wxEVT_DESTROY,
- wxWindowDestroyEventHandler(wxPopupTransientWindow::OnDestroy),
- NULL, this);
-#ifdef __WXMSW__
- // Assume that the mouse is currently outside of the popup window
- CaptureMouse();
-
- // Connect the child Enter/Leave events too, incase the child completly
- // covers the popup (because then the popup's enter/leave events won't be
- // sent.
- if (m_child != this)
- {
- m_child->Connect(wxEVT_ENTER_WINDOW,
- wxMouseEventHandler(wxPopupTransientWindow::OnChildEnter),
+ if (m_focus)
+ m_focus->Connect(wxEVT_DESTROY,
+ wxWindowDestroyEventHandler(wxPopupTransientWindow::OnDestroy),
NULL, this);
- m_child->Connect(wxEVT_LEAVE_WINDOW,
- wxMouseEventHandler(wxPopupTransientWindow::OnChildLeave),
- NULL, this);
- }
-#endif
}
bool wxPopupTransientWindow::Show( bool show )
}
#endif
+#ifdef __WXMSW__
+ if (!show && m_child && m_child->HasCapture())
+ {
+ m_child->ReleaseMouse();
+ }
+#endif
+
bool ret = wxPopupWindow::Show( show );
#ifdef __WXGTK__
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();
}
m_focus = NULL;
}
-void wxPopupTransientWindow::OnEnter(wxMouseEvent& /*event*/)
-{
- if ( HasCapture() )
- {
- ReleaseCapture();
- }
-}
-
-void wxPopupTransientWindow::OnLeave(wxMouseEvent& /*event*/)
+#ifdef __WXMSW__
+void wxPopupTransientWindow::OnIdle(wxIdleEvent& event)
{
- CaptureMouse();
-}
+ event.Skip();
-void wxPopupTransientWindow::OnLeftDown(wxMouseEvent& event)
-{
- if (m_handlerPopup && m_child && m_child != this)
+ if (IsShown() && m_child)
{
- m_child->GetEventHandler()->ProcessEvent(event);
- }
-}
-
+ wxPoint pos = ScreenToClient(wxGetMousePosition());
+ wxRect rect(GetSize());
-
-// If the child is the same size as the popup window then handle the event,
-// otherwise assume that there is enough of the popup showing that it will get
-// it's own Enter/Leave events. A more reliable way to detect this situation
-// would be appreciated...
-
-void wxPopupTransientWindow::OnChildEnter(wxMouseEvent& event)
-{
- if (m_child)
- {
- wxSize cs = m_child->GetSize();
- wxSize ps = GetClientSize();
-
- if ((cs.x * cs.y) >= (ps.x * ps.y))
- OnEnter(event);
+ if ( rect.Inside(pos) )
+ {
+ if ( m_child->HasCapture() )
+ {
+ m_child->ReleaseMouse();
+ }
+ }
+ else
+ {
+ if ( !m_child->HasCapture() )
+ {
+ m_child->CaptureMouse();
+ }
+ }
}
}
+#endif
-void wxPopupTransientWindow::OnChildLeave(wxMouseEvent& event)
-{
- if (m_child)
- {
- wxSize cs = m_child->GetSize();
- wxSize ps = GetClientSize();
-
- if ((cs.x * cs.y) >= (ps.x * ps.y))
- OnLeave(event);
- }
-}
#if wxUSE_COMBOBOX && defined(__WXUNIVERSAL__)