Invalid conversion compile error fix (GTK+ 2.12.9)

[wxWidgets.git] / src / gtk / statbox.cpp

diff --git a/src/gtk/statbox.cpp b/src/gtk/statbox.cpp
index a5314ce4df5d875bb04487ee1db5314d93886596..139a678df29fe90441e902deb81e99e856562f11 100644 (file)
--- a/src/gtk/statbox.cpp
+++ b/src/gtk/statbox.cpp
@@ -1,5 +1,5 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        gtk/statbox.cpp
+// Name:        src/gtk/statbox.cpp

 // Purpose:
 // Author:      Robert Roebling
 // Id:          $Id$
 // Purpose:
 // Author:      Robert Roebling
 // Id:          $Id$


@@ -13,60 +13,45 @@
 #if wxUSE_STATBOX
 
 #include "wx/statbox.h"
 #if wxUSE_STATBOX
 
 #include "wx/statbox.h"

-#include "wx/gtk/private.h"

 
 

-#include "gdk/gdk.h"
-#include "gtk/gtk.h"
-
-
-// ============================================================================
-// implementation
-// ============================================================================
+#include <gtk/gtk.h>
+#include "wx/gtk/private/gtk2-compat.h"
+#include "wx/gtk/private/win_gtk.h"

 
 // constants taken from GTK sources
 #define LABEL_PAD 1
 #define LABEL_SIDE_PAD 2
 
 //-----------------------------------------------------------------------------
 
 // constants taken from GTK sources
 #define LABEL_PAD 1
 #define LABEL_SIDE_PAD 2
 
 //-----------------------------------------------------------------------------

-// "gtk_frame_size_allocate" signal
+// "size_allocate" from m_widget

 //-----------------------------------------------------------------------------
 
 //-----------------------------------------------------------------------------
 

+#ifndef __WXGTK3__

 extern "C" {
 extern "C" {

-
-static void
-gtk_frame_size_allocate (GtkWidget     *widget,
-                         GtkAllocation *allocation,
-                         wxStaticBox *p)
+static void size_allocate(GtkWidget* widget, GtkAllocation* alloc, void*)

 {
 {

-    GtkFrame *frame = GTK_FRAME (widget);
-
-    // this handler gets called _after_ the GTK+'s own signal handler; thus we
-    // need to fix only the width of the GtkLabel
-    // (everything else has already been handled by the GTK+ signal handler).
-
-    if (frame->label_widget && GTK_WIDGET_VISIBLE (frame->label_widget))
+    // clip label as GTK >= 2.12 does
+    GtkWidget* label_widget = gtk_frame_get_label_widget(GTK_FRAME(widget));
+    int w = alloc->width -
+        2 * gtk_widget_get_style(widget)->xthickness - 2 * LABEL_PAD - 2 * LABEL_SIDE_PAD;
+    if (w < 0)
+        w = 0;
+
+    GtkAllocation a;
+    gtk_widget_get_allocation(label_widget, &a);
+    if (a.width > w)

     {
     {

-        GtkAllocation ca = frame->label_widget->allocation;
-
-        // we want the GtkLabel to not exceed maxWidth:
-        int maxWidth = allocation->width - 2*LABEL_SIDE_PAD - 2*LABEL_PAD;
-        maxWidth = wxMax(2, maxWidth);      // maxWidth must always be positive!
-
-        // truncate the label to the GtkFrame width...
-        ca.width = wxMin(ca.width, maxWidth);
-        gtk_widget_size_allocate(frame->label_widget, &ca);
+        a.width = w;
+        gtk_widget_size_allocate(label_widget, &a);

     }
 }
     }
 }

-

 }
 }

-
+#endif

 
 //-----------------------------------------------------------------------------
 // wxStaticBox
 //-----------------------------------------------------------------------------
 
 
 //-----------------------------------------------------------------------------
 // wxStaticBox
 //-----------------------------------------------------------------------------
 

-IMPLEMENT_DYNAMIC_CLASS(wxStaticBox, wxControl)
-

 wxStaticBox::wxStaticBox()
 {
 }
 wxStaticBox::wxStaticBox()
 {
 }


@@ -90,16 +75,17 @@ bool wxStaticBox::Create( wxWindow *parent,
                           long style,
                           const wxString& name )
 {
                           long style,
                           const wxString& name )
 {

-    m_needParent = TRUE;
-

     if (!PreCreation( parent, pos, size ) ||
         !CreateBase( parent, id, pos, size, style, wxDefaultValidator, name ))
     {
         wxFAIL_MSG( wxT("wxStaticBox creation failed") );
     if (!PreCreation( parent, pos, size ) ||
         !CreateBase( parent, id, pos, size, style, wxDefaultValidator, name ))
     {
         wxFAIL_MSG( wxT("wxStaticBox creation failed") );

-        return FALSE;
+        return false;

     }
 
     m_widget = GTKCreateFrame(label);
     }
 
     m_widget = GTKCreateFrame(label);

+    g_object_ref(m_widget);
+
+    // only base SetLabel needs to be called after GTKCreateFrame

     wxControl::SetLabel(label);
 
     m_parent->DoAddChild( this );
     wxControl::SetLabel(label);
 
     m_parent->DoAddChild( this );


@@ -107,23 +93,37 @@ bool wxStaticBox::Create( wxWindow *parent,
     PostCreation(size);
 
     // need to set non default alignment?
     PostCreation(size);
 
     // need to set non default alignment?

-    gfloat xalign;
+    gfloat xalign = 0;

     if ( style & wxALIGN_CENTER )
         xalign = 0.5;
     else if ( style & wxALIGN_RIGHT )
         xalign = 1.0;
     if ( style & wxALIGN_CENTER )
         xalign = 0.5;
     else if ( style & wxALIGN_RIGHT )
         xalign = 1.0;

-    else // wxALIGN_LEFT
-        xalign = 0.0;

 
 

-    if ( style & (wxALIGN_RIGHT | wxALIGN_CENTER) ) // left alignment is default
-        gtk_frame_set_label_align(GTK_FRAME( m_widget ), xalign, 0.5);
+    gtk_frame_set_label_align(GTK_FRAME(m_widget), xalign, 0.5);

 
 

-    // in order to clip the label widget, we must connect to the size allocate
-    // signal of this GtkFrame after the default GTK+'s allocate size function
-    g_signal_connect_after (m_widget, "size_allocate",
-                            G_CALLBACK (gtk_frame_size_allocate), this);
+#ifndef __WXGTK3__
+    if (gtk_check_version(2, 12, 0))
+    {
+        // we connect this signal to perform label-clipping as GTK >= 2.12 does
+        g_signal_connect(m_widget, "size_allocate", G_CALLBACK(size_allocate), NULL);
+    }
+#endif

 
 

-    return TRUE;
+    return true;
+}
+
+void wxStaticBox::AddChild( wxWindowBase *child )
+{
+    if (!m_wxwindow)
+    {
+        // make this window a container of other wxWindows by instancing a wxPizza
+        // and packing it into the GtkFrame:
+        m_wxwindow = wxPizza::New();
+        gtk_widget_show( m_wxwindow );
+        gtk_container_add( GTK_CONTAINER (m_widget), m_wxwindow );
+    }
+
+    wxWindow::AddChild( child );

 }
 
 void wxStaticBox::SetLabel( const wxString& label )
 }
 
 void wxStaticBox::SetLabel( const wxString& label )


@@ -152,16 +152,14 @@ void wxStaticBox::GTKWidgetDoSetMnemonic(GtkWidget* w)
 wxVisualAttributes
 wxStaticBox::GetClassDefaultAttributes(wxWindowVariant WXUNUSED(variant))
 {
 wxVisualAttributes
 wxStaticBox::GetClassDefaultAttributes(wxWindowVariant WXUNUSED(variant))
 {

-    return GetDefaultAttributesFromGTKWidget(gtk_frame_new);
+    return GetDefaultAttributesFromGTKWidget(gtk_frame_new(""));

 }
 
 
 void wxStaticBox::GetBordersForSizer(int *borderTop, int *borderOther) const
 {
 }
 
 
 void wxStaticBox::GetBordersForSizer(int *borderTop, int *borderOther) const
 {

-    const int BORDER = 5; // FIXME: hardcoded value
-
-    *borderTop = GetLabel().empty() ? 2*BORDER : GetCharHeight();
-    *borderOther = BORDER;
+    *borderTop = GetCharHeight();
+    *borderOther = GetCharWidth()/2;

 }
 
 #endif // wxUSE_STATBOX
 }
 
 #endif // wxUSE_STATBOX