fix for bug reported by Robert and Julian, probably causes other problems

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

diff --git a/src/mac/carbon/stattext.cpp b/src/mac/carbon/stattext.cpp
index 86c4fd7158a7c81ada8f0736be3d8c47b64369fb..c31530e181e3256210fa313cd44f50917efceda1 100644 (file)
--- a/src/mac/carbon/stattext.cpp
+++ b/src/mac/carbon/stattext.cpp
@@ -6,7 +6,7 @@
 // Created:     04/01/98
 // RCS-ID:      $Id$
 // Copyright:   (c) AUTHOR
 // Created:     04/01/98
 // RCS-ID:      $Id$
 // Copyright:   (c) AUTHOR

-// Licence:    wxWindows licence
+// Licence:     wxWindows licence

 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__
 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__


@@ -19,6 +19,8 @@
 #include "wx/tabctrl.h"
 #include "wx/dc.h"
 #include "wx/dcclient.h"
 #include "wx/tabctrl.h"
 #include "wx/dc.h"
 #include "wx/dcclient.h"

+#include "wx/utils.h"
+#include "wx/settings.h"

 
 #include <stdio.h>
 
 
 #include <stdio.h>
 


@@ -26,7 +28,7 @@
 IMPLEMENT_DYNAMIC_CLASS(wxStaticText, wxControl)
 #endif
 
 IMPLEMENT_DYNAMIC_CLASS(wxStaticText, wxControl)
 #endif
 

-#include <wx/mac/uma.h>
+#include "wx/mac/uma.h"

 
 BEGIN_EVENT_TABLE(wxStaticText, wxStaticTextBase)
     EVT_PAINT(wxStaticText::OnPaint)
 
 BEGIN_EVENT_TABLE(wxStaticText, wxStaticTextBase)
     EVT_PAINT(wxStaticText::OnPaint)


@@ -44,144 +46,129 @@ bool wxStaticText::Create(wxWindow *parent, wxWindowID id,
     m_foregroundColour = parent->GetForegroundColour() ;
 
     if ( id == -1 )
     m_foregroundColour = parent->GetForegroundColour() ;
 
     if ( id == -1 )

-           m_windowId = (int)NewControlId();
+        m_windowId = (int)NewControlId();

     else
     else

-           m_windowId = id;
+        m_windowId = id;

 
     m_windowStyle = style;
     m_label = label ;
 
 
     m_windowStyle = style;
     m_label = label ;
 

-       bool ret = wxControl::Create( parent, id, pos, size, style , wxDefaultValidator , name );
-       SetBestSize( size ) ;
-    
+    bool ret = wxControl::Create( parent, id, pos, size, style , wxDefaultValidator , name );
+    SetBestSize( size ) ;
+

     return ret;
 }
 
 const wxString punct = " ,.-;:!?";
 
     return ret;
 }
 
 const wxString punct = " ,.-;:!?";
 

-void wxStaticText::DrawParagraph(wxDC &dc, wxString paragraph)
+void wxStaticText::DrawParagraph(wxDC &dc, wxString paragraph, int &y)

 {
   int x = 0 ;
 {
   int x = 0 ;

-  int y = 0 ;
+

   int i = 0 ;
   long width, height ;
   bool linedrawn = true;
   while( paragraph.Length() > 0 )
   {
   int i = 0 ;
   long width, height ;
   bool linedrawn = true;
   while( paragraph.Length() > 0 )
   {

-               dc.GetTextExtent( paragraph , &width , &height ) ;
-                                       
-               if ( width > m_width )
-               {
-                       for ( int p = paragraph.Length() -1 ; p > 0 ; --p )
-                       {                                 
-                         if ((punct.Find(paragraph[p]) != wxNOT_FOUND) || !linedrawn)
-                               {
-                                 int blank = (paragraph[p] == ' ') ? 0 : 1;
-                                 
-                                       dc.GetTextExtent( paragraph.Left(p + blank) , &width , &height ) ;
-                                       
-                                       if ( width <= m_width )
-                                       {       
-                                               int pos = x ;
-                                               if ( HasFlag( wxALIGN_CENTER ) )
-                                               {
-                                                       pos += ( m_width - width ) / 2 ;
-                                         }
-                                               else if ( HasFlag( wxALIGN_RIGHT ) )
-                                               {
-                                                       pos += ( m_width - width ) ;
-                                         }
-                                                       
-                                       dc.DrawText( paragraph.Left(p + blank), pos , y) ;
-                                       y += height ;
-                                               paragraph = paragraph.Mid(p+1) ;
-                                               linedrawn = true;
-                                               break ;
-                                       }
-                               }
-                       }
-                       
-                       linedrawn = false;
-               }
-               else
-               {
-                       int pos = x ;
-                       if ( HasFlag( wxALIGN_CENTER ) )
-                       {
-                               pos += ( m_width - width ) / 2 ;
-                 }
-                       else if ( HasFlag( wxALIGN_RIGHT ) )
-                       {
-                               pos += ( m_width - width ) ;
-                 }
-                               
-               dc.DrawText( paragraph, pos , y) ;
-               paragraph="";
-               y += height ;
-               }
-       }
+        dc.GetTextExtent( paragraph , &width , &height ) ;
+
+        if ( width > m_width )
+        {
+            for ( int p = paragraph.Length() -1 ; p > 0 ; --p )
+            {
+              if ((punct.Find(paragraph[p]) != wxNOT_FOUND) || !linedrawn)
+                {
+                  int blank = (paragraph[p] == ' ') ? 0 : 1;
+
+                    dc.GetTextExtent( paragraph.Left(p + blank) , &width , &height ) ;
+
+                    if ( width <= m_width )
+                    {
+                        int pos = x ;
+                        if ( HasFlag( wxALIGN_CENTER ) )
+                        {
+                            pos += ( m_width - width ) / 2 ;
+                      }
+                        else if ( HasFlag( wxALIGN_RIGHT ) )
+                        {
+                            pos += ( m_width - width ) ;
+                      }
+
+                    dc.DrawText( paragraph.Left(p + blank), pos , y) ;
+                    y += height ;
+                        paragraph = paragraph.Mid(p+1) ;
+                        linedrawn = true;
+                        break ;
+                    }
+                }
+            }
+
+            linedrawn = false;
+        }
+        else
+        {
+            int pos = x ;
+            if ( HasFlag( wxALIGN_CENTER ) )
+            {
+                pos += ( m_width - width ) / 2 ;
+          }
+            else if ( HasFlag( wxALIGN_RIGHT ) )
+            {
+                pos += ( m_width - width ) ;
+          }
+
+        dc.DrawText( paragraph, pos , y) ;
+        paragraph="";
+        y += height ;
+        }
+    }

 }
 
 void wxStaticText::OnDraw( wxDC &dc )
 {
     if (m_width <= 0 || m_height <= 0)
         return;
 }
 
 void wxStaticText::OnDraw( wxDC &dc )
 {
     if (m_width <= 0 || m_height <= 0)
         return;

+/*
+    dc.Clear() ;
+    wxRect rect(0,0,m_width,m_height) ;
+    dc.SetFont(*wxSMALL_FONT) ;
+
+    dc.DrawRectangle(rect) ;
+*/
+  if ( !IsWindowHilited( (WindowRef) MacGetRootWindow() ) &&
+    ( GetBackgroundColour() == wxSystemSettings::GetColour(wxSYS_COLOUR_3DFACE )
+      || GetBackgroundColour() == wxSystemSettings::GetColour(wxSYS_COLOUR_APPWORKSPACE) ) )
+  {
+    dc.SetTextForeground( wxColour( 0x80 , 0x80 , 0x80 ) ) ;
+  }
+  else
+  {
+    dc.SetTextForeground( GetForegroundColour() ) ;
+  }

 
   wxString paragraph;
   int i = 0 ;
   wxString text = m_label;
 
   wxString paragraph;
   int i = 0 ;
   wxString text = m_label;

+  int y = 0 ;
+  while (i < text.Length())
+  {

 
 

-    PrepareDC(dc);
-    
-    bool doClear = true ;
-       WindowRef window = GetMacRootWindow() ;
-       if ( window )
-       {
-               wxWindow* win = wxFindWinFromMacWindow( window ) ;
-               if ( win )
-               {
-                       wxWindow* parent = GetParent() ;
-                       while ( parent )
-                       {
-                               if( parent->MacGetWindowData() )
-                               {
-                                       break ;
-                               }
-                               
-                               if( parent->IsKindOf( CLASSINFO( wxNotebook ) ) ||  parent->IsKindOf( CLASSINFO( wxTabCtrl ) ))
-                               {
-                                       if ( ((wxControl*)parent)->GetMacControl() ) {
-                                               Rect rect = { -32767 , -32767 , 32767 , 32767 } ;
-                                               if ( DrawThemeTabPane != (void*)kUnresolvedCFragSymbolAddress )
-                                               {
-                                                 DrawThemeTabPane ( &rect, kThemeStateActive);
-                                                 doClear = false ;
-                                               }
-                                       }
-                                       break ;
-                               }
-                               
-                               parent = parent->GetParent() ;
-                       } 
-               }
+       if (text[i] == 13 || text[i] == 10)
+       {
+           DrawParagraph(dc, paragraph,y);
+           paragraph = "" ;

        }
        }

-       if ( doClear )
-               dc.Clear() ;
-               
-       while (i < text.Length())
+       else

        {
        {

-         paragraph += text[i];
-         
-       if (text[i] == 13 || text[i] == 10)
-           DrawParagraph(dc, paragraph);
-           
+        paragraph += text[i];
+    }

     ++i;
   }
   if (paragraph.Length() > 0)
     ++i;
   }
   if (paragraph.Length() > 0)

-         DrawParagraph(dc, paragraph);
+         DrawParagraph(dc, paragraph,y);

 }
 
 }
 

-void wxStaticText::OnPaint( wxPaintEvent &event ) 
+void wxStaticText::OnPaint( wxPaintEvent &event )

 {
     wxPaintDC dc(this);
     OnDraw( dc ) ;
 {
     wxPaintDC dc(this);
     OnDraw( dc ) ;


@@ -189,10 +176,9 @@ void wxStaticText::OnPaint( wxPaintEvent &event )
 
 wxSize wxStaticText::DoGetBestSize() const
 {
 
 wxSize wxStaticText::DoGetBestSize() const
 {

-       int x,y ;

     int widthTextMax = 0, widthLine,
         heightTextTotal = 0, heightLineDefault = 0, heightLine = 0;
     int widthTextMax = 0, widthLine,
         heightTextTotal = 0, heightLineDefault = 0, heightLine = 0;

-        
+

     wxString curLine;
     for ( const wxChar *pc = m_label; ; pc++ )
     {
     wxString curLine;
     for ( const wxChar *pc = m_label; ; pc++ )
     {


@@ -209,7 +195,7 @@ wxSize wxStaticText::DoGetBestSize() const
                     GetTextExtent(_T("W"), NULL, &heightLineDefault);
 
                 heightTextTotal += heightLineDefault;
                     GetTextExtent(_T("W"), NULL, &heightLineDefault);
 
                 heightTextTotal += heightLineDefault;

-                
+

                 heightTextTotal++;  // FIXME: why is this necessary?
             }
             else
                 heightTextTotal++;  // FIXME: why is this necessary?
             }
             else


@@ -218,7 +204,7 @@ wxSize wxStaticText::DoGetBestSize() const
                 if ( widthLine > widthTextMax )
                     widthTextMax = widthLine;
                 heightTextTotal += heightLine;
                 if ( widthLine > widthTextMax )
                     widthTextMax = widthLine;
                 heightTextTotal += heightLine;

-                
+

                 heightTextTotal++;  // FIXME: why is this necessary?
             }
 
                 heightTextTotal++;  // FIXME: why is this necessary?
             }
 


@@ -240,13 +226,23 @@ wxSize wxStaticText::DoGetBestSize() const
 
 void wxStaticText::SetLabel(const wxString& st )
 {
 
 void wxStaticText::SetLabel(const wxString& st )
 {

-       SetTitle( st ) ;
-       m_label = st ;
-       if ( !(GetWindowStyle() & wxST_NO_AUTORESIZE) )
-               SetSize( GetBestSize() ) ;
-
-       Refresh() ;     
-       MacUpdateImmediately() ;
-//    wxClientDC dc(this);
-//    OnDraw( dc ) ;
+    SetTitle( st ) ;
+    m_label = st ;
+    if ( !(GetWindowStyle() & wxST_NO_AUTORESIZE) )
+        SetSize( GetBestSize() ) ;
+
+    Refresh() ;
+    Update() ;
+}
+
+bool wxStaticText::SetFont(const wxFont& font)
+{
+    bool ret = wxControl::SetFont(font);
+
+    // adjust the size of the window to fit to the label unless autoresizing is
+    // disabled
+    if ( !(GetWindowStyle() & wxST_NO_AUTORESIZE) )
+        SetSize( GetBestSize() );
+
+    return ret;

 }
 }