Correct slider width when labels are used.

[wxWidgets.git] / src / mac / carbon / combobox.cpp

diff --git a/src/mac/carbon/combobox.cpp b/src/mac/carbon/combobox.cpp
index b1b55589f0fa0a054bd3a50dcb0cdac195ffcafe..7df584688994378bc34e5a5a8ba67a9275f7c06d 100644 (file)
--- a/src/mac/carbon/combobox.cpp
+++ b/src/mac/carbon/combobox.cpp
@@ -44,9 +44,12 @@ MenuHandle NewUniqueMenu()
 // around text control.
 static const wxCoord MARGIN = 6;
 static const int    POPUPWIDTH = 24;
 // around text control.
 static const wxCoord MARGIN = 6;
 static const int    POPUPWIDTH = 24;

+// this is the border a focus rect on OSX is needing
+static const int    TEXTFOCUSBORDER = 3 ;

 #else
 static const wxCoord MARGIN = 2;
 static const int    POPUPWIDTH = 18;
 #else
 static const wxCoord MARGIN = 2;
 static const int    POPUPWIDTH = 18;

+static const int    TEXTFOCUSBORDER = 0 ;

 #endif
 static const int    POPUPHEIGHT = 23;
 
 #endif
 static const int    POPUPHEIGHT = 23;
 


@@ -62,9 +65,6 @@ public:
         : wxTextCtrl( cb , 1 )
     {
         m_cb = cb;
         : wxTextCtrl( cb , 1 )
     {
         m_cb = cb;

-
-        // remove the default minsize, the combobox will have one instead
-        SetSizeHints(-1,-1);

     }
 
 protected:
     }
 
 protected:


@@ -145,9 +145,6 @@ public:
         : wxChoice( cb , 1 )
     {
         m_cb = cb;
         : wxChoice( cb , 1 )
     {
         m_cb = cb;

-
-        // remove the default minsize, the combobox will have one instead
-        SetSizeHints(-1,-1);

     }
 
 protected:
     }
 
 protected:


@@ -205,7 +202,7 @@ wxComboBox::~wxComboBox()
 
 wxSize wxComboBox::DoGetBestSize() const
 {
 
 wxSize wxComboBox::DoGetBestSize() const
 {

-    if (!m_choice || !m_text)
+    if (!m_choice && !m_text)

         return GetSize();
     wxSize size = m_choice->GetBestSize();
     
         return GetSize();
     wxSize size = m_choice->GetBestSize();
     


@@ -215,22 +212,15 @@ wxSize wxComboBox::DoGetBestSize() const
         if (sizeText.y > size.y)
             size.y = sizeText.y;
         size.x = POPUPWIDTH + sizeText.x + MARGIN;
         if (sizeText.y > size.y)
             size.y = sizeText.y;
         size.x = POPUPWIDTH + sizeText.x + MARGIN;

+        size.x += TEXTFOCUSBORDER ;
+        size.y += 2 * TEXTFOCUSBORDER ;

     }
     }

-

     return size;
 }
 
 void wxComboBox::DoMoveWindow(int x, int y, int width, int height) {
     return size;
 }
 
 void wxComboBox::DoMoveWindow(int x, int y, int width, int height) {

-    height = POPUPHEIGHT;
-    int origin = 0;
-#if TARGET_API_MAC_OSX
-    // give the controls some padding so that the text ctrl's borders
-    // and blue highlight can appear
-    origin = 4;
-#endif
-
-    wxControl::DoMoveWindow(x, y, width + origin, height + origin);
-
+    wxControl::DoMoveWindow(x, y, width , height );
+    

     if ( m_text == NULL )
     {
         // we might not be fully constructed yet, therefore watch out...
     if ( m_text == NULL )
     {
         // we might not be fully constructed yet, therefore watch out...


@@ -240,14 +230,8 @@ void wxComboBox::DoMoveWindow(int x, int y, int width, int height) {
     else
     {
         wxCoord wText = width - POPUPWIDTH - MARGIN;
     else
     {
         wxCoord wText = width - POPUPWIDTH - MARGIN;

-#if TARGET_API_MAC_OSX
-        // also, we need to shrink the size of the wxTextCtrl a bit
-        // to make it appear properly on OS X.
-        height -= 8;
-        wText -= 8;
-#endif
-        m_text->SetSize(origin, origin, wText, height);
-        m_choice->SetSize(origin + wText + MARGIN, 0, POPUPWIDTH, -1);
+        m_text->SetSize(TEXTFOCUSBORDER, TEXTFOCUSBORDER, wText, -1 );
+        m_choice->SetSize(TEXTFOCUSBORDER + wText + MARGIN, TEXTFOCUSBORDER, POPUPWIDTH, -1);

     }    
 }
 
     }    
 }
 


@@ -325,7 +309,6 @@ bool wxComboBox::Create(wxWindow *parent, wxWindowID id,
     }
 
     m_choice = new wxComboBoxChoice(this, style );
     }
 
     m_choice = new wxComboBoxChoice(this, style );

-    m_choice->SetSizeHints( wxSize( POPUPWIDTH , POPUPHEIGHT ) ) ;

     wxSize csize = size;
     if ( style & wxCB_READONLY )
     {
     wxSize csize = size;
     if ( style & wxCB_READONLY )
     {


@@ -334,8 +317,10 @@ bool wxComboBox::Create(wxWindow *parent, wxWindowID id,
     else
     {
         m_text = new wxComboBoxText(this);
     else
     {
         m_text = new wxComboBoxText(this);

-        if ( size.y == -1 ) {
-          csize.y = m_text->GetSize().y ;
+        if ( size.y == -1 ) 
+        {
+            csize.y = m_text->GetSize().y ;
+            csize.y += 2 * TEXTFOCUSBORDER ;

         }
     }
     
         }
     }
     


@@ -346,7 +331,7 @@ bool wxComboBox::Create(wxWindow *parent, wxWindowID id,
         m_choice->DoAppend( choices[ i ] );
     }
 
         m_choice->DoAppend( choices[ i ] );
     }
 

-    SetBestSize(csize);   // Needed because it is a wxControlWithItems
+    SetBestSize(size);   // Needed because it is a wxControlWithItems

 
     return TRUE;
 }
 
     return TRUE;
 }