2006-07-01 [colin] 2.3.1cvs50
[claws.git] / src / prefs_account.c
index afa4c6ee49f5ae0598bdcfd22e91d4123a3c23a6..16bf3626ee8710cafd1ba970e883777852343a87 100644 (file)
@@ -54,6 +54,7 @@
 #include "imap.h"
 #include "remotefolder.h"
 #include "base64.h"
+#include "combobox.h"
 
 static gboolean cancelled;
 static gboolean new_account;
@@ -69,6 +70,12 @@ static GtkWidget *signature_edit_button;
 
 static GSList *prefs_pages = NULL;
 
+struct BasicProtocol {
+       GtkWidget *combobox;
+       GtkWidget *label;
+       GtkWidget *descrlabel;
+};
+
 static struct Basic {
        GtkWidget *acname_entry;
        GtkWidget *default_chkbtn;
@@ -79,7 +86,7 @@ static struct Basic {
 
        GtkWidget *serv_frame;
        GtkWidget *serv_table;
-       GtkWidget *protocol_optmenu;
+       gpointer *protocol_optmenu;
        GtkWidget *recvserv_label;
        GtkWidget *smtpserv_label;
        GtkWidget *nntpserv_label;
@@ -220,11 +227,20 @@ static struct Advanced {
        GtkWidget *trash_folder_entry;
 } advanced;
 
-static void prefs_account_fix_size                     (void);
+static char *protocol_names[] = {
+       N_("POP3"),
+       NULL,           /* APOP, deprecated */
+       NULL,           /* RPOP, deprecated */
+       N_("IMAP4"),
+       N_("News (NNTP)"),
+       N_("Local mbox file"),
+       N_("None (SMTP only)")
+};
 
 static void prefs_account_protocol_set_data_from_optmenu(PrefParam *pparam);
 static void prefs_account_protocol_set_optmenu         (PrefParam *pparam);
-static void prefs_account_protocol_activated           (GtkMenuItem *menuitem);
+static void prefs_account_protocol_changed             (GtkComboBox *combobox,
+                                                       gpointer data);
 
 static void prefs_account_set_string_from_optmenu      (PrefParam *pparam);
 static void prefs_account_set_optmenu_from_string      (PrefParam *pparam);
@@ -268,7 +284,7 @@ static PrefParam param[] = {
         &basic.org_entry, prefs_set_data_from_entry, prefs_set_entry},
 
        {"protocol", NULL, &tmp_ac_prefs.protocol, P_ENUM,
-        &basic.protocol_optmenu,
+        (GtkWidget **)&basic.protocol_optmenu,
         prefs_account_protocol_set_data_from_optmenu,
         prefs_account_protocol_set_optmenu},
 
@@ -709,12 +725,6 @@ static void create_widget_func(PrefsPage * _page,
                        gtk_entry_set_text(GTK_ENTRY(basic.org_entry),
                                           def_ac->organization ? def_ac->organization : "");
                }
-               menu_set_sensitive_all
-                       (GTK_MENU_SHELL
-                               (gtk_option_menu_get_menu
-                                       (GTK_OPTION_MENU
-                                               (basic.protocol_optmenu))),
-                        TRUE);
        } else
                prefs_set_dialog(param);
 
@@ -1028,34 +1038,6 @@ static void prefs_account_create(void)
 #endif /* USE_OPENSSL */
        prefs_account_advanced_create();
        SET_NOTEBOOK_LABEL(notebook, _("A_dvanced"), page++);
-
-       prefs_account_fix_size();
-}
-
-/**
- * prefs_account_fix_size:
- * 
- * Fix the window size after creating widgets by selecting "Local"
- * protocol (currently it has the largest size of parameter widgets).
- * Without this the window gets too large.
- **/
-static void prefs_account_fix_size(void)
-{
-       GtkOptionMenu *optmenu = GTK_OPTION_MENU (basic.protocol_optmenu);
-       GtkWidget *menu;
-       GtkWidget *menuitem;
-
-       gtk_option_menu_set_history (optmenu, 4); /* local */
-       menu = gtk_option_menu_get_menu (optmenu);
-       menuitem = gtk_menu_get_active (GTK_MENU (menu));
-       gtk_menu_item_activate (GTK_MENU_ITEM (menuitem));
-}
-
-#define SET_ACTIVATE(menuitem) \
-{ \
-       g_signal_connect(G_OBJECT(menuitem), "activate", \
-                        G_CALLBACK(prefs_account_protocol_activated), \
-                        NULL); \
 }
 
 #define TABLE_YPAD 2
@@ -1075,9 +1057,9 @@ static void prefs_account_basic_create(void)
 
        GtkWidget *serv_frame;
        GtkWidget *vbox2;
+       GtkWidget *optmenubox;
        GtkWidget *optmenu;
-       GtkWidget *optmenu_menu;
-       GtkWidget *menuitem;
+       GtkWidget *optlabel;
        GtkWidget *serv_table;
        GtkWidget *recvserv_label;
        GtkWidget *smtpserv_label;
@@ -1096,6 +1078,11 @@ static void prefs_account_basic_create(void)
        GtkWidget *pass_label;
        GtkWidget *uid_entry;
        GtkWidget *pass_entry;
+       GtkListStore *menu;
+       GtkTreeIter iter;
+
+       struct BasicProtocol *protocol_optmenu;
+       gint i;
 
        vbox1 = gtk_vbox_new (FALSE, VSPACING);
        gtk_widget_show (vbox1);
@@ -1179,24 +1166,34 @@ static void prefs_account_basic_create(void)
        gtk_widget_show (label);
        gtk_box_pack_start (GTK_BOX (hbox), label, FALSE, FALSE, 0);
 
-       optmenu = gtk_option_menu_new ();
-       gtk_widget_show (optmenu);
-       gtk_box_pack_start (GTK_BOX (hbox), optmenu, FALSE, FALSE, 0);
+       /* Create GtkHBox for protocol combobox and label */
+       optmenubox = gtk_hbox_new(FALSE, 20);
+       gtk_widget_show(optmenubox);
+       gtk_box_pack_start (GTK_BOX (hbox), optmenubox, FALSE, FALSE, 0);
 
-       optmenu_menu = gtk_menu_new ();
+       /* Create and populate the combobox */
+       optmenu = gtkut_sc_combobox_create(NULL, FALSE);
+       gtk_box_pack_start(GTK_BOX (optmenubox), optmenu, FALSE, FALSE, 0);
 
-       MENUITEM_ADD (optmenu_menu, menuitem, _("POP3"),  A_POP3);
-       SET_ACTIVATE (menuitem);
-       MENUITEM_ADD (optmenu_menu, menuitem, _("IMAP4"), A_IMAP4);
-       SET_ACTIVATE (menuitem);
-       MENUITEM_ADD (optmenu_menu, menuitem, _("News (NNTP)"), A_NNTP);
-       SET_ACTIVATE (menuitem);
-       MENUITEM_ADD (optmenu_menu, menuitem, _("Local mbox file"), A_LOCAL);
-       SET_ACTIVATE (menuitem);
-       MENUITEM_ADD (optmenu_menu, menuitem, _("None (SMTP only)"), A_NONE);
-       SET_ACTIVATE (menuitem);
+       menu = GTK_LIST_STORE(gtk_combo_box_get_model(GTK_COMBO_BOX(optmenu)));
+       for( i = 0; i < NUM_RECV_PROTOCOLS; i++ )
+               if( protocol_names[i] != NULL )
+                       COMBOBOX_ADD (menu, protocol_names[i], i);
 
-       gtk_option_menu_set_menu (GTK_OPTION_MENU (optmenu), optmenu_menu);
+       g_signal_connect(G_OBJECT(optmenu), "changed",
+                       G_CALLBACK(prefs_account_protocol_changed), NULL);
+
+       /* Create protocol label, empty for now */
+       optlabel = gtk_label_new("");
+       gtk_label_set_use_markup(GTK_LABEL(optlabel), TRUE);
+       gtk_label_set_justify(GTK_LABEL(optlabel), GTK_JUSTIFY_CENTER);
+       gtk_box_pack_start(GTK_BOX (optmenubox), optlabel, FALSE, FALSE, 0);
+
+       /* Set up a struct to store pointers to necessary widgets */
+       protocol_optmenu = g_new(struct BasicProtocol, 1);
+       protocol_optmenu->combobox = optmenu;
+       protocol_optmenu->label = optlabel;
+       protocol_optmenu->descrlabel = label;
 
        serv_table = gtk_table_new (6, 4, FALSE);
        gtk_widget_show (serv_table);
@@ -1334,7 +1331,7 @@ static void prefs_account_basic_create(void)
 
        basic.serv_frame       = serv_frame;
        basic.serv_table       = serv_table;
-       basic.protocol_optmenu = optmenu;
+       basic.protocol_optmenu = (gpointer)protocol_optmenu;
        basic.recvserv_label   = recvserv_label;
        basic.recvserv_entry   = recvserv_entry;
        basic.smtpserv_label   = smtpserv_label;
@@ -2481,15 +2478,10 @@ static void prefs_account_advanced_create(void)
 static gint prefs_account_apply(void)
 {
        RecvProtocol protocol;
-       GtkWidget *menu;
-       GtkWidget *menuitem;
        gchar *old_id = NULL;
        gchar *new_id = NULL;
        
-       menu = gtk_option_menu_get_menu(GTK_OPTION_MENU(basic.protocol_optmenu));
-       menuitem = gtk_menu_get_active(GTK_MENU(menu));
-       protocol = GPOINTER_TO_INT
-               (g_object_get_data(G_OBJECT(menuitem), MENU_VAL_ID));
+       protocol = combobox_get_active_data(GTK_COMBO_BOX(basic.protocol_optmenu));
 
        if (*gtk_entry_get_text(GTK_ENTRY(basic.acname_entry)) == '\0') {
                alertpanel_error(_("Account name is not entered."));
@@ -2663,35 +2655,43 @@ static void prefs_account_enum_set_radiobtn(PrefParam *pparam)
 
 static void prefs_account_protocol_set_data_from_optmenu(PrefParam *pparam)
 {
-       GtkWidget *menu;
-       GtkWidget *menuitem;
+       struct BasicProtocol *protocol_optmenu =
+               (struct BasicProtocol *)*pparam->widget;
+       GtkWidget *optmenu = protocol_optmenu->combobox;
 
-       menu = gtk_option_menu_get_menu(GTK_OPTION_MENU(*pparam->widget));
-       menuitem = gtk_menu_get_active(GTK_MENU(menu));
-       *((RecvProtocol *)pparam->data) = GPOINTER_TO_INT
-               (g_object_get_data(G_OBJECT(menuitem), MENU_VAL_ID));
+       *((RecvProtocol *)pparam->data) =
+               combobox_get_active_data(GTK_COMBO_BOX(optmenu));
 }
 
 static void prefs_account_protocol_set_optmenu(PrefParam *pparam)
 {
        RecvProtocol protocol;
-       GtkOptionMenu *optmenu = GTK_OPTION_MENU(*pparam->widget);
-       GtkWidget *menu;
-       GtkWidget *menuitem;
-       gint index;
+       struct BasicProtocol *protocol_optmenu =
+               (struct BasicProtocol *)*pparam->widget;
+       GtkWidget *optmenu = protocol_optmenu->combobox;
+       GtkWidget *optlabel = protocol_optmenu->label;
+       GtkWidget *descrlabel = protocol_optmenu->descrlabel;
+       gchar *label = NULL;
 
        protocol = *((RecvProtocol *)pparam->data);
-       index = menu_find_option_menu_index
-               (optmenu, GINT_TO_POINTER(protocol), NULL);
-       if (index < 0) return;
-       gtk_option_menu_set_history(optmenu, index);
-
-       menu = gtk_option_menu_get_menu(optmenu);
-       menu_set_insensitive_all(GTK_MENU_SHELL(menu));
 
-       menuitem = gtk_menu_get_active(GTK_MENU(menu));
-       gtk_widget_set_sensitive(menuitem, TRUE);
-       gtk_menu_item_activate(GTK_MENU_ITEM(menuitem));
+       /* Set combobox to correct value even if it will be hidden, so
+        * we won't break existing accounts when saving. */
+       combobox_select_by_data(GTK_COMBO_BOX(optmenu), protocol);
+
+       /* Set up widgets accordingly */
+       if( new_account ) {
+               gtk_label_set_text(GTK_LABEL(descrlabel), _("Protocol"));
+               gtk_widget_hide(optlabel);
+               gtk_widget_show(optmenu);
+       } else {
+               gtk_label_set_text(GTK_LABEL(descrlabel), _("Protocol:"));
+               label = g_markup_printf_escaped("<b>%s</b>", protocol_names[protocol]);
+               gtk_label_set_markup(GTK_LABEL(optlabel), label);
+               g_free(label);
+               gtk_widget_hide(optmenu);
+               gtk_widget_show(optlabel);
+       }
 }
 
 static void prefs_account_imap_auth_type_set_data_from_optmenu(PrefParam *pparam)
@@ -2838,12 +2838,11 @@ static void prefs_account_set_optmenu_from_string(PrefParam *pparam)
        g_list_free(children);
 }
 
-static void prefs_account_protocol_activated(GtkMenuItem *menuitem)
+static void prefs_account_protocol_changed(GtkComboBox *combobox, gpointer data)
 {
        RecvProtocol protocol;
 
-       protocol = GPOINTER_TO_INT
-               (g_object_get_data(G_OBJECT(menuitem), MENU_VAL_ID));
+       protocol = combobox_get_active_data(combobox);
 
        switch(protocol) {
        case A_NNTP: