# HG changeset patch # User Richard Laager # Date 1156818128 0 # Node ID c2fe0f54c389a3d0f7022a0ebb8b19e8c0ee5723 # Parent 66cb461d6079b25e0640248bb1f22d15cfbcd8b1 [gaim-migrate @ 17076] SF Patch #1547720 from Hilbert Fixes SF Bug #1541097 If you search, leave the Find dialog open, switch tabs, and search again, the search happens in the original tab. This patch fixes that bug. Now when you search, it searches in the active conversation window. committer: Tailor Script diff -r 66cb461d6079 -r c2fe0f54c389 COPYRIGHT --- a/COPYRIGHT Tue Aug 29 01:58:14 2006 +0000 +++ b/COPYRIGHT Tue Aug 29 02:22:08 2006 +0000 @@ -129,6 +129,7 @@ Mike Heffner Benjamin Herrenschmidt Fernando Herrera +Hilbert Casey Ho Iain Holmes Joshua Honeycutt diff -r 66cb461d6079 -r c2fe0f54c389 doc/CREDITS --- a/doc/CREDITS Tue Aug 29 01:58:14 2006 +0000 +++ b/doc/CREDITS Tue Aug 29 02:22:08 2006 +0000 @@ -1,5 +1,8 @@ gaim: The Pimpin' Penguin IM Clone that's Good For The Soul! +This file is very old. Newer developers and contributors are +credited in the AUTHORS and COPYRIGHT files. + CREDITS ------- Authors: diff -r 66cb461d6079 -r c2fe0f54c389 doc/ChangeLog.API --- a/doc/ChangeLog.API Tue Aug 29 01:58:14 2006 +0000 +++ b/doc/ChangeLog.API Tue Aug 29 02:22:08 2006 +0000 @@ -193,6 +193,7 @@ GaimConversation * Removed ui_ops from GaimBuddyList. Use gaim_blist_get_ui_ops() instead * GaimGtkLogViewer hidden... You weren't using it anyway. + * GaimGtkConversation: dialogs (dialogs.search moved to GaimGtkWindow) Added: * gaim_prefs_disconnect_by_handle() @@ -315,6 +316,7 @@ query. * gaim_util_fetch_url_cancel(): Can be used to cancel a pending call to gaim_util_fetch_url() or gaim_util_fetch_url_request(). + * GaimGtkWindow: dialogs.search (previously in GaimGtkConversation) Signals - Changed: (See the Doxygen docs for details on all signals.) * Signal propagation now stops after a handler returns a non-NULL value. diff -r 66cb461d6079 -r c2fe0f54c389 gtk/gtkconv.c --- a/gtk/gtkconv.c Tue Aug 29 01:58:14 2006 +0000 +++ b/gtk/gtkconv.c Tue Aug 29 02:22:08 2006 +0000 @@ -1039,25 +1039,53 @@ } struct _search { - GaimGtkConversation *gtkconv; + GaimGtkWindow *gtkwin; GtkWidget *entry; }; static void do_search_cb(GtkWidget *widget, gint resp, struct _search *s) { - switch (resp) { - case GTK_RESPONSE_OK: - gtk_imhtml_search_find(GTK_IMHTML(s->gtkconv->imhtml), - gtk_entry_get_text(GTK_ENTRY(s->entry))); - break; - - case GTK_RESPONSE_DELETE_EVENT: - case GTK_RESPONSE_CLOSE: - gtk_imhtml_search_clear(GTK_IMHTML(s->gtkconv->imhtml)); - gtk_widget_destroy(s->gtkconv->dialogs.search); - s->gtkconv->dialogs.search = NULL; - g_free(s); - break; + GaimConversation *conv; + GaimGtkConversation *gtk_active_conv; + GList *iter; + + conv = gaim_gtk_conv_window_get_active_conversation(s->gtkwin); + gtk_active_conv = GAIM_GTK_CONVERSATION(conv); + + switch (resp) + { + case GTK_RESPONSE_OK: + /* clear highlighting except the active conversation window + * highlight the keywords in the active conversation window */ + for (iter = gaim_gtk_conv_window_get_gtkconvs(s->gtkwin) ; iter ; iter = iter->next) + { + GaimGtkConversation *gtkconv = iter->data; + + if (gtkconv != gtk_active_conv) + { + gtk_imhtml_search_clear(GTK_IMHTML(gtkconv->imhtml)); + } + else + { + gtk_imhtml_search_find(GTK_IMHTML(gtk_active_conv->imhtml), + gtk_entry_get_text(GTK_ENTRY(s->entry))); + } + } + break; + + case GTK_RESPONSE_DELETE_EVENT: + case GTK_RESPONSE_CLOSE: + /* clear the keyword highlighting in all the conversation windows */ + for (iter = gaim_gtk_conv_window_get_gtkconvs(s->gtkwin); iter; iter=iter->next) + { + GaimGtkConversation *gconv = iter->data; + gtk_imhtml_search_clear(GTK_IMHTML(gconv->imhtml)); + } + + gtk_widget_destroy(s->gtkwin->dialogs.search); + s->gtkwin->dialogs.search = NULL; + g_free(s); + break; } } @@ -1065,45 +1093,43 @@ menu_find_cb(gpointer data, guint action, GtkWidget *widget) { GaimGtkWindow *gtkwin = data; - GaimConversation *conv = gaim_gtk_conv_window_get_active_conversation(gtkwin); - GaimGtkConversation *gtkconv = GAIM_GTK_CONVERSATION(conv); GtkWidget *hbox; GtkWidget *img = gtk_image_new_from_stock(GAIM_STOCK_DIALOG_QUESTION, GTK_ICON_SIZE_DIALOG); GtkWidget *label; struct _search *s; - if (gtkconv->dialogs.search) { - gtk_window_present(GTK_WINDOW(gtkconv->dialogs.search)); + if (gtkwin->dialogs.search) { + gtk_window_present(GTK_WINDOW(gtkwin->dialogs.search)); return; } s = g_malloc(sizeof(struct _search)); - s->gtkconv = gtkconv; - - gtkconv->dialogs.search = gtk_dialog_new_with_buttons(_("Find"), + s->gtkwin = gtkwin; + + gtkwin->dialogs.search = gtk_dialog_new_with_buttons(_("Find"), GTK_WINDOW(gtkwin->window), GTK_DIALOG_DESTROY_WITH_PARENT, GTK_STOCK_CLOSE, GTK_RESPONSE_CLOSE, GTK_STOCK_FIND, GTK_RESPONSE_OK, NULL); - gtk_dialog_set_default_response(GTK_DIALOG(gtkconv->dialogs.search), + gtk_dialog_set_default_response(GTK_DIALOG(gtkwin->dialogs.search), GTK_RESPONSE_OK); - g_signal_connect(G_OBJECT(gtkconv->dialogs.search), "response", + g_signal_connect(G_OBJECT(gtkwin->dialogs.search), "response", G_CALLBACK(do_search_cb), s); - gtk_container_set_border_width(GTK_CONTAINER(gtkconv->dialogs.search), GAIM_HIG_BOX_SPACE); - gtk_window_set_resizable(GTK_WINDOW(gtkconv->dialogs.search), FALSE); - gtk_dialog_set_has_separator(GTK_DIALOG(gtkconv->dialogs.search), FALSE); - gtk_box_set_spacing(GTK_BOX(GTK_DIALOG(gtkconv->dialogs.search)->vbox), GAIM_HIG_BORDER); + gtk_container_set_border_width(GTK_CONTAINER(gtkwin->dialogs.search), GAIM_HIG_BOX_SPACE); + gtk_window_set_resizable(GTK_WINDOW(gtkwin->dialogs.search), FALSE); + gtk_dialog_set_has_separator(GTK_DIALOG(gtkwin->dialogs.search), FALSE); + gtk_box_set_spacing(GTK_BOX(GTK_DIALOG(gtkwin->dialogs.search)->vbox), GAIM_HIG_BORDER); gtk_container_set_border_width( - GTK_CONTAINER(GTK_DIALOG(gtkconv->dialogs.search)->vbox), GAIM_HIG_BOX_SPACE); + GTK_CONTAINER(GTK_DIALOG(gtkwin->dialogs.search)->vbox), GAIM_HIG_BOX_SPACE); hbox = gtk_hbox_new(FALSE, GAIM_HIG_BORDER); - gtk_container_add(GTK_CONTAINER(GTK_DIALOG(gtkconv->dialogs.search)->vbox), + gtk_container_add(GTK_CONTAINER(GTK_DIALOG(gtkwin->dialogs.search)->vbox), hbox); gtk_box_pack_start(GTK_BOX(hbox), img, FALSE, FALSE, 0); gtk_misc_set_alignment(GTK_MISC(img), 0, 0); - gtk_dialog_set_response_sensitive(GTK_DIALOG(gtkconv->dialogs.search), + gtk_dialog_set_response_sensitive(GTK_DIALOG(gtkwin->dialogs.search), GTK_RESPONSE_OK, FALSE); label = gtk_label_new(NULL); @@ -1115,10 +1141,10 @@ gtk_label_set_mnemonic_widget(GTK_LABEL(label), GTK_WIDGET(s->entry)); g_signal_connect(G_OBJECT(s->entry), "changed", G_CALLBACK(gaim_gtk_set_sensitive_if_input), - gtkconv->dialogs.search); + gtkwin->dialogs.search); gtk_box_pack_start(GTK_BOX(hbox), s->entry, FALSE, FALSE, 0); - gtk_widget_show_all(gtkconv->dialogs.search); + gtk_widget_show_all(gtkwin->dialogs.search); gtk_widget_grab_focus(s->entry); } @@ -4474,10 +4500,6 @@ gaim_request_close_with_handle(gtkconv); gaim_notify_close_with_handle(gtkconv); - /* Close the "Find" dialog if it's open */ - if (gtkconv->dialogs.search) - gtk_widget_destroy(gtkconv->dialogs.search); - gtk_widget_destroy(gtkconv->tab_cont); g_object_unref(gtkconv->tab_cont); @@ -7481,6 +7503,10 @@ gaim_prefs_disconnect_by_handle(win); window_list = g_list_remove(window_list, win); + /* Close the "Find" dialog if it's open */ + if (win->dialogs.search) + gtk_widget_destroy(win->dialogs.search); + if (win->gtkconvs) { while (win->gtkconvs) { GList *nextgtk = win->gtkconvs->next; diff -r 66cb461d6079 -r c2fe0f54c389 gtk/gtkconv.h --- a/gtk/gtkconv.h Tue Aug 29 01:58:14 2006 +0000 +++ b/gtk/gtkconv.h Tue Aug 29 02:22:08 2006 +0000 @@ -142,13 +142,6 @@ GaimUnseenState unseen_state; guint unseen_count; - struct - { - GtkWidget *image; - GtkWidget *search; - - } dialogs; - union { GaimGtkImPane *im; diff -r 66cb461d6079 -r c2fe0f54c389 gtk/gtkconvwin.h --- a/gtk/gtkconvwin.h Tue Aug 29 01:58:14 2006 +0000 +++ b/gtk/gtkconvwin.h Tue Aug 29 02:22:08 2006 +0000 @@ -78,6 +78,12 @@ } menu; + struct + { + GtkWidget *search; + + } dialogs; + /* Tab dragging stuff. */ gboolean in_drag; gboolean in_predrag;