Do not look into email addresses for "Check contacts"
[modest] / src / hildon2 / modest-hildon2-global-settings-dialog.c
index 99ecd07..2215133 100644 (file)
@@ -32,6 +32,7 @@
 #endif /*HAVE_CONFIG_H*/
 
 #include <modest-hildon-includes.h>
+#include <modest-scrollable.h>
 #include <modest-maemo-utils.h>
 
 #include <glib/gi18n.h>
 #include <gtk/gtkcheckbutton.h>
 #include "modest-runtime.h"
 #include "widgets/modest-global-settings-dialog-priv.h"
-#include "modest-selector-picker.h"
-#include "hildon/hildon-pannable-area.h"
 #include "modest-hildon2-global-settings-dialog.h"
 #include "widgets/modest-ui-constants.h"
 #include "modest-text-utils.h"
 #include "modest-defs.h"
 #include <tny-account-store.h>
 #include <modest-account-mgr-helpers.h>
+#include <modest-toolkit-utils.h>
 
 
 #define MSG_SIZE_MAX_VAL 5000
@@ -68,14 +68,6 @@ static ModestConnectedVia current_connection (void);
 
 static GtkWidget* create_updating_page   (ModestHildon2GlobalSettingsDialog *self);
 
-static gboolean   on_range_error         (ModestNumberEditor *editor, 
-                                         ModestNumberEditorErrorType type,
-                                         gpointer user_data);
-
-static void       on_size_notify         (ModestNumberEditor *editor, 
-                                         GParamSpec *arg1,
-                                         gpointer user_data);
-
 static void       on_auto_update_clicked (GtkButton *button,
                                          gpointer user_data);
 static void       update_sensitive       (ModestGlobalSettingsDialog *dialog);
@@ -142,11 +134,16 @@ modest_hildon2_global_settings_dialog_init (ModestHildon2GlobalSettingsDialog *s
 {
        ModestHildon2GlobalSettingsDialogPrivate *priv;
        ModestGlobalSettingsDialogPrivate *ppriv;
+       GtkWidget *align;
+       GtkWidget *top_vbox;
 
        priv  = MODEST_HILDON2_GLOBAL_SETTINGS_DIALOG_GET_PRIVATE (self);
        ppriv = MODEST_GLOBAL_SETTINGS_DIALOG_GET_PRIVATE (self);
 
        ppriv->updating_page = create_updating_page (self);
+       top_vbox = gtk_vbox_new (FALSE, 0);
+       align = gtk_alignment_new (0.0, 0.0, 1.0, 1.0);
+       gtk_alignment_set_padding (GTK_ALIGNMENT (align), 0, 0, MODEST_MARGIN_DOUBLE, 0);
 
        /* Add the buttons: */
        gtk_dialog_add_button (GTK_DIALOG (self), _HL("wdgt_bd_save"), GTK_RESPONSE_OK);
@@ -155,9 +152,14 @@ modest_hildon2_global_settings_dialog_init (ModestHildon2GlobalSettingsDialog *s
        g_object_set_data (G_OBJECT(ppriv->updating_page), DEFAULT_FOCUS_WIDGET,
                           (gpointer)ppriv->auto_update);
 
-       gtk_container_add (GTK_CONTAINER (GTK_DIALOG (self)->vbox), ppriv->updating_page);
+       gtk_container_add (GTK_CONTAINER (top_vbox), ppriv->updating_page);
+       gtk_container_add (GTK_CONTAINER (align), top_vbox);
+       gtk_container_add (GTK_CONTAINER (GTK_DIALOG (self)->vbox), align);
        gtk_container_set_border_width (GTK_CONTAINER (GTK_DIALOG (self)->vbox), MODEST_MARGIN_HALF);
-       gtk_window_set_default_size (GTK_WINDOW (self), -1, 340);
+       gtk_window_set_default_size (GTK_WINDOW (self), -1, MODEST_DIALOG_WINDOW_MAX_HEIGHT);
+
+       gtk_widget_show (align);
+       gtk_widget_show (top_vbox);
 }
 
 static void
@@ -190,62 +192,25 @@ modest_hildon2_global_settings_dialog_new (void)
 static GtkWidget*
 create_updating_page (ModestHildon2GlobalSettingsDialog *self)
 {
-       GtkWidget *vbox, *vbox_update, *vbox_limit, *label, *hbox;
+       GtkWidget *vbox;
        GtkSizeGroup *title_size_group;
        GtkSizeGroup *value_size_group;
        ModestGlobalSettingsDialogPrivate *ppriv;
-       GtkWidget *pannable;
+       GtkWidget *scrollable, *separator;
        ModestHildon2GlobalSettingsDialogPrivate *priv;
 
        priv = MODEST_HILDON2_GLOBAL_SETTINGS_DIALOG_GET_PRIVATE (self);
        ppriv = MODEST_GLOBAL_SETTINGS_DIALOG_GET_PRIVATE (self);
        vbox = gtk_vbox_new (FALSE, MODEST_MARGIN_HALF);
 
-       vbox_update = gtk_vbox_new (FALSE, MODEST_MARGIN_HALF);
        title_size_group = gtk_size_group_new (GTK_SIZE_GROUP_HORIZONTAL);
        value_size_group = gtk_size_group_new (GTK_SIZE_GROUP_HORIZONTAL);
 
-       /* Auto update */
-       ppriv->auto_update = hildon_check_button_new (MODEST_EDITABLE_SIZE);
-       gtk_button_set_label (GTK_BUTTON (ppriv->auto_update), _("mcen_fi_options_autoupdate"));
-       gtk_button_set_alignment (GTK_BUTTON (ppriv->auto_update), 0.0, 0.5);
-       gtk_box_pack_start (GTK_BOX (vbox_update), ppriv->auto_update, FALSE, FALSE, MODEST_MARGIN_HALF);
-       g_signal_connect (ppriv->auto_update, "clicked", G_CALLBACK (on_auto_update_clicked), self);
-
-       /* Connected via */
-
-       /* Note: This ModestPairList* must exist for as long as the picker
-        * that uses it, because the ModestSelectorPicker uses the ID opaquely, 
-        * so it can't know how to manage its memory. */ 
-       ppriv->connect_via_list = _modest_global_settings_dialog_get_connected_via ();
-       ppriv->connect_via = modest_selector_picker_new (MODEST_EDITABLE_SIZE,
-                                                        HILDON_BUTTON_ARRANGEMENT_VERTICAL,
-                                                        ppriv->connect_via_list, g_int_equal);
-       modest_maemo_utils_set_vbutton_layout (title_size_group, 
-                                              _("mcen_fi_options_connectiontype"),
-                                              ppriv->connect_via);
-       gtk_box_pack_start (GTK_BOX (vbox_update), ppriv->connect_via, FALSE, FALSE, MODEST_MARGIN_HALF);
-
-       /* Update interval */
-
-       /* Note: This ModestPairList* must exist for as long as the picker
-        * that uses it, because the ModestSelectorPicker uses the ID opaquely, 
-        * so it can't know how to manage its memory. */ 
-       ppriv->update_interval_list = _modest_global_settings_dialog_get_update_interval ();
-       ppriv->update_interval = modest_selector_picker_new (MODEST_EDITABLE_SIZE,
-                                                            HILDON_BUTTON_ARRANGEMENT_VERTICAL,
-                                                            ppriv->update_interval_list, g_int_equal);
-       modest_maemo_utils_set_vbutton_layout (title_size_group, 
-                                              _("mcen_fi_options_updateinterval"), 
-                                              ppriv->update_interval);
-       gtk_box_pack_start (GTK_BOX (vbox_update), ppriv->update_interval, FALSE, FALSE, MODEST_MARGIN_HALF);
-
        /* Default account selector */
        ppriv->accounts_list = get_accounts_list ();
-       ppriv->default_account_selector = modest_selector_picker_new (MODEST_EDITABLE_SIZE,
-                                                                     HILDON_BUTTON_ARRANGEMENT_VERTICAL,
-                                                                     ppriv->accounts_list,
-                                                                     g_str_equal);
+       ppriv->default_account_selector = modest_toolkit_factory_create_selector (modest_runtime_get_toolkit_factory (),
+                                                                                 ppriv->accounts_list,
+                                                                                 g_str_equal);
        if (ppriv->accounts_list == NULL) {
                gtk_widget_set_sensitive (GTK_WIDGET (ppriv->default_account_selector), FALSE);
        } else {
@@ -254,65 +219,127 @@ create_updating_page (ModestHildon2GlobalSettingsDialog *self)
                default_account = modest_account_mgr_get_default_account (
                        modest_runtime_get_account_mgr ());
                if (default_account) {
-                       modest_selector_picker_set_active_id (
-                               MODEST_SELECTOR_PICKER (ppriv->default_account_selector),
-                               default_account);
+                       modest_selector_set_active_id (ppriv->default_account_selector,
+                                                      default_account);
                        ppriv->initial_state.default_account = default_account;
                }
        }
-       modest_maemo_utils_set_vbutton_layout (title_size_group, 
-                                              _("mcen_ti_default_account"), 
-                                              ppriv->default_account_selector);
-       gtk_box_pack_start (GTK_BOX (vbox_update), ppriv->default_account_selector, 
-                           FALSE, FALSE, MODEST_MARGIN_HALF);
+       if (GTK_IS_COMBO_BOX (ppriv->default_account_selector)) {
+               GtkWidget *caption;
+
+               caption = modest_toolkit_utils_create_vcaptioned (title_size_group,
+                                                                 _("mcen_ti_default_account"), FALSE,
+                                                                 ppriv->default_account_selector);
+               gtk_widget_show (caption);
+               gtk_box_pack_start (GTK_BOX (vbox), caption,
+                                   FALSE, FALSE, 0);
+       } else {
+               modest_toolkit_utils_set_vbutton_layout (title_size_group,
+                                                        _("mcen_ti_default_account"),
+                                                        ppriv->default_account_selector);
+               gtk_box_pack_start (GTK_BOX (vbox), ppriv->default_account_selector,
+                                   FALSE, FALSE, 0);
+       }
 
-       /* Add to vbox */
-       gtk_box_pack_start (GTK_BOX (vbox), vbox_update, FALSE, FALSE, MODEST_MARGIN_HALF);
+       /* Message format */
+       /* Note: This ModestPairList* must exist for as long as the picker
+        * that uses it, because the ModestSelectorPicker uses the ID opaquely,
+        * so it can't know how to manage its memory. */
+       ppriv->msg_format_list = _modest_global_settings_dialog_get_msg_formats ();
+       ppriv->msg_format = modest_toolkit_factory_create_selector (modest_runtime_get_toolkit_factory (),
+                                                                   ppriv->msg_format_list, g_int_equal);
+       if (GTK_IS_COMBO_BOX (ppriv->msg_format)) {
+               GtkWidget *caption;
+               caption = modest_toolkit_utils_create_vcaptioned (title_size_group,
+                                                                 _("mcen_fi_options_messageformat"), FALSE,
+                                                                 ppriv->msg_format);
+               gtk_widget_show (caption);
+               gtk_box_pack_start (GTK_BOX (vbox), caption, FALSE, FALSE, 0);
+       } else {
+               modest_toolkit_utils_set_vbutton_layout (title_size_group,
+                                                        _("mcen_fi_options_messageformat"),
+                                                        ppriv->msg_format);
+               gtk_box_pack_start (GTK_BOX (vbox), ppriv->msg_format, FALSE, FALSE, 0);
+       }
 
-       g_object_unref (title_size_group);
-       g_object_unref (value_size_group);
 
-       /* Limits */
-       vbox_limit = gtk_vbox_new (FALSE, MODEST_MARGIN_HALF);
-       title_size_group = gtk_size_group_new (GTK_SIZE_GROUP_HORIZONTAL);
-       value_size_group = gtk_size_group_new (GTK_SIZE_GROUP_HORIZONTAL);
+       /* Incoming notifications */
+       ppriv->notifications = hildon_check_button_new (MODEST_EDITABLE_SIZE);
+       gtk_button_set_label (GTK_BUTTON (ppriv->notifications), _("mcen_fi_options_incoming_notifications"));
+       gtk_button_set_alignment (GTK_BUTTON (ppriv->notifications), 0.0, 0.5);
+       gtk_box_pack_start (GTK_BOX (vbox), ppriv->notifications, FALSE, FALSE, 0);
+
+       /* Automatic add to contacts */
+       ppriv->add_to_contacts = hildon_check_button_new (MODEST_EDITABLE_SIZE);
+       gtk_button_set_label (GTK_BUTTON (ppriv->add_to_contacts), _("mcen_fi_options_automatic_add"));
+       gtk_button_set_alignment (GTK_BUTTON (ppriv->add_to_contacts), 0.0, 0.5);
+       gtk_box_pack_start (GTK_BOX (vbox), ppriv->add_to_contacts, FALSE, FALSE, 0);
+
+       /* Separator label */
+       separator = gtk_label_new (_("mcen_ti_updating"));
+       gtk_label_set_justify ((GtkLabel *) separator, GTK_JUSTIFY_CENTER);
+       gtk_box_pack_start (GTK_BOX (vbox), separator, FALSE, FALSE, MODEST_MARGIN_DEFAULT);
+
+       /* Auto update */
+       ppriv->auto_update = modest_toolkit_factory_create_check_button (modest_runtime_get_toolkit_factory (),
+                                                                        _("mcen_fi_options_autoupdate"));
+       gtk_box_pack_start (GTK_BOX (vbox), ppriv->auto_update, FALSE, FALSE, 0);
+       g_signal_connect (ppriv->auto_update, "clicked", G_CALLBACK (on_auto_update_clicked), self);
 
-       /* Size limit */
-       ppriv->size_limit = modest_number_editor_new (MSG_SIZE_MIN_VAL, MSG_SIZE_MAX_VAL);
-       modest_number_editor_set_value (MODEST_NUMBER_EDITOR (ppriv->size_limit), MSG_SIZE_DEF_VAL);
-       g_signal_connect (ppriv->size_limit, "range_error", G_CALLBACK (on_range_error), self);
-       g_signal_connect (ppriv->size_limit, "notify", G_CALLBACK (on_size_notify), self);
-       label = gtk_label_new (_("mcen_fi_advsetup_sizelimit"));
-       hbox = gtk_hbox_new (FALSE, MODEST_MARGIN_HALF);
-       gtk_box_pack_start (GTK_BOX (hbox), label, FALSE, FALSE, 0);
-       gtk_box_pack_start (GTK_BOX (hbox), ppriv->size_limit, TRUE, TRUE, 0);
-       gtk_box_pack_start (GTK_BOX (vbox_limit), hbox, FALSE, FALSE, MODEST_MARGIN_HALF);
-       gtk_box_pack_start (GTK_BOX (vbox), vbox_limit, FALSE, FALSE, MODEST_MARGIN_HALF);
-       gtk_widget_show_all (vbox_limit);
+       /* Connected via */
 
        /* Note: This ModestPairList* must exist for as long as the picker
         * that uses it, because the ModestSelectorPicker uses the ID opaquely, 
         * so it can't know how to manage its memory. */ 
-       ppriv->msg_format_list = _modest_global_settings_dialog_get_msg_formats ();
-       ppriv->msg_format = modest_selector_picker_new (MODEST_EDITABLE_SIZE,
-                                                       HILDON_BUTTON_ARRANGEMENT_VERTICAL,
-                                                       ppriv->msg_format_list, g_int_equal);
-       modest_maemo_utils_set_vbutton_layout (title_size_group, 
-                                              _("mcen_fi_options_messageformat"), 
-                                              ppriv->msg_format);
+       ppriv->connect_via_list = _modest_global_settings_dialog_get_connected_via ();
+       ppriv->connect_via = modest_toolkit_factory_create_selector (modest_runtime_get_toolkit_factory (),
+                                                                    ppriv->connect_via_list, g_int_equal);
+       if (GTK_IS_COMBO_BOX (ppriv->connect_via)) {
+               GtkWidget *caption;
+               caption = modest_toolkit_utils_create_vcaptioned (title_size_group,
+                                                                 _("mcen_fi_options_connectiontype"), FALSE,
+                                                                 ppriv->connect_via);
+               gtk_widget_show (caption);
+               gtk_box_pack_start (GTK_BOX (vbox), caption, FALSE, FALSE, 0);
+       } else {
+               modest_toolkit_utils_set_vbutton_layout (title_size_group, 
+                                                        _("mcen_fi_options_connectiontype"),
+                                                        ppriv->connect_via);
+               gtk_box_pack_start (GTK_BOX (vbox), ppriv->connect_via, FALSE, FALSE, 0);
+       }
+
+       /* Update interval */
 
-       gtk_box_pack_start (GTK_BOX (vbox), ppriv->msg_format, FALSE, FALSE, MODEST_MARGIN_HALF);
+       /* Note: This ModestPairList* must exist for as long as the picker
+        * that uses it, because the ModestSelectorPicker uses the ID opaquely, 
+        * so it can't know how to manage its memory. */ 
+       ppriv->update_interval_list = _modest_global_settings_dialog_get_update_interval ();
+       ppriv->update_interval = modest_toolkit_factory_create_selector (modest_runtime_get_toolkit_factory (),
+                                                                        ppriv->update_interval_list, g_int_equal);
+       if (GTK_IS_COMBO_BOX (ppriv->update_interval)) {
+               GtkWidget *caption;
+               caption = modest_toolkit_utils_create_vcaptioned (title_size_group,
+                                                                 _("mcen_fi_options_updateinterval"), FALSE,
+                                                                 ppriv->update_interval);
+               gtk_widget_show (caption);
+               gtk_box_pack_start (GTK_BOX (vbox), caption, FALSE, FALSE, 0);
+       } else {
+               modest_toolkit_utils_set_vbutton_layout (title_size_group, 
+                                                        _("mcen_fi_options_updateinterval"), 
+                                                        ppriv->update_interval);
+               gtk_box_pack_start (GTK_BOX (vbox), ppriv->update_interval, FALSE, FALSE, 0);
+       }
 
-       pannable = g_object_new (HILDON_TYPE_PANNABLE_AREA, "initial-hint", TRUE, NULL);
+       scrollable = modest_toolkit_factory_create_scrollable (modest_runtime_get_toolkit_factory ());
 
-       hildon_pannable_area_add_with_viewport (HILDON_PANNABLE_AREA (pannable), vbox);
+       modest_scrollable_add_with_viewport (MODEST_SCROLLABLE (scrollable), vbox);
        gtk_widget_show (vbox);
-       gtk_widget_show (pannable);
+       gtk_widget_show (scrollable);
 
        g_object_unref (title_size_group);
        g_object_unref (value_size_group);
 
-       return pannable;
+       return scrollable;
 }
 
 
@@ -324,7 +351,7 @@ update_sensitive (ModestGlobalSettingsDialog *dialog)
        g_return_if_fail (MODEST_IS_GLOBAL_SETTINGS_DIALOG (dialog));
        ppriv = MODEST_GLOBAL_SETTINGS_DIALOG_GET_PRIVATE (dialog);
 
-       if (hildon_check_button_get_active (HILDON_CHECK_BUTTON (ppriv->auto_update))) {
+       if (modest_togglable_get_active (ppriv->auto_update)) {
                gtk_widget_set_sensitive (ppriv->connect_via, TRUE);
                gtk_widget_set_sensitive (ppriv->update_interval, TRUE);
        } else {
@@ -340,56 +367,6 @@ on_auto_update_clicked (GtkButton *button,
        g_return_if_fail (MODEST_IS_GLOBAL_SETTINGS_DIALOG (user_data));
        update_sensitive ((ModestGlobalSettingsDialog *) user_data);
 }
-static gboolean
-on_range_error (ModestNumberEditor *editor, 
-               ModestNumberEditorErrorType type,
-               gpointer user_data)
-{
-       gchar *msg;
-       gint new_val;
-
-       switch (type) {
-       case MODEST_NUMBER_EDITOR_ERROR_MAXIMUM_VALUE_EXCEED:
-               msg = g_strdup_printf (dgettext("hildon-libs", "ckct_ib_maximum_value"), MSG_SIZE_MAX_VAL);
-               new_val = MSG_SIZE_MAX_VAL;
-               break;
-       case MODEST_NUMBER_EDITOR_ERROR_MINIMUM_VALUE_EXCEED:
-               msg = g_strdup_printf (dgettext("hildon-libs", "ckct_ib_minimum_value"), MSG_SIZE_MIN_VAL);
-               new_val = MSG_SIZE_MIN_VAL;
-               break;
-       case MODEST_NUMBER_EDITOR_ERROR_ERRONEOUS_VALUE:
-               msg = g_strdup_printf (dgettext("hildon-libs", "ckct_ib_set_a_value_within_range"), 
-                                      MSG_SIZE_MIN_VAL, 
-                                      MSG_SIZE_MAX_VAL);
-               /* FIXME: use the previous */
-               new_val = MSG_SIZE_DEF_VAL;
-               break;
-       default:
-               g_return_val_if_reached (FALSE);
-       }
-
-       /* Restore value */
-       modest_number_editor_set_value (editor, new_val);
-
-       /* Show error */
-       hildon_banner_show_information (GTK_WIDGET (user_data), NULL, msg);
-
-       /* Free */
-       g_free (msg);
-
-       return TRUE;
-}
-
-static void
-on_size_notify         (ModestNumberEditor *editor, 
-                       GParamSpec *arg1,
-                       gpointer user_data)
-{
-       ModestHildon2GlobalSettingsDialog *dialog = MODEST_HILDON2_GLOBAL_SETTINGS_DIALOG (user_data);
-       gint value = modest_number_editor_get_value (editor);
-
-       gtk_dialog_set_response_sensitive (GTK_DIALOG (dialog), GTK_RESPONSE_OK, value > 0);
-}
 
 static ModestConnectedVia
 current_connection (void)
@@ -440,6 +417,7 @@ get_accounts_list (void)
                settings = modest_account_mgr_load_account_settings (account_mgr, account_name);
                if (!settings) {
                        g_printerr ("modest: failed to get account data for %s\n", account_name);
+                       cursor = cursor->next;
                        continue;
                }
                store_settings = modest_account_settings_get_store_settings (settings);
@@ -472,13 +450,33 @@ modest_hildon2_global_settings_dialog_load_settings (ModestGlobalSettingsDialog
 {
        ModestConf *conf;
        gboolean checked;
-       gint combo_id, value;
+       gint combo_id;
        GError *error = NULL;
        ModestGlobalSettingsDialogPrivate *ppriv;
 
        ppriv = MODEST_GLOBAL_SETTINGS_DIALOG_GET_PRIVATE (self);
        conf = modest_runtime_get_conf ();
 
+       /* Incoming notifications */
+       checked = modest_conf_get_bool (conf, MODEST_CONF_NOTIFICATIONS, &error);
+       if (error) {
+               g_clear_error (&error);
+               error = NULL;
+               checked = FALSE;
+       }
+       hildon_check_button_set_active (HILDON_CHECK_BUTTON (ppriv->notifications), checked);
+       ppriv->initial_state.notifications = checked;
+
+       /* Add to contacts */
+       checked = modest_conf_get_bool (conf, MODEST_CONF_AUTO_ADD_TO_CONTACTS, &error);
+       if (error) {
+               g_clear_error (&error);
+               error = NULL;
+               checked = FALSE;
+       }
+       hildon_check_button_set_active (HILDON_CHECK_BUTTON (ppriv->add_to_contacts), checked);
+       ppriv->initial_state.add_to_contacts = checked;
+
        /* Autoupdate */
        checked = modest_conf_get_bool (conf, MODEST_CONF_AUTO_UPDATE, &error);
        if (error) {
@@ -486,7 +484,7 @@ modest_hildon2_global_settings_dialog_load_settings (ModestGlobalSettingsDialog
                error = NULL;
                checked = FALSE;
        }
-       hildon_check_button_set_active (HILDON_CHECK_BUTTON (ppriv->auto_update), checked);
+       modest_togglable_set_active (ppriv->auto_update, checked);
        ppriv->initial_state.auto_update = checked;
 
        /* Connected by */
@@ -496,13 +494,10 @@ modest_hildon2_global_settings_dialog_load_settings (ModestGlobalSettingsDialog
                error = NULL;
                combo_id = MODEST_CONNECTED_VIA_WLAN_OR_WIMAX;
        }
-       modest_selector_picker_set_active_id (MODEST_SELECTOR_PICKER (ppriv->connect_via),
-                                             (gpointer) &combo_id);
+       modest_selector_set_active_id (ppriv->connect_via,
+                                      (gpointer) &combo_id);
        ppriv->initial_state.connect_via = combo_id;
 
-       /* Emit toggled to update the visibility of connect_by caption */
-       gtk_toggle_button_toggled (GTK_TOGGLE_BUTTON (ppriv->auto_update));
-
        /* Update interval */
        combo_id = modest_conf_get_int (conf, MODEST_CONF_UPDATE_INTERVAL, &error);
        if (error) {
@@ -510,22 +505,10 @@ modest_hildon2_global_settings_dialog_load_settings (ModestGlobalSettingsDialog
                error = NULL;
                combo_id = MODEST_UPDATE_INTERVAL_15_MIN;
        }
-       modest_selector_picker_set_active_id (MODEST_SELECTOR_PICKER (ppriv->update_interval),
-                                       (gpointer) &combo_id);
+       modest_selector_set_active_id (ppriv->update_interval,
+                                      (gpointer) &combo_id);
        ppriv->initial_state.update_interval = combo_id;
 
-       /* Size limit */
-       value  = modest_conf_get_int (conf, MODEST_CONF_MSG_SIZE_LIMIT, &error);
-       if (error) {
-               g_error_free (error);
-               error = NULL;
-               value = 1000;
-       }
-       /* It's better to do this in the subclasses, but it's just one
-          line, so we'll leave it here for the moment */
-       modest_number_editor_set_value (MODEST_NUMBER_EDITOR (ppriv->size_limit), value);
-       ppriv->initial_state.size_limit = value;
-
        /* Play sound */
        checked = modest_conf_get_bool (conf, MODEST_CONF_PLAY_SOUND_MSG_ARRIVE, &error);
        if (error) {
@@ -533,7 +516,6 @@ modest_hildon2_global_settings_dialog_load_settings (ModestGlobalSettingsDialog
                error = NULL;
                checked = FALSE;
        }
-       gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (ppriv->play_sound), checked);
        ppriv->initial_state.play_sound = checked;
 
        /* Msg format */
@@ -544,8 +526,8 @@ modest_hildon2_global_settings_dialog_load_settings (ModestGlobalSettingsDialog
                combo_id = MODEST_FILE_FORMAT_FORMATTED_TEXT;
        }
        combo_id = (checked) ? MODEST_FILE_FORMAT_FORMATTED_TEXT : MODEST_FILE_FORMAT_PLAIN_TEXT;
-       modest_selector_picker_set_active_id (MODEST_SELECTOR_PICKER (ppriv->msg_format),
-                                             (gpointer) &combo_id);
+       modest_selector_set_active_id (ppriv->msg_format,
+                                      (gpointer) &combo_id);
        ppriv->initial_state.prefer_formatted_text = checked;
 
        /* force update of sensitiveness */