]> git.saurik.com Git - wxWidgets.git/blobdiff - src/mac/control.cpp
applied Paul A. Thiessen's patches for correcting coordinates when having borders...
[wxWidgets.git] / src / mac / control.cpp
index eb83a83ee005c16dfdead7029b36cb1609539e32..5c2a5d785d6f519cfc5ccbced383e2a12fca5c4b 100644 (file)
@@ -13,6 +13,8 @@
 #pragma implementation "control.h"
 #endif
 
 #pragma implementation "control.h"
 #endif
 
+#include "wx/defs.h"
+
 #include "wx/control.h"
 #include "wx/panel.h"
 #include "wx/app.h"
 #include "wx/control.h"
 #include "wx/panel.h"
 #include "wx/app.h"
@@ -38,7 +40,7 @@ BEGIN_EVENT_TABLE(wxControl, wxWindow)
 END_EVENT_TABLE()
 #endif
 
 END_EVENT_TABLE()
 #endif
 
-#include <wx/mac/uma.h>
+#include "wx/mac/uma.h"
 
 // Item members
 
 
 // Item members
 
@@ -70,7 +72,7 @@ wxControl::wxControl()
 
        if ( wxMacLiveScrollbarActionUPP == NULL )
        {
 
        if ( wxMacLiveScrollbarActionUPP == NULL )
        {
-#ifdef __UNIX__
+#if defined(UNIVERSAL_INTERFACES_VERSION) && (UNIVERSAL_INTERFACES_VERSION >= 0x0340)
         wxMacLiveScrollbarActionUPP = NewControlActionUPP( wxMacLiveScrollbarActionProc );
 #else
                wxMacLiveScrollbarActionUPP = NewControlActionProc( wxMacLiveScrollbarActionProc ) ;
         wxMacLiveScrollbarActionUPP = NewControlActionUPP( wxMacLiveScrollbarActionProc );
 #else
                wxMacLiveScrollbarActionUPP = NewControlActionProc( wxMacLiveScrollbarActionProc ) ;
@@ -109,7 +111,7 @@ wxControl::~wxControl()
     }
     if ( m_macControl )
     {
     }
     if ( m_macControl )
     {
-       UMADisposeControl( m_macControl ) ;
+       ::DisposeControl( m_macControl ) ;
        m_macControl = NULL ;
     }
 }
        m_macControl = NULL ;
     }
 }
@@ -136,11 +138,55 @@ void wxControl::SetLabel(const wxString& title)
 #endif
                ::SetControlTitle( m_macControl , maclabel ) ;
        }
 #endif
                ::SetControlTitle( m_macControl , maclabel ) ;
        }
+       Refresh() ;
 }
 
 wxSize wxControl::DoGetBestSize() const
 {
 }
 
 wxSize wxControl::DoGetBestSize() const
 {
-    return wxSize(20, 20);
+       Rect    bestsize = { 0 , 0 , 0 , 0 } ;
+       short   baselineoffset ;
+       int bestWidth, bestHeight ;
+       ::GetBestControlRect( m_macControl , &bestsize , &baselineoffset ) ;
+
+       if ( EmptyRect( &bestsize ) )
+       {
+               baselineoffset = 0;
+               bestsize.left = bestsize.top = 0 ;
+               bestsize.right = 16 ;
+               bestsize.bottom = 16 ;
+               if ( IsKindOf( CLASSINFO( wxScrollBar ) ) )
+               {
+                       bestsize.bottom = 16 ;
+               }
+               else if ( IsKindOf( CLASSINFO( wxSpinButton ) ) )
+               {
+                       bestsize.bottom = 24 ; 
+               }
+       }
+
+       if ( IsKindOf( CLASSINFO( wxButton ) ) )
+       {
+               bestWidth = m_label.Length() * 8 + 12 ;
+               if ( bestWidth < 70 )
+                 bestWidth = 70 ;
+       }
+       else if ( IsKindOf( CLASSINFO( wxStaticText ) ) )
+       {
+               bestWidth = m_label.Length() * 8 ;
+       }
+       else
+               bestWidth = bestsize.right - bestsize.left ;
+               
+       bestWidth += 2 * m_macHorizontalBorder ;
+
+       bestHeight = bestsize.bottom - bestsize.top ;
+       if ( bestHeight < 10 )
+               bestHeight = 13 ;
+
+       bestHeight += 2 * m_macVerticalBorder;
+
+               
+    return wxSize(bestWidth, bestHeight);
 }
 
 bool wxControl::ProcessCommand (wxCommandEvent & event)
 }
 
 bool wxControl::ProcessCommand (wxCommandEvent & event)
@@ -200,7 +246,7 @@ void wxControl::MacPreControlCreate( wxWindow *parent, wxWindowID id, wxString l
        SetValidator(validator);
 
   m_windowStyle = style;
        SetValidator(validator);
 
   m_windowStyle = style;
-  parent->AddChild((wxButton *)this);
+  parent->AddChild(this);
 
   m_backgroundColour = parent->GetBackgroundColour() ;
   m_foregroundColour = parent->GetForegroundColour() ;
 
   m_backgroundColour = parent->GetBackgroundColour() ;
   m_foregroundColour = parent->GetForegroundColour() ;
@@ -210,20 +256,17 @@ void wxControl::MacPreControlCreate( wxWindow *parent, wxWindowID id, wxString l
   else
       m_windowId = id;
 
   else
       m_windowId = id;
 
+   // These sizes will be adjusted in MacPostControlCreate
+
        m_width = size.x ;
        m_height = size.y ;
        m_width = size.x ;
        m_height = size.y ;
-       int x = pos.x ;
-       int y = pos.y ;
-       AdjustForParentClientOrigin(x, y, wxSIZE_USE_EXISTING);
-       m_x = x ;
-       m_y = y ;
+       m_x = pos.x ;
+       m_y = pos.y ;
                
                
-
-       parent->MacClientToRootWindow( &x , &y ) ;
-       outBounds->top = y + m_macVerticalBorder ;
-       outBounds->left = x + m_macHorizontalBorder ;
-       outBounds->bottom = outBounds->top + m_height - 2 * m_macVerticalBorder;
-       outBounds->right = outBounds->left + m_width - 2 * m_macHorizontalBorder ;
+       outBounds->top = -10;
+       outBounds->left = -10;
+       outBounds->bottom = 0;
+       outBounds->right = 0;
 
        char c_text[255];
        strcpy( c_text , label ) ;
 
        char c_text[255];
        strcpy( c_text , label ) ;
@@ -254,7 +297,7 @@ void wxControl::MacPostControlCreate()
                controlstyle.flags = kControlUseFontMask ;
                controlstyle.font = kControlFontSmallBoldSystemFont ;
                
                controlstyle.flags = kControlUseFontMask ;
                controlstyle.font = kControlFontSmallBoldSystemFont ;
                
-               ::UMASetControlFontStyle( m_macControl , &controlstyle ) ;
+               ::SetControlFontStyle( m_macControl , &controlstyle ) ;
        }
        else
        {
        }
        else
        {
@@ -262,25 +305,44 @@ void wxControl::MacPostControlCreate()
                controlstyle.flags = kControlUseFontMask ;
                controlstyle.font = kControlFontSmallSystemFont ;
                
                controlstyle.flags = kControlUseFontMask ;
                controlstyle.font = kControlFontSmallSystemFont ;
                
-               ::UMASetControlFontStyle( m_macControl , &controlstyle ) ;
+               ::SetControlFontStyle( m_macControl , &controlstyle ) ;
        }
        ControlHandle container = GetParent()->MacGetContainerForEmbedding() ;
        }
        ControlHandle container = GetParent()->MacGetContainerForEmbedding() ;
-       wxASSERT_MSG( container != NULL , "No valid mac container control" ) ;
-       ::UMAEmbedControl( m_macControl , container ) ;
+       wxASSERT_MSG( container != NULL , wxT("No valid mac container control") ) ;
+       ::EmbedControl( m_macControl , container ) ;
        m_macControlIsShown  = true ;
        m_macControlIsShown  = true ;
-       MacAdjustControlRect() ;
+
        wxAssociateControlWithMacControl( m_macControl , this ) ;
        wxAssociateControlWithMacControl( m_macControl , this ) ;
+
+     // Adjust the controls size and position
+     wxPoint pos(m_x, m_y);
+     wxSize best_size( DoGetBestSize() );
+     wxSize new_size( m_width, m_height );
+     m_x = m_y = m_width = m_height = -1;  // Forces SetSize to move/size the control
+     if (new_size.x == -1) {
+        new_size.x = best_size.x;
+     }
+     if (new_size.y == -1) {
+         new_size.y = best_size.y;
+     }
+     SetSize(pos.x, pos.y, new_size.x, new_size.y);
+     UMAShowControl( m_macControl ) ;
 }
 
 void wxControl::MacAdjustControlRect() 
 {
 }
 
 void wxControl::MacAdjustControlRect() 
 {
-       wxASSERT_MSG( m_macControl != NULL , "No valid mac control" ) ;
+       wxASSERT_MSG( m_macControl != NULL , wxT("No valid mac control") ) ;
        if ( m_width == -1 || m_height == -1 )
        {
                Rect    bestsize = { 0 , 0 , 0 , 0 } ;
                short   baselineoffset ;
                
        if ( m_width == -1 || m_height == -1 )
        {
                Rect    bestsize = { 0 , 0 , 0 , 0 } ;
                short   baselineoffset ;
                
-               UMAGetBestControlRect( m_macControl , &bestsize , &baselineoffset ) ;
+               ::GetBestControlRect( m_macControl , &bestsize , &baselineoffset ) ;
 
                if ( EmptyRect( &bestsize ) )
                {
 
                if ( EmptyRect( &bestsize ) )
                {
@@ -353,27 +415,27 @@ void wxControl::MacSuperChangedPosition()
                
                WindowRef rootwindow = GetMacRootWindow() ;
                wxWindow* wxrootwindow = wxFindWinFromMacWindow( rootwindow ) ;
                
                WindowRef rootwindow = GetMacRootWindow() ;
                wxWindow* wxrootwindow = wxFindWinFromMacWindow( rootwindow ) ;
-               UMASetThemeWindowBackground( rootwindow , kThemeBrushDialogBackgroundActive , false ) ;
+               ::SetThemeWindowBackground( rootwindow , kThemeBrushDialogBackgroundActive , false ) ;
                wxMacDrawingHelper focus( wxrootwindow ) ;
        
                if ( mac_x != former_mac_x || mac_y != former_mac_y )
                {
                wxMacDrawingHelper focus( wxrootwindow ) ;
        
                if ( mac_x != former_mac_x || mac_y != former_mac_y )
                {
-                       {
-                               Rect inval = { former_mac_y , former_mac_x , former_mac_y + m_height , former_mac_x + m_width } ;
-                               InvalWindowRect( rootwindow , &inval ) ;
-                       }
+                       {
+                               Rect inval = { former_mac_y , former_mac_x , former_mac_y + m_height , former_mac_x + m_width } ;
+                               InvalWindowRect( rootwindow , &inval ) ;
+                       }
                        UMAMoveControl( m_macControl , mac_x + m_macHorizontalBorder , mac_y + m_macVerticalBorder ) ;
                        UMAMoveControl( m_macControl , mac_x + m_macHorizontalBorder , mac_y + m_macVerticalBorder ) ;
-                       {
-                               Rect inval = { mac_y , mac_x , mac_y + m_height , mac_x + m_width } ;
-                               InvalWindowRect( rootwindow , &inval ) ;
-                       }
+                       {
+                               Rect inval = { mac_y , mac_x , mac_y + m_height , mac_x + m_width } ;
+                               InvalWindowRect( rootwindow , &inval ) ;
+                       }
                }
                if ( wxrootwindow->IsKindOf( CLASSINFO( wxDialog ) ) )
                {
                }
                else
                {
                }
                if ( wxrootwindow->IsKindOf( CLASSINFO( wxDialog ) ) )
                {
                }
                else
                {
-                       UMASetThemeWindowBackground( rootwindow , kThemeBrushDocumentWindowBackground , false ) ;
+                       ::SetThemeWindowBackground( rootwindow , kThemeBrushDocumentWindowBackground , false ) ;
                }
        }
 
                }
        }
 
@@ -422,7 +484,7 @@ void  wxControl::MacSuperShown( bool show )
                {
                        if ( m_macControlIsShown )
                        {
                {
                        if ( m_macControlIsShown )
                        {
-                               ::UMAHideControl( m_macControl ) ;
+                               ::HideControl( m_macControl ) ;
                                m_macControlIsShown = false ;
                        }
                }
                                m_macControlIsShown = false ;
                        }
                }
@@ -449,100 +511,127 @@ void  wxControl::DoSetSize(int x, int y,
                return ;
        }
 
                return ;
        }
 
-       WindowRef rootwindow = GetMacRootWindow() ;
-       wxWindow* wxrootwindow = wxFindWinFromMacWindow( rootwindow ) ;
-       UMASetThemeWindowBackground( rootwindow , kThemeBrushDialogBackgroundActive , false ) ;
-               
-       int former_x = m_x ;
-       int former_y = m_y ;
-       int former_w = m_width ;
-       int former_h = m_height ;
-       
-       Rect contrlRect ;               
-       GetControlBounds( m_macControl , &contrlRect ) ; 
-       int former_mac_x = contrlRect.left ;
-       int former_mac_y = contrlRect.top ;
-       
-  int currentX, currentY;
-  GetPosition(&currentX, &currentY);
-  int currentW,currentH;
-  GetSize(&currentW, &currentH);
-
-  int actualWidth = width;
-  int actualHeight = height;
-  int actualX = x;
-  int actualY = y;
-  if (x == -1 && !(sizeFlags & wxSIZE_ALLOW_MINUS_ONE))
-      actualX = currentX;
-  if (y == -1 && !(sizeFlags & wxSIZE_ALLOW_MINUS_ONE))
-      actualY = currentY;
-  if (width == -1)
-      actualWidth = currentW ;
-  if (height == -1)
-      actualHeight = currentH ;
-
-       if ( actualX == currentX && actualY == currentY && actualWidth == currentW && actualHeight == currentH)
-               return ;
-               
-       AdjustForParentClientOrigin(actualX, actualY, sizeFlags);
-       WindowRef macrootwindow = GetMacRootWindow() ;
-       wxMacDrawingHelper focus( wxFindWinFromMacWindow( macrootwindow ) ) ;
-
-       int mac_x = actualX ;
-       int mac_y = actualY ;
-       GetParent()->MacClientToRootWindow( & mac_x , & mac_y ) ;
-       
-       if ( mac_x != former_mac_x || mac_y != former_mac_y )
-       {
-               {
-                       Rect inval = { former_mac_y , former_mac_x , former_mac_y + m_height , former_mac_x + m_width } ;
-                       InvalWindowRect( macrootwindow, &inval ) ;
-               }
-               UMAMoveControl( m_macControl , mac_x + m_macHorizontalBorder , mac_y  + m_macVerticalBorder ) ;
-               {
-                       Rect inval = { mac_y , mac_x , mac_y + m_height , mac_x + m_width } ;
-                       InvalWindowRect(macrootwindow, &inval ) ;
-               }
-       }
-
-       if ( actualX != former_x || actualY != former_y )
-       {
-               m_x = actualX ;
-               m_y = actualY ;
-
-               MacRepositionScrollBars() ;
-               // To consider -> should the parameters be the effective or the virtual coordinates (AdjustForParent..)
-    wxMoveEvent event(wxPoint(m_x, m_y), m_windowId);
-    event.SetEventObject(this);
-    GetEventHandler()->ProcessEvent(event);
-       }               
-       if ( actualWidth != former_w || actualHeight != former_h )
-       {
-               {
-                       Rect inval = { mac_y , mac_x , mac_y + former_h , mac_x + former_w } ;
-                       InvalWindowRect( macrootwindow, &inval ) ;
-               }
-               m_width = actualWidth ;
-               m_height = actualHeight ;
-
-               UMASizeControl( m_macControl ,  m_width - 2 * m_macHorizontalBorder, m_height -  2 * m_macVerticalBorder ) ;
-               {
-                       Rect inval = { mac_y , mac_x , mac_y + m_height , mac_x + m_width } ;
-                       InvalWindowRect( macrootwindow , &inval ) ;
-               }
-
-               MacRepositionScrollBars() ;
-    wxSizeEvent event(wxSize(m_width, m_height), m_windowId);
-    event.SetEventObject(this);
-    GetEventHandler()->ProcessEvent(event);
-       }
-       if ( wxrootwindow->IsKindOf( CLASSINFO( wxDialog ) ) )
-       {
-       }
-       else
-       {
-               UMASetThemeWindowBackground( rootwindow , kThemeBrushDocumentWindowBackground , false ) ;
-       }
+     Rect oldbounds, newbounds;
+     int new_x, new_y, new_width, new_height;
+     int mac_x, mac_y;
+     new_x = m_x;
+     new_y = m_y;
+     new_width  = m_width;
+     new_height = m_height;
+     if (sizeFlags & wxSIZE_ALLOW_MINUS_ONE)
+     {
+         new_x      = x;
+         new_y      = y;
+         new_width  = width;
+         new_height = height;
+     }
+     else
+     {
+         if (x != -1)      new_x      = x;
+         if (y != -1)      new_y      = y;
+         if (width != -1)  new_width  = width;
+         if (height != -1) new_height = height;
+     }
+     if(sizeFlags & wxSIZE_AUTO)
+     {
+         wxSize size = GetBestSize();
+         if (sizeFlags & wxSIZE_AUTO_WIDTH)
+         {
+             if (width == -1)   new_width = size.x;
+         }
+         if (sizeFlags & wxSIZE_AUTO_HEIGHT)
+         {
+             if (height == -1)   new_height = size.y;
+         }
+     }
+     AdjustForParentClientOrigin(new_x, new_y, sizeFlags);
+     mac_x = new_x;
+     mac_y = new_y;
+     if(GetParent()) {
+         GetParent()->MacClientToRootWindow(&mac_x, &mac_y);
+     }
+     GetControlBounds(m_macControl, &oldbounds);
+     oldbounds.right = oldbounds.left + m_width;
+     oldbounds.bottom = oldbounds.top + m_height;
+     bool doMove = false;
+     bool doResize = false;
+     if ( mac_x != (oldbounds.left - m_macHorizontalBorder) ||
+          mac_y != (oldbounds.top - m_macVerticalBorder) )
+     {
+         doMove = true ;
+     }
+     if ( new_width != oldbounds.right - oldbounds.left - 2 * m_macHorizontalBorder ||
+          new_height != oldbounds.bottom - oldbounds.top - 2 * m_macVerticalBorder)
+     {
+         doResize = true ;
+     }
+     if ( doMove || doResize )
+     {
+         // Ensure resize is within constraints
+         if ((m_minWidth != -1) && (new_width < m_minWidth)) {
+             new_width = m_minWidth;
+         }
+         if ((m_minHeight != -1) && (new_height < m_minHeight)) {
+             new_height = m_minHeight;
+         }
+         if ((m_maxWidth != -1) && (new_width > m_maxWidth)) {
+             new_width = m_maxWidth;
+         }
+         if ((m_maxHeight != -1) && (new_height > m_maxHeight)) {
+             new_height = m_maxHeight;
+         }
+         if ( doMove )
+         {
+             m_x = new_x;
+             m_y = new_y;
+             UMAMoveControl(m_macControl,
+                            mac_x + m_macHorizontalBorder, mac_y + m_macVerticalBorder);
+             wxMoveEvent event(wxPoint(m_x, m_y), m_windowId);
+             event.SetEventObject(this);
+             GetEventHandler()->ProcessEvent(event) ;
+         }
+         if ( doResize )
+         {
+             m_width = new_width;
+             m_height = new_height;
+             UMASizeControl( m_macControl,
+                             m_width - 2 * m_macHorizontalBorder,
+                             m_height -  2 * m_macVerticalBorder ) ;
+             wxSizeEvent event(wxSize(m_width, m_height), m_windowId);
+             event.SetEventObject(this);
+             GetEventHandler()->ProcessEvent(event);
+         }
+         // Set up port
+         WindowRef rootwindow = GetMacRootWindow() ;
+         wxWindow* wxrootwindow = wxFindWinFromMacWindow( rootwindow ) ;
+         wxMacDrawingHelper focus( wxrootwindow );
+         ::SetThemeWindowBackground( rootwindow , kThemeBrushDialogBackgroundActive , false ) ;
+         // Update window at old and new positions
+         SetRect(&newbounds, m_x, m_y, m_x + m_width, m_y + m_height);
+         InvalWindowRect( rootwindow , &oldbounds );
+         InvalWindowRect( rootwindow , &newbounds );
+         MacRepositionScrollBars() ;
+         if ( !wxrootwindow->IsKindOf( CLASSINFO( wxDialog ) ) )
+         {
+             ::SetThemeWindowBackground( rootwindow, kThemeBrushDocumentWindowBackground, false );
+         }
+     }
 }
 
 bool  wxControl::Show(bool show) 
 }
 
 bool  wxControl::Show(bool show) 
@@ -556,7 +645,7 @@ bool  wxControl::Show(bool show)
                {
                        if ( m_macControlIsShown )
                        {
                {
                        if ( m_macControlIsShown )
                        {
-                               ::UMAHideControl( m_macControl ) ;
+                               ::HideControl( m_macControl ) ;
                                m_macControlIsShown = false ;
                        }
                }
                                m_macControlIsShown = false ;
                        }
                }
@@ -579,21 +668,10 @@ bool  wxControl::Enable(bool enable)
 
        if ( m_macControl )
        {
 
        if ( m_macControl )
        {
-               
-               if ( UMAHasAppearance() )
-               {
-                       if ( enable )
-                               ::ActivateControl( m_macControl ) ;
-                       else
-                               ::DeactivateControl( m_macControl ) ;
-               }
+               if ( enable )
+                   UMAActivateControl( m_macControl ) ;
                else
                else
-               {
-                       if ( enable )
-                               ::HiliteControl( m_macControl , 0 ) ;
-                       else
-                               ::HiliteControl( m_macControl , 255 ) ;
-               }
+                   UMADeactivateControl( m_macControl ) ;
        }
        return TRUE ;
 }
        }
        return TRUE ;
 }
@@ -624,13 +702,12 @@ void wxControl::MacRedrawControl()
                                // the mac control manager always assumes to have the origin at 0,0
                                SetOrigin( 0 , 0 ) ;
                                
                                // the mac control manager always assumes to have the origin at 0,0
                                SetOrigin( 0 , 0 ) ;
                                
-                               bool                    hasTabBehind = false ;
                                wxWindow* parent = GetParent() ;
                                while ( parent )
                                {
                                        if( parent->MacGetWindowData() )
                                        {
                                wxWindow* parent = GetParent() ;
                                while ( parent )
                                {
                                        if( parent->MacGetWindowData() )
                                        {
-                                               UMASetThemeWindowBackground( win->MacGetWindowData()->m_macWindow , kThemeBrushDialogBackgroundActive , false ) ;
+                                               ::SetThemeWindowBackground( win->MacGetWindowData()->m_macWindow , kThemeBrushDialogBackgroundActive , false ) ;
                                                break ;
                                        }
                                        
                                                break ;
                                        }
                                        
@@ -645,8 +722,7 @@ void wxControl::MacRedrawControl()
                                } 
                                
                                UMADrawControl( m_macControl ) ;
                                } 
                                
                                UMADrawControl( m_macControl ) ;
-                               UMASetThemeWindowBackground( win->MacGetWindowData()->m_macWindow , win->MacGetWindowData()->m_macWindowBackgroundTheme , false ) ;
-                               wxDC::MacInvalidateSetup() ;
+                               ::SetThemeWindowBackground( win->MacGetWindowData()->m_macWindow , win->MacGetWindowData()->m_macWindowBackgroundTheme , false ) ;
                        }
                }
        }
                        }
                }
        }
@@ -665,14 +741,13 @@ void wxControl::OnPaint(wxPaintEvent& event)
                                wxMacDrawingHelper help( win ) ;
                                // the mac control manager always assumes to have the origin at 0,0
                                SetOrigin( 0 , 0 ) ;
                                wxMacDrawingHelper help( win ) ;
                                // the mac control manager always assumes to have the origin at 0,0
                                SetOrigin( 0 , 0 ) ;
-                               
-                               bool                    hasTabBehind = false ;
+
                                wxWindow* parent = GetParent() ;
                                while ( parent )
                                {
                                        if( parent->MacGetWindowData() )
                                        {
                                wxWindow* parent = GetParent() ;
                                while ( parent )
                                {
                                        if( parent->MacGetWindowData() )
                                        {
-                                               UMASetThemeWindowBackground( win->MacGetWindowData()->m_macWindow , kThemeBrushDialogBackgroundActive , false ) ;
+                                               ::SetThemeWindowBackground( win->MacGetWindowData()->m_macWindow , kThemeBrushDialogBackgroundActive , false ) ;
                                                break ;
                                        }
                                        
                                                break ;
                                        }
                                        
@@ -685,10 +760,9 @@ void wxControl::OnPaint(wxPaintEvent& event)
                                        
                                        parent = parent->GetParent() ;
                                } 
                                        
                                        parent = parent->GetParent() ;
                                } 
-                               
+
                                UMADrawControl( m_macControl ) ;
                                UMADrawControl( m_macControl ) ;
-                               UMASetThemeWindowBackground( win->MacGetWindowData()->m_macWindow , win->MacGetWindowData()->m_macWindowBackgroundTheme , false ) ;
-                               wxDC::MacInvalidateSetup() ;
+                               ::SetThemeWindowBackground( win->MacGetWindowData()->m_macWindow , win->MacGetWindowData()->m_macWindowBackgroundTheme , false ) ;
                        }
                }
        }
                        }
                }
        }
@@ -717,7 +791,7 @@ void  wxControl::OnKeyDown( wxKeyEvent &event )
        keychar = short(ev->message & charCodeMask);
        keycode = short(ev->message & keyCodeMask) >> 8 ;
 
        keychar = short(ev->message & charCodeMask);
        keycode = short(ev->message & keyCodeMask) >> 8 ;
 
-       UMAHandleControlKey( m_macControl , keycode , keychar , ev->modifiers ) ;
+       ::HandleControlKey( m_macControl , keycode , keychar , ev->modifiers ) ;
 }
 
 void  wxControl::OnMouseEvent( wxMouseEvent &event ) 
 }
 
 void  wxControl::OnMouseEvent( wxMouseEvent &event ) 
@@ -738,7 +812,6 @@ void  wxControl::OnMouseEvent( wxMouseEvent &event )
                        
                ControlHandle   control ;
                Point           localwhere ;
                        
                ControlHandle   control ;
                Point           localwhere ;
-               GrafPtr         port ;
                SInt16          controlpart ;
                WindowRef       window = GetMacRootWindow() ;
                
                SInt16          controlpart ;
                WindowRef       window = GetMacRootWindow() ;
                
@@ -761,8 +834,13 @@ void  wxControl::OnMouseEvent( wxMouseEvent &event )
        
                if ( event.m_metaDown )
                        modifiers |= cmdKey ;
        
                if ( event.m_metaDown )
                        modifiers |= cmdKey ;
-       
+/*
+#if TARGET_CARBON
+        control = FindControlUnderMouse( localwhere , window , &controlpart ) ;
+#else
                controlpart = FindControl( localwhere , window , &control ) ;
                controlpart = FindControl( localwhere , window , &control ) ;
+#endif
+*/
                {
                /*
                        if ( AcceptsFocus() && FindFocus() != this )
                {
                /*
                        if ( AcceptsFocus() && FindFocus() != this )
@@ -770,18 +848,17 @@ void  wxControl::OnMouseEvent( wxMouseEvent &event )
                                SetFocus() ;
                        }
                */
                                SetFocus() ;
                        }
                */
-                       if ( control && UMAIsControlActive( control ) )
+                   control = m_macControl ;
+                       if ( control && ::IsControlActive( control ) )
                        {
                                {
                        {
                                {
-                                       if ( controlpart == kControlIndicatorPart && !UMAHasAppearance() )
-                                               controlpart = UMAHandleControlClick( control , localwhere , modifiers , (ControlActionUPP) NULL ) ;
-                                       else
-                                               controlpart = UMAHandleControlClick( control , localwhere , modifiers , (ControlActionUPP) -1 ) ;
+                                       controlpart = ::HandleControlClick( control , localwhere , modifiers , (ControlActionUPP) -1 ) ;
                                        wxTheApp->s_lastMouseDown = 0 ;
                                        wxTheApp->s_lastMouseDown = 0 ;
-                                       if ( controlpart && ! ( ( UMAHasAppearance() || (controlpart != kControlIndicatorPart) ) 
-                                               && (IsKindOf( CLASSINFO( wxScrollBar ) ) ) ) ) // otherwise we will get the event twice
+                                       if ( control && controlpart != kControlNoPart &&
+                                           ! IsKindOf( CLASSINFO( wxScrollBar ) ) 
+                                       ) // otherwise we will get the event twice for scrollbar
                                        {
                                        {
-                                               MacHandleControlClick( control , controlpart ) ;
+                                           MacHandleControlClick( control , controlpart ) ;
                                        }
                                }
                        }
                                        }
                                }
                        }