# HG changeset patch
# User Richard Laager <rlaager@wiktel.com>
# Date 1143358866 0
# Node ID 6034444bbcf5d88edc11a0fb393200f289de7d25
# Parent  724119af64f3f3cf5c98b4adab5af46222893fc3
[gaim-migrate @ 15947]
Rest of SF Patch #1453583 from Sadrul

Sadrul's description of this patch:

This tweaks a few things in the log viewer:
 - clear the imhtml when you press 'Find'
 - select the first entry (if any) in the treeview
 after search, or after clearing search
 - make autoselect the first entry in the treeview
 work for GTK < 2.2
 - "activate" on buttons shouldn't be used.
 ('Applications should never connect to this signal, but
 use the "clicked" signal')
 - uses glist functions instead of doing things manually
 - fixes a small memleak when there's no log

committer: Tailor Script <tailor@pidgin.im>

diff -r 724119af64f3 -r 6034444bbcf5 src/gtklog.c
--- a/src/gtklog.c	Sun Mar 26 07:33:03 2006 +0000
+++ b/src/gtklog.c	Sun Mar 26 07:41:06 2006 +0000
@@ -83,6 +83,29 @@
 	return ret;
 }
 
+static void select_first_log(GaimGtkLogViewer *lv)
+{
+	GtkTreeModel *model;
+	GtkTreeIter iter, it;
+	GtkTreePath *path;
+
+	model = GTK_TREE_MODEL(lv->treestore);
+
+	if (!gtk_tree_model_get_iter_first(model, &iter))
+		return;
+
+	path = gtk_tree_model_get_path(model, &iter);
+	if (gtk_tree_model_iter_children(model, &it, &iter))
+	{
+		gtk_tree_view_expand_row(GTK_TREE_VIEW(lv->treeview), path, TRUE);
+		path = gtk_tree_model_get_path(model, &it);
+	}
+
+	gtk_tree_selection_select_path(gtk_tree_view_get_selection(GTK_TREE_VIEW(lv->treeview)), path);
+
+	gtk_tree_path_free(path);
+}
+
 static void search_cb(GtkWidget *button, GaimGtkLogViewer *lv)
 {
 	const char *search_term = gtk_entry_get_text(GTK_ENTRY(lv->entry));
@@ -97,10 +120,12 @@
 		populate_log_tree(lv);
 		lv->search = NULL;
 		gtk_imhtml_search_clear(GTK_IMHTML(lv->imhtml));
+		select_first_log(lv);
 		return;
 	}
 
 	lv->search = g_strdup(search_term);
+	gtk_imhtml_clear(GTK_IMHTML(lv->imhtml));
 
 	gaim_gtk_set_cursor(lv->window, GDK_WATCH);
 
@@ -118,6 +143,7 @@
 		g_free(read);
 	}
 
+	select_first_log(lv);
 	gaim_gtk_clear_cursor(lv->window);
 }
 
@@ -135,15 +161,8 @@
 	} else
 		syslog_viewer = NULL;
 
-	while (lv->logs != NULL) {
-		GList *logs2;
-
-		gaim_log_free((GaimLog *)lv->logs->data);
-
-		logs2 = lv->logs->next;
-		g_list_free_1(lv->logs);
-		lv->logs = logs2;
-	}
+	g_list_foreach(lv->logs, (GFunc)gaim_log_free, NULL);
+	g_list_free(lv->logs);
 
 	if (lv->search != NULL)
 		g_free(lv->search);
@@ -273,18 +292,12 @@
 	GtkCellRenderer *rend;
 	GtkTreeViewColumn *col;
 	GtkTreeSelection *sel;
-#if GTK_CHECK_VERSION(2,2,0)
-	GtkTreePath *path_to_first_log;
-#endif
 	GtkWidget *vbox;
 	GtkWidget *frame;
 	GtkWidget *hbox;
 	GtkWidget *button;
 	GtkWidget *size_label;
 
-	lv = g_new0(GaimGtkLogViewer, 1);
-	lv->logs = logs;
-
 	if (logs == NULL)
 	{
 		/* No logs were found. */
@@ -307,6 +320,9 @@
 		return NULL;
 	}
 
+	lv = g_new0(GaimGtkLogViewer, 1);
+	lv->logs = logs;
+
 	if (ht != NULL)
 		g_hash_table_insert(log_viewers, ht, lv);
 
@@ -408,19 +424,9 @@
 	button = gtk_button_new_from_stock(GTK_STOCK_FIND);
 	gtk_box_pack_start(GTK_BOX(hbox), button, FALSE, FALSE, 0);
 	g_signal_connect(GTK_ENTRY(lv->entry), "activate", G_CALLBACK(search_cb), lv);
-	g_signal_connect(GTK_BUTTON(button), "activate", G_CALLBACK(search_cb), lv);
 	g_signal_connect(GTK_BUTTON(button), "clicked", G_CALLBACK(search_cb), lv);
 
-#if GTK_CHECK_VERSION(2,2,0)
-	/* Show most recent log **********/
-	path_to_first_log = gtk_tree_path_new_from_string("0:0");
-	if (path_to_first_log)
-	{
-		gtk_tree_view_expand_to_path(GTK_TREE_VIEW(lv->treeview), path_to_first_log);
-		gtk_tree_selection_select_path(sel, path_to_first_log);
-		gtk_tree_path_free(path_to_first_log);
-	}
-#endif
+	select_first_log(lv);
 
 	gtk_widget_show_all(lv->window);