Fix accelerstors with down and left

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

diff --git a/src/gtk/checklst.cpp b/src/gtk/checklst.cpp
index e47e7bd8054147c25ddf5c11b5589e5a1a528157..169e56785b24b63d70af67d6abc780a1c7fce70c 100644 (file)
--- a/src/gtk/checklst.cpp
+++ b/src/gtk/checklst.cpp
@@ -1,20 +1,47 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

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

 // Purpose:
 // Author:      Robert Roebling
 // Purpose:
 // Author:      Robert Roebling

+// Modified by: Ryan Norton (Native GTK2.0+ checklist)

 // Id:          $Id$
 // Copyright:   (c) 1998 Robert Roebling
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 // Id:          $Id$
 // Copyright:   (c) 1998 Robert Roebling
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 

-#ifdef __GNUG__
-#pragma implementation "checklst.h"
-#endif
+// For compilers that support precompilation, includes "wx.h".
+#include "wx/wxprec.h"
+
+#if wxUSE_CHECKLISTBOX

 
 #include "wx/checklst.h"
 
 #include "wx/checklst.h"

+#include "wx/gtk/private.h"
+#include "wx/gtk/treeentry_gtk.h"
+
+#include <gdk/gdk.h>
+#include <gtk/gtk.h>

 
 

-#include "gdk/gdk.h"
-#include "gtk/gtk.h"
+//-----------------------------------------------------------------------------
+// "toggled"
+//-----------------------------------------------------------------------------
+#if wxUSE_NATIVEGTKCHECKLIST
+extern "C" {
+static void gtk_checklist_toggled(GtkCellRendererToggle *renderer,
+                                  gchar                 *stringpath,
+                                  wxCheckListBox        *listbox)
+{
+    wxCHECK_RET( listbox->m_treeview != NULL, wxT("invalid listbox") );
+
+    GtkTreePath* path = gtk_tree_path_new_from_string(stringpath);
+    wxCommandEvent new_event( wxEVT_COMMAND_CHECKLISTBOX_TOGGLED,
+                              listbox->GetId() );
+    new_event.SetEventObject( listbox );
+    new_event.SetInt( gtk_tree_path_get_indices(path)[0] );
+    gtk_tree_path_free(path);
+    listbox->Check( new_event.GetInt(), !listbox->IsChecked(new_event.GetInt()));
+    listbox->GetEventHandler()->ProcessEvent( new_event );
+}
+}
+#endif

 
 //-----------------------------------------------------------------------------
 // wxCheckListBox
 
 //-----------------------------------------------------------------------------
 // wxCheckListBox


@@ -22,69 +49,149 @@
 
 IMPLEMENT_DYNAMIC_CLASS(wxCheckListBox,wxListBox)
 
 
 IMPLEMENT_DYNAMIC_CLASS(wxCheckListBox,wxListBox)
 

-wxCheckListBox::wxCheckListBox() :
-   wxListBox()
+wxCheckListBox::wxCheckListBox() : wxListBox()

 {
 {

-    m_hasCheckBoxes = TRUE;
+    m_hasCheckBoxes = true;

 }
 
 wxCheckListBox::wxCheckListBox(wxWindow *parent, wxWindowID id,
 }
 
 wxCheckListBox::wxCheckListBox(wxWindow *parent, wxWindowID id,

-                 const wxPoint& pos,
-                 const wxSize& size,
-                 int nStrings, 
-                 const wxString *choices,
-                 long style,
-                 const wxValidator& validator,
-                 const wxString& name )
+                               const wxPoint& pos,
+                               const wxSize& size,
+                               int nStrings,
+                               const wxString *choices,
+                               long style,
+                               const wxValidator& validator,
+                               const wxString& name )

 {
 {

-    m_hasCheckBoxes = TRUE;
+    m_hasCheckBoxes = true;

     wxListBox::Create( parent, id, pos, size, nStrings, choices, style, validator, name );
 }
 
     wxListBox::Create( parent, id, pos, size, nStrings, choices, style, validator, name );
 }
 

+wxCheckListBox::wxCheckListBox(wxWindow *parent, wxWindowID id,
+                               const wxPoint& pos,
+                               const wxSize& size,
+                               const wxArrayString& choices,
+                               long style,
+                               const wxValidator& validator,
+                               const wxString& name )
+{
+    m_hasCheckBoxes = true;
+    wxListBox::Create( parent, id, pos, size, choices,
+                       style, validator, name );
+}
+
+#if wxUSE_NATIVEGTKCHECKLIST
+void wxCheckListBox::DoCreateCheckList()
+{
+    //Create the checklist in our treeview and set up events for it
+    GtkCellRenderer* renderer =
+        gtk_cell_renderer_toggle_new();
+    GtkTreeViewColumn* column =
+        gtk_tree_view_column_new_with_attributes( "", renderer,
+                                                  "active", 0,
+                                                  NULL );
+    gtk_tree_view_column_set_fixed_width(column, 20);
+    gtk_tree_view_column_set_sizing(column, GTK_TREE_VIEW_COLUMN_FIXED);
+    gtk_tree_view_column_set_clickable(column, TRUE);
+
+    g_signal_connect (renderer, "toggled",
+                      G_CALLBACK (gtk_checklist_toggled),
+                      this);
+
+    gtk_tree_view_append_column(m_treeview, column);
+}
+

 bool wxCheckListBox::IsChecked( int index ) const
 {
 bool wxCheckListBox::IsChecked( int index ) const
 {

-    wxCHECK_MSG( m_list != NULL, FALSE, "invalid checklistbox" );
+    wxCHECK_MSG( m_treeview != NULL, FALSE, wxT("invalid checklistbox") );
+
+    GtkTreeIter iter;
+    gboolean res = gtk_tree_model_iter_nth_child(
+                        GTK_TREE_MODEL(m_liststore),
+                        &iter, NULL, //NULL = parent = get first
+                        index
+                                                );
+    if(!res)
+        return false;
+
+    GValue value = {0, };
+    gtk_tree_model_get_value(GTK_TREE_MODEL(m_liststore),
+                             &iter,
+                             0, //column
+                             &value);
+
+    return g_value_get_boolean(&value) == TRUE ? true : false;
+}

 
 

-    GList *child = g_list_nth( m_list->children, index );
-    if (child)
-    {
-        GtkBin *bin = GTK_BIN( child->data );
-        GtkLabel *label = GTK_LABEL( bin->child );
+void wxCheckListBox::Check( int index, bool check )
+{
+    wxCHECK_RET( m_treeview != NULL, wxT("invalid checklistbox") );
+
+    GtkTreeIter iter;
+    gboolean res = gtk_tree_model_iter_nth_child(
+                        GTK_TREE_MODEL(m_liststore),
+                        &iter, NULL, //NULL = parent = get first
+                        index
+                                                );
+    if(!res)
+        return;
+
+    gtk_list_store_set(m_liststore,
+                       &iter,
+                       0, //column
+                       check ? TRUE : FALSE, -1);
+}
+
+int wxCheckListBox::GetItemHeight() const
+{
+    wxCHECK_MSG( m_treeview != NULL, 0, wxT("invalid listbox"));
+
+    gint height;
+    gtk_tree_view_column_cell_get_size(
+        gtk_tree_view_get_column(m_treeview, 0),
+                                       NULL, NULL, NULL, NULL,
+                                       &height);
+    return height;
+}

 
 

-        wxString str = label->label;
+#else //NON-NATIVE

 
 

-        return (str[1] == 'X');
+bool wxCheckListBox::IsChecked( int index ) const
+{
+    wxCHECK_MSG( m_treeview != NULL, false, wxT("invalid checklistbox") );
+
+    GtkTreeEntry* entry = GtkGetEntry(index);
+    if (entry)
+    {
+        wxString str( wxGTK_CONV_BACK( gtk_tree_entry_get_label(entry) ) );
+
+        return str.GetChar(1) == wxCHECKLBOX_CHECKED;

     }
 
     }
 

-    wxFAIL_MSG("wrong checklistbox index");
-    return FALSE;
+    wxFAIL_MSG(wxT("wrong checklistbox index"));
+    return false;

 }
 
 void wxCheckListBox::Check( int index, bool check )
 {
 }
 
 void wxCheckListBox::Check( int index, bool check )
 {

-    wxCHECK_RET( m_list != NULL, "invalid checklistbox" );
+    wxCHECK_RET( m_treeview != NULL, wxT("invalid checklistbox") );

 
 

-    GList *child = g_list_nth( m_list->children, index );
-    if (child)
+    GtkTreeEntry* entry = GtkGetEntry(index);
+    if (entry)

     {
     {

-        GtkBin *bin = GTK_BIN( child->data );
-        GtkLabel *label = GTK_LABEL( bin->child );
-
-        wxString str = label->label;
+        wxString str( wxGTK_CONV_BACK( gtk_tree_entry_get_label(entry) ) );

 
 

-        if (check == (str[1] == 'X')) return;
+        if (check == (str.GetChar(1) == wxCHECKLBOX_CHECKED))
+            return;

 
 

-        if (check)
-            str.SetChar( 1, 'X' );
-        else
-            str.SetChar( 1, '-' );
+        str.SetChar( 1, check ? wxCHECKLBOX_CHECKED : wxCHECKLBOX_UNCHECKED );

 
 

-        gtk_label_set( label, str );
+        gtk_tree_entry_set_label( entry, wxGTK_CONV( str ) );

 
         return;
     }
 
 
         return;
     }
 

-    wxFAIL_MSG("wrong checklistbox index");
+    wxFAIL_MSG(wxT("wrong checklistbox index"));

 }
 
 int wxCheckListBox::GetItemHeight() const
 }
 
 int wxCheckListBox::GetItemHeight() const


@@ -92,3 +199,7 @@ int wxCheckListBox::GetItemHeight() const
     // FIXME
     return 22;
 }
     // FIXME
     return 22;
 }

+
+#endif //wxUSE_NATIVEGTKCHECKLIST
+
+#endif //wxUSE_CHECKLISTBOX