]> git.saurik.com Git - wxWidgets.git/blobdiff - src/mac/carbon/control.cpp
projects / wxWidgets.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
Applied [ 590985 ] fix for radiobox under motif
[wxWidgets.git] / src / mac / carbon / control.cpp
diff --git a/src/mac/carbon/control.cpp b/src/mac/carbon/control.cpp
index 5c2a5d785d6f519cfc5ccbced383e2a12fca5c4b..fc90b8ea04ce697de35e424fc7a1f16ecafd05ec 100644 (file)
--- a/src/mac/carbon/control.cpp
+++ b/src/mac/carbon/control.cpp
@@ -6,7 +6,7 @@
 // Created:     ??/??/98
 // RCS-ID:      $Id$
 // Copyright:   (c) AUTHOR
 // Created:     ??/??/98
 // RCS-ID:      $Id$
 // Copyright:   (c) AUTHOR
-// Licence:    wxWindows licence
+// Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__
 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__
@@ -19,6 +19,7 @@
 #include "wx/panel.h"
 #include "wx/app.h"
 #include "wx/dc.h"
 #include "wx/panel.h"
 #include "wx/app.h"
 #include "wx/dc.h"
+#include "wx/dcclient.h"
 #include "wx/notebook.h"
 #include "wx/tabctrl.h"
 #include "wx/radiobox.h"
 #include "wx/notebook.h"
 #include "wx/tabctrl.h"
 #include "wx/radiobox.h"
@@ -34,13 +35,14 @@
 IMPLEMENT_ABSTRACT_CLASS(wxControl, wxWindow)
 
 BEGIN_EVENT_TABLE(wxControl, wxWindow)
 IMPLEMENT_ABSTRACT_CLASS(wxControl, wxWindow)
 
 BEGIN_EVENT_TABLE(wxControl, wxWindow)
-       EVT_MOUSE_EVENTS( wxControl::OnMouseEvent ) 
-       EVT_CHAR( wxControl::OnKeyDown ) 
-       EVT_PAINT( wxControl::OnPaint ) 
+    EVT_MOUSE_EVENTS( wxControl::OnMouseEvent ) 
+//    EVT_CHAR( wxControl::OnKeyDown ) 
+    EVT_PAINT( wxControl::OnPaint ) 
 END_EVENT_TABLE()
 #endif
 
 #include "wx/mac/uma.h"
 END_EVENT_TABLE()
 #endif
 
 #include "wx/mac/uma.h"
+#include "wx/mac/private.h"
 
 // Item members
 
 
 // Item members
 
@@ -49,35 +51,63 @@ ControlActionUPP wxMacLiveScrollbarActionUPP = NULL ;
 pascal void wxMacLiveScrollbarActionProc( ControlHandle control , ControlPartCode partCode ) ;
 pascal void wxMacLiveScrollbarActionProc( ControlHandle control , ControlPartCode partCode )
 {
 pascal void wxMacLiveScrollbarActionProc( ControlHandle control , ControlPartCode partCode ) ;
 pascal void wxMacLiveScrollbarActionProc( ControlHandle control , ControlPartCode partCode )
 {
-       if ( partCode != 0)
+    if ( partCode != 0)
+    {
+        wxControl*  wx = (wxControl*) GetControlReference( control ) ;
+        if ( wx )
+        {
+            wx->MacHandleControlClick( control , partCode ) ;
+        }
+    }
+}
+
+ControlColorUPP wxMacSetupControlBackgroundUPP = NULL ;
+
+pascal OSStatus wxMacSetupControlBackground( ControlRef iControl , SInt16 iMessage , SInt16 iDepth , Boolean iIsColor )
+{
+       OSStatus status = noErr ;
+       switch( iMessage )
        {
        {
-               wxControl*      wx = (wxControl*) GetControlReference( control ) ;
-               if ( wx )
-               {
-                       wx->MacHandleControlClick( control , partCode ) ;
-               }
+               case kControlMsgSetUpBackground :
+                       {
+                       wxControl*  wx = (wxControl*) GetControlReference( iControl ) ;
+                       if ( wx != NULL && wx->IsKindOf( CLASSINFO( wxControl ) ) )
+                       {
+                               wxDC::MacSetupBackgroundForCurrentPort( wx->MacGetBackgroundBrush() ) ;
+       //                              SetThemeBackground( iDepth , iIsColor ) ;
+                               }
+                               else
+                               {
+                                       status = paramErr ;
+                               }
+                       }
+                       break ;
+               default :
+                       status = paramErr ;
+                       break ;
        }
        }
+       return status ;
 }
 }
-
+ 
 wxControl::wxControl()
 {
 wxControl::wxControl()
 {
-       m_macControl = NULL ;
-       m_macHorizontalBorder = 0 ; // additional pixels around the real control
-       m_macVerticalBorder = 0 ;
+    m_macControl = NULL ;
+    m_macHorizontalBorder = 0 ; // additional pixels around the real control
+    m_macVerticalBorder = 0 ;
     m_backgroundColour = *wxWHITE;
     m_foregroundColour = *wxBLACK;
 #if WXWIN_COMPATIBILITY
   m_callback = 0;
 #endif // WXWIN_COMPATIBILITY
 
     m_backgroundColour = *wxWHITE;
     m_foregroundColour = *wxBLACK;
 #if WXWIN_COMPATIBILITY
   m_callback = 0;
 #endif // WXWIN_COMPATIBILITY
 
-       if ( wxMacLiveScrollbarActionUPP == NULL )
-       {
+    if ( wxMacLiveScrollbarActionUPP == NULL )
+    {
 #if defined(UNIVERSAL_INTERFACES_VERSION) && (UNIVERSAL_INTERFACES_VERSION >= 0x0340)
         wxMacLiveScrollbarActionUPP = NewControlActionUPP( wxMacLiveScrollbarActionProc );
 #else
 #if defined(UNIVERSAL_INTERFACES_VERSION) && (UNIVERSAL_INTERFACES_VERSION >= 0x0340)
         wxMacLiveScrollbarActionUPP = NewControlActionUPP( wxMacLiveScrollbarActionProc );
 #else
-               wxMacLiveScrollbarActionUPP = NewControlActionProc( wxMacLiveScrollbarActionProc ) ;
+        wxMacLiveScrollbarActionUPP = NewControlActionProc( wxMacLiveScrollbarActionProc ) ;
 #endif
 #endif
-       }
+    }
 }
 
 bool wxControl::Create(wxWindow *parent, wxWindowID id,
 }
 
 bool wxControl::Create(wxWindow *parent, wxWindowID id,
@@ -86,10 +116,16 @@ bool wxControl::Create(wxWindow *parent, wxWindowID id,
                        const wxValidator& validator,
                        const wxString& name)
 {
                        const wxValidator& validator,
                        const wxString& name)
 {
-       m_macControl = NULL ;
-       m_macHorizontalBorder = 0 ; // additional pixels around the real control
-       m_macVerticalBorder = 0 ;
+    m_macControl = NULL ;
+    m_macHorizontalBorder = 0 ; // additional pixels around the real control
+    m_macVerticalBorder = 0 ;
+    
     bool rval = wxWindow::Create(parent, id, pos, size, style, name);
     bool rval = wxWindow::Create(parent, id, pos, size, style, name);
+    if ( parent )
+    {
+        m_backgroundColour = parent->GetBackgroundColour() ;
+        m_foregroundColour = parent->GetForegroundColour() ;
+    }
     if (rval) {
 #if wxUSE_VALIDATORS
         SetValidator(validator);
     if (rval) {
 #if wxUSE_VALIDATORS
         SetValidator(validator);
@@ -103,89 +139,89 @@ wxControl::~wxControl()
     m_isBeingDeleted = TRUE;
     // If we delete an item, we should initialize the parent panel,
     // because it could now be invalid.
     m_isBeingDeleted = TRUE;
     // If we delete an item, we should initialize the parent panel,
     // because it could now be invalid.
-    wxPanel *panel = wxDynamicCast(GetParent(), wxPanel);
-    if ( panel )
+    wxWindow *parent = GetParent() ;
+    if ( parent )
     {
     {
-        if (panel->GetDefaultItem() == (wxButton*) this)
-            panel->SetDefaultItem(NULL);
+        if (parent->GetDefaultItem() == (wxButton*) this)
+            parent->SetDefaultItem(NULL);
     }
     }
-    if ( m_macControl )
+    if ( (ControlHandle) m_macControl )
     {
     {
-       ::DisposeControl( m_macControl ) ;
-       m_macControl = NULL ;
+        ::DisposeControl( (ControlHandle) m_macControl ) ;
+        m_macControl = NULL ;
     }
 }
 
 void wxControl::SetLabel(const wxString& title)
 {
     }
 }
 
 void wxControl::SetLabel(const wxString& title)
 {
-       m_label = title ;
+    m_label = wxStripMenuCodes(title) ;
 
 
-       if ( m_macControl )
-       {
-               Str255 maclabel ;
-               wxString label ;
-       
-               if( wxApp::s_macDefaultEncodingIsPC )
-                       label = wxMacMakeMacStringFromPC( title ) ;
-               else
-                       label = title ;
-               
+    if ( (ControlHandle) m_macControl )
+    {
+        Str255 maclabel ;
+        wxString label ;
+    
+        if( wxApp::s_macDefaultEncodingIsPC )
+            label = wxMacMakeMacStringFromPC( m_label ) ;
+        else
+            label = m_label ;
+        
 #if TARGET_CARBON
 #if TARGET_CARBON
-               c2pstrcpy( (StringPtr) maclabel , label ) ;
+        c2pstrcpy( (StringPtr) maclabel , label ) ;
 #else
 #else
-               strcpy( (char *) maclabel , label ) ;
-               c2pstr( (char *) maclabel ) ;
+        strcpy( (char *) maclabel , label ) ;
+        c2pstr( (char *) maclabel ) ;
 #endif
 #endif
-               ::SetControlTitle( m_macControl , maclabel ) ;
-       }
-       Refresh() ;
+        ::SetControlTitle( (ControlHandle) m_macControl , maclabel ) ;
+    }
+    Refresh() ;
 }
 
 wxSize wxControl::DoGetBestSize() const
 {
 }
 
 wxSize wxControl::DoGetBestSize() const
 {
-       Rect    bestsize = { 0 , 0 , 0 , 0 } ;
-       short   baselineoffset ;
-       int bestWidth, bestHeight ;
-       ::GetBestControlRect( m_macControl , &bestsize , &baselineoffset ) ;
+    Rect    bestsize = { 0 , 0 , 0 , 0 } ;
+    short   baselineoffset ;
+    int bestWidth, bestHeight ;
+    ::GetBestControlRect( (ControlHandle) 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 ( 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 ;
+    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 = bestsize.bottom - bestsize.top ;
+    if ( bestHeight < 10 )
+        bestHeight = 13 ;
 
 
-       bestHeight += 2 * m_macVerticalBorder;
+    bestHeight += 2 * m_macVerticalBorder;
 
 
-               
+        
     return wxSize(bestWidth, bestHeight);
 }
 
     return wxSize(bestWidth, bestHeight);
 }
 
@@ -235,15 +271,15 @@ void wxRemoveMacControlAssociation(wxControl *control)
 }
 
 void wxControl::MacPreControlCreate( wxWindow *parent, wxWindowID id, wxString label , 
 }
 
 void wxControl::MacPreControlCreate( wxWindow *parent, wxWindowID id, wxString label , 
-                        const wxPoint& pos,
-                        const wxSize& size, long style,
-                        const wxValidator& validator,
-                        const wxString& name , Rect *outBounds , StringPtr maclabel ) 
+             const wxPoint& pos,
+             const wxSize& size, long style,
+             const wxValidator& validator,
+             const wxString& name , WXRECTPTR outBounds , unsigned char* maclabel ) 
 {
 {
-       m_label = label ;
+    m_label = label ;
   SetName(name);
   if ( &validator )
   SetName(name);
   if ( &validator )
-       SetValidator(validator);
+    SetValidator(validator);
 
   m_windowStyle = style;
   parent->AddChild(this);
 
   m_windowStyle = style;
   parent->AddChild(this);
@@ -258,62 +294,67 @@ void wxControl::MacPreControlCreate( wxWindow *parent, wxWindowID id, wxString l
 
    // These sizes will be adjusted in MacPostControlCreate
 
 
    // These sizes will be adjusted in MacPostControlCreate
 
-       m_width = size.x ;
-       m_height = size.y ;
-       m_x = pos.x ;
-       m_y = pos.y ;
-               
-       outBounds->top = -10;
-       outBounds->left = -10;
-       outBounds->bottom = 0;
-       outBounds->right = 0;
-
-       char c_text[255];
-       strcpy( c_text , label ) ;
-       if( wxApp::s_macDefaultEncodingIsPC )
-       {
-               wxMacConvertFromPCForControls( c_text ) ;
-       }
+    m_width = size.x ;
+    m_height = size.y ;
+    m_x = pos.x ;
+    m_y = pos.y ;
+        
+    ((Rect*)outBounds)->top = -10;
+    ((Rect*)outBounds)->left = -10;
+    ((Rect*)outBounds)->bottom = 0;
+    ((Rect*)outBounds)->right = 0;
+
+    char c_text[255];
+    strcpy( c_text , label ) ;
+    if( wxApp::s_macDefaultEncodingIsPC )
+    {
+        wxMacConvertFromPCForControls( c_text ) ;
+    }
 
 #if TARGET_CARBON
 
 #if TARGET_CARBON
-       c2pstrcpy( (StringPtr) maclabel , c_text ) ;
+    c2pstrcpy( (StringPtr) maclabel , c_text ) ;
 #else
 #else
-       strcpy( (char *) maclabel , c_text ) ;
-       c2pstr( (char *) maclabel ) ;
+    strcpy( (char *) maclabel , c_text ) ;
+    c2pstr( (char *) maclabel ) ;
 #endif
 }
 
 void wxControl::MacPostControlCreate()
 {
 #endif
 }
 
 void wxControl::MacPostControlCreate()
 {
-       wxASSERT_MSG( m_macControl != NULL , "No valid mac control" ) ;
-       
-       if ( IsKindOf( CLASSINFO( wxScrollBar ) ) )
-       {
-               // no font
-       }
-       else if ( IsKindOf( CLASSINFO( wxStaticBox ) ) || IsKindOf( CLASSINFO( wxRadioBox ) ) || IsKindOf( CLASSINFO( wxButton ) ) )
-       {
-               ControlFontStyleRec             controlstyle ;
-               controlstyle.flags = kControlUseFontMask ;
-               controlstyle.font = kControlFontSmallBoldSystemFont ;
-               
-               ::SetControlFontStyle( m_macControl , &controlstyle ) ;
-       }
-       else
-       {
-               ControlFontStyleRec             controlstyle ;
-               controlstyle.flags = kControlUseFontMask ;
-               controlstyle.font = kControlFontSmallSystemFont ;
-               
-               ::SetControlFontStyle( m_macControl , &controlstyle ) ;
-       }
-       ControlHandle container = GetParent()->MacGetContainerForEmbedding() ;
-       wxASSERT_MSG( container != NULL , wxT("No valid mac container control") ) ;
-       ::EmbedControl( m_macControl , container ) ;
-       m_macControlIsShown  = true ;
+    wxASSERT_MSG( (ControlHandle) m_macControl != NULL , "No valid mac control" ) ;
+    
+    if ( IsKindOf( CLASSINFO( wxScrollBar ) ) )
+    {
+        // no font
+    }
+    else if ( !UMAHasAquaLayout() && (IsKindOf( CLASSINFO( wxStaticBox ) ) || IsKindOf( CLASSINFO( wxRadioBox ) ) || IsKindOf( CLASSINFO( wxButton ) ) ) )
+    {
+        ControlFontStyleRec     controlstyle ;
+        controlstyle.flags = kControlUseFontMask ;
+        controlstyle.font = kControlFontSmallBoldSystemFont ;
+        
+        ::SetControlFontStyle( (ControlHandle) m_macControl , &controlstyle ) ;
+    }
+    else
+    {
+        ControlFontStyleRec     controlstyle ;
+        controlstyle.flags = kControlUseFontMask ;
+        controlstyle.font = kControlFontSmallSystemFont ;
+        
+        ::SetControlFontStyle( (ControlHandle) m_macControl , &controlstyle ) ;
+    }
+    ControlHandle container = (ControlHandle) GetParent()->MacGetContainerForEmbedding() ;
+    wxASSERT_MSG( container != NULL , wxT("No valid mac container control") ) ;
+    ::EmbedControl( (ControlHandle) m_macControl , container ) ;
+    m_macControlIsShown  = true ;
 
 
-       wxAssociateControlWithMacControl( m_macControl , this ) ;
+    wxAssociateControlWithMacControl( (ControlHandle) m_macControl , this ) ;
 
 
+       if ( wxMacSetupControlBackgroundUPP == NULL )
+       {
+               wxMacSetupControlBackgroundUPP = NewControlColorUPP( wxMacSetupControlBackground ) ;
+       }
+       SetControlColorProc( (ControlHandle) m_macControl , wxMacSetupControlBackgroundUPP ) ;
  
      // Adjust the controls size and position
      wxPoint pos(m_x, m_y);
  
      // Adjust the controls size and position
      wxPoint pos(m_x, m_y);
@@ -331,187 +372,166 @@ void wxControl::MacPostControlCreate()
  
      SetSize(pos.x, pos.y, new_size.x, new_size.y);
  
  
      SetSize(pos.x, pos.y, new_size.x, new_size.y);
  
-     UMAShowControl( m_macControl ) ;
+     UMAShowControl( (ControlHandle) m_macControl ) ;
+     Refresh() ;
 }
 
 void wxControl::MacAdjustControlRect() 
 {
 }
 
 void wxControl::MacAdjustControlRect() 
 {
-       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 ;
-               
-               ::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 ( m_width == -1 )
-               {
-                       if ( IsKindOf( CLASSINFO( wxButton ) ) )
-                       {
-                               m_width = m_label.Length() * 8 + 12 ;
-                               if ( m_width < 70 )
-                                 m_width = 70 ;
-                       }
-                       else if ( IsKindOf( CLASSINFO( wxStaticText ) ) )
-                       {
-                               m_width = m_label.Length() * 8 ;
-                       }
-                       else
-                               m_width = bestsize.right - bestsize.left ;
-                       
-                       m_width += 2 * m_macHorizontalBorder ;
-               }
-               if ( m_height == -1 )
-               {
-                       m_height = bestsize.bottom - bestsize.top ;
-                       if ( m_height < 10 )
-                               m_height = 13 ;
-
-                       m_height += 2 * m_macVerticalBorder;
-               }
-               
-               wxMacDrawingHelper helper ( wxFindWinFromMacWindow( GetMacRootWindow() ) ) ;
-               if ( helper.Ok() )
-               {
-               UMASizeControl( m_macControl , m_width - 2 * m_macHorizontalBorder, m_height -  2 * m_macVerticalBorder ) ;
-               }
-       }
+    wxASSERT_MSG( (ControlHandle) m_macControl != NULL , wxT("No valid mac control") ) ;
+    if ( m_width == -1 || m_height == -1 )
+    {
+        Rect    bestsize = { 0 , 0 , 0 , 0 } ;
+        short   baselineoffset ;
+        
+        ::GetBestControlRect( (ControlHandle) 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 ( m_width == -1 )
+        {
+            if ( IsKindOf( CLASSINFO( wxButton ) ) )
+            {
+                m_width = m_label.Length() * 8 + 12 ;
+                if ( m_width < 70 )
+                  m_width = 70 ;
+            }
+            else if ( IsKindOf( CLASSINFO( wxStaticText ) ) )
+            {
+                m_width = m_label.Length() * 8 ;
+            }
+            else
+                m_width = bestsize.right - bestsize.left ;
+            
+            m_width += 2 * m_macHorizontalBorder + MacGetLeftBorderSize() + MacGetRightBorderSize() ;
+        }
+        if ( m_height == -1 )
+        {
+            m_height = bestsize.bottom - bestsize.top ;
+            if ( m_height < 10 )
+                m_height = 13 ;
+
+            m_height += 2 * m_macVerticalBorder + MacGetTopBorderSize() + MacGetBottomBorderSize() ;
+        }
+               MacUpdateDimensions() ;      
+//        UMASizeControl( (ControlHandle) m_macControl , m_width - 2 * m_macHorizontalBorder, m_height -  2 * m_macVerticalBorder ) ;
+    }
 }
 }
-ControlHandle wxControl::MacGetContainerForEmbedding() 
+
+WXWidget wxControl::MacGetContainerForEmbedding() 
 {
 {
-       if ( m_macControl )
-               return m_macControl ;
+    if ( m_macControl )
+        return m_macControl ;
 
 
-       return wxWindow::MacGetContainerForEmbedding() ;
+    return wxWindow::MacGetContainerForEmbedding() ;
 }
 
 }
 
-void wxControl::MacSuperChangedPosition() 
+void wxControl::MacUpdateDimensions() 
 {
 {
-       if ( m_macControl )
-       {
-               Rect contrlRect ;               
-               GetControlBounds( m_macControl , &contrlRect ) ; 
-               int former_mac_x = contrlRect.left ;
-               int former_mac_y = contrlRect.top ;
-               int mac_x = m_x ;
-               int mac_y = m_y ;
-               GetParent()->MacClientToRootWindow( & mac_x , & mac_y ) ;
-               
-               WindowRef rootwindow = GetMacRootWindow() ;
-               wxWindow* wxrootwindow = wxFindWinFromMacWindow( rootwindow ) ;
-               ::SetThemeWindowBackground( rootwindow , kThemeBrushDialogBackgroundActive , false ) ;
-               wxMacDrawingHelper focus( wxrootwindow ) ;
+       // actually in the current systems this should never be possible, but later reparenting
+       // may become a reality
        
        
-               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 ) ;
-                       }
-                       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 ) ;
-                       }
-               }
-               if ( wxrootwindow->IsKindOf( CLASSINFO( wxDialog ) ) )
+       if ( (ControlHandle) m_macControl == NULL )
+               return ;
+               
+       if ( GetParent() == NULL )
+               return ;
+               
+    WindowRef rootwindow = (WindowRef) MacGetRootWindow() ;
+    if ( rootwindow == NULL )
+       return ;
+       
+    Rect oldBounds ;       
+    GetControlBounds( (ControlHandle) m_macControl , &oldBounds ) ; 
+    
+    int new_x = m_x + MacGetLeftBorderSize() + m_macHorizontalBorder ;
+    int new_y = m_y + MacGetTopBorderSize() + m_macVerticalBorder ;
+    int new_width = m_width - MacGetLeftBorderSize() - MacGetRightBorderSize() - 2 * m_macHorizontalBorder ;
+    int new_height = m_height - MacGetTopBorderSize() - MacGetBottomBorderSize() - 2 * m_macVerticalBorder ;
+    
+    GetParent()->MacWindowToRootWindow( & new_x , & new_y ) ;
+    bool doMove = new_x != oldBounds.left || new_y != oldBounds.top ;
+    bool doResize =  ( oldBounds.right - oldBounds.left ) != new_width || (oldBounds.bottom - oldBounds.top ) != new_height ;
+       if ( doMove || doResize )
+       {
+               InvalWindowRect( rootwindow, &oldBounds ) ;
+               if ( doMove )
                {
                {
+                       UMAMoveControl( (ControlHandle) m_macControl , new_x , new_y ) ;
                }
                }
-               else
+               if ( doResize )
                {
                {
-                       ::SetThemeWindowBackground( rootwindow , kThemeBrushDocumentWindowBackground , false ) ;
+                       UMASizeControl( (ControlHandle) m_macControl , new_width , new_height ) ;
                }
        }
                }
        }
+}
 
 
-       wxWindow::MacSuperChangedPosition() ;
+void wxControl::MacSuperChangedPosition() 
+{
+       MacUpdateDimensions() ;
+    wxWindow::MacSuperChangedPosition() ;
 }
 
 void wxControl::MacSuperEnabled( bool enabled ) 
 {
 }
 
 void wxControl::MacSuperEnabled( bool enabled ) 
 {
-/*
-       if ( m_macControl )
-       {
-               if ( UMAHasAppearance() )
-               {
-                       if ( !enabled )
-                       {
-                               ::DeactivateControl( m_macControl ) ; 
-                       }
-                       else
-                       {
-                               if ( m_macEnabled )
-                                       ::ActivateControl( m_macControl ) ;
-                       }
-               }
-               else
-               {
-                       if ( !enabled )
-                       {
-                               ::HiliteControl( m_macControl , 255 ) ;
-                       }
-                       else
-                       {
-                               if ( m_macEnabled )
-                                       ::HiliteControl( m_macControl , 0 ) ;
-                       }
-               }
-       }
-       wxWindow::MacSuperEnabled( enabled ) ;
-*/
+    Refresh(FALSE) ;
+    wxWindow::MacSuperEnabled( enabled ) ;
 }
 
 void  wxControl::MacSuperShown( bool show ) 
 {
 }
 
 void  wxControl::MacSuperShown( bool show ) 
 {
-       if ( m_macControl )
-       {
-               if ( !show )
-               {
-                       if ( m_macControlIsShown )
-                       {
-                               ::HideControl( m_macControl ) ;
-                               m_macControlIsShown = false ;
-                       }
-               }
-               else
-               {
-                       if ( MacIsReallyShown() && !m_macControlIsShown )
-                       {
-                               ::UMAShowControl( m_macControl ) ;
-                               m_macControlIsShown = true ;
-                       }
-               }
-       }
-               
-       wxWindow::MacSuperShown( show ) ;
+    if ( (ControlHandle) m_macControl )
+    {
+        if ( !show )
+        {
+            if ( m_macControlIsShown )
+            {
+                ::UMAHideControl( (ControlHandle) m_macControl ) ;
+                m_macControlIsShown = false ;
+            }
+        }
+        else
+        {
+            if ( MacIsReallyShown() && !m_macControlIsShown )
+            {
+                ::UMAShowControl( (ControlHandle) m_macControl ) ;
+                m_macControlIsShown = true ;
+            }
+        }
+    }
+        
+    wxWindow::MacSuperShown( show ) ;
 }
 
 void  wxControl::DoSetSize(int x, int y,
             int width, int height,
             int sizeFlags )
 {
 }
 
 void  wxControl::DoSetSize(int x, int y,
             int width, int height,
             int sizeFlags )
 {
-       if ( m_macControl == NULL )
-       {
-               wxWindow::DoSetSize( x , y ,width , height ,sizeFlags ) ;
-               return ;
-       }
+    wxWindow::DoSetSize( x , y ,width , height ,sizeFlags ) ;
+    return ;
+/*
 
 
-     Rect oldbounds, newbounds;
+    if ( (ControlHandle) m_macControl == NULL )
+    {
+        wxWindow::DoSetSize( x , y ,width , height ,sizeFlags ) ;
+        return ;
+    }
+
+     Rect oldbounds;
      int new_x, new_y, new_width, new_height;
      int mac_x, mac_y;
  
      int new_x, new_y, new_width, new_height;
      int mac_x, mac_y;
  
@@ -552,9 +572,9 @@ void  wxControl::DoSetSize(int x, int y,
      mac_x = new_x;
      mac_y = new_y;
      if(GetParent()) {
      mac_x = new_x;
      mac_y = new_y;
      if(GetParent()) {
-         GetParent()->MacClientToRootWindow(&mac_x, &mac_y);
+         GetParent()->MacWindowToRootWindow(&mac_x, &mac_y);
      }
      }
-     GetControlBounds(m_macControl, &oldbounds);
+     GetControlBounds( (ControlHandle) m_macControl, &oldbounds);
      oldbounds.right = oldbounds.left + m_width;
      oldbounds.bottom = oldbounds.top + m_height;
  
      oldbounds.right = oldbounds.left + m_width;
      oldbounds.bottom = oldbounds.top + m_height;
  
@@ -574,6 +594,8 @@ void  wxControl::DoSetSize(int x, int y,
  
      if ( doMove || doResize )
      {
  
      if ( doMove || doResize )
      {
+         Refresh() ;
+     
          // Ensure resize is within constraints
          if ((m_minWidth != -1) && (new_width < m_minWidth)) {
              new_width = m_minWidth;
          // Ensure resize is within constraints
          if ((m_minWidth != -1) && (new_width < m_minWidth)) {
              new_width = m_minWidth;
@@ -593,9 +615,9 @@ void  wxControl::DoSetSize(int x, int y,
              m_x = new_x;
              m_y = new_y;
  
              m_x = new_x;
              m_y = new_y;
  
-             UMAMoveControl(m_macControl,
+             UMAMoveControl( (ControlHandle) m_macControl,
                             mac_x + m_macHorizontalBorder, mac_y + m_macVerticalBorder);
                             mac_x + m_macHorizontalBorder, mac_y + m_macVerticalBorder);
- 
+
              wxMoveEvent event(wxPoint(m_x, m_y), m_windowId);
              event.SetEventObject(this);
              GetEventHandler()->ProcessEvent(event) ;
              wxMoveEvent event(wxPoint(m_x, m_y), m_windowId);
              event.SetEventObject(this);
              GetEventHandler()->ProcessEvent(event) ;
@@ -605,60 +627,46 @@ void  wxControl::DoSetSize(int x, int y,
              m_width = new_width;
              m_height = new_height;
  
              m_width = new_width;
              m_height = new_height;
  
-             UMASizeControl( m_macControl,
+             UMASizeControl( (ControlHandle) m_macControl,
                              m_width - 2 * m_macHorizontalBorder,
                              m_height -  2 * m_macVerticalBorder ) ;
  
                              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);
          }
  
              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 );
-         }
+          Refresh() ;
      }
      }
+*/
 }
 
 bool  wxControl::Show(bool show) 
 {
 }
 
 bool  wxControl::Show(bool show) 
 {
-       if ( !wxWindow::Show( show ) )
-               return FALSE ;
-               
-       if ( m_macControl )
-       {
-               if ( !show )
-               {
-                       if ( m_macControlIsShown )
-                       {
-                               ::HideControl( m_macControl ) ;
-                               m_macControlIsShown = false ;
-                       }
-               }
-               else
-               {
-                       if ( MacIsReallyShown() && !m_macControlIsShown )
-                       {
-                               ::UMAShowControl( m_macControl ) ;
-                               m_macControlIsShown = true ;
-                       }
-               }
-       }
-       return TRUE ;
+    if ( !wxWindow::Show( show ) )
+        return FALSE ;
+        
+    if ( (ControlHandle) m_macControl )
+    {
+        if ( !show )
+        {
+            if ( m_macControlIsShown )
+            {
+                ::UMAHideControl( (ControlHandle) m_macControl ) ;
+                m_macControlIsShown = false ;
+            }
+        }
+        else
+        {
+            if ( MacIsReallyShown() && !m_macControlIsShown )
+            {
+                ::UMAShowControl( (ControlHandle) m_macControl ) ;
+                m_macControlIsShown = true ;
+            }
+        }
+    }
+    return TRUE ;
 }
 
 bool  wxControl::Enable(bool enable) 
 }
 
 bool  wxControl::Enable(bool enable) 
@@ -666,217 +674,192 @@ bool  wxControl::Enable(bool enable)
     if ( !wxWindow::Enable(enable) )
         return FALSE;
 
     if ( !wxWindow::Enable(enable) )
         return FALSE;
 
-       if ( m_macControl )
-       {
-               if ( enable )
-                   UMAActivateControl( m_macControl ) ;
-               else
-                   UMADeactivateControl( m_macControl ) ;
-       }
-       return TRUE ;
+    if ( (ControlHandle) m_macControl )
+    {
+        if ( enable )
+            UMAActivateControl( (ControlHandle) m_macControl ) ;
+        else
+            UMADeactivateControl( (ControlHandle) m_macControl ) ;
+    }
+    return TRUE ;
 }
 
 void wxControl::Refresh(bool eraseBack, const wxRect *rect)
 {
 }
 
 void wxControl::Refresh(bool eraseBack, const wxRect *rect)
 {
-  if ( m_macControl )
-  {
-       wxWindow::Refresh( eraseBack , rect ) ;
-       }
-  else
-  {
-       wxWindow::Refresh( eraseBack , rect ) ;
-  }
+    wxWindow::Refresh( eraseBack , rect ) ;
 }
 
 void wxControl::MacRedrawControl()
 {
 }
 
 void wxControl::MacRedrawControl()
 {
-       if ( m_macControl )
-       {
-               WindowRef window = GetMacRootWindow() ;
-               if ( window )
-               {
-                       wxWindow* win = wxFindWinFromMacWindow( window ) ;
-                       if ( win )
-                       {
-                               wxMacDrawingHelper help( win ) ;
-                               // the mac control manager always assumes to have the origin at 0,0
-                               SetOrigin( 0 , 0 ) ;
-                               
-                               wxWindow* parent = GetParent() ;
-                               while ( parent )
-                               {
-                                       if( parent->MacGetWindowData() )
-                                       {
-                                               ::SetThemeWindowBackground( win->MacGetWindowData()->m_macWindow , kThemeBrushDialogBackgroundActive , false ) ;
-                                               break ;
-                                       }
-                                       
-                                       if( parent->IsKindOf( CLASSINFO( wxNotebook ) ) ||  parent->IsKindOf( CLASSINFO( wxTabCtrl ) ))
-                                       {
-                                               if ( ((wxControl*)parent)->m_macControl )
-                                                       SetUpControlBackground( ((wxControl*)parent)->m_macControl , -1 , true ) ;
-                                               break ;
-                                       }
-                                       
-                                       parent = parent->GetParent() ;
-                               } 
-                               
-                               UMADrawControl( m_macControl ) ;
-                               ::SetThemeWindowBackground( win->MacGetWindowData()->m_macWindow , win->MacGetWindowData()->m_macWindowBackgroundTheme , false ) ;
-                       }
-               }
-       }
+    if ( (ControlHandle) m_macControl && MacGetRootWindow() )
+    {
+        wxClientDC dc(this) ;
+        wxMacPortSetter helper(&dc) ;
+/*
+        Rect r = { 0 , 0 , 32000 , 32000 } ;
+        ClipRect( &r ) ;
+*/
+        int x = 0 , y = 0;
+        wxWindow *parent = GetParent() ;
+        parent->MacWindowToRootWindow( &x,&y ) ;
+        RgnHandle clrgn = NewRgn() ;
+        RgnHandle insidergn = NewRgn() ;
+        wxSize size = parent->GetSize() ;
+        SetRectRgn( insidergn , parent->MacGetLeftBorderSize() , parent->MacGetTopBorderSize() , 
+         size.x - parent->MacGetLeftBorderSize() - parent->MacGetRightBorderSize(), 
+         size.y - parent->MacGetTopBorderSize() - parent->MacGetBottomBorderSize()) ;
+
+        CopyRgn( (RgnHandle) parent->MacGetVisibleRegion(false).GetWXHRGN() , clrgn ) ;
+               SectRgn( clrgn , insidergn , clrgn ) ;
+        OffsetRgn( clrgn , x , y ) ;
+        SetClip( clrgn ) ;
+               DisposeRgn( clrgn ) ;
+               DisposeRgn( insidergn ) ;
+
+        wxDC::MacSetupBackgroundForCurrentPort( MacGetBackgroundBrush() ) ;
+        UMADrawControl( (ControlHandle) m_macControl ) ;
+    }
 }
 
 void wxControl::OnPaint(wxPaintEvent& event)
 {
 }
 
 void wxControl::OnPaint(wxPaintEvent& event)
 {
-       if ( m_macControl )
-       {
-               WindowRef window = GetMacRootWindow() ;
-               if ( window )
-               {
-                       wxWindow* win = wxFindWinFromMacWindow( window ) ;
-                       if ( win )
-                       {
-                               wxMacDrawingHelper help( win ) ;
-                               // the mac control manager always assumes to have the origin at 0,0
-                               SetOrigin( 0 , 0 ) ;
+    if ( (ControlHandle) m_macControl )
+    {
+        wxPaintDC dc(this) ;
+        wxMacPortSetter helper(&dc) ;
+/*
+        Rect r = { 0 , 0 , 32000 , 32000 } ;
+        ClipRect( &r ) ;
+*/
 
 
-                               wxWindow* parent = GetParent() ;
-                               while ( parent )
-                               {
-                                       if( parent->MacGetWindowData() )
-                                       {
-                                               ::SetThemeWindowBackground( win->MacGetWindowData()->m_macWindow , kThemeBrushDialogBackgroundActive , false ) ;
-                                               break ;
-                                       }
-                                       
-                                       if( parent->IsKindOf( CLASSINFO( wxNotebook ) ) ||  parent->IsKindOf( CLASSINFO( wxTabCtrl ) ))
-                                       {
-                                               if ( ((wxControl*)parent)->m_macControl )
-                                                       SetUpControlBackground( ((wxControl*)parent)->m_macControl , -1 , true ) ;
-                                               break ;
-                                       }
-                                       
-                                       parent = parent->GetParent() ;
-                               } 
-
-                               UMADrawControl( m_macControl ) ;
-                               ::SetThemeWindowBackground( win->MacGetWindowData()->m_macWindow , win->MacGetWindowData()->m_macWindowBackgroundTheme , false ) ;
-                       }
-               }
-       }
-       else
-       {
-               // wxWindow::OnPaint( event ) ;
-       }
+        int x = 0 , y = 0;
+        wxWindow *parent = GetParent() ;
+        parent->MacWindowToRootWindow( &x,&y ) ;
+        RgnHandle clrgn = NewRgn() ;
+        RgnHandle insidergn = NewRgn() ;
+        wxSize size = parent->GetSize() ;
+        SetRectRgn( insidergn , parent->MacGetLeftBorderSize() , parent->MacGetTopBorderSize() , 
+         size.x - parent->MacGetLeftBorderSize() - parent->MacGetRightBorderSize(), 
+         size.y - parent->MacGetTopBorderSize() - parent->MacGetBottomBorderSize()) ;
+
+        CopyRgn( (RgnHandle) parent->MacGetVisibleRegion(false).GetWXHRGN() , clrgn ) ;
+               SectRgn( clrgn , insidergn , clrgn ) ;
+        OffsetRgn( clrgn , x , y ) ;
+        SetClip( clrgn ) ;
+               DisposeRgn( clrgn ) ;
+               DisposeRgn( insidergn ) ;
+
+        wxDC::MacSetupBackgroundForCurrentPort( MacGetBackgroundBrush() ) ;
+        UMADrawControl( (ControlHandle) m_macControl ) ;
+    }
+    else
+    {
+      event.Skip() ;
+    }
 }
 void wxControl::OnEraseBackground(wxEraseEvent& event)
 {
 }
 void wxControl::OnEraseBackground(wxEraseEvent& event)
 {
-    // In general, you don't want to erase the background of a control,
-    // or you'll get a flicker.
-    // TODO: move this 'null' function into each control that
-    // might flicker.
+    wxWindow::OnEraseBackground( event ) ; 
 }
 
 
 void  wxControl::OnKeyDown( wxKeyEvent &event ) 
 {
 }
 
 
 void  wxControl::OnKeyDown( wxKeyEvent &event ) 
 {
-       if ( m_macControl == NULL )
-               return ;
-       
-       EventRecord *ev = wxTheApp->MacGetCurrentEvent() ;
-       short keycode ;
-       short keychar ;
-       keychar = short(ev->message & charCodeMask);
-       keycode = short(ev->message & keyCodeMask) >> 8 ;
-
-       ::HandleControlKey( m_macControl , keycode , keychar , ev->modifiers ) ;
+    if ( (ControlHandle) m_macControl == NULL )
+        return ;
+    
+    EventRecord *ev = (EventRecord*) wxTheApp->MacGetCurrentEvent() ;
+    short keycode ;
+    short keychar ;
+    keychar = short(ev->message & charCodeMask);
+    keycode = short(ev->message & keyCodeMask) >> 8 ;
+
+    ::HandleControlKey( (ControlHandle) m_macControl , keycode , keychar , ev->modifiers ) ;
 }
 
 void  wxControl::OnMouseEvent( wxMouseEvent &event ) 
 {
 }
 
 void  wxControl::OnMouseEvent( wxMouseEvent &event ) 
 {
-       if ( m_macControl == NULL )
-       {
-               event.Skip() ;
-               return ;
-       }
-               
-       if (event.GetEventType() == wxEVT_LEFT_DOWN || event.GetEventType() == wxEVT_LEFT_DCLICK )
-       {
-                       
-               int x = event.m_x ;
-               int y = event.m_y ;
-               
-               MacClientToRootWindow( &x , &y ) ;
-                       
-               ControlHandle   control ;
-               Point           localwhere ;
-               SInt16          controlpart ;
-               WindowRef       window = GetMacRootWindow() ;
-               
-               localwhere.h = x ;
-               localwhere.v = y ;
-       
-               short modifiers = 0;
-               
-               if ( !event.m_leftDown && !event.m_rightDown )
-                       modifiers  |= btnState ;
-       
-               if ( event.m_shiftDown )
-                       modifiers |= shiftKey ;
-                       
-               if ( event.m_controlDown )
-                       modifiers |= controlKey ;
-       
-               if ( event.m_altDown )
-                       modifiers |= optionKey ;
-       
-               if ( event.m_metaDown )
-                       modifiers |= cmdKey ;
+    if ( (ControlHandle) m_macControl == NULL )
+    {
+        event.Skip() ;
+        return ;
+    }
+        
+    if (event.GetEventType() == wxEVT_LEFT_DOWN || event.GetEventType() == wxEVT_LEFT_DCLICK )
+    {
+            
+        int x = event.m_x ;
+        int y = event.m_y ;
+        
+        MacClientToRootWindow( &x , &y ) ;
+            
+        ControlHandle   control ;
+        Point       localwhere ;
+        SInt16      controlpart ;
+        WindowRef   window = (WindowRef) MacGetRootWindow() ;
+        
+        localwhere.h = x ;
+        localwhere.v = y ;
+    
+        short modifiers = 0;
+        
+        if ( !event.m_leftDown && !event.m_rightDown )
+            modifiers  |= btnState ;
+    
+        if ( event.m_shiftDown )
+            modifiers |= shiftKey ;
+            
+        if ( event.m_controlDown )
+            modifiers |= controlKey ;
+    
+        if ( event.m_altDown )
+            modifiers |= optionKey ;
+    
+        if ( event.m_metaDown )
+            modifiers |= cmdKey ;
 /*
 #if TARGET_CARBON
         control = FindControlUnderMouse( localwhere , window , &controlpart ) ;
 #else
 /*
 #if TARGET_CARBON
         control = FindControlUnderMouse( localwhere , window , &controlpart ) ;
 #else
-               controlpart = FindControl( localwhere , window , &control ) ;
+        controlpart = FindControl( localwhere , window , &control ) ;
 #endif
 */
 #endif
 */
-               {
-               /*
-                       if ( AcceptsFocus() && FindFocus() != this )
-                       {
-                               SetFocus() ;
-                       }
-               */
-                   control = m_macControl ;
-                       if ( control && ::IsControlActive( control ) )
-                       {
-                               {
-                                       controlpart = ::HandleControlClick( control , localwhere , modifiers , (ControlActionUPP) -1 ) ;
-                                       wxTheApp->s_lastMouseDown = 0 ;
-                                       if ( control && controlpart != kControlNoPart &&
-                                           ! IsKindOf( CLASSINFO( wxScrollBar ) ) 
-                                       ) // otherwise we will get the event twice for scrollbar
-                                       {
-                                           MacHandleControlClick( control , controlpart ) ;
-                                       }
-                               }
-                       }
-               }
-       }
+        {
+        /*
+            if ( AcceptsFocus() && FindFocus() != this )
+            {
+                SetFocus() ;
+            }
+        */
+            control = (ControlHandle) m_macControl ;
+            if ( control && ::IsControlActive( control ) )
+            {
+                {
+                    controlpart = ::HandleControlClick( control , localwhere , modifiers , (ControlActionUPP) -1 ) ;
+                    wxTheApp->s_lastMouseDown = 0 ;
+                    if ( control && controlpart != kControlNoPart && 
+                        ! IsKindOf( CLASSINFO( wxScrollBar ) ) 
+                    ) // otherwise we will get the event twice for scrollbar
+                    {
+                        MacHandleControlClick( control , controlpart ) ;
+                    }
+                }
+            }
+        }
+    }
 }
 
 bool wxControl::MacCanFocus() const
 {
 }
 
 bool wxControl::MacCanFocus() const
 {
-               { if ( m_macControl == NULL ) 
-                               return true ; 
-                       else
-                       return false ; 
-               }
+        { if ( (ControlHandle) m_macControl == NULL ) 
+                return true ; 
+            else
+            return false ; 
+        }
 }
 
 }
 
-void wxControl::MacHandleControlClick( ControlHandle control , SInt16 controlpart ) 
+void wxControl::MacHandleControlClick( WXWidget control , wxInt16 controlpart ) 
 {
 {
-       wxASSERT_MSG( m_macControl != NULL , "No valid mac control" ) ;
+    wxASSERT_MSG( (ControlHandle) m_macControl != NULL , "No valid mac control" ) ;
 }
 
 }
 
wxWidgets (Impactor)