]> git.saurik.com Git - wxWidgets.git/blobdiff - src/gtk/listbox.cpp
no message
[wxWidgets.git] / src / gtk / listbox.cpp
index 1e2e80ebc82b0cb25de040be918505bb0b88541a..d434764ba4277d61418161d0ca9d8ff738ae5598 100644 (file)
@@ -32,9 +32,9 @@ static void gtk_listitem_select_callback( GtkWidget *WXUNUSED(widget), wxListBox
 {
   if (!listbox->HasVMT()) return;
   if (g_blockEventsOnDrag) return;
-  
+
   wxCommandEvent event(wxEVT_COMMAND_LISTBOX_SELECTED, listbox->GetId() );
-  
+
     wxArrayInt aSelections;
     int count = listbox->GetSelections(aSelections);
     if ( count > 0 )
@@ -68,59 +68,64 @@ wxListBox::wxListBox(void)
   m_list = (GtkList *) NULL;
 }
 
-bool wxListBox::Create( wxWindow *parent, wxWindowID id, 
-      const wxPoint &pos, const wxSize &size, 
+bool wxListBox::Create( wxWindow *parent, wxWindowID id,
+      const wxPoint &pos, const wxSize &size,
       int n, const wxString choices[],
       long style, const wxValidator& validator, const wxString &name )
 {
   m_needParent = TRUE;
-  
+
   PreCreation( parent, id, pos, size, style, name );
-  
+
   SetValidator( validator );
 
   m_widget = gtk_scrolled_window_new( (GtkAdjustment *) NULL, (GtkAdjustment *) NULL );
   gtk_scrolled_window_set_policy( GTK_SCROLLED_WINDOW(m_widget),
     GTK_POLICY_AUTOMATIC, GTK_POLICY_AUTOMATIC );
-  
+
   m_list = GTK_LIST( gtk_list_new() );
-  
-  GtkSelectionMode mode = GTK_SELECTION_SINGLE;
+
+  GtkSelectionMode mode = GTK_SELECTION_BROWSE;
   if (style & wxLB_MULTIPLE)
     mode = GTK_SELECTION_MULTIPLE;
   else if (style & wxLB_EXTENDED)
     mode = GTK_SELECTION_EXTENDED;
 
   gtk_list_set_selection_mode( GTK_LIST(m_list), mode );
-  
+
   gtk_container_add (GTK_CONTAINER(m_widget), GTK_WIDGET(m_list) );
   gtk_widget_show( GTK_WIDGET(m_list) );
-  
+
+  wxSize newSize = size;
+  if (newSize.x == -1) newSize.x = 100;
+  if (newSize.y == -1) newSize.y = 110;
+  SetSize( newSize.x, newSize.y );
+
   for (int i = 0; i < n; i++)
   {
     GtkWidget *list_item;
-    list_item = gtk_list_item_new_with_label( choices[i] ); 
-  
+    list_item = gtk_list_item_new_with_label( choices[i] );
+
     gtk_container_add( GTK_CONTAINER(m_list), list_item );
-    
-    gtk_signal_connect( GTK_OBJECT(list_item), "select", 
+
+    gtk_signal_connect( GTK_OBJECT(list_item), "select",
       GTK_SIGNAL_FUNC(gtk_listitem_select_callback), (gpointer)this );
-  
+
     if (style & wxLB_MULTIPLE)
-      gtk_signal_connect( GTK_OBJECT(list_item), "deselect", 
+      gtk_signal_connect( GTK_OBJECT(list_item), "deselect",
         GTK_SIGNAL_FUNC(gtk_listitem_select_callback), (gpointer)this );
-  
+
     m_clientData.Append( (wxObject*)NULL );
-    
+
     gtk_widget_show( list_item );
   }
+
   PostCreation();
-  
+
   gtk_widget_realize( GTK_WIDGET(m_list) );
-  
+
   Show( TRUE );
-  
+
   return TRUE;
 }
 
@@ -131,56 +136,56 @@ void wxListBox::Append( const wxString &item )
 
 void wxListBox::Append( const wxString &item, char *clientData )
 {
-  GtkWidget *list_item = gtk_list_item_new_with_label( item ); 
-  
+  GtkWidget *list_item = gtk_list_item_new_with_label( item );
+
   if (m_hasOwnStyle)
   {
     GtkBin *bin = GTK_BIN( list_item );
-    gtk_widget_set_style( bin->child, 
+    gtk_widget_set_style( bin->child,
       gtk_style_ref(
-        gtk_widget_get_style( m_widget ) ) ); 
+        gtk_widget_get_style( m_widget ) ) );
   }
-  
-  gtk_signal_connect( GTK_OBJECT(list_item), "select", 
+
+  gtk_signal_connect( GTK_OBJECT(list_item), "select",
     GTK_SIGNAL_FUNC(gtk_listitem_select_callback), (gpointer)this );
-  
+
   if (GetWindowStyleFlag() & wxLB_MULTIPLE)
-    gtk_signal_connect( GTK_OBJECT(list_item), "deselect", 
+    gtk_signal_connect( GTK_OBJECT(list_item), "deselect",
       GTK_SIGNAL_FUNC(gtk_listitem_select_callback), (gpointer)this );
-  
+
   m_clientData.Append( (wxObject*)clientData );
-  
+
   gtk_container_add( GTK_CONTAINER(m_list), list_item );
-  
+
   gtk_widget_show( list_item );
-  
+
   ConnectWidget( list_item );
-  
+
   ConnectDnDWidget( list_item );
-  
+
 }
 
 void wxListBox::Clear(void)
 {
   gtk_list_clear_items( m_list, 0, Number() );
-  
+
   m_clientData.Clear();
 }
 
 void wxListBox::Delete( int n )
 {
   GList *child = g_list_nth( m_list->children, n );
-  
+
   if (!child)
   {
     wxFAIL_MSG("wrong listbox index");
     return;
   }
-  
+
   GList *list = g_list_append( NULL, child->data );
   gtk_list_remove_items( m_list, list );
   g_list_free( list );
-  
+
   wxNode *node = m_clientData.Nth( n );
   if (!node)
   {
@@ -207,7 +212,11 @@ int wxListBox::FindString( const wxString &item ) const
     count++;
     child = child->next;
   }
-  wxFAIL_MSG("wrong listbox index");
+
+  // it's not an error if the string is not found - this function may be used to
+  // test for existence of the string in the listbox, so don't give any
+  // errors/assert failures.
+
   return -1;
 }
 
@@ -215,28 +224,21 @@ char *wxListBox::GetClientData( int n ) const
 {
   wxNode *node = m_clientData.Nth( n );
   if (node) return ((char*)node->Data());
-  
+
   wxFAIL_MSG("wrong listbox index");
   return (char *) NULL;
 }
 
 int wxListBox::GetSelection(void) const
 {
-  GList *selection = m_list->selection;
-  if (selection)
+  GList *child = m_list->children;
+  int count = 0;
+  while (child)
   {
-    GList *child = m_list->children;
-    int count = 0;
-    while (child)
-    {
-      if (child->data == selection->data) return count;
-      count++;
-      child = child->next;
-    }
+    if (GTK_WIDGET(child->data)->state == GTK_STATE_SELECTED) return count;
+    count++;
+    child = child->next;
   }
-  // No, I think it's reasonable to return -1 to indicate
-  // there is no selection. -- JACS
-//  wxFAIL_MSG("wrong listbox index");
   return -1;
 }
 
@@ -245,25 +247,25 @@ int wxListBox::GetSelections(wxArrayInt& aSelections) const
   // get the number of selected items first
   GList *child = m_list->children;
   int count = 0;
-  for ( child = m_list->children; child != NULL; child = child->next ) 
+  for ( child = m_list->children; child != NULL; child = child->next )
   {
-    if ( GTK_WIDGET(child->data)->state == GTK_STATE_SELECTED ) 
+    if ( GTK_WIDGET(child->data)->state == GTK_STATE_SELECTED )
       count++;
   }
 
   aSelections.Empty();
-  
+
   if ( count > 0 ) {
     // now fill the list
     aSelections.Alloc(count); // optimization attempt
     int i = 0;
-    for ( child = m_list->children; child != NULL; child = child->next, i++ ) 
+    for ( child = m_list->children; child != NULL; child = child->next, i++ )
     {
-      if ( GTK_WIDGET(child->data)->state == GTK_STATE_SELECTED ) 
+      if ( GTK_WIDGET(child->data)->state == GTK_STATE_SELECTED )
         aSelections.Add(i);
     }
   }
-  
+
   return count;
 }
 
@@ -326,7 +328,7 @@ void wxListBox::SetClientData( int n, char *clientData )
 {
   wxNode *node = m_clientData.Nth( n );
   if (node)
-  { 
+  {
     node->SetData( (wxObject*)clientData );
   }
   else
@@ -397,9 +399,9 @@ void wxListBox::SetDropTarget( wxDropTarget *dropTarget )
     DisconnectDnDWidget( GTK_WIDGET( child->data ) );
     child = child->next;
   }
-  
+
   wxWindow::SetDropTarget( dropTarget  );
-  
+
   child = m_list->children;
   while (child)
   {
@@ -421,11 +423,26 @@ void wxListBox::SetFont( const wxFont &font )
   while (child)
   {
     GtkBin *bin = (GtkBin*) child->data;
-    gtk_widget_set_style( bin->child, 
+    gtk_widget_set_style( bin->child,
       gtk_style_ref(
-        gtk_widget_get_style( m_widget ) ) ); 
+        gtk_widget_get_style( m_widget ) ) );
+    child = child->next;
+  }
+}
+
+bool wxListBox::IsOwnGtkWindow( GdkWindow *window )
+{
+  if (wxWindow::IsOwnGtkWindow( window )) return TRUE;
+
+  GList *child = m_list->children;
+  while (child)
+  {
+    GtkBin *bin = (GtkBin*) child->data;
+    if (bin->child->window == window) return TRUE;
     child = child->next;
   }
+
+  return FALSE;
 }