///////////////////////////////////////////////////////////////////////////////
-// Name: common/popupcmn.cpp
+// Name: src/common/popupcmn.cpp
// Purpose: implementation of wxPopupTransientWindow
// Author: Vadim Zeitlin
// Modified by:
// Created: 06.01.01
// RCS-ID: $Id$
// Copyright: (c) 2001 Vadim Zeitlin <zeitlin@dptmaths.ens-cachan.fr>
-// License: wxWindows license
+// License: wxWindows licence
///////////////////////////////////////////////////////////////////////////////
// ============================================================================
// headers
// ----------------------------------------------------------------------------
-#ifdef __GNUG__
- #pragma implementation "popupwinbase.h"
-#endif
-
// For compilers that support precompilation, includes "wx.h".
#include "wx/wxprec.h"
#pragma hdrstop
#endif
-#if wxUSE_POPUPWIN && !defined(__WXMOTIF__)
+#if wxUSE_POPUPWIN
#include "wx/popupwin.h"
#ifndef WX_PRECOMP
- #include "wx/combobox.h" // wxComboControl
+ #include "wx/combobox.h" // wxComboCtrl
+ #include "wx/app.h" // wxPostEvent
+ #include "wx/log.h"
#endif //WX_PRECOMP
+#include "wx/recguard.h"
+
#ifdef __WXUNIVERSAL__
#include "wx/univ/renderer.h"
+ #include "wx/scrolbar.h"
#endif // __WXUNIVERSAL__
-// there is no src/{msw,mgl}/popupwin.cpp to put this in, so we do it here - BTW we
-// probably could do it for all ports here just as well
-#if defined(__WXMSW__) || defined(__WXMGL__)
- IMPLEMENT_DYNAMIC_CLASS(wxPopupWindow, wxWindow)
-#endif // __WXMSW__
+#ifdef __WXGTK__
+ #include <gtk/gtk.h>
+#endif
+#ifdef __WXX11__
+#include "wx/x11/private.h"
+#endif
+IMPLEMENT_DYNAMIC_CLASS(wxPopupWindow, wxWindow)
IMPLEMENT_DYNAMIC_CLASS(wxPopupTransientWindow, wxPopupWindow)
#if wxUSE_COMBOBOX && defined(__WXUNIVERSAL__)
class wxPopupWindowHandler : public wxEvtHandler
{
public:
- wxPopupWindowHandler(wxPopupTransientWindow *popup) { m_popup = popup; }
+ wxPopupWindowHandler(wxPopupTransientWindow *popup) : m_popup(popup) {}
protected:
// event handlers
wxPopupTransientWindow *m_popup;
DECLARE_EVENT_TABLE()
+ DECLARE_NO_COPY_CLASS(wxPopupWindowHandler)
};
class wxPopupFocusHandler : public wxEvtHandler
{
public:
- wxPopupFocusHandler(wxPopupTransientWindow *popup)
- {
- m_popup = popup;
-
-#ifdef __WXGTK__
- // ignore the next few OnKillFocus() calls
- m_creationTime = time(NULL);
-#endif // __WXGTK__
- }
+ wxPopupFocusHandler(wxPopupTransientWindow *popup) : m_popup(popup) {}
protected:
- // event handlers
void OnKillFocus(wxFocusEvent& event);
- void OnKeyUp(wxKeyEvent& event);
+ void OnKeyDown(wxKeyEvent& event);
private:
wxPopupTransientWindow *m_popup;
- // hack around wxGTK bug: we always get several kill focus events
- // immediately after creation!
-#ifdef __WXGTK__
- time_t m_creationTime;
-#endif // __WXGTK__
-
DECLARE_EVENT_TABLE()
+ DECLARE_NO_COPY_CLASS(wxPopupFocusHandler)
};
// ----------------------------------------------------------------------------
BEGIN_EVENT_TABLE(wxPopupFocusHandler, wxEvtHandler)
EVT_KILL_FOCUS(wxPopupFocusHandler::OnKillFocus)
- EVT_KEY_UP(wxPopupFocusHandler::OnKeyUp)
+ EVT_KEY_DOWN(wxPopupFocusHandler::OnKeyDown)
+END_EVENT_TABLE()
+
+BEGIN_EVENT_TABLE(wxPopupTransientWindow, wxPopupWindow)
+#ifdef __WXMSW__
+ EVT_IDLE(wxPopupTransientWindow::OnIdle)
+#endif
END_EVENT_TABLE()
// ============================================================================
bool wxPopupWindowBase::Create(wxWindow* WXUNUSED(parent), int WXUNUSED(flags))
{
- return TRUE;
+ return true;
}
void wxPopupWindowBase::Position(const wxPoint& ptOrigin,
}
// now check left/right too
- wxCoord x = ptOrigin.x + size.x;
+ wxCoord x = ptOrigin.x;
+
+ if ( wxTheApp->GetLayoutDirection() == wxLayout_RightToLeft )
+ {
+ // shift the window to the left instead of the right.
+ x -= size.x;
+ x -= sizeSelf.x; // also shift it by window width.
+ }
+ else
+ x += size.x;
+
+
if ( x + sizeSelf.x > sizeScreen.x )
{
// check if there is enough space to the left
{
m_child =
m_focus = (wxWindow *)NULL;
+
+ m_handlerFocus = NULL;
+ m_handlerPopup = NULL;
}
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 )
{
- m_child->PopEventHandler(TRUE /* delete it */);
- m_child->ReleaseMouse();
+ 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;
}
if ( m_focus )
{
- m_focus->PopEventHandler(TRUE /* delete it */);
- m_focus = NULL;
+ if ( !m_focus->RemoveEventHandler(m_handlerFocus) )
+ {
+ // see above
+ m_handlerFocus = NULL;
+ }
}
+ m_focus = NULL;
}
void wxPopupTransientWindow::Popup(wxWindow *winFocus)
m_child = this;
}
- // we can't capture mouse before the window is shown in wxGTK, so do it
- // first
Show();
- m_child->CaptureMouse();
- m_child->PushEventHandler(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->PushEventHandler(m_handlerPopup);
m_focus = winFocus ? winFocus : this;
m_focus->SetFocus();
#ifdef __WXMSW__
- // FIXME: I don't know why does this happen but sometimes SetFocus() simply
- // refuses to work under MSW - no error happens but the focus is not
- // given to the window, i.e. the assert below is triggered
- //
- // Try work around this as we can...
-
- //wxASSERT_MSG( FindFocus() == m_focus, _T("setting focus failed") );
+ // MSW doesn't allow to set focus to the popup window, but we need to
+ // 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 )
-#endif // __WXMSW__
{
- m_focus->PushEventHandler(new wxPopupFocusHandler(this));
+ if (!m_handlerFocus)
+ m_handlerFocus = new wxPopupFocusHandler(this);
+
+ m_focus->PushEventHandler(m_handlerFocus);
}
}
-void wxPopupTransientWindow::Dismiss()
+bool wxPopupTransientWindow::Show( bool show )
{
- PopHandlers();
+#ifdef __WXGTK__
+ if (!show)
+ {
+ gdk_pointer_ungrab( (guint32)GDK_CURRENT_TIME );
+
+ gtk_grab_remove( m_widget );
+ }
+#endif
+
+#ifdef __WXX11__
+ if (!show)
+ {
+ XUngrabPointer( wxGlobalDisplay(), CurrentTime );
+ }
+#endif
+
+#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 |
+ GDK_BUTTON_RELEASE_MASK |
+ GDK_POINTER_MOTION_HINT_MASK |
+ GDK_POINTER_MOTION_MASK),
+ (GdkWindow *) NULL,
+ (GdkCursor *) NULL,
+ (guint32)GDK_CURRENT_TIME );
+ }
+#endif
+
+#ifdef __WXX11__
+ if (show)
+ {
+ Window xwindow = (Window) m_clientWindow;
+
+ /* int res =*/ XGrabPointer(wxGlobalDisplay(), xwindow,
+ True,
+ ButtonPressMask | ButtonReleaseMask | ButtonMotionMask | EnterWindowMask | LeaveWindowMask | PointerMotionMask,
+ GrabModeAsync,
+ GrabModeAsync,
+ None,
+ None,
+ 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()
+{
Hide();
+ PopHandlers();
}
void wxPopupTransientWindow::DismissAndNotify()
{
Dismiss();
-
OnDismiss();
}
bool wxPopupTransientWindow::ProcessLeftDown(wxMouseEvent& WXUNUSED(event))
{
// no special processing here
- return FALSE;
+ return false;
}
+#ifdef __WXMSW__
+void wxPopupTransientWindow::OnIdle(wxIdleEvent& event)
+{
+ event.Skip();
+
+ if (IsShown() && m_child)
+ {
+ wxPoint pos = ScreenToClient(wxGetMousePosition());
+ wxRect rect(GetSize());
+
+ if ( rect.Contains(pos) )
+ {
+ if ( m_child->HasCapture() )
+ {
+ m_child->ReleaseMouse();
+ }
+ }
+ else
+ {
+ if ( !m_child->HasCapture() )
+ {
+ m_child->CaptureMouse();
+ }
+ }
+ }
+}
+#endif // __WXMSW__
+
+
#if wxUSE_COMBOBOX && defined(__WXUNIVERSAL__)
// ----------------------------------------------------------------------------
// wxPopupComboWindow
// ----------------------------------------------------------------------------
-wxPopupComboWindow::wxPopupComboWindow(wxComboControl *parent)
+BEGIN_EVENT_TABLE(wxPopupComboWindow, wxPopupTransientWindow)
+ EVT_KEY_DOWN(wxPopupComboWindow::OnKeyDown)
+END_EVENT_TABLE()
+
+wxPopupComboWindow::wxPopupComboWindow(wxComboCtrl *parent)
: wxPopupTransientWindow(parent)
{
m_combo = parent;
}
-bool wxPopupComboWindow::Create(wxComboControl *parent)
+bool wxPopupComboWindow::Create(wxComboCtrl *parent)
{
m_combo = 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
void wxPopupComboWindow::OnDismiss()
{
- m_combo->OnDismiss();
+ m_combo->OnPopupDismiss();
+}
+
+void wxPopupComboWindow::OnKeyDown(wxKeyEvent& event)
+{
+ m_combo->ProcessEvent(event);
}
#endif // wxUSE_COMBOBOX && defined(__WXUNIVERSAL__)
wxPoint pos = event.GetPosition();
- // scrollbar on which the click occured
+ // in non-Univ ports the system manages scrollbars for us
+#if defined(__WXUNIVERSAL__) && wxUSE_SCROLLBAR
+ // scrollbar on which the click occurred
wxWindow *sbar = NULL;
+#endif // __WXUNIVERSAL__ && wxUSE_SCROLLBAR
wxWindow *win = (wxWindow *)event.GetEventObject();
+
switch ( win->HitTest(pos.x, pos.y) )
{
case wxHT_WINDOW_OUTSIDE:
- // clicking outside a popup dismisses it
- m_popup->DismissAndNotify();
+ {
+ // do the coords translation now as after DismissAndNotify()
+ // m_popup may be destroyed
+ wxMouseEvent event2(event);
+
+ m_popup->ClientToScreen(&event2.m_x, &event2.m_y);
+
+ // clicking outside a popup dismisses it
+ m_popup->DismissAndNotify();
+
+ // dismissing a tooltip shouldn't waste a click, i.e. you
+ // should be able to dismiss it and press the button with the
+ // same click, so repost this event to the window beneath us
+ wxWindow *winUnder = wxFindWindowAtPoint(event2.GetPosition());
+ if ( winUnder )
+ {
+ // translate the event coords to the ones of the window
+ // which is going to get the event
+ winUnder->ScreenToClient(&event2.m_x, &event2.m_y);
+
+ event2.SetEventObject(winUnder);
+ wxPostEvent(winUnder, event2);
+ }
+ }
break;
-#ifdef __WXUNIVERSAL__
+#if defined(__WXUNIVERSAL__) && wxUSE_SCROLLBAR
case wxHT_WINDOW_HORZ_SCROLLBAR:
sbar = win->GetScrollbar(wxHORIZONTAL);
break;
case wxHT_WINDOW_VERT_SCROLLBAR:
sbar = win->GetScrollbar(wxVERTICAL);
break;
-#endif
+#endif // __WXUNIVERSAL__ && wxUSE_SCROLLBAR
default:
// forgot to update the switch after adding a new hit test code?
break;
}
+#if defined(__WXUNIVERSAL__) && wxUSE_SCROLLBAR
if ( sbar )
{
// translate the event coordinates to the scrollbar ones
(void)sbar->GetEventHandler()->ProcessEvent(event2);
}
+#endif // __WXUNIVERSAL__ && wxUSE_SCROLLBAR
}
// ----------------------------------------------------------------------------
void wxPopupFocusHandler::OnKillFocus(wxFocusEvent& event)
{
-#ifdef __WXGTK__
- // ignore the next OnKillFocus() call
- if ( time(NULL) < m_creationTime + 1 )
+ // when we lose focus we always disappear - unless it goes to the popup (in
+ // which case we don't really lose it)
+ wxWindow *win = event.GetWindow();
+ while ( win )
{
- event.Skip();
-
- return;
+ if ( win == m_popup )
+ return;
+ win = win->GetParent();
}
-#endif // __WXGTK__
- // when we lose focus we always disappear - unless it goes to the popup (in
- // which case we don't really lose it)
- if ( event.GetWindow() != m_popup )
- m_popup->DismissAndNotify();
+ m_popup->DismissAndNotify();
}
-void wxPopupFocusHandler::OnKeyUp(wxKeyEvent& event)
+void wxPopupFocusHandler::OnKeyDown(wxKeyEvent& event)
{
+ // we can be associated with the popup itself in which case we should avoid
+ // infinite recursion
+ static int s_inside;
+ wxRecursionGuard guard(s_inside);
+ if ( guard.IsInside() )
+ {
+ event.Skip();
+ return;
+ }
+
// let the window have it first, it might process the keys
- if ( !m_popup->ProcessEvent(event) )
+ if ( !m_popup->GetEventHandler()->ProcessEvent(event) )
{
// by default, dismiss the popup
m_popup->DismissAndNotify();
}
#endif // wxUSE_POPUPWIN
-
projects / wxWidgets.git / blobdiff