Update the current row after item deletion in generic wxDataViewCtrl.

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

diff --git a/src/gtk/choice.cpp b/src/gtk/choice.cpp
index 9eefcf06da0e0cbe1a4bd071cc3b94413043cad2..28832ef3e4f0d4d34c4b7fc2f4cf5cdd6cb18da2 100644 (file)
--- a/src/gtk/choice.cpp
+++ b/src/gtk/choice.cpp
@@ -17,8 +17,9 @@
     #include "wx/arrstr.h"
 #endif
 
     #include "wx/arrstr.h"
 #endif
 

+#include <gtk/gtk.h>

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

-
+#include "wx/gtk/private/gtk2-compat.h"

 
 // ----------------------------------------------------------------------------
 // GTK callbacks
 
 // ----------------------------------------------------------------------------
 // GTK callbacks


@@ -38,11 +39,10 @@ gtk_choice_changed_callback( GtkWidget *WXUNUSED(widget), wxChoice *choice )
 // wxChoice
 //-----------------------------------------------------------------------------
 
 // wxChoice
 //-----------------------------------------------------------------------------
 

-IMPLEMENT_DYNAMIC_CLASS(wxChoice, wxControlWithItems)
-
-wxChoice::wxChoice()
-    : m_strings(NULL)
+void wxChoice::Init()

 {
 {

+    m_strings = NULL;
+    m_stringCellIndex = 0;

 }
 
 bool wxChoice::Create( wxWindow *parent, wxWindowID id,
 }
 
 bool wxChoice::Create( wxWindow *parent, wxWindowID id,


@@ -63,8 +63,6 @@ bool wxChoice::Create( wxWindow *parent, wxWindowID id,
                        long style, const wxValidator& validator,
                        const wxString &name )
 {
                        long style, const wxValidator& validator,
                        const wxString &name )
 {

-    m_strings = NULL;
-

     if (!PreCreation( parent, pos, size ) ||
         !CreateBase( parent, id, pos, size, style, validator, name ))
     {
     if (!PreCreation( parent, pos, size ) ||
         !CreateBase( parent, id, pos, size, style, validator, name ))
     {


@@ -76,10 +74,15 @@ bool wxChoice::Create( wxWindow *parent, wxWindowID id,
     {
         // if our m_strings != NULL, Append() will check for it and insert
         // items in the correct order
     {
         // if our m_strings != NULL, Append() will check for it and insert
         // items in the correct order

-        m_strings = new wxSortedArrayString;
+        m_strings = new wxGtkCollatedArrayString;

     }
 
     }
 

+#ifdef __WXGTK3__
+    m_widget = gtk_combo_box_text_new();
+#else

     m_widget = gtk_combo_box_new_text();
     m_widget = gtk_combo_box_new_text();

+#endif
+    g_object_ref(m_widget);

 
     Append(n, choices);
 
 
     Append(n, choices);
 


@@ -100,9 +103,6 @@ wxChoice::~wxChoice()
 
 void wxChoice::SendSelectionChangedEvent(wxEventType evt_type)
 {
 
 void wxChoice::SendSelectionChangedEvent(wxEventType evt_type)
 {

-    if (!m_hasVMT)
-        return;
-

     if (GetSelection() == -1)
         return;
 
     if (GetSelection() == -1)
         return;
 


@@ -117,6 +117,15 @@ void wxChoice::SendSelectionChangedEvent(wxEventType evt_type)
     HandleWindowEvent( event );
 }
 
     HandleWindowEvent( event );
 }
 

+void wxChoice::GTKInsertComboBoxTextItem( unsigned int n, const wxString& text )
+{
+#ifdef __WXGTK3__
+    gtk_combo_box_text_insert_text(GTK_COMBO_BOX_TEXT(m_widget), n, wxGTK_CONV(text));
+#else
+    gtk_combo_box_insert_text( GTK_COMBO_BOX( m_widget ), n, wxGTK_CONV( text ) );
+#endif
+}
+

 int wxChoice::DoInsertItems(const wxArrayStringsAdapter & items,
                             unsigned int pos,
                             void **clientData, wxClientDataType type)
 int wxChoice::DoInsertItems(const wxArrayStringsAdapter & items,
                             unsigned int pos,
                             void **clientData, wxClientDataType type)


@@ -124,22 +133,21 @@ int wxChoice::DoInsertItems(const wxArrayStringsAdapter & items,
     wxCHECK_MSG( m_widget != NULL, -1, wxT("invalid control") );
 
     wxASSERT_MSG( !IsSorted() || (pos == GetCount()),
     wxCHECK_MSG( m_widget != NULL, -1, wxT("invalid control") );
 
     wxASSERT_MSG( !IsSorted() || (pos == GetCount()),

-                 _T("In a sorted choice data could only be appended"));
+                 wxT("In a sorted choice data could only be appended"));

 
     const int count = items.GetCount();
 
     int n = wxNOT_FOUND;
 
 
     const int count = items.GetCount();
 
     int n = wxNOT_FOUND;
 

-    GtkComboBox* combobox = GTK_COMBO_BOX( m_widget );

     for ( int i = 0; i < count; ++i )
     {
         n = pos + i;
         // If sorted, use this wxSortedArrayStrings to determine
         // the right insertion point
     for ( int i = 0; i < count; ++i )
     {
         n = pos + i;
         // If sorted, use this wxSortedArrayStrings to determine
         // the right insertion point

-        if(m_strings)
+        if (m_strings)

             n = m_strings->Add(items[i]);
 
             n = m_strings->Add(items[i]);
 

-        gtk_combo_box_insert_text( combobox, n, wxGTK_CONV( items[i] ) );
+        GTKInsertComboBoxTextItem( n, items[i] );

 
         m_clientData.Insert( NULL, n );
         AssignNewItemClientData(n, clientData, i, type);
 
         m_clientData.Insert( NULL, n );
         AssignNewItemClientData(n, clientData, i, type);


@@ -164,18 +172,18 @@ void wxChoice::DoClear()
 {
     wxCHECK_RET( m_widget != NULL, wxT("invalid control") );
 
 {
     wxCHECK_RET( m_widget != NULL, wxT("invalid control") );
 

-    DisableEvents();
+    GTKDisableEvents();

 
 

-    const unsigned int count = GetCount();
-    for (unsigned int i = 0; i < count; i++)
-        gtk_combo_box_remove_text( GTK_COMBO_BOX(m_widget), 0 );
+    GtkComboBox* combobox = GTK_COMBO_BOX( m_widget );
+    GtkTreeModel* model = gtk_combo_box_get_model( combobox );
+    gtk_list_store_clear(GTK_LIST_STORE(model));

 
     m_clientData.Clear();
 
     if (m_strings)
         m_strings->Clear();
 
 
     m_clientData.Clear();
 
     if (m_strings)
         m_strings->Clear();
 

-    EnableEvents();
+    GTKEnableEvents();

 
     InvalidateBestSize();
 }
 
     InvalidateBestSize();
 }


@@ -183,9 +191,16 @@ void wxChoice::DoClear()
 void wxChoice::DoDeleteOneItem(unsigned int n)
 {
     wxCHECK_RET( m_widget != NULL, wxT("invalid control") );
 void wxChoice::DoDeleteOneItem(unsigned int n)
 {
     wxCHECK_RET( m_widget != NULL, wxT("invalid control") );

-    wxCHECK_RET( IsValid(n), _T("invalid index in wxChoice::Delete") );
+    wxCHECK_RET( IsValid(n), wxT("invalid index in wxChoice::Delete") );
+
+    GtkComboBox* combobox = GTK_COMBO_BOX( m_widget );
+    GtkTreeModel* model = gtk_combo_box_get_model( combobox );
+    GtkListStore* store = GTK_LIST_STORE(model);
+    GtkTreeIter iter;
+    gtk_tree_model_iter_nth_child( model, &iter,
+                                   NULL, (gint) n );
+    gtk_list_store_remove( store, &iter );

 
 

-    gtk_combo_box_remove_text( GTK_COMBO_BOX( m_widget ), n );

     m_clientData.RemoveAt( n );
     if ( m_strings )
         m_strings->RemoveAt( n );
     m_clientData.RemoveAt( n );
     if ( m_strings )
         m_strings->RemoveAt( n );


@@ -207,7 +222,7 @@ int wxChoice::FindString( const wxString &item, bool bCase ) const
     do
     {
         GValue value = { 0, };
     do
     {
         GValue value = { 0, };

-        gtk_tree_model_get_value( model, &iter, 0, &value );
+        gtk_tree_model_get_value( model, &iter, m_stringCellIndex, &value );

         wxString str = wxGTK_CONV_BACK( g_value_get_string( &value ) );
         g_value_unset( &value );
 
         wxString str = wxGTK_CONV_BACK( g_value_get_string( &value ) );
         g_value_unset( &value );
 


@@ -240,7 +255,7 @@ void wxChoice::SetString(unsigned int n, const wxString &text)
         GValue value = { 0, };
         g_value_init( &value, G_TYPE_STRING );
         g_value_set_string( &value, wxGTK_CONV( text ) );
         GValue value = { 0, };
         g_value_init( &value, G_TYPE_STRING );
         g_value_set_string( &value, wxGTK_CONV( text ) );

-        gtk_list_store_set_value( GTK_LIST_STORE(model), &iter, 0, &value );
+        gtk_list_store_set_value( GTK_LIST_STORE(model), &iter, m_stringCellIndex, &value );

         g_value_unset( &value );
     }
 
         g_value_unset( &value );
     }
 


@@ -259,7 +274,7 @@ wxString wxChoice::GetString(unsigned int n) const
     if (gtk_tree_model_iter_nth_child (model, &iter, NULL, n))
     {
         GValue value = { 0, };
     if (gtk_tree_model_iter_nth_child (model, &iter, NULL, n))
     {
         GValue value = { 0, };

-        gtk_tree_model_get_value( model, &iter, 0, &value );
+        gtk_tree_model_get_value( model, &iter, m_stringCellIndex, &value );

         wxString tmp = wxGTK_CONV_BACK( g_value_get_string( &value ) );
         g_value_unset( &value );
         return tmp;
         wxString tmp = wxGTK_CONV_BACK( g_value_get_string( &value ) );
         g_value_unset( &value );
         return tmp;


@@ -288,21 +303,35 @@ void wxChoice::SetSelection( int n )
 {
     wxCHECK_RET( m_widget != NULL, wxT("invalid control") );
 
 {
     wxCHECK_RET( m_widget != NULL, wxT("invalid control") );
 

-    DisableEvents();
+    GTKDisableEvents();

 
     GtkComboBox* combobox = GTK_COMBO_BOX( m_widget );
     gtk_combo_box_set_active( combobox, n );
 
 
     GtkComboBox* combobox = GTK_COMBO_BOX( m_widget );
     gtk_combo_box_set_active( combobox, n );
 

-    EnableEvents();
+    GTKEnableEvents();
+}
+
+void wxChoice::SetColumns(int n)
+{
+    gtk_combo_box_set_wrap_width(GTK_COMBO_BOX(m_widget), n);
+}
+
+int wxChoice::GetColumns() const
+{
+    // gtk_combo_box_get_wrap_width() was added in gtk 2.6
+    gint intval;
+    g_object_get(G_OBJECT(m_widget), "wrap-width", &intval, NULL);
+    return intval;

 }
 
 }
 

-void wxChoice::DisableEvents()
+
+void wxChoice::GTKDisableEvents()

 {
     g_signal_handlers_block_by_func(m_widget,
                                 (gpointer) gtk_choice_changed_callback, this);
 }
 
 {
     g_signal_handlers_block_by_func(m_widget,
                                 (gpointer) gtk_choice_changed_callback, this);
 }
 

-void wxChoice::EnableEvents()
+void wxChoice::GTKEnableEvents()

 {
     g_signal_handlers_unblock_by_func(m_widget,
                                 (gpointer) gtk_choice_changed_callback, this);
 {
     g_signal_handlers_unblock_by_func(m_widget,
                                 (gpointer) gtk_choice_changed_callback, this);


@@ -311,9 +340,25 @@ void wxChoice::EnableEvents()
 
 GdkWindow *wxChoice::GTKGetWindow(wxArrayGdkWindows& WXUNUSED(windows)) const
 {
 
 GdkWindow *wxChoice::GTKGetWindow(wxArrayGdkWindows& WXUNUSED(windows)) const
 {

-    return m_widget->window;
+    return gtk_widget_get_window(m_widget);
+}
+
+wxSize wxChoice::DoGetBestSize() const
+{
+    // Get the height of the control from GTK+ itself, but use our own version
+    // to compute the width large enough to show all our strings as GTK+
+    // doesn't seem to take the control contents into account.
+    return wxSize(wxChoiceBase::DoGetBestSize().x + 40,
+                  wxControl::DoGetBestSize().y);

 }
 
 }
 

+void wxChoice::DoApplyWidgetStyle(GtkRcStyle *style)
+{
+    GTKApplyStyle(m_widget, style);
+    GTKApplyStyle(gtk_bin_get_child(GTK_BIN(m_widget)), style);
+}
+
+

 // static
 wxVisualAttributes
 wxChoice::GetClassDefaultAttributes(wxWindowVariant WXUNUSED(variant))
 // static
 wxVisualAttributes
 wxChoice::GetClassDefaultAttributes(wxWindowVariant WXUNUSED(variant))