gdm r6166 - in trunk: . gui/simple-greeter

[halfline svn gnome org]

Author: halfline
Date: Mon Apr 21 21:08:14 2008
New Revision: 6166
URL: http://svn.gnome.org/viewvc/gdm?rev=6166&view=rev

Log:
2008-04-21  Ray Strode  <rstrode redhat com>

	* gui/simple-greeter/gdm-chooser-widget.c
	(set_inactive_items_visible): Don't unconditionally
	show separator when showing all other inactive items


Modified:
   trunk/ChangeLog
   trunk/gui/simple-greeter/gdm-chooser-widget.c

Modified: trunk/gui/simple-greeter/gdm-chooser-widget.c
==============================================================================
--- trunk/gui/simple-greeter/gdm-chooser-widget.c	(original)
+++ trunk/gui/simple-greeter/gdm-chooser-widget.c	Mon Apr 21 21:08:14 2008
@@ -415,6 +415,35 @@
 }
 
 static void
+update_separator_visibility (GdmChooserWidget *widget)
+{
+        GtkTreePath *separator_path;
+        GtkTreeIter  iter;
+        gboolean     is_visible;
+
+        separator_path = gtk_tree_row_reference_get_path (widget->priv->separator_row);
+
+        if (separator_path == NULL) {
+                return;
+        }
+
+        gtk_tree_model_get_iter (GTK_TREE_MODEL (widget->priv->list_store),
+                                 &iter, separator_path);
+
+        if (widget->priv->number_of_normal_rows > 0 &&
+            widget->priv->number_of_separated_rows > 0) {
+                is_visible = TRUE;
+        } else {
+                is_visible = FALSE;
+        }
+
+        gtk_list_store_set (widget->priv->list_store,
+                            &iter,
+                            CHOOSER_ITEM_IS_VISIBLE_COLUMN, is_visible,
+                            -1);
+}
+
+static void
 set_inactive_items_visible (GdmChooserWidget *widget,
                             gboolean          should_show)
 {
@@ -463,6 +492,8 @@
         } while (gtk_tree_model_iter_next (model, &iter));
 
         g_free (active_item_id);
+
+        update_separator_visibility (widget);
 }
 
 static void
@@ -1383,35 +1414,6 @@
 }
 
 static void
-update_separator_visibility (GdmChooserWidget *widget)
-{
-        GtkTreePath *separator_path;
-        GtkTreeIter  iter;
-        gboolean     is_visible;
-
-        separator_path = gtk_tree_row_reference_get_path (widget->priv->separator_row);
-
-        if (separator_path == NULL) {
-                return;
-        }
-
-        gtk_tree_model_get_iter (GTK_TREE_MODEL (widget->priv->list_store),
-                                 &iter, separator_path);
-
-        if (widget->priv->number_of_normal_rows > 0 &&
-            widget->priv->number_of_separated_rows > 0) {
-                is_visible = TRUE;
-        } else {
-                is_visible = FALSE;
-        }
-
-        gtk_list_store_set (widget->priv->list_store,
-                            &iter,
-                            CHOOSER_ITEM_IS_VISIBLE_COLUMN, is_visible,
-                            -1);
-}
-
-static void
 clear_canceled_visibility_update (GdmChooserWidget *widget)
 {
         widget->priv->update_idle_id = 0;