[Xfce4-commits] <exo:xfce-4.6> Make the treeview work again with gtk 2.20 (bug #6230).

[Nick Schermer]

Updating branch refs/heads/xfce-4.6
         to c85fffcfa78dec9ae12251b9d2974927745ba7ce (commit)
       from f38ceb4d9358e51fe5f310e62a92de96bb60cdbf (commit)

commit c85fffcfa78dec9ae12251b9d2974927745ba7ce
Author: Nick Schermer <nick at xfce.org>
Date:   Tue May 11 20:57:11 2010 +0200

    Make the treeview work again with gtk 2.20 (bug #6230).

 exo/exo-tree-view.c |   18 ++++++++++--------
 1 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/exo/exo-tree-view.c b/exo/exo-tree-view.c
index cfc5b1b..61bafc7 100644
--- a/exo/exo-tree-view.c
+++ b/exo/exo-tree-view.c
@@ -378,14 +378,7 @@ exo_tree_view_button_press_event (GtkWidget      *widget,
       && path != NULL && gtk_tree_selection_path_is_selected (selection, path))
     {
       /* check if we have to restore paths */
-      if (G_LIKELY (selection->user_func == (GtkTreeSelectionFunc) exo_noop_false))
-        {
-          /* just reset the select function (previously set to exo_noop_false),
-           * there's no clean way to do this, so what the heck.
-           */
-          selection->user_func = NULL;
-        }
-      else
+      if (G_LIKELY (selection->user_func != (GtkTreeSelectionFunc) exo_noop_false))
         {
           /* select all previously selected paths */
           for (lp = selected_paths; lp != NULL; lp = lp->next)
@@ -393,6 +386,15 @@ exo_tree_view_button_press_event (GtkWidget      *widget,
         }
     }
 
+  /* see bug http://bugzilla.xfce.org/show_bug.cgi?id=6230 for more information */
+  if (G_LIKELY (selection->user_func == (GtkTreeSelectionFunc) exo_noop_false))
+    {
+      /* just reset the select function (previously set to exo_noop_false),
+       * there's no clean way to do this, so what the heck.
+       */
+      selection->user_func = NULL;
+    }
+
   /* release the path (if any) */
   if (G_LIKELY (path != NULL))
     gtk_tree_path_free (path);