gimp r25979 - in trunk: . app/widgets

[mitch svn gnome org]

Author: mitch
Date: Tue Jun 24 08:42:49 2008
New Revision: 25979
URL: http://svn.gnome.org/viewvc/gimp?rev=25979&view=rev

Log:
2008-06-24  Michael Natterer  <mitch gimp org>

	* app/widgets/gimpcontainercombobox.[ch]: made the model column
	enum public and namespaced it.

	* app/widgets/gimpsettingsbox.c: use the enum value instead of a
	magic number.

	* app/widgets/gimpsettingseditor.c: add a separator between
	recently used settings and favorites.



Modified:
   trunk/ChangeLog
   trunk/app/widgets/gimpcontainercombobox.c
   trunk/app/widgets/gimpcontainercombobox.h
   trunk/app/widgets/gimpsettingsbox.c
   trunk/app/widgets/gimpsettingseditor.c

Modified: trunk/app/widgets/gimpcontainercombobox.c
==============================================================================
--- trunk/app/widgets/gimpcontainercombobox.c	(original)
+++ trunk/app/widgets/gimpcontainercombobox.c	Tue Jun 24 08:42:49 2008
@@ -43,12 +43,6 @@
   PROP_ELLIPSIZE = GIMP_CONTAINER_VIEW_PROP_LAST + 1
 };
 
-enum
-{
-  COLUMN_RENDERER,
-  COLUMN_NAME,
-  NUM_COLUMNS
-};
 
 static void     gimp_container_combo_box_view_iface_init (GimpContainerViewInterface *iface);
 
@@ -144,7 +138,7 @@
   GtkCellLayout   *layout;
   GtkCellRenderer *cell;
 
-  store = gtk_list_store_new (NUM_COLUMNS,
+  store = gtk_list_store_new (GIMP_CONTAINER_COMBO_BOX_N_COLUMNS,
                               GIMP_TYPE_VIEW_RENDERER,
                               G_TYPE_STRING);
 
@@ -157,7 +151,8 @@
   cell = gimp_cell_renderer_viewable_new ();
   gtk_cell_layout_pack_start (layout, cell, FALSE);
   gtk_cell_layout_set_attributes (layout, cell,
-                                  "renderer", COLUMN_RENDERER,
+                                  "renderer",
+                                  GIMP_CONTAINER_COMBO_BOX_COLUMN_RENDERER,
                                   NULL);
 
   combo_box->viewable_renderer = cell;
@@ -165,7 +160,8 @@
   cell = gtk_cell_renderer_text_new ();
   gtk_cell_layout_pack_start (layout, cell, TRUE);
   gtk_cell_layout_set_attributes (layout, cell,
-                                  "text", COLUMN_NAME,
+                                  "text",
+                                  GIMP_CONTAINER_COMBO_BOX_COLUMN_NAME,
                                   NULL);
 
   combo_box->text_renderer = cell;
@@ -277,8 +273,8 @@
                     view);
 
   gtk_list_store_set (GTK_LIST_STORE (model), iter,
-                      COLUMN_RENDERER, renderer,
-                      COLUMN_NAME,     name,
+                      GIMP_CONTAINER_COMBO_BOX_COLUMN_RENDERER, renderer,
+                      GIMP_CONTAINER_COMBO_BOX_COLUMN_NAME,     name,
                       -1);
 
   g_object_unref (renderer);
@@ -308,7 +304,7 @@
           GimpViewRenderer *renderer;
 
           gtk_tree_model_get (model, &iter,
-                              COLUMN_RENDERER, &renderer,
+                              GIMP_CONTAINER_COMBO_BOX_COLUMN_RENDERER, &renderer,
                               -1);
 
           gimp_view_renderer_set_context (renderer, context);
@@ -429,7 +425,7 @@
       gchar *name = gimp_viewable_get_description (viewable, NULL);
 
       gtk_list_store_set (GTK_LIST_STORE (model), iter,
-                          COLUMN_NAME, name,
+                          GIMP_CONTAINER_COMBO_BOX_COLUMN_NAME, name,
                           -1);
 
       g_free (name);
@@ -494,7 +490,7 @@
       GimpViewRenderer *renderer;
 
       gtk_tree_model_get (model, &iter,
-                          COLUMN_RENDERER, &renderer,
+                          GIMP_CONTAINER_COMBO_BOX_COLUMN_RENDERER, &renderer,
                           -1);
 
       gimp_view_renderer_set_size (renderer, view_size, border_width);
@@ -516,7 +512,7 @@
       GimpViewRenderer *renderer;
 
       gtk_tree_model_get (gtk_combo_box_get_model (combo_box), &iter,
-                          COLUMN_RENDERER, &renderer,
+                          GIMP_CONTAINER_COMBO_BOX_COLUMN_RENDERER, &renderer,
                           -1);
 
       gimp_container_view_item_selected (view, renderer->viewable);

Modified: trunk/app/widgets/gimpcontainercombobox.h
==============================================================================
--- trunk/app/widgets/gimpcontainercombobox.h	(original)
+++ trunk/app/widgets/gimpcontainercombobox.h	Tue Jun 24 08:42:49 2008
@@ -23,6 +23,14 @@
 #define __GIMP_CONTAINER_COMBO_BOX_H__
 
 
+enum
+{
+  GIMP_CONTAINER_COMBO_BOX_COLUMN_RENDERER,
+  GIMP_CONTAINER_COMBO_BOX_COLUMN_NAME,
+  GIMP_CONTAINER_COMBO_BOX_N_COLUMNS
+};
+
+
 #define GIMP_TYPE_CONTAINER_COMBO_BOX            (gimp_container_combo_box_get_type ())
 #define GIMP_CONTAINER_COMBO_BOX(obj)            (G_TYPE_CHECK_INSTANCE_CAST ((obj), GIMP_TYPE_CONTAINER_COMBO_BOX, GimpContainerComboBox))
 #define GIMP_CONTAINER_COMBO_BOX_CLASS(klass)    (G_TYPE_CHECK_CLASS_CAST ((klass), GIMP_TYPE_CONTAINER_COMBO_BOX, GimpContainerComboBoxClass))

Modified: trunk/app/widgets/gimpsettingsbox.c
==============================================================================
--- trunk/app/widgets/gimpsettingsbox.c	(original)
+++ trunk/app/widgets/gimpsettingsbox.c	Tue Jun 24 08:42:49 2008
@@ -459,13 +459,9 @@
 {
   gchar *name = NULL;
 
-#ifdef __GNUC__
-#warning FIXME: dont use magic model column
-#endif
   gtk_tree_model_get (model, iter,
-                      1, &name,
+                      GIMP_CONTAINER_COMBO_BOX_COLUMN_NAME, &name,
                       -1);
-
   g_free (name);
 
   return name == NULL;

Modified: trunk/app/widgets/gimpsettingseditor.c
==============================================================================
--- trunk/app/widgets/gimpsettingseditor.c	(original)
+++ trunk/app/widgets/gimpsettingseditor.c	Tue Jun 24 08:42:49 2008
@@ -63,6 +63,11 @@
                                                      GValue            *value,
                                                      GParamSpec        *pspec);
 
+static gboolean
+            gimp_settings_editor_row_separator_func (GtkTreeModel      *model,
+                                                     GtkTreeIter       *iter,
+                                                     gpointer           data);
+
 
 G_DEFINE_TYPE (GimpSettingsEditor, gimp_settings_editor, GTK_TYPE_VBOX)
 
@@ -127,6 +132,9 @@
   view = gimp_container_tree_view_new (editor->container,
                                        gimp_get_user_context (editor->gimp),
                                        16, 0);
+  gtk_tree_view_set_row_separator_func (GIMP_CONTAINER_TREE_VIEW (view)->view,
+                                        gimp_settings_editor_row_separator_func,
+                                        view, NULL);
   gtk_container_add (GTK_CONTAINER (editor), view);
   gtk_widget_show (view);
 
@@ -209,6 +217,22 @@
     }
 }
 
+static gboolean
+gimp_settings_editor_row_separator_func (GtkTreeModel *model,
+                                         GtkTreeIter  *iter,
+                                         gpointer      data)
+{
+  GimpContainerTreeView *view = GIMP_CONTAINER_TREE_VIEW (data);
+  gchar                 *name = NULL;
+
+  gtk_tree_model_get (model, iter,
+                      view->model_column_name, &name,
+                      -1);
+  g_free (name);
+
+  return name == NULL;
+}
+
 
 /*  public functions  */