Mercurial > pidgin
diff src/gtkconv.c @ 13505:7a6763bbf82b
[gaim-migrate @ 15881]
On Mon, 2006-03-13 at 15:10 -0500, Adil wrote:
> Would it be `better' to not print an error message for the event
> (because it is possible to have a chat-window for a disconnected
> account), and popup a menu with disabled entries instead?
This commit implements that. I've also removed some code duplication by having create_chat_menu() look up the prpl_info from the gc.
Also, I think we should grey out the Send File menu item when a particular user doesn't have the ability to receive files, rather than not show it. It's more consistent with what we do elsewhere. I've made that change as well.
committer: Tailor Script <tailor@pidgin.im>
author | Richard Laager <rlaager@wiktel.com> |
---|---|
date | Tue, 14 Mar 2006 06:15:31 +0000 |
parents | 2d2a52b0bfa5 |
children | cd21f1738063 |
line wrap: on
line diff
--- a/src/gtkconv.c Tue Mar 14 05:18:37 2006 +0000 +++ b/src/gtkconv.c Tue Mar 14 06:15:31 2006 +0000 @@ -1535,14 +1535,17 @@ } static GtkWidget * -create_chat_menu(GaimConversation *conv, const char *who, - GaimPluginProtocolInfo *prpl_info, GaimConnection *gc) +create_chat_menu(GaimConversation *conv, const char *who, GaimConnection *gc) { static GtkWidget *menu = NULL; + GaimPluginProtocolInfo *prpl_info = NULL; GaimConvChat *chat = GAIM_CONV_CHAT(conv); gboolean is_me = FALSE; GtkWidget *button; + if (gc != NULL) + prpl_info = GAIM_PLUGIN_PROTOCOL_INFO(gc->prpl); + /* * If a menu already exists, destroy it before creating a new one, * thus freeing-up the memory it occupied. @@ -1558,44 +1561,73 @@ if (!is_me) { button = gaim_new_item_from_stock(menu, _("IM"), GAIM_STOCK_IM, G_CALLBACK(menu_chat_im_cb), GAIM_GTK_CONVERSATION(conv), 0, 0, NULL); + + if (gc == NULL) + gtk_widget_set_sensitive(button, FALSE); + g_object_set_data_full(G_OBJECT(button), "user_data", g_strdup(who), g_free); - if (prpl_info && prpl_info->send_file - && (!prpl_info->can_receive_file || prpl_info->can_receive_file(gc, who))) { + + if (prpl_info && prpl_info->send_file) + { button = gaim_new_item_from_stock(menu, _("Send File"), GAIM_STOCK_FILE_TRANSFER, G_CALLBACK(menu_chat_send_file_cb), GAIM_GTK_CONVERSATION(conv), 0, 0, NULL); + + if (gc == NULL || prpl_info == NULL || + !(!prpl_info->can_receive_file || prpl_info->can_receive_file(gc, who))) + { + gtk_widget_set_sensitive(button, FALSE); + } + g_object_set_data_full(G_OBJECT(button), "user_data", g_strdup(who), g_free); } + if (gaim_conv_chat_is_user_ignored(GAIM_CONV_CHAT(conv), who)) button = gaim_new_item_from_stock(menu, _("Un-Ignore"), GAIM_STOCK_IGNORE, G_CALLBACK(ignore_cb), GAIM_GTK_CONVERSATION(conv), 0, 0, NULL); else button = gaim_new_item_from_stock(menu, _("Ignore"), GAIM_STOCK_IGNORE, G_CALLBACK(ignore_cb), GAIM_GTK_CONVERSATION(conv), 0, 0, NULL); + + if (gc == NULL) + gtk_widget_set_sensitive(button, FALSE); + g_object_set_data_full(G_OBJECT(button), "user_data", g_strdup(who), g_free); } if (prpl_info && (prpl_info->get_info || prpl_info->get_cb_info)) { button = gaim_new_item_from_stock(menu, _("Info"), GAIM_STOCK_INFO, G_CALLBACK(menu_chat_info_cb), GAIM_GTK_CONVERSATION(conv), 0, 0, NULL); + + if (gc == NULL) + gtk_widget_set_sensitive(button, FALSE); + g_object_set_data_full(G_OBJECT(button), "user_data", g_strdup(who), g_free); } if (prpl_info && prpl_info->get_cb_away) { button = gaim_new_item_from_stock(menu, _("Get Away Message"), GAIM_STOCK_AWAY, G_CALLBACK(menu_chat_get_away_cb), GAIM_GTK_CONVERSATION(conv), 0, 0, NULL); + + if (gc == NULL) + gtk_widget_set_sensitive(button, FALSE); + g_object_set_data_full(G_OBJECT(button), "user_data", g_strdup(who), g_free); } if (!is_me && prpl_info && !(prpl_info->options & OPT_PROTO_UNIQUE_CHATNAME)) { - if (gaim_find_buddy(gc->account, who)) + if (gaim_find_buddy(conv->account, who)) button = gaim_new_item_from_stock(menu, _("Remove"), GTK_STOCK_REMOVE, G_CALLBACK(menu_chat_add_remove_cb), GAIM_GTK_CONVERSATION(conv), 0, 0, NULL); else button = gaim_new_item_from_stock(menu, _("Add"), GTK_STOCK_ADD, G_CALLBACK(menu_chat_add_remove_cb), GAIM_GTK_CONVERSATION(conv), 0, 0, NULL); + + if (gc == NULL) + gtk_widget_set_sensitive(button, FALSE); + g_object_set_data_full(G_OBJECT(button), "user_data", g_strdup(who), g_free); } @@ -1613,7 +1645,6 @@ gtkconv_chat_popup_menu_cb(GtkWidget *widget, GaimGtkConversation *gtkconv) { GaimConversation *conv = gtkconv->active_conv; - GaimPluginProtocolInfo *prpl_info = NULL; GaimGtkChatPane *gtkchat; GaimConnection *gc; GaimAccount *account; @@ -1628,18 +1659,14 @@ account = gaim_conversation_get_account(conv); gc = account->gc; - g_return_val_if_fail(gc != NULL, FALSE); - model = gtk_tree_view_get_model(GTK_TREE_VIEW(gtkchat->list)); - prpl_info = GAIM_PLUGIN_PROTOCOL_INFO(gc->prpl); - sel = gtk_tree_view_get_selection(GTK_TREE_VIEW(gtkchat->list)); if(!gtk_tree_selection_get_selected(sel, NULL, &iter)) return FALSE; gtk_tree_model_get(GTK_TREE_MODEL(model), &iter, CHAT_USERS_NAME_COLUMN, &who, -1); - menu = create_chat_menu (conv, who, prpl_info, gc); + menu = create_chat_menu (conv, who, gc); gtk_menu_popup(GTK_MENU(menu), NULL, NULL, gaim_gtk_treeview_popup_menu_position_func, widget, 0, GDK_CURRENT_TIME); @@ -1654,7 +1681,6 @@ GaimGtkConversation *gtkconv) { GaimConversation *conv = gtkconv->active_conv; - GaimPluginProtocolInfo *prpl_info = NULL; GaimGtkChatPane *gtkchat; GaimConnection *gc; GaimAccount *account; @@ -1677,9 +1703,6 @@ if (path == NULL) return FALSE; - if (gc != NULL) - prpl_info = GAIM_PLUGIN_PROTOCOL_INFO(gc->prpl); - gtk_tree_selection_select_path(GTK_TREE_SELECTION( gtk_tree_view_get_selection(GTK_TREE_VIEW(gtkchat->list))), path); @@ -1695,7 +1718,7 @@ if(mark != NULL) gtk_text_view_scroll_to_mark(GTK_TEXT_VIEW(gtkconv->imhtml), mark, 0.1, FALSE, 0, 0); } else if (event->button == 3 && event->type == GDK_BUTTON_PRESS) { - GtkWidget *menu = create_chat_menu (conv, who, prpl_info, gc); + GtkWidget *menu = create_chat_menu (conv, who, gc); gtk_menu_popup(GTK_MENU(menu), NULL, NULL, NULL, NULL, event->button, event->time); } @@ -4508,18 +4531,12 @@ if (!gtk_text_buffer_get_selection_bounds( gtk_text_iter_get_buffer(arg2), &start, &end)) { - GaimPluginProtocolInfo *prpl_info = NULL; GtkWidget *menu = NULL; GaimConnection *gc = gaim_conversation_get_gc(conv); - if (gc != NULL) - prpl_info = GAIM_PLUGIN_PROTOCOL_INFO( - gc->prpl); - - menu = create_chat_menu(conv, buddyname, - prpl_info, gc); + menu = create_chat_menu(conv, buddyname, gc); gtk_menu_popup(GTK_MENU(menu), NULL, NULL, NULL, GTK_WIDGET(imhtml), btn_event->button,