Added OnEraseBackground to wxNotebook on wxMSW to avoid black background;

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

diff --git a/src/gtk/listbox.cpp b/src/gtk/listbox.cpp
index d434764ba4277d61418161d0ca9d8ff738ae5598..f3aefb791a2f660c1dfb7f6290b67c0b004efd34 100644 (file)
--- a/src/gtk/listbox.cpp
+++ b/src/gtk/listbox.cpp
@@ -2,9 +2,8 @@
 // Name:        listbox.cpp
 // Purpose:
 // Author:      Robert Roebling
-// Created:     01/02/97
-// Id:
-// Copyright:   (c) 1998 Robert Roebling, Julian Smart and Markus Holzem
+// Id:          $Id$
+// Copyright:   (c) 1998 Robert Roebling
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
@@ -124,6 +123,8 @@ bool wxListBox::Create( wxWindow *parent, wxWindowID id,
 
   gtk_widget_realize( GTK_WIDGET(m_list) );
 
+  SetBackgroundColour( parent->GetBackgroundColour() );
+
   Show( TRUE );
 
   return TRUE;
@@ -136,6 +137,8 @@ void wxListBox::Append( const wxString &item )
 
 void wxListBox::Append( const wxString &item, char *clientData )
 {
+  wxCHECK_RET( m_list != NULL, "invalid listbox" );
+
   GtkWidget *list_item = gtk_list_item_new_with_label( item );
 
   if (m_hasOwnStyle)
@@ -145,7 +148,7 @@ void wxListBox::Append( const wxString &item, char *clientData )
       gtk_style_ref(
         gtk_widget_get_style( m_widget ) ) );
   }
-
+  
   gtk_signal_connect( GTK_OBJECT(list_item), "select",
     GTK_SIGNAL_FUNC(gtk_listitem_select_callback), (gpointer)this );
 
@@ -162,11 +165,12 @@ void wxListBox::Append( const wxString &item, char *clientData )
   ConnectWidget( list_item );
 
   ConnectDnDWidget( list_item );
-
 }
 
 void wxListBox::Clear(void)
 {
+  wxCHECK_RET( m_list != NULL, "invalid listbox" );
+
   gtk_list_clear_items( m_list, 0, Number() );
 
   m_clientData.Clear();
@@ -174,6 +178,8 @@ void wxListBox::Clear(void)
 
 void wxListBox::Delete( int n )
 {
+  wxCHECK_RET( m_list != NULL, "invalid listbox" );
+
   GList *child = g_list_nth( m_list->children, n );
 
   if (!child)
@@ -197,11 +203,15 @@ void wxListBox::Delete( int n )
 
 void wxListBox::Deselect( int n )
 {
+  wxCHECK_RET( m_list != NULL, "invalid listbox" );
+
   gtk_list_unselect_item( m_list, n );
 }
 
 int wxListBox::FindString( const wxString &item ) const
 {
+  wxCHECK_MSG( m_list != NULL, -1, "invalid listbox" );
+  
   GList *child = m_list->children;
   int count = 0;
   while (child)
@@ -222,6 +232,8 @@ int wxListBox::FindString( const wxString &item ) const
 
 char *wxListBox::GetClientData( int n ) const
 {
+  wxCHECK_MSG( m_list != NULL, (char*) NULL, "invalid listbox" );
+  
   wxNode *node = m_clientData.Nth( n );
   if (node) return ((char*)node->Data());
 
@@ -231,6 +243,8 @@ char *wxListBox::GetClientData( int n ) const
 
 int wxListBox::GetSelection(void) const
 {
+  wxCHECK_MSG( m_list != NULL, -1, "invalid listbox" );
+  
   GList *child = m_list->children;
   int count = 0;
   while (child)
@@ -244,6 +258,8 @@ int wxListBox::GetSelection(void) const
 
 int wxListBox::GetSelections(wxArrayInt& aSelections) const
 {
+  wxCHECK_MSG( m_list != NULL, -1, "invalid listbox" );
+  
   // get the number of selected items first
   GList *child = m_list->children;
   int count = 0;
@@ -271,6 +287,8 @@ int wxListBox::GetSelections(wxArrayInt& aSelections) const
 
 wxString wxListBox::GetString( int n ) const
 {
+  wxCHECK_MSG( m_list != NULL, "", "invalid listbox" );
+  
   GList *child = g_list_nth( m_list->children, n );
   if (child)
   {
@@ -284,6 +302,8 @@ wxString wxListBox::GetString( int n ) const
 
 wxString wxListBox::GetStringSelection(void) const
 {
+  wxCHECK_MSG( m_list != NULL, "", "invalid listbox" );
+  
   GList *selection = m_list->selection;
   if (selection)
   {
@@ -297,6 +317,8 @@ wxString wxListBox::GetStringSelection(void) const
 
 int wxListBox::Number(void)
 {
+  wxCHECK_MSG( m_list != NULL, -1, "invalid listbox" );
+  
   GList *child = m_list->children;
   int count = 0;
   while (child) { count++; child = child->next; }
@@ -305,6 +327,8 @@ int wxListBox::Number(void)
 
 bool wxListBox::Selected( int n )
 {
+  wxCHECK_MSG( m_list != NULL, FALSE, "invalid listbox" );
+  
   GList *target = g_list_nth( m_list->children, n );
   if (target)
   {
@@ -326,6 +350,8 @@ void wxListBox::Set( int WXUNUSED(n), const wxString *WXUNUSED(choices) )
 
 void wxListBox::SetClientData( int n, char *clientData )
 {
+  wxCHECK_RET( m_list != NULL, "invalid listbox" );
+  
   wxNode *node = m_clientData.Nth( n );
   if (node)
   {
@@ -349,6 +375,8 @@ void wxListBox::SetFirstItem( const wxString &WXUNUSED(item) )
 
 void wxListBox::SetSelection( int n, bool select )
 {
+  wxCHECK_RET( m_list != NULL, "invalid listbox" );
+  
   if (select)
     gtk_list_select_item( m_list, n );
   else
@@ -357,6 +385,8 @@ void wxListBox::SetSelection( int n, bool select )
 
 void wxListBox::SetString( int n, const wxString &string )
 {
+  wxCHECK_RET( m_list != NULL, "invalid listbox" );
+  
   GList *child = g_list_nth( m_list->children, n );
   if (child)
   {
@@ -372,6 +402,8 @@ void wxListBox::SetString( int n, const wxString &string )
 
 void wxListBox::SetStringSelection( const wxString &string, bool select )
 {
+  wxCHECK_RET( m_list != NULL, "invalid listbox" );
+  
   SetSelection( FindString(string), select );
 }
 
@@ -393,6 +425,8 @@ int wxListBox::GetIndex( GtkWidget *item ) const
 
 void wxListBox::SetDropTarget( wxDropTarget *dropTarget )
 {
+  wxCHECK_RET( m_list != NULL, "invalid listbox" );
+  
   GList *child = m_list->children;
   while (child)
   {
@@ -415,34 +449,66 @@ GtkWidget *wxListBox::GetConnectWidget(void)
   return GTK_WIDGET(m_list);
 }
 
-void wxListBox::SetFont( const wxFont &font )
+bool wxListBox::IsOwnGtkWindow( GdkWindow *window )
 {
-  wxWindow::SetFont( font );
+  if (wxWindow::IsOwnGtkWindow( window )) return TRUE;
 
   GList *child = m_list->children;
   while (child)
   {
-    GtkBin *bin = (GtkBin*) child->data;
-    gtk_widget_set_style( bin->child,
-      gtk_style_ref(
-        gtk_widget_get_style( m_widget ) ) );
+    GtkBin *bin = GTK_BIN( child->data );
+    if (bin->child->window == window) return TRUE;
     child = child->next;
   }
+
+  return FALSE;
 }
 
-bool wxListBox::IsOwnGtkWindow( GdkWindow *window )
+void wxListBox::SetFont( const wxFont &font )
 {
-  if (wxWindow::IsOwnGtkWindow( window )) return TRUE;
+  wxCHECK_RET( m_list != NULL, "invalid listbox" );
+  
+  wxControl::SetFont( font );
 
   GList *child = m_list->children;
   while (child)
   {
-    GtkBin *bin = (GtkBin*) child->data;
-    if (bin->child->window == window) return TRUE;
+    gtk_widget_set_style( GTK_BIN(child->data)->child, 
+      gtk_style_ref( 
+        gtk_widget_get_style( m_widget ) ) );
+       
     child = child->next;
   }
-
-  return FALSE;
 }
 
+void wxListBox::SetBackgroundColour( const wxColour &colour )
+{
+  return;
+
+  wxCHECK_RET( m_list != NULL, "invalid listbox" );
+  
+  wxControl::SetBackgroundColour( colour );
+  
+  return;
+  
+  if (!m_backgroundColour.Ok()) return;
+  
+  gtk_widget_set_style( GTK_WIDGET(m_list), 
+    gtk_style_ref(
+      gtk_widget_get_style( m_widget ) ) );
+      
+  GList *child = m_list->children;
+  while (child)
+  {
+    gtk_widget_set_style( GTK_WIDGET(child->data), 
+      gtk_style_ref( 
+        gtk_widget_get_style( m_widget ) ) );
+       
+    gtk_widget_set_style( GTK_BIN(child->data)->child, 
+      gtk_style_ref( 
+        gtk_widget_get_style( m_widget ) ) );
+       
+    child = child->next;
+  }
+}