Add entry for 2.8.0.1

[wxWidgets.git] / src / common / combocmn.cpp

diff --git a/src/common/combocmn.cpp b/src/common/combocmn.cpp
index 0b53234ca985052020c6099c204e3863c5d8aced..f47619998ea5541fdafc8f0544e9a0452941b9bd 100644 (file)
--- a/src/common/combocmn.cpp
+++ b/src/common/combocmn.cpp
@@ -44,10 +44,6 @@
 // constants
 // ----------------------------------------------------------------------------
 
 // constants
 // ----------------------------------------------------------------------------
 

-// Milliseconds to wait for two mouse-ups after focus inorder
-// to trigger a double-click.
-#define DOUBLE_CLICK_CONVERSION_TRESHOLD        500
-

 #define DEFAULT_DROPBUTTON_WIDTH                19
 
 #define BMP_BUTTON_MARGIN                       4
 #define DEFAULT_DROPBUTTON_WIDTH                19
 
 #define BMP_BUTTON_MARGIN                       4


@@ -66,6 +62,7 @@
                                         // native controls work on it like normal.
 #define POPUPWIN_IS_PERFECT           0 // Same, but for non-transient popup window.
 #define TEXTCTRL_TEXT_CENTERED        0 // 1 if text in textctrl is vertically centered
                                         // native controls work on it like normal.
 #define POPUPWIN_IS_PERFECT           0 // Same, but for non-transient popup window.
 #define TEXTCTRL_TEXT_CENTERED        0 // 1 if text in textctrl is vertically centered

+#define FOCUS_RING                    0 // No focus ring on wxMSW

 
 //#undef wxUSE_POPUPWIN
 //#define wxUSE_POPUPWIN 0
 
 //#undef wxUSE_POPUPWIN
 //#define wxUSE_POPUPWIN 0


@@ -83,6 +80,7 @@
                                         // native controls work on it like normal.
 #define POPUPWIN_IS_PERFECT           1 // Same, but for non-transient popup window.
 #define TEXTCTRL_TEXT_CENTERED        1 // 1 if text in textctrl is vertically centered
                                         // native controls work on it like normal.
 #define POPUPWIN_IS_PERFECT           1 // Same, but for non-transient popup window.
 #define TEXTCTRL_TEXT_CENTERED        1 // 1 if text in textctrl is vertically centered

+#define FOCUS_RING                    0 // No focus ring on wxGTK

 
 #elif defined(__WXMAC__)
 
 
 #elif defined(__WXMAC__)
 


@@ -91,6 +89,12 @@
                                         // native controls work on it like normal.
 #define POPUPWIN_IS_PERFECT           0 // Same, but for non-transient popup window.
 #define TEXTCTRL_TEXT_CENTERED        1 // 1 if text in textctrl is vertically centered
                                         // native controls work on it like normal.
 #define POPUPWIN_IS_PERFECT           0 // Same, but for non-transient popup window.
 #define TEXTCTRL_TEXT_CENTERED        1 // 1 if text in textctrl is vertically centered

+#define FOCUS_RING                    3 // Reserve room for the textctrl's focus ring to display
+
+#undef DEFAULT_DROPBUTTON_WIDTH
+#define DEFAULT_DROPBUTTON_WIDTH      22
+#undef COMBO_MARGIN
+#define COMBO_MARGIN                  FOCUS_RING

 
 #else
 
 
 #else
 


@@ -99,6 +103,7 @@
                                         // native controls work on it like normal.
 #define POPUPWIN_IS_PERFECT           0 // Same, but for non-transient popup window.
 #define TEXTCTRL_TEXT_CENTERED        1 // 1 if text in textctrl is vertically centered
                                         // native controls work on it like normal.
 #define POPUPWIN_IS_PERFECT           0 // Same, but for non-transient popup window.
 #define TEXTCTRL_TEXT_CENTERED        1 // 1 if text in textctrl is vertically centered

+#define FOCUS_RING                    0

 
 #endif
 
 
 #endif
 


@@ -344,20 +349,51 @@ public:
                                                 style)
     #endif
     {
                                                 style)
     #endif
     {

+        m_inShow = 0;

     }
 
 #if USES_WXPOPUPTRANSIENTWINDOW
     }
 
 #if USES_WXPOPUPTRANSIENTWINDOW

+    virtual bool Show( bool show );

     virtual bool ProcessLeftDown(wxMouseEvent& event);
     virtual bool ProcessLeftDown(wxMouseEvent& event);

+protected:

     virtual void OnDismiss();
 #endif
 
     virtual void OnDismiss();
 #endif
 

+private:
+    wxByte      m_inShow;

 };
 
 
 #if USES_WXPOPUPTRANSIENTWINDOW
 };
 
 
 #if USES_WXPOPUPTRANSIENTWINDOW

-bool wxComboPopupWindow::ProcessLeftDown(wxMouseEvent& event )
+bool wxComboPopupWindow::Show( bool show )

 {
 {

-    return wxComboPopupWindowBase::ProcessLeftDown(event);
+    // Guard against recursion
+    if ( m_inShow )
+        return wxComboPopupWindowBase::Show(show);
+
+    m_inShow++;
+
+    wxASSERT( IsKindOf(CLASSINFO(wxPopupTransientWindow)) );
+
+    wxPopupTransientWindow* ptw = (wxPopupTransientWindow*) this;
+    wxComboCtrlBase* combo = (wxComboCtrlBase*) GetParent();
+
+    if ( show != ptw->IsShown() )
+    {
+        if ( show )
+            ptw->Popup(combo->GetPopupControl()->GetControl());
+        else
+            ptw->Dismiss();
+    }
+
+    m_inShow--;
+
+    return true;
+}
+
+bool wxComboPopupWindow::ProcessLeftDown(wxMouseEvent& event)
+{
+    return wxPopupTransientWindow::ProcessLeftDown(event);

 }
 
 // First thing that happens when a transient popup closes is that this method gets called.
 }
 
 // First thing that happens when a transient popup closes is that this method gets called.


@@ -387,6 +423,7 @@ public:
         m_combo = parent;
     }
 
         m_combo = parent;
     }
 

+    void OnSizeEvent( wxSizeEvent& event );

     void OnKeyEvent(wxKeyEvent& event);
 #if USES_WXDIALOG
     void OnActivate( wxActivateEvent& event );
     void OnKeyEvent(wxKeyEvent& event);
 #if USES_WXDIALOG
     void OnActivate( wxActivateEvent& event );


@@ -405,9 +442,15 @@ BEGIN_EVENT_TABLE(wxComboPopupWindowEvtHandler, wxEvtHandler)
 #if USES_WXDIALOG
     EVT_ACTIVATE(wxComboPopupWindowEvtHandler::OnActivate)
 #endif
 #if USES_WXDIALOG
     EVT_ACTIVATE(wxComboPopupWindowEvtHandler::OnActivate)
 #endif

+    EVT_SIZE(wxComboPopupWindowEvtHandler::OnSizeEvent)

 END_EVENT_TABLE()
 
 
 END_EVENT_TABLE()
 
 

+void wxComboPopupWindowEvtHandler::OnSizeEvent( wxSizeEvent& WXUNUSED(event) )
+{
+    // Block the event so that the popup control does not get auto-resized.
+}
+

 void wxComboPopupWindowEvtHandler::OnKeyEvent( wxKeyEvent& event )
 {
     // Relay keyboard event to the main child controls
 void wxComboPopupWindowEvtHandler::OnKeyEvent( wxKeyEvent& event )
 {
     // Relay keyboard event to the main child controls


@@ -621,7 +664,7 @@ void wxComboPopupExtraEventHandler::OnMouseEvent( wxMouseEvent& event )
          evtType == wxEVT_RIGHT_DOWN )
     {
         // Block motion and click events outside the popup
          evtType == wxEVT_RIGHT_DOWN )
     {
         // Block motion and click events outside the popup

-        if ( !isInside )
+        if ( !isInside || !m_combo->IsPopupShown() )

         {
             event.Skip(false);
             return;
         {
             event.Skip(false);
             return;


@@ -629,11 +672,10 @@ void wxComboPopupExtraEventHandler::OnMouseEvent( wxMouseEvent& event )
     }
     else if ( evtType == wxEVT_LEFT_UP )
     {
     }
     else if ( evtType == wxEVT_LEFT_UP )
     {

-        // Don't let left-down events in if outside
-        if ( evtType == wxEVT_LEFT_DOWN )
+        if ( !m_combo->IsPopupShown() )

         {
         {

-            if ( !isInside )
-                return;
+            event.Skip(false);
+            return;

         }
 
         if ( !m_beenInside )
         }
 
         if ( !m_beenInside )


@@ -675,6 +717,7 @@ BEGIN_EVENT_TABLE(wxComboCtrlBase, wxControl)
     EVT_SIZE(wxComboCtrlBase::OnSizeEvent)
     EVT_SET_FOCUS(wxComboCtrlBase::OnFocusEvent)
     EVT_KILL_FOCUS(wxComboCtrlBase::OnFocusEvent)
     EVT_SIZE(wxComboCtrlBase::OnSizeEvent)
     EVT_SET_FOCUS(wxComboCtrlBase::OnFocusEvent)
     EVT_KILL_FOCUS(wxComboCtrlBase::OnFocusEvent)

+    EVT_IDLE(wxComboCtrlBase::OnIdleEvent)

     //EVT_BUTTON(wxID_ANY,wxComboCtrlBase::OnButtonClickEvent)
     EVT_KEY_DOWN(wxComboCtrlBase::OnKeyEvent)
     EVT_TEXT_ENTER(wxID_ANY,wxComboCtrlBase::OnTextCtrlEvent)
     //EVT_BUTTON(wxID_ANY,wxComboCtrlBase::OnButtonClickEvent)
     EVT_KEY_DOWN(wxComboCtrlBase::OnKeyEvent)
     EVT_TEXT_ENTER(wxID_ANY,wxComboCtrlBase::OnTextCtrlEvent)


@@ -688,7 +731,7 @@ void wxComboCtrlBase::Init()
 {
     m_winPopup = (wxWindow *)NULL;
     m_popup = (wxWindow *)NULL;
 {
     m_winPopup = (wxWindow *)NULL;
     m_popup = (wxWindow *)NULL;

-    m_isPopupShown = false;
+    m_popupWinState = Hidden;

     m_btn = (wxWindow*) NULL;
     m_text = (wxTextCtrl*) NULL;
     m_popupInterface = (wxComboPopup*) NULL;
     m_btn = (wxWindow*) NULL;
     m_text = (wxTextCtrl*) NULL;
     m_popupInterface = (wxComboPopup*) NULL;


@@ -700,6 +743,8 @@ void wxComboCtrlBase::Init()
     m_toplevEvtHandler = (wxEvtHandler*) NULL;
 #endif
 
     m_toplevEvtHandler = (wxEvtHandler*) NULL;
 #endif
 

+    m_mainCtrlWnd = this;
+

     m_heightPopup = -1;
     m_widthMinPopup = -1;
     m_anchorSide = 0;
     m_heightPopup = -1;
     m_widthMinPopup = -1;
     m_anchorSide = 0;


@@ -720,6 +765,8 @@ void wxComboCtrlBase::Init()
     m_absIndent = -1;
     m_iFlags = 0;
     m_timeCanAcceptClick = 0;
     m_absIndent = -1;
     m_iFlags = 0;
     m_timeCanAcceptClick = 0;

+
+    m_resetFocus = false;

 }
 
 bool wxComboCtrlBase::Create(wxWindow *parent,
 }
 
 bool wxComboCtrlBase::Create(wxWindow *parent,


@@ -794,17 +841,19 @@ wxComboCtrlBase::CreateTextCtrl(int style, const wxValidator& validator)
             m_ignoreEvtText = 0;
 
         m_text = new wxTextCtrl(this, wxID_ANY, m_valueString,
             m_ignoreEvtText = 0;
 
         m_text = new wxTextCtrl(this, wxID_ANY, m_valueString,

-                                wxDefaultPosition, wxDefaultSize,
+                                wxDefaultPosition, wxSize(10,-1),

                                 style, validator);
                                 style, validator);

-
-        // This is required for some platforms (GTK+ atleast)
-        m_text->SetSizeHints(2,4);

     }
 }
 
 void wxComboCtrlBase::OnThemeChange()
 {
     }
 }
 
 void wxComboCtrlBase::OnThemeChange()
 {

-    SetBackgroundColour(wxSystemSettings::GetColour(wxSYS_COLOUR_WINDOW));
+    // Leave the default bg on the Mac so the area used by the focus ring will
+    // be the correct colour and themed brush.  Instead we'll use
+    // wxSYS_COLOUR_WINDOW in the EVT_PAINT handler as needed.
+#ifndef __WXMAC__
+    SetOwnBackgroundColour(wxSystemSettings::GetColour(wxSYS_COLOUR_WINDOW));
+#endif

 }
 
 wxComboCtrlBase::~wxComboCtrlBase()
 }
 
 wxComboCtrlBase::~wxComboCtrlBase()


@@ -932,14 +981,14 @@ void wxComboCtrlBase::CalculateAreas( int btnWidth )
     m_btnSize.y = butHeight;
 
     m_btnArea.x = ( m_btnSide==wxRIGHT ? sz.x - butAreaWid - btnBorder : btnBorder );
     m_btnSize.y = butHeight;
 
     m_btnArea.x = ( m_btnSide==wxRIGHT ? sz.x - butAreaWid - btnBorder : btnBorder );

-    m_btnArea.y = btnBorder;
+    m_btnArea.y = btnBorder + FOCUS_RING;

     m_btnArea.width = butAreaWid;
     m_btnArea.width = butAreaWid;

-    m_btnArea.height = sz.y - (btnBorder*2);
+    m_btnArea.height = sz.y - ((btnBorder+FOCUS_RING)*2);

 
 

-    m_tcArea.x = ( m_btnSide==wxRIGHT ? 0 : butAreaWid ) + customBorder;
-    m_tcArea.y = customBorder;
-    m_tcArea.width = sz.x - butAreaWid - (customBorder*2);
-    m_tcArea.height = sz.y - (customBorder*2);
+    m_tcArea.x = ( m_btnSide==wxRIGHT ? 0 : butAreaWid ) + customBorder + FOCUS_RING;
+    m_tcArea.y = customBorder + FOCUS_RING;
+    m_tcArea.width = sz.x - butAreaWid - (customBorder*2) - (FOCUS_RING*2);
+    m_tcArea.height = sz.y - ((customBorder+FOCUS_RING)*2);

 
 /*
     if ( m_text )
 
 /*
     if ( m_text )


@@ -955,10 +1004,11 @@ void wxComboCtrlBase::PositionTextCtrl( int textCtrlXAdjust, int textCtrlYAdjust
     if ( !m_text )
         return;
 
     if ( !m_text )
         return;
 

+#if !TEXTCTRL_TEXT_CENTERED
+

     wxSize sz = GetClientSize();
     wxSize sz = GetClientSize();

-    int customBorder = m_widthCustomBorder;

 
 

-#if !TEXTCTRL_TEXT_CENTERED
+    int customBorder = m_widthCustomBorder;

     if ( (m_text->GetWindowStyleFlag() & wxBORDER_MASK) == wxNO_BORDER )
     {
         // Centre textctrl
     if ( (m_text->GetWindowStyleFlag() & wxBORDER_MASK) == wxNO_BORDER )
     {
         // Centre textctrl


@@ -985,16 +1035,16 @@ void wxComboCtrlBase::PositionTextCtrl( int textCtrlXAdjust, int textCtrlYAdjust
         }
     }
     else
         }
     }
     else

-#else
+#else // TEXTCTRL_TEXT_CENTERED

     wxUnusedVar(textCtrlXAdjust);
     wxUnusedVar(textCtrlYAdjust);
     wxUnusedVar(textCtrlXAdjust);
     wxUnusedVar(textCtrlYAdjust);

-#endif
+#endif // !TEXTCTRL_TEXT_CENTERED/TEXTCTRL_TEXT_CENTERED

     {
         // If it has border, have textctrl will the entire text field.
         m_text->SetSize( m_tcArea.x + m_widthCustomPaint,
     {
         // If it has border, have textctrl will the entire text field.
         m_text->SetSize( m_tcArea.x + m_widthCustomPaint,

-                         customBorder,
-                         sz.x - m_btnArea.width - m_widthCustomPaint - customBorder,
-                         sz.y-(customBorder*2) );
+                         m_tcArea.y,
+                         m_tcArea.width - m_widthCustomPaint,
+                         m_tcArea.height );

     }
 }
 
     }
 }
 


@@ -1038,9 +1088,27 @@ wxSize wxComboCtrlBase::DoGetBestSize() const
     fhei += 1;
 #endif
 
     fhei += 1;
 #endif
 

-    wxSize ret(sizeText.x + COMBO_MARGIN + DEFAULT_DROPBUTTON_WIDTH,
-               fhei);
+#ifdef __WXMAC__
+    // these are the numbers from the HIG:
+    switch ( m_windowVariant )
+    {
+        case wxWINDOW_VARIANT_NORMAL:
+        default :
+            fhei = 22;
+            break;
+        case wxWINDOW_VARIANT_SMALL:
+            fhei = 19;
+            break;
+        case wxWINDOW_VARIANT_MINI:
+            fhei = 15;
+            break;
+    }
+#endif

 
 

+    fhei += 2 * FOCUS_RING;
+    int width = sizeText.x + FOCUS_RING + COMBO_MARGIN + DEFAULT_DROPBUTTON_WIDTH;
+
+    wxSize ret(width, fhei);

     CacheBestSize(ret);
     return ret;
 }
     CacheBestSize(ret);
     return ret;
 }


@@ -1118,6 +1186,26 @@ void wxComboCtrlBase::DoSetToolTip(wxToolTip *tooltip)
 }
 #endif // wxUSE_TOOLTIPS
 
 }
 #endif // wxUSE_TOOLTIPS
 

+#if wxUSE_VALIDATORS
+void wxComboCtrlBase::SetValidator(const wxValidator& validator)
+{
+    wxTextCtrl* textCtrl = GetTextCtrl();
+
+    if ( textCtrl )
+        textCtrl->SetValidator( validator );
+}
+
+wxValidator* wxComboCtrlBase::GetValidator()
+{
+    wxTextCtrl* textCtrl = GetTextCtrl();
+
+    if ( textCtrl )
+        return textCtrl->GetValidator();
+
+    return wxControl::GetValidator();
+}
+#endif // wxUSE_VALIDATORS
+

 // ----------------------------------------------------------------------------
 // painting
 // ----------------------------------------------------------------------------
 // ----------------------------------------------------------------------------
 // painting
 // ----------------------------------------------------------------------------


@@ -1181,13 +1269,21 @@ void wxComboCtrlBase::PrepareBackground( wxDC& dc, const wxRect& rect, int flags
         else
         {
             dc.SetTextForeground( wxSystemSettings::GetColour(wxSYS_COLOUR_WINDOWTEXT) );
         else
         {
             dc.SetTextForeground( wxSystemSettings::GetColour(wxSYS_COLOUR_WINDOWTEXT) );

+#ifndef __WXMAC__  // see note in OnThemeChange

             bgCol = GetBackgroundColour();
             bgCol = GetBackgroundColour();

+#else
+            bgCol = wxSystemSettings::GetColour(wxSYS_COLOUR_WINDOW);
+#endif

         }
     }
     else
     {
         dc.SetTextForeground( wxSystemSettings::GetColour(wxSYS_COLOUR_GRAYTEXT) );
         }
     }
     else
     {
         dc.SetTextForeground( wxSystemSettings::GetColour(wxSYS_COLOUR_GRAYTEXT) );

+#ifndef __WXMAC__  // see note in OnThemeChange

         bgCol = GetBackgroundColour();
         bgCol = GetBackgroundColour();

+#else
+        bgCol = wxSystemSettings::GetColour(wxSYS_COLOUR_WINDOW);
+#endif

     }
 
     dc.SetBrush( bgCol );
     }
 
     dc.SetBrush( bgCol );


@@ -1207,12 +1303,12 @@ void wxComboCtrlBase::PrepareBackground( wxDC&, const wxRect&, int ) const
 }
 #endif
 
 }
 #endif
 

-void wxComboCtrlBase::DrawButton( wxDC& dc, const wxRect& rect, bool paintBg )
+void wxComboCtrlBase::DrawButton( wxDC& dc, const wxRect& rect, int paintBg )

 {
     int drawState = m_btnState;
 
 #ifdef __WXGTK__
 {
     int drawState = m_btnState;
 
 #ifdef __WXGTK__

-    if ( m_isPopupShown )
+    if ( GetPopupWindowState() >= Animating )

         drawState |= wxCONTROL_PRESSED;
 #endif
 
         drawState |= wxCONTROL_PRESSED;
 #endif
 


@@ -1365,7 +1461,7 @@ bool wxComboCtrlBase::HandleButtonMouseEvent( wxMouseEvent& event,
             Refresh();
         }
     }
             Refresh();
         }
     }

-    else if ( type == wxEVT_LEFT_DOWN )
+    else if ( type == wxEVT_LEFT_DOWN || type == wxEVT_LEFT_DCLICK )

     {
         if ( flags & (wxCC_MF_ON_CLICK_AREA|wxCC_MF_ON_BUTTON) )
         {
     {
         if ( flags & (wxCC_MF_ON_CLICK_AREA|wxCC_MF_ON_BUTTON) )
         {


@@ -1406,7 +1502,7 @@ bool wxComboCtrlBase::HandleButtonMouseEvent( wxMouseEvent& event,
             m_btnState &= ~(wxCONTROL_CURRENT);
 
             // Mouse hover ends
             m_btnState &= ~(wxCONTROL_CURRENT);
 
             // Mouse hover ends

-            if ( !m_isPopupShown )
+            if ( IsPopupWindowState(Hidden) )

             {
                 m_btnState &= ~(wxCONTROL_PRESSED);
                 Refresh();
             {
                 m_btnState &= ~(wxCONTROL_PRESSED);
                 Refresh();


@@ -1429,7 +1525,7 @@ bool wxComboCtrlBase::PreprocessMouseEvent( wxMouseEvent& event,
 #if USES_WXPOPUPWINDOW || USES_WXDIALOG
     if ( m_popupWinType != POPUPWIN_WXPOPUPTRANSIENTWINDOW )
     {
 #if USES_WXPOPUPWINDOW || USES_WXDIALOG
     if ( m_popupWinType != POPUPWIN_WXPOPUPTRANSIENTWINDOW )
     {

-        if ( m_isPopupShown &&
+        if ( IsPopupWindowState(Visible) &&

              ( evtType == wxEVT_LEFT_DOWN || evtType == wxEVT_RIGHT_DOWN ) )
         {
             HidePopup();
              ( evtType == wxEVT_LEFT_DOWN || evtType == wxEVT_RIGHT_DOWN ) )
         {
             HidePopup();


@@ -1455,7 +1551,7 @@ void wxComboCtrlBase::HandleNormalMouseEvent( wxMouseEvent& event )
     if ( (evtType == wxEVT_LEFT_DOWN || evtType == wxEVT_LEFT_DCLICK) &&
          (m_windowStyle & wxCB_READONLY) )
     {
     if ( (evtType == wxEVT_LEFT_DOWN || evtType == wxEVT_LEFT_DCLICK) &&
          (m_windowStyle & wxCB_READONLY) )
     {

-        if ( m_isPopupShown )
+        if ( GetPopupWindowState() >= Animating )

         {
     #if USES_WXPOPUPWINDOW
             // Click here always hides the popup.
         {
     #if USES_WXPOPUPWINDOW
             // Click here always hides the popup.


@@ -1481,7 +1577,7 @@ void wxComboCtrlBase::HandleNormalMouseEvent( wxMouseEvent& event )
         }
     }
     else
         }
     }
     else

-    if ( m_isPopupShown )
+    if ( IsPopupShown() )

     {
         // relay (some) mouse events to the popup
         if ( evtType == wxEVT_MOUSEWHEEL )
     {
         // relay (some) mouse events to the popup
         if ( evtType == wxEVT_MOUSEWHEEL )


@@ -1505,11 +1601,15 @@ void wxComboCtrlBase::OnKeyEvent(wxKeyEvent& event)
         if ( keycode == WXK_TAB )
         {
             wxNavigationKeyEvent evt;
         if ( keycode == WXK_TAB )
         {
             wxNavigationKeyEvent evt;

+
+            wxWindow* mainCtrl = GetMainWindowOfCompositeControl();
+

             evt.SetFlags(wxNavigationKeyEvent::FromTab|
                          (!event.ShiftDown() ? wxNavigationKeyEvent::IsForward
                                              : wxNavigationKeyEvent::IsBackward));
             evt.SetFlags(wxNavigationKeyEvent::FromTab|
                          (!event.ShiftDown() ? wxNavigationKeyEvent::IsForward
                                              : wxNavigationKeyEvent::IsBackward));

-            evt.SetEventObject(this);
-            GetParent()->GetEventHandler()->AddPendingEvent(evt);
+            evt.SetEventObject(mainCtrl);
+            evt.SetCurrentFocus(mainCtrl);
+            mainCtrl->GetParent()->GetEventHandler()->AddPendingEvent(evt);

             return;
         }
 
             return;
         }
 


@@ -1542,13 +1642,29 @@ void wxComboCtrlBase::OnFocusEvent( wxFocusEvent& event )
 {
     if ( event.GetEventType() == wxEVT_SET_FOCUS )
     {
 {
     if ( event.GetEventType() == wxEVT_SET_FOCUS )
     {

-        if ( m_text && m_text != ::wxWindow::FindFocus() )
-            m_text->SetFocus();
+        wxWindow* tc = GetTextCtrl();
+        if ( tc && tc != DoFindFocus() )
+#ifdef __WXMAC__
+            m_resetFocus = true;
+#else
+            tc->SetFocus();
+#endif

     }
 
     Refresh();
 }
 
     }
 
     Refresh();
 }
 

+void wxComboCtrlBase::OnIdleEvent( wxIdleEvent& WXUNUSED(event) )
+{
+    if ( m_resetFocus )
+    {
+        m_resetFocus = false;
+        wxWindow* tc = GetTextCtrl();
+        if ( tc )
+            tc->SetFocus();
+    }
+}
+

 void wxComboCtrlBase::OnSysColourChanged(wxSysColourChangedEvent& WXUNUSED(event))
 {
     OnThemeChange();
 void wxComboCtrlBase::OnSysColourChanged(wxSysColourChangedEvent& WXUNUSED(event))
 {
     OnThemeChange();


@@ -1677,7 +1793,10 @@ void wxComboCtrlBase::OnButtonClick()
 void wxComboCtrlBase::ShowPopup()
 {
     EnsurePopupControl();
 void wxComboCtrlBase::ShowPopup()
 {
     EnsurePopupControl();

-    wxCHECK_RET( !IsPopupShown(), wxT("popup window already shown") );
+    wxCHECK_RET( !IsPopupWindowState(Visible), wxT("popup window already shown") );
+
+    if ( IsPopupWindowState(Animating) )
+        return;

 
     SetFocus();
 
 
     SetFocus();
 


@@ -1734,6 +1853,8 @@ void wxComboCtrlBase::ShowPopup()
         popup = m_popup;
     }
 
         popup = m_popup;
     }
 

+    winPopup->Enable();
+    

     wxASSERT( !m_popup || m_popup == popup ); // Consistency check.
 
     wxSize adjustedSize = m_popupInterface->GetAdjustedSize(widthPopup,
     wxASSERT( !m_popup || m_popup == popup ); // Consistency check.
 
     wxSize adjustedSize = m_popupInterface->GetAdjustedSize(widthPopup,


@@ -1793,22 +1914,27 @@ void wxComboCtrlBase::ShowPopup()
     else
         popupX = 0;
 
     else
         popupX = 0;
 

+    int showFlags = CanDeferShow;
+

     if ( spaceBelow < szp.y )
     {
         popupY = scrPos.y - szp.y;
     if ( spaceBelow < szp.y )
     {
         popupY = scrPos.y - szp.y;

+        showFlags |= ShowAbove;

     }
 
     }
 

-    // Move to position
-    //wxLogDebug(wxT("popup scheduled position1: %i,%i"),ptp.x,ptp.y);
-    //wxLogDebug(wxT("popup position1: %i,%i"),winPopup->GetPosition().x,winPopup->GetPosition().y);
-
-    // Some platforms (GTK) may need these two to be separate
-    winPopup->SetSize( szp.x, szp.y );
-    winPopup->Move( popupX, popupY );
-
-    //wxLogDebug(wxT("popup position2: %i,%i"),winPopup->GetPosition().x,winPopup->GetPosition().y);
+#if INSTALL_TOPLEV_HANDLER
+    // Put top level window event handler into place
+    if ( m_popupWinType == POPUPWIN_WXPOPUPWINDOW )
+    {
+        if ( !m_toplevEvtHandler )
+            m_toplevEvtHandler = new wxComboFrameEventHandler(this);

 
 

-    m_popup = popup;
+        wxWindow* toplev = ::wxGetTopLevelParent( this );
+        wxASSERT( toplev );
+        ((wxComboFrameEventHandler*)m_toplevEvtHandler)->OnPopup();
+        toplev->PushEventHandler( m_toplevEvtHandler );
+    }
+#endif

 
     // Set string selection (must be this way instead of SetStringSelection)
     if ( m_text )
 
     // Set string selection (must be this way instead of SetStringSelection)
     if ( m_text )


@@ -1825,51 +1951,62 @@ void wxComboCtrlBase::ShowPopup()
     }
 
     // This must be after SetStringValue
     }
 
     // This must be after SetStringValue

-    m_isPopupShown = true;
+    m_popupWinState = Animating;

 
 

-    // Show it
-#if USES_WXPOPUPTRANSIENTWINDOW
-    if ( m_popupWinType == POPUPWIN_WXPOPUPTRANSIENTWINDOW )
-        ((wxPopupTransientWindow*)winPopup)->Popup(popup);
-    else
-#endif
-        winPopup->Show();
+    wxRect popupWinRect( popupX, popupY, szp.x, szp.y );

 
 

-#if INSTALL_TOPLEV_HANDLER
-    // Put top level window event handler into place
-    if ( m_popupWinType == POPUPWIN_WXPOPUPWINDOW )
+    m_popup = popup;
+    if ( (m_iFlags & wxCC_IFLAG_DISABLE_POPUP_ANIM) ||
+         AnimateShow( popupWinRect, showFlags ) )

     {
     {

-        if ( !m_toplevEvtHandler )
-            m_toplevEvtHandler = new wxComboFrameEventHandler(this);
+        DoShowPopup( popupWinRect, showFlags );
+    }
+}

 
 

-        wxWindow* toplev = ::wxGetTopLevelParent( this );
-        wxASSERT( toplev );
-        ((wxComboFrameEventHandler*)m_toplevEvtHandler)->OnPopup();
-        toplev->PushEventHandler( m_toplevEvtHandler );
+bool wxComboCtrlBase::AnimateShow( const wxRect& WXUNUSED(rect), int WXUNUSED(flags) )
+{
+    return true;
+}
+
+void wxComboCtrlBase::DoShowPopup( const wxRect& rect, int WXUNUSED(flags) )
+{
+    wxWindow* winPopup = m_winPopup;
+
+    if ( IsPopupWindowState(Animating) )
+    {
+        // Make sure the popup window is shown in the right position.
+        // Should not matter even if animation already did this.
+
+        // Some platforms (GTK) may like SetSize and Move to be separate
+        // (though the bug was probably fixed).
+        winPopup->SetSize( rect );
+
+        winPopup->Show();
+
+        m_popupWinState = Visible;

     }
     }

-#endif
+    else if ( IsPopupWindowState(Hidden) )
+    {
+        // Animation was aborted
+
+        wxASSERT( !winPopup->IsShown() );

 
 

+        m_popupWinState = Hidden;
+    }

 }
 
 void wxComboCtrlBase::OnPopupDismiss()
 }
 
 void wxComboCtrlBase::OnPopupDismiss()

-{ 
+{

     // Just in case, avoid double dismiss
     // Just in case, avoid double dismiss

-    if ( !m_isPopupShown )
+    if ( IsPopupWindowState(Hidden) )

         return;
 
         return;
 

-    // NB: Focus setting is really funny, atleast on wxMSW. First of all,
-    //     we need to have SetFocus at the end. Otherwise wxTextCtrl may
-    //     freeze until focus goes somewhere else. Second, wxTreeCtrl as
-    //     popup, when dismissing, "steals" focus back to itself unless
-    //     SetFocus is called also here, exactly before m_isPopupShown
-    //     is set to false. Which is truly weird since SetFocus is just
-    //     wxWindowMSW method and does not call event handler or anything like
-    //     that (ie. does not care about m_isPopupShown).
-
-    SetFocus();
+    // This must be set before focus - otherwise there will be recursive
+    // OnPopupDismisses.
+    m_popupWinState = Hidden;

 
 

-    // This should preferably be set before focus.
-    m_isPopupShown = false;
+    //SetFocus();
+    m_winPopup->Disable();

 
     // Inform popup control itself
     m_popupInterface->OnDismiss();
 
     // Inform popup control itself
     m_popupInterface->OnDismiss();


@@ -1887,7 +2024,10 @@ void wxComboCtrlBase::OnPopupDismiss()
     }
 #endif
 
     }
 #endif
 

-    m_timeCanAcceptClick = ::wxGetLocalTimeMillis() + 150;
+    m_timeCanAcceptClick = ::wxGetLocalTimeMillis();
+
+    if ( m_popupWinType == POPUPWIN_WXPOPUPTRANSIENTWINDOW )
+        m_timeCanAcceptClick += 150;

 
     // If cursor not on dropdown button, then clear its state
     // (technically not required by all ports, but do it for all just in case)
 
     // If cursor not on dropdown button, then clear its state
     // (technically not required by all ports, but do it for all just in case)


@@ -1912,19 +2052,14 @@ void wxComboCtrlBase::OnPopupDismiss()
 void wxComboCtrlBase::HidePopup()
 {
     // Should be able to call this without popup interface
 void wxComboCtrlBase::HidePopup()
 {
     // Should be able to call this without popup interface

-    //wxCHECK_RET( m_popupInterface, _T("no popup interface") );
-    if ( !m_isPopupShown )
+    if ( IsPopupWindowState(Hidden) )

         return;
 
     // transfer value and show it in textctrl, if any
         return;
 
     // transfer value and show it in textctrl, if any

-    SetValue( m_popupInterface->GetStringValue() );
+    if ( !IsPopupWindowState(Animating) )
+        SetValue( m_popupInterface->GetStringValue() );

 
 

-#if USES_WXPOPUPTRANSIENTWINDOW
-    if ( m_popupWinType == POPUPWIN_WXPOPUPTRANSIENTWINDOW )
-        ((wxPopupTransientWindow*)m_winPopup)->Dismiss();
-    else
-#endif
-        m_winPopup->Hide();
+    m_winPopup->Hide();

 
     OnPopupDismiss();
 }
 
     OnPopupDismiss();
 }