X-Git-Url: http://git.maemo.org/git/?p=modest;a=blobdiff_plain;f=src%2Fhildon2%2Fmodest-easysetup-wizard-dialog.c;h=37ef5aaa0c1f40e4e6780cd72db10763cd4c4438;hp=9f1040fdfd0f24bb0ccb7b176ca55483fedbac9c;hb=431405fd7c2c7ab14d9a478024a001920a192bee;hpb=2c42e651cc7c81b3b48adbab9a9a3c80cc32b215 diff --git a/src/hildon2/modest-easysetup-wizard-dialog.c b/src/hildon2/modest-easysetup-wizard-dialog.c index 9f1040f..37ef5aa 100644 --- a/src/hildon2/modest-easysetup-wizard-dialog.c +++ b/src/hildon2/modest-easysetup-wizard-dialog.c @@ -27,7 +27,7 @@ * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. */ - + #include #include "modest-easysetup-wizard-dialog.h" #include @@ -89,34 +89,41 @@ struct _ModestEasysetupWizardDialogPrivate /* Remember what fields the user edited manually to not prefill them * again. */ ModestEasysetupWizardDialogServerChanges server_changes; - + /* Check if the user changes a field to show a confirmation dialog */ gboolean dirty; /* If we have a pending load of settings or not. */ gboolean pending_load_settings; - + /* Used by derived widgets to query existing accounts, * and to create new accounts: */ ModestAccountMgr *account_manager; ModestAccountSettings *settings; - + /* notebook pages: */ GtkWidget *page_welcome; - + GtkWidget *check_support_progress; + gboolean check_support_done; + guint check_support_show_progress_id; + guint check_support_progress_pulse_id; + gint pending_check_support; + gboolean destroyed; + GtkWidget *page_account_details; GtkWidget *account_country_picker; GtkWidget *account_serviceprovider_picker; GtkWidget *entry_account_title; - + GtkWidget *caption_account_title; + GtkWidget *page_user_details; GtkWidget *entry_user_name; GtkWidget *entry_user_username; GtkWidget *entry_user_password; GtkWidget *entry_user_email; - + GtkWidget *page_complete_easysetup; - + GtkWidget *page_custom_incoming; GtkWidget *incoming_servertype_picker; GtkWidget *caption_incoming; @@ -130,7 +137,7 @@ struct _ModestEasysetupWizardDialogPrivate GtkWidget *entry_outgoingserver; GtkWidget *checkbox_outgoing_smtp_specific; GtkWidget *button_outgoing_smtp_servers; - + GtkWidget *page_complete_customsetup; ModestProtocolType last_plugin_protocol_selected; @@ -139,8 +146,16 @@ struct _ModestEasysetupWizardDialogPrivate static void save_to_settings (ModestEasysetupWizardDialog *self); static void real_enable_buttons (ModestWizardDialog *dialog, gboolean enable_next); +static void on_update_model (ModestWizardDialog *dialog); +static gboolean on_save (ModestWizardDialog *dialog); static GList* check_for_supported_auth_methods (ModestEasysetupWizardDialog* self); static gboolean check_has_supported_auth_methods(ModestEasysetupWizardDialog* self); +static void check_support_callback (ModestAccountProtocol *protocol, + gboolean supported, + gpointer userdata); +static void check_support_of_protocols (ModestEasysetupWizardDialog *self); +static gboolean check_support_show_progress (gpointer userdata); +static gboolean check_support_progress_pulse (gpointer userdata); static gboolean on_delete_event (GtkWidget *widget, @@ -162,6 +177,19 @@ on_easysetup_changed(GtkWidget* widget, ModestEasysetupWizardDialog* wizard) static void modest_easysetup_wizard_dialog_dispose (GObject *object) { + ModestEasysetupWizardDialogPrivate *priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (object); + priv->destroyed = TRUE; + + if (priv->check_support_show_progress_id > 0) { + g_source_remove (priv->check_support_show_progress_id); + priv->check_support_show_progress_id = 0; + } + + if (priv->check_support_progress_pulse_id > 0) { + g_source_remove (priv->check_support_progress_pulse_id); + priv->check_support_progress_pulse_id = 0; + } + if (G_OBJECT_CLASS (modest_easysetup_wizard_dialog_parent_class)->dispose) G_OBJECT_CLASS (modest_easysetup_wizard_dialog_parent_class)->dispose (object); } @@ -171,10 +199,10 @@ modest_easysetup_wizard_dialog_finalize (GObject *object) { ModestEasysetupWizardDialog *self = MODEST_EASYSETUP_WIZARD_DIALOG (object); ModestEasysetupWizardDialogPrivate *priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (self); - + if (priv->account_manager) g_object_unref (G_OBJECT (priv->account_manager)); - + if (priv->presets) modest_presets_destroy (priv->presets); @@ -185,7 +213,7 @@ modest_easysetup_wizard_dialog_finalize (GObject *object) modest_signal_mgr_disconnect_all_and_destroy (priv->missing_data_signals); priv->missing_data_signals = NULL; } - + G_OBJECT_CLASS (modest_easysetup_wizard_dialog_parent_class)->finalize (object); } @@ -197,7 +225,7 @@ set_default_custom_servernames(ModestEasysetupWizardDialog *dialog); static void on_servertype_selector_changed(HildonTouchSelector *selector, gint column, gpointer user_data); -static gint +static gint get_port_from_protocol (ModestProtocolType server_type, gboolean alternate_port) { @@ -220,16 +248,16 @@ static void invoke_enable_buttons_vfunc (ModestEasysetupWizardDialog *wizard_dialog) { ModestWizardDialogClass *klass = MODEST_WIZARD_DIALOG_GET_CLASS (wizard_dialog); - + /* Call the vfunc, which may be overridden by derived classes: */ if (klass->enable_buttons) { GtkNotebook *notebook = NULL; g_object_get (wizard_dialog, "wizard-notebook", ¬ebook, NULL); - + const gint current_page_num = gtk_notebook_get_current_page (notebook); if (current_page_num == -1) return; - + GtkWidget* current_page_widget = gtk_notebook_get_nth_page (notebook, current_page_num); (*(klass->enable_buttons))(MODEST_WIZARD_DIALOG (wizard_dialog), current_page_widget); } @@ -238,8 +266,12 @@ invoke_enable_buttons_vfunc (ModestEasysetupWizardDialog *wizard_dialog) static void on_caption_entry_changed (GtkEditable *editable, gpointer user_data) { - ModestEasysetupWizardDialog *self = MODEST_EASYSETUP_WIZARD_DIALOG (user_data); - g_assert(self); + ModestEasysetupWizardDialog *self; + + g_return_if_fail (user_data); + + self = MODEST_EASYSETUP_WIZARD_DIALOG (user_data); + invoke_enable_buttons_vfunc(self); } @@ -267,6 +299,10 @@ on_serviceprovider_picker_button_value_changed (HildonPickerButton *widget, gpoi ModestEasysetupWizardDialog *self; ModestEasysetupWizardDialogPrivate *priv; ModestProviderPickerIdType provider_id_type; + ModestProtocol *protocol; + gchar *proto_name; + ModestProtocolType proto_type; + gboolean hide_account_title; self = MODEST_EASYSETUP_WIZARD_DIALOG (user_data); priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (self); @@ -281,15 +317,40 @@ on_serviceprovider_picker_button_value_changed (HildonPickerButton *widget, gpoi GtkWidget *selector; selector = GTK_WIDGET (hildon_picker_button_get_selector (HILDON_PICKER_BUTTON (widget))); - default_account_name_start = + default_account_name_start = g_strdup (hildon_touch_selector_get_current_text (HILDON_TOUCH_SELECTOR (selector))); - + } default_account_name = modest_account_mgr_get_unused_account_display_name ( priv->account_manager, default_account_name_start); g_free (default_account_name_start); default_account_name_start = NULL; + hide_account_title = FALSE; + proto_name = modest_provider_picker_get_active_provider_id (MODEST_PROVIDER_PICKER (priv->account_serviceprovider_picker)); + if (proto_name != NULL) { + protocol = modest_protocol_registry_get_protocol_by_name (modest_runtime_get_protocol_registry (), + MODEST_PROTOCOL_REGISTRY_PROVIDER_PROTOCOLS, + proto_name); + if (protocol != NULL) { + proto_type = modest_protocol_get_type_id (protocol); + + hide_account_title = (modest_protocol_registry_protocol_type_has_tag + (modest_runtime_get_protocol_registry (), + proto_type, + MODEST_PROTOCOL_REGISTRY_SINGLETON_PROVIDER_PROTOCOLS) || + modest_protocol_registry_protocol_type_has_tag + (modest_runtime_get_protocol_registry (), + proto_type, + MODEST_PROTOCOL_REGISTRY_MULTI_MAILBOX_PROVIDER_PROTOCOLS)); + } + } + + if (hide_account_title) + gtk_widget_hide (priv->caption_account_title); + else + gtk_widget_show (priv->caption_account_title); + hildon_entry_set_text (HILDON_ENTRY (priv->entry_account_title), default_account_name); g_free (default_account_name); } @@ -297,7 +358,7 @@ on_serviceprovider_picker_button_value_changed (HildonPickerButton *widget, gpoi /** This is a convenience function to create a caption containing a mandatory widget. * When the widget is edited, the enable_buttons() vfunc will be called. */ -static GtkWidget* +static GtkWidget* create_captioned (ModestEasysetupWizardDialog *self, GtkSizeGroup *title_size_group, GtkSizeGroup *value_size_group, @@ -310,35 +371,47 @@ create_captioned (ModestEasysetupWizardDialog *self, result = modest_maemo_utils_create_captioned (title_size_group, value_size_group, value, use_markup, control); - /* Connect to the appropriate changed signal for the widget, + /* Connect to the appropriate changed signal for the widget, * so we can ask for the prev/next buttons to be enabled/disabled appropriately: */ if (GTK_IS_ENTRY (control)) { g_signal_connect (G_OBJECT (control), "changed", G_CALLBACK (on_caption_entry_changed), self); - + } else if (GTK_IS_COMBO_BOX (control)) { g_signal_connect (G_OBJECT (control), "changed", G_CALLBACK (on_caption_combobox_changed), self); } - + return result; } - + static GtkWidget* create_page_welcome (ModestEasysetupWizardDialog *self) { - GtkWidget *box = gtk_vbox_new (FALSE, MODEST_MARGIN_NONE); - GtkWidget *label = gtk_label_new(_("mcen_ia_emailsetup_intro")); + GtkWidget *box; + GtkWidget *align; + GtkWidget *label; + ModestEasysetupWizardDialogPrivate *priv; + + priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (self); + box = gtk_vbox_new (FALSE, MODEST_MARGIN_NONE); + label = gtk_label_new(_("mcen_ia_emailsetup_intro")); + priv->check_support_progress = gtk_progress_bar_new (); + 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); + gtk_widget_set_size_request (label, LABELS_WIDTH, -1); gtk_label_set_line_wrap (GTK_LABEL (label), TRUE); /* So that it is not truncated: */ - gtk_widget_set_size_request (label, LABELS_WIDTH, -1); gtk_misc_set_alignment (GTK_MISC (label), 0.0, 0.0); - gtk_box_pack_start (GTK_BOX (box), label, FALSE, FALSE, 0); + gtk_misc_set_padding (GTK_MISC (label), MODEST_MARGIN_DOUBLE, MODEST_MARGIN_DOUBLE); + gtk_box_pack_start (GTK_BOX (box), label, TRUE, TRUE, 0); + gtk_box_pack_end (GTK_BOX (box), priv->check_support_progress, FALSE, FALSE, 0); + gtk_container_add (GTK_CONTAINER (align), box); gtk_widget_show (label); gtk_widget_show (GTK_WIDGET (box)); - return GTK_WIDGET (box); + return GTK_WIDGET (align); } static void @@ -347,9 +420,9 @@ on_account_country_selector_changed (HildonTouchSelector *widget, gint column, g ModestEasysetupWizardDialog *self = MODEST_EASYSETUP_WIZARD_DIALOG (user_data); g_assert(self); ModestEasysetupWizardDialogPrivate *priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (self); - + priv->dirty = TRUE; - + /* Fill the providers picker, based on the selected country: */ if (priv->presets != NULL) { gint mcc = modest_country_picker_get_active_country_mcc ( @@ -362,9 +435,8 @@ on_account_country_selector_changed (HildonTouchSelector *widget, gint column, g static void update_user_email_from_provider (ModestEasysetupWizardDialog *self) { - ModestEasysetupWizardDialogPrivate *priv; - gchar* provider_id; - gchar* domain_name = NULL; + ModestEasysetupWizardDialogPrivate *priv; + gchar* provider_id = NULL, *with_at = NULL; g_assert(self); priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (self); @@ -372,18 +444,21 @@ update_user_email_from_provider (ModestEasysetupWizardDialog *self) /* Fill the providers combo, based on the selected country: */ provider_id = modest_provider_picker_get_active_provider_id ( MODEST_PROVIDER_PICKER (priv->account_serviceprovider_picker)); - - if(provider_id) - domain_name = modest_presets_get_domain (priv->presets, provider_id); - - if(!domain_name) - domain_name = g_strdup (MODEST_EXAMPLE_EMAIL_ADDRESS); - + + if (provider_id) { + gchar *domain_name = modest_presets_get_domain (priv->presets, provider_id); + with_at = g_strdup_printf ("%c%s", '@', domain_name); + g_free (domain_name); + } + + /* The sample address already contains the '@' */ + if (!with_at) + with_at = g_strdup (MODEST_EXAMPLE_EMAIL_ADDRESS); + if (priv->entry_user_email) - hildon_entry_set_text (HILDON_ENTRY (priv->entry_user_email), domain_name); - - g_free (domain_name); - + hildon_entry_set_text (HILDON_ENTRY (priv->entry_user_email), with_at); + + g_free (with_at); g_free (provider_id); } @@ -393,7 +468,7 @@ on_account_serviceprovider_selector_changed (HildonTouchSelector *widget, gint c ModestEasysetupWizardDialog *self = MODEST_EASYSETUP_WIZARD_DIALOG (user_data); g_assert(self); ModestEasysetupWizardDialogPrivate *priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (self); - + priv->dirty = TRUE; update_user_email_from_provider (self); @@ -409,14 +484,15 @@ on_entry_max (ModestValidatingEntry *self, gpointer user_data) static GtkWidget* create_page_account_details (ModestEasysetupWizardDialog *self) { - GtkWidget *caption; + GtkWidget *align; GtkWidget *box = gtk_vbox_new (FALSE, MODEST_MARGIN_NONE); GtkWidget *label = gtk_label_new(_("mcen_ia_accountdetails")); ModestEasysetupWizardDialogPrivate* priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE(self); gtk_label_set_line_wrap (GTK_LABEL (label), TRUE); - gtk_widget_set_size_request (label, LABELS_WIDTH, -1); - gtk_box_pack_start (GTK_BOX (box), label, FALSE, FALSE, MODEST_MARGIN_HALF); + gtk_misc_set_alignment (GTK_MISC (label), 0.0, 0.0); + gtk_misc_set_padding (GTK_MISC (label), MODEST_MARGIN_DOUBLE, MODEST_MARGIN_DOUBLE); + gtk_box_pack_start (GTK_BOX (box), label, FALSE, FALSE, 0); gtk_widget_show (label); /* Create a size group to be used by all captions. @@ -428,36 +504,32 @@ create_page_account_details (ModestEasysetupWizardDialog *self) /* The country widgets: */ priv->account_country_picker = GTK_WIDGET (modest_country_picker_new (MODEST_EDITABLE_SIZE, HILDON_BUTTON_ARRANGEMENT_HORIZONTAL)); - modest_maemo_utils_set_hbutton_layout (title_sizegroup, value_sizegroup, + modest_maemo_utils_set_hbutton_layout (title_sizegroup, value_sizegroup, _("mcen_fi_country"), priv->account_country_picker); g_signal_connect (G_OBJECT (priv->account_country_picker), "value-changed", G_CALLBACK (on_picker_button_value_changed), self); gtk_box_pack_start (GTK_BOX (box), priv->account_country_picker, FALSE, FALSE, MODEST_MARGIN_HALF); gtk_widget_show (priv->account_country_picker); - - GtkWidget *separator = gtk_hseparator_new (); - gtk_box_pack_start (GTK_BOX (box), separator, FALSE, FALSE, MODEST_MARGIN_HALF); - gtk_widget_show (separator); - - /* The service provider widgets: */ + + /* The service provider widgets: */ priv->account_serviceprovider_picker = GTK_WIDGET (modest_provider_picker_new (MODEST_EDITABLE_SIZE, HILDON_BUTTON_ARRANGEMENT_HORIZONTAL)); modest_maemo_utils_set_hbutton_layout (title_sizegroup, value_sizegroup, - _("mcen_fi_serviceprovider"), + _("mcen_fi_serviceprovider"), priv->account_serviceprovider_picker); g_signal_connect (G_OBJECT (priv->account_serviceprovider_picker), "value-changed", G_CALLBACK (on_serviceprovider_picker_button_value_changed), self); - gtk_box_pack_start (GTK_BOX (box), priv->account_serviceprovider_picker, FALSE, FALSE, MODEST_MARGIN_HALF); + gtk_box_pack_start (GTK_BOX (box), priv->account_serviceprovider_picker, FALSE, FALSE, 0); gtk_widget_show (priv->account_serviceprovider_picker); - - /* The description widgets: */ + + /* The description widgets: */ priv->entry_account_title = GTK_WIDGET (modest_validating_entry_new ()); g_signal_connect(G_OBJECT(priv->entry_account_title), "changed", G_CALLBACK(on_easysetup_changed), self); /* Do use auto-capitalization: */ - hildon_gtk_entry_set_input_mode (GTK_ENTRY (priv->entry_account_title), + hildon_gtk_entry_set_input_mode (GTK_ENTRY (priv->entry_account_title), HILDON_GTK_INPUT_MODE_FULL | HILDON_GTK_INPUT_MODE_AUTOCAP); - + /* Set a default account title, choosing one that does not already exist: */ /* Note that this is irrelevant to the non-user visible name, which we will create later. */ gchar* default_account_name_start = g_strdup (_("mcen_ia_emailsetup_defaultname")); @@ -465,18 +537,20 @@ create_page_account_details (ModestEasysetupWizardDialog *self) priv->account_manager, default_account_name_start); g_free (default_account_name_start); default_account_name_start = NULL; - + hildon_entry_set_text (HILDON_ENTRY (priv->entry_account_title), default_account_name); g_free (default_account_name); default_account_name = NULL; - caption = create_captioned (self, title_sizegroup, value_sizegroup, _("mcen_fi_account_title"), FALSE, - priv->entry_account_title); + priv->caption_account_title = create_captioned (self, title_sizegroup, value_sizegroup, + _("mcen_fi_account_title"), FALSE, + priv->entry_account_title); gtk_widget_show (priv->entry_account_title); - gtk_box_pack_start (GTK_BOX (box), caption, FALSE, FALSE, MODEST_MARGIN_HALF); - gtk_widget_show (caption); - - /* Prevent the use of some characters in the account title, + gtk_box_pack_start (GTK_BOX (box), priv->caption_account_title, FALSE, FALSE, 0); + gtk_widget_show (priv->caption_account_title); + gtk_widget_set_no_show_all (priv->caption_account_title, TRUE); + + /* Prevent the use of some characters in the account title, * as required by our UI specification: */ GList *list_prevent = NULL; list_prevent = g_list_append (list_prevent, "\\"); @@ -486,29 +560,34 @@ create_page_account_details (ModestEasysetupWizardDialog *self) list_prevent = g_list_append (list_prevent, "?"); list_prevent = g_list_append (list_prevent, "\""); /* The UI spec mentions “, but maybe means ", maybe both. */ list_prevent = g_list_append (list_prevent, "“"); - list_prevent = g_list_append (list_prevent, "<"); - list_prevent = g_list_append (list_prevent, ">"); + list_prevent = g_list_append (list_prevent, "<"); + list_prevent = g_list_append (list_prevent, ">"); list_prevent = g_list_append (list_prevent, "|"); - list_prevent = g_list_append (list_prevent, "^"); + list_prevent = g_list_append (list_prevent, "^"); modest_validating_entry_set_unallowed_characters ( MODEST_VALIDATING_ENTRY (priv->entry_account_title), list_prevent); g_list_free (list_prevent); list_prevent = NULL; modest_validating_entry_set_func(MODEST_VALIDATING_ENTRY(priv->entry_account_title), modest_utils_on_entry_invalid_character, self); - + /* Set max length as in the UI spec: * The UI spec seems to want us to show a dialog if we hit the maximum. */ gtk_entry_set_max_length (GTK_ENTRY (priv->entry_account_title), 64); - modest_validating_entry_set_max_func (MODEST_VALIDATING_ENTRY (priv->entry_account_title), + modest_validating_entry_set_max_func (MODEST_VALIDATING_ENTRY (priv->entry_account_title), on_entry_max, self); - + gtk_widget_show (GTK_WIDGET (box)); g_object_unref (title_sizegroup); g_object_unref (value_sizegroup); - - return GTK_WIDGET (box); + + align = gtk_alignment_new (0.0, 0.0, 1.0, 1.0); + gtk_alignment_set_padding (GTK_ALIGNMENT (align), 0, MODEST_MARGIN_DOUBLE, MODEST_MARGIN_DOUBLE, 0); + gtk_widget_show (align); + gtk_container_add (GTK_CONTAINER (align), box); + + return GTK_WIDGET (align); } static GtkWidget* @@ -518,35 +597,36 @@ create_page_user_details (ModestEasysetupWizardDialog *self) GtkSizeGroup* value_sizegroup; GtkWidget *box; ModestEasysetupWizardDialogPrivate *priv; + GtkWidget *align; priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE(self); - + /* Create a size group to be used by all captions. * Note that HildonCaption does not create a default size group if we do not specify one. * We use GTK_SIZE_GROUP_HORIZONTAL, so that the widths are the same. */ box = gtk_vbox_new (FALSE, MODEST_MARGIN_NONE); title_sizegroup = gtk_size_group_new(GTK_SIZE_GROUP_HORIZONTAL); value_sizegroup = gtk_size_group_new(GTK_SIZE_GROUP_HORIZONTAL); - + /* The name widgets: (use auto cap) */ priv->entry_user_name = GTK_WIDGET (modest_validating_entry_new ()); - hildon_gtk_entry_set_input_mode (GTK_ENTRY (priv->entry_user_name), + hildon_gtk_entry_set_input_mode (GTK_ENTRY (priv->entry_user_name), HILDON_GTK_INPUT_MODE_FULL | HILDON_GTK_INPUT_MODE_AUTOCAP); - + /* Set max length as in the UI spec: * The UI spec seems to want us to show a dialog if we hit the maximum. */ gtk_entry_set_max_length (GTK_ENTRY (priv->entry_user_name), 64); - modest_validating_entry_set_max_func (MODEST_VALIDATING_ENTRY (priv->entry_user_name), + modest_validating_entry_set_max_func (MODEST_VALIDATING_ENTRY (priv->entry_user_name), on_entry_max, self); GtkWidget *caption = create_captioned (self, title_sizegroup, value_sizegroup, _("mcen_li_emailsetup_name"), FALSE, priv->entry_user_name); - g_signal_connect(G_OBJECT(priv->entry_user_name), "changed", + g_signal_connect(G_OBJECT(priv->entry_user_name), "changed", G_CALLBACK(on_easysetup_changed), self); gtk_widget_show (priv->entry_user_name); - gtk_box_pack_start (GTK_BOX (box), caption, FALSE, FALSE, MODEST_MARGIN_HALF); + gtk_box_pack_start (GTK_BOX (box), caption, FALSE, FALSE, 0); gtk_widget_show (caption); - - /* Prevent the use of some characters in the name, + + /* Prevent the use of some characters in the name, * as required by our UI specification: */ GList *list_prevent = NULL; list_prevent = g_list_append (list_prevent, "<"); @@ -556,47 +636,47 @@ create_page_user_details (ModestEasysetupWizardDialog *self) modest_validating_entry_set_func(MODEST_VALIDATING_ENTRY(priv->entry_user_name), modest_utils_on_entry_invalid_character, self); g_list_free (list_prevent); - - /* The username widgets: */ + + /* The username widgets: */ priv->entry_user_username = GTK_WIDGET (modest_validating_entry_new ()); /* Auto-capitalization is the default, so let's turn it off: */ - hildon_gtk_entry_set_input_mode (GTK_ENTRY (priv->entry_user_username), + hildon_gtk_entry_set_input_mode (GTK_ENTRY (priv->entry_user_username), HILDON_GTK_INPUT_MODE_FULL); caption = create_captioned (self, title_sizegroup, value_sizegroup, _("mail_fi_username"), FALSE, priv->entry_user_username); gtk_widget_show (priv->entry_user_username); - gtk_box_pack_start (GTK_BOX (box), caption, FALSE, FALSE, MODEST_MARGIN_HALF); - g_signal_connect(G_OBJECT(priv->entry_user_username), "changed", + gtk_box_pack_start (GTK_BOX (box), caption, FALSE, FALSE, 0); + g_signal_connect(G_OBJECT(priv->entry_user_username), "changed", G_CALLBACK(on_easysetup_changed), self); gtk_widget_show (caption); - - /* Prevent the use of some characters in the username, + + /* Prevent the use of some characters in the username, * as required by our UI specification: */ modest_validating_entry_set_func(MODEST_VALIDATING_ENTRY(priv->entry_user_username), modest_utils_on_entry_invalid_character, self); - + /* Set max length as in the UI spec: * The UI spec seems to want us to show a dialog if we hit the maximum. */ gtk_entry_set_max_length (GTK_ENTRY (priv->entry_user_username), 64); - modest_validating_entry_set_max_func (MODEST_VALIDATING_ENTRY (priv->entry_user_username), + modest_validating_entry_set_max_func (MODEST_VALIDATING_ENTRY (priv->entry_user_username), on_entry_max, self); - - /* The password widgets: */ + + /* The password widgets: */ priv->entry_user_password = hildon_entry_new (MODEST_EDITABLE_SIZE); /* Auto-capitalization is the default, so let's turn it off: */ - hildon_gtk_entry_set_input_mode (GTK_ENTRY (priv->entry_user_password), + hildon_gtk_entry_set_input_mode (GTK_ENTRY (priv->entry_user_password), HILDON_GTK_INPUT_MODE_FULL | HILDON_GTK_INPUT_MODE_INVISIBLE); gtk_entry_set_visibility (GTK_ENTRY (priv->entry_user_password), FALSE); /* gtk_entry_set_invisible_char (GTK_ENTRY (priv->entry_user_password), '*'); */ caption = create_captioned (self, title_sizegroup, value_sizegroup, _("mail_fi_password"), FALSE, priv->entry_user_password); - g_signal_connect(G_OBJECT(priv->entry_user_password), "changed", + g_signal_connect(G_OBJECT(priv->entry_user_password), "changed", G_CALLBACK(on_easysetup_changed), self); gtk_widget_show (priv->entry_user_password); - gtk_box_pack_start (GTK_BOX (box), caption, FALSE, FALSE, MODEST_MARGIN_HALF); + gtk_box_pack_start (GTK_BOX (box), caption, FALSE, FALSE, 0); gtk_widget_show (caption); - - /* The email address widgets: */ + + /* The email address widgets: */ priv->entry_user_email = GTK_WIDGET (modest_validating_entry_new ()); /* Auto-capitalization is the default, so let's turn it off: */ hildon_gtk_entry_set_input_mode (GTK_ENTRY (priv->entry_user_email), HILDON_GTK_INPUT_MODE_FULL); @@ -604,60 +684,72 @@ create_page_user_details (ModestEasysetupWizardDialog *self) _("mcen_li_emailsetup_email_address"), FALSE, priv->entry_user_email); update_user_email_from_provider (self); gtk_widget_show (priv->entry_user_email); - gtk_box_pack_start (GTK_BOX (box), caption, FALSE, FALSE, MODEST_MARGIN_HALF); - g_signal_connect(G_OBJECT(priv->entry_user_email), "changed", + gtk_box_pack_start (GTK_BOX (box), caption, FALSE, FALSE, 0); + g_signal_connect(G_OBJECT(priv->entry_user_email), "changed", G_CALLBACK(on_easysetup_changed), self); gtk_widget_show (caption); - + /* Set max length as in the UI spec: * The UI spec seems to want us to show a dialog if we hit the maximum. */ gtk_entry_set_max_length (GTK_ENTRY (priv->entry_user_email), 64); - modest_validating_entry_set_max_func (MODEST_VALIDATING_ENTRY (priv->entry_user_email), + modest_validating_entry_set_max_func (MODEST_VALIDATING_ENTRY (priv->entry_user_email), on_entry_max, self); - - + + gtk_widget_show (GTK_WIDGET (box)); g_object_unref (title_sizegroup); g_object_unref (value_sizegroup); - - return GTK_WIDGET (box); + + align = gtk_alignment_new (0.0, 0.0, 1.0, 1.0); + gtk_alignment_set_padding (GTK_ALIGNMENT (align), 0, MODEST_MARGIN_DOUBLE, MODEST_MARGIN_DOUBLE, 0); + gtk_widget_show (align); + gtk_container_add (GTK_CONTAINER (align), box); + + return GTK_WIDGET (align); } -static GtkWidget* +static GtkWidget* create_page_complete_easysetup (ModestEasysetupWizardDialog *self) { + GtkWidget *align; GtkWidget *box = gtk_vbox_new (FALSE, MODEST_MARGIN_NONE); - + GtkWidget *label = gtk_label_new(_("mcen_ia_emailsetup_setup_complete")); gtk_label_set_line_wrap (GTK_LABEL (label), TRUE); gtk_widget_set_size_request (label, LABELS_WIDTH, -1); + gtk_misc_set_padding (GTK_MISC (label), MODEST_MARGIN_DOUBLE, MODEST_MARGIN_DOUBLE); gtk_misc_set_alignment (GTK_MISC (label), 0, 0.5); - /* The documentation for gtk_label_set_line_wrap() says that we must - * call gtk_widget_set_size_request() with a hard-coded width, - * though I wonder why gtk_label_set_max_width_chars() isn't enough. */ + gtk_box_pack_start (GTK_BOX (box), label, FALSE, FALSE, 0); gtk_widget_show (label); - + label = gtk_label_new (_("mcen_ia_easysetup_complete")); gtk_label_set_line_wrap (GTK_LABEL (label), TRUE); gtk_widget_set_size_request (label, LABELS_WIDTH, -1); - - gtk_misc_set_alignment (GTK_MISC (label), 0, 0.5); - gtk_box_pack_start (GTK_BOX (box), label, FALSE, FALSE, 0); + + gtk_misc_set_padding (GTK_MISC (label), MODEST_MARGIN_DOUBLE, MODEST_MARGIN_DOUBLE); + gtk_misc_set_alignment (GTK_MISC (label), 0.0, 1.0); + gtk_box_pack_start (GTK_BOX (box), label, TRUE, TRUE, 0); gtk_widget_show (label); - + gtk_widget_show (GTK_WIDGET (box)); - return GTK_WIDGET (box); + + align = gtk_alignment_new (0.0, 0.0, 1.0, 1.0); + gtk_alignment_set_padding (GTK_ALIGNMENT (align), 0, MODEST_MARGIN_DOUBLE, MODEST_MARGIN_DOUBLE, 0); + gtk_widget_show (align); + gtk_container_add (GTK_CONTAINER (align), box); + + return GTK_WIDGET (align); } -/** Change the caption title for the incoming server, +/** Change the caption title for the incoming server, * as specified in the UI spec: */ -static void +static void update_incoming_server_title (ModestEasysetupWizardDialog *self) { - ModestEasysetupWizardDialogPrivate* priv; - ModestProtocolType protocol_type; + ModestEasysetupWizardDialogPrivate* priv; + ModestProtocolType protocol_type; ModestProtocolRegistry *protocol_registry; priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE(self); @@ -672,7 +764,7 @@ update_incoming_server_title (ModestEasysetupWizardDialog *self) const gchar *protocol_display_name; ModestProtocol *protocol; - protocol = modest_protocol_registry_get_protocol_by_type (protocol_registry, + protocol = modest_protocol_registry_get_protocol_by_type (protocol_registry, protocol_type); protocol_display_name = modest_protocol_get_display_name (protocol); @@ -685,10 +777,10 @@ update_incoming_server_title (ModestEasysetupWizardDialog *self) } } -/** Change the caption title for the incoming server, +/** Change the caption title for the incoming server, * as specified in the UI spec: */ -static void +static void update_incoming_server_security_choices (ModestEasysetupWizardDialog *self) { ModestEasysetupWizardDialogPrivate *priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE(self); @@ -696,18 +788,18 @@ update_incoming_server_security_choices (ModestEasysetupWizardDialog *self) ModestProtocolType protocol_type; ModestSecurityOptionsView *view; - server_type_picker = + server_type_picker = MODEST_SERVERTYPE_PICKER (priv->incoming_servertype_picker); - protocol_type = + protocol_type = modest_servertype_picker_get_active_servertype (server_type_picker); - + /* Fill the combo with appropriately titled choices for all those protocols */ view = MODEST_SECURITY_OPTIONS_VIEW (priv->incoming_security); modest_security_options_view_set_server_type (view, protocol_type); } -static void +static void on_servertype_selector_changed(HildonTouchSelector *selector, gint column, gpointer user_data) { ModestEasysetupWizardDialog *self = MODEST_EASYSETUP_WIZARD_DIALOG (user_data); @@ -716,7 +808,7 @@ on_servertype_selector_changed(HildonTouchSelector *selector, gint column, gpoin ModestProtocolType protocol_type; priv->dirty = TRUE; - + /* Update title */ update_incoming_server_title (self); @@ -729,7 +821,7 @@ on_servertype_selector_changed(HildonTouchSelector *selector, gint column, gpoin set_default_custom_servernames (self); } -static void +static void on_entry_incoming_servername_changed(GtkEntry *entry, gpointer user_data) { ModestEasysetupWizardDialog *self = MODEST_EASYSETUP_WIZARD_DIALOG (user_data); @@ -738,13 +830,14 @@ on_entry_incoming_servername_changed(GtkEntry *entry, gpointer user_data) priv->server_changes |= MODEST_EASYSETUP_WIZARD_DIALOG_INCOMING_CHANGED; } -static GtkWidget* +static GtkWidget* create_page_custom_incoming (ModestEasysetupWizardDialog *self) { ModestProtocolRegistry *protocol_registry; - ModestEasysetupWizardDialogPrivate* priv; - GtkWidget *box; - GtkWidget *scrolled_window; + ModestEasysetupWizardDialogPrivate* priv; + GtkWidget *box; + GtkWidget *align; + GtkWidget *pannable; GtkWidget *label; GtkSizeGroup *title_sizegroup; GtkSizeGroup *value_sizegroup; @@ -753,35 +846,35 @@ create_page_custom_incoming (ModestEasysetupWizardDialog *self) protocol_registry = modest_runtime_get_protocol_registry (); box = gtk_vbox_new (FALSE, MODEST_MARGIN_NONE); - scrolled_window = gtk_scrolled_window_new (NULL, NULL); - - gtk_scrolled_window_set_policy (GTK_SCROLLED_WINDOW (scrolled_window), - GTK_POLICY_NEVER, - GTK_POLICY_AUTOMATIC); + pannable = hildon_pannable_area_new (); /* Show note that account type cannot be changed in future: */ label = gtk_label_new (_("mcen_ia_emailsetup_account_type")); gtk_label_set_line_wrap (GTK_LABEL (label), TRUE); + gtk_misc_set_padding (GTK_MISC (label), MODEST_MARGIN_DOUBLE, MODEST_MARGIN_DOUBLE); + gtk_misc_set_alignment (GTK_MISC (label), 0.0, 0.0); gtk_widget_set_size_request (label, LABELS_WIDTH, -1); gtk_box_pack_start (GTK_BOX (box), label, FALSE, FALSE, 0); gtk_widget_show (label); - + /* Create a size group to be used by all captions. * Note that HildonCaption does not create a default size group if we do not specify one. * We use GTK_SIZE_GROUP_HORIZONTAL, so that the widths are the same. */ title_sizegroup = gtk_size_group_new(GTK_SIZE_GROUP_HORIZONTAL); value_sizegroup = gtk_size_group_new(GTK_SIZE_GROUP_HORIZONTAL); - + /* The incoming server widgets: */ priv->incoming_servertype_picker = GTK_WIDGET (modest_servertype_picker_new (MODEST_EDITABLE_SIZE, HILDON_BUTTON_ARRANGEMENT_HORIZONTAL, TRUE)); - hildon_button_set_title (HILDON_BUTTON (priv->incoming_servertype_picker), _("mcen_li_emailsetup_type")); + modest_maemo_utils_set_hbutton_layout (title_sizegroup, value_sizegroup, + _("mcen_li_emailsetup_type"), + priv->incoming_servertype_picker); g_signal_connect (G_OBJECT (priv->incoming_servertype_picker), "value-changed", G_CALLBACK (on_picker_button_value_changed), self); - gtk_box_pack_start (GTK_BOX (box), priv->incoming_servertype_picker, FALSE, FALSE, MODEST_MARGIN_HALF); + gtk_box_pack_start (GTK_BOX (box), priv->incoming_servertype_picker, FALSE, FALSE, 0); gtk_widget_show (priv->incoming_servertype_picker); - + priv->entry_incomingserver = hildon_entry_new (MODEST_EDITABLE_SIZE); g_signal_connect(G_OBJECT(priv->entry_incomingserver), "changed", G_CALLBACK(on_easysetup_changed), self); /* Auto-capitalization is the default, so let's turn it off: */ @@ -791,16 +884,16 @@ create_page_custom_incoming (ModestEasysetupWizardDialog *self) /* The caption title will be updated in update_incoming_server_title(). * so this default text will never be seen: */ priv->caption_incoming = create_captioned (self, title_sizegroup, value_sizegroup, - "This will be removed", + "This will be removed", FALSE, priv->entry_incomingserver); update_incoming_server_title (self); gtk_widget_show (priv->entry_incomingserver); - gtk_box_pack_start (GTK_BOX (box), priv->caption_incoming, FALSE, FALSE, MODEST_MARGIN_HALF); + gtk_box_pack_start (GTK_BOX (box), priv->caption_incoming, FALSE, FALSE, 0); gtk_widget_show (priv->caption_incoming); - - /* Change the caption title when the servertype changes, + + /* Change the caption title when the servertype changes, * as in the UI spec: */ - g_signal_connect (G_OBJECT (hildon_picker_button_get_selector (HILDON_PICKER_BUTTON (priv->incoming_servertype_picker))), + g_signal_connect (G_OBJECT (hildon_picker_button_get_selector (HILDON_PICKER_BUTTON (priv->incoming_servertype_picker))), "changed", G_CALLBACK (on_servertype_selector_changed), self); @@ -809,36 +902,39 @@ create_page_custom_incoming (ModestEasysetupWizardDialog *self) G_CALLBACK (on_entry_incoming_servername_changed), self); /* The secure connection widgets. These are only valid for - protocols with security */ - priv->incoming_security = + protocols with security */ + priv->incoming_security = modest_maemo_security_options_view_new (MODEST_SECURITY_OPTIONS_INCOMING, FALSE, title_sizegroup, value_sizegroup); - gtk_box_pack_start (GTK_BOX (box), priv->incoming_security, - FALSE, FALSE, MODEST_MARGIN_HALF); - gtk_widget_show_all (priv->incoming_security); + gtk_box_pack_start (GTK_BOX (box), priv->incoming_security, + FALSE, FALSE, 0); + gtk_widget_show (priv->incoming_security); /* Set default selection */ modest_servertype_picker_set_active_servertype ( - MODEST_SERVERTYPE_PICKER (priv->incoming_servertype_picker), + MODEST_SERVERTYPE_PICKER (priv->incoming_servertype_picker), MODEST_PROTOCOLS_STORE_POP); - gtk_scrolled_window_add_with_viewport (GTK_SCROLLED_WINDOW (scrolled_window), box); - gtk_container_set_focus_vadjustment (GTK_CONTAINER (box), - gtk_scrolled_window_get_vadjustment (GTK_SCROLLED_WINDOW (scrolled_window))); + 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); + gtk_widget_show (align); + gtk_container_add (GTK_CONTAINER (align), box); + + hildon_pannable_area_add_with_viewport (HILDON_PANNABLE_AREA (pannable), align); gtk_widget_show (GTK_WIDGET (box)); - gtk_widget_show (scrolled_window); + gtk_widget_show (pannable); g_object_unref (title_sizegroup); g_object_unref (value_sizegroup); - return GTK_WIDGET (scrolled_window); + return GTK_WIDGET (pannable); } static void on_check_button_changed (HildonCheckButton *button, gpointer user_data) { GtkWidget *widget = GTK_WIDGET (user_data); - + /* Enable the widget only if the check button is active: */ const gboolean enable = hildon_check_button_get_active (button); gtk_widget_set_sensitive (widget, enable); @@ -851,7 +947,7 @@ enable_widget_for_checkbutton (GtkWidget *widget, HildonCheckButton* button) { g_signal_connect (G_OBJECT (button), "toggled", G_CALLBACK (on_check_button_changed), widget); - + /* Set the starting sensitivity: */ on_check_button_changed (button, widget); } @@ -862,11 +958,11 @@ on_button_outgoing_smtp_servers (GtkButton *button, gpointer user_data) ModestEasysetupWizardDialog * self = MODEST_EASYSETUP_WIZARD_DIALOG (user_data); ModestEasysetupWizardDialogPrivate* priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE(self); GtkWidget *specific_window; - + /* We set dirty here because setting it depending on the connection specific dialog seems overkill */ priv->dirty = TRUE; - + /* Create the window, if necessary: */ specific_window = (GtkWidget *) modest_connection_specific_smtp_window_new (); modest_connection_specific_smtp_window_fill_with_connections (MODEST_CONNECTION_SPECIFIC_SMTP_WINDOW (specific_window), priv->account_manager); @@ -876,7 +972,7 @@ on_button_outgoing_smtp_servers (GtkButton *button, gpointer user_data) gtk_widget_show (specific_window); } -static void +static void on_entry_outgoing_servername_changed (GtkEntry *entry, gpointer user_data) { ModestEasysetupWizardDialog *self = MODEST_EASYSETUP_WIZARD_DIALOG (user_data); @@ -884,19 +980,23 @@ on_entry_outgoing_servername_changed (GtkEntry *entry, gpointer user_data) priv->server_changes |= MODEST_EASYSETUP_WIZARD_DIALOG_OUTGOING_CHANGED; } -static GtkWidget* +static GtkWidget* create_page_custom_outgoing (ModestEasysetupWizardDialog *self) { ModestEasysetupWizardDialogPrivate *priv; gchar *smtp_caption_label; + GtkWidget *pannable; + GtkWidget *align; GtkWidget *box = gtk_vbox_new (FALSE, MODEST_MARGIN_NONE); - + + pannable = hildon_pannable_area_new (); + /* Create a size group to be used by all captions. * Note that HildonCaption does not create a default size group if we do not specify one. * We use GTK_SIZE_GROUP_HORIZONTAL, so that the widths are the same. */ GtkSizeGroup *title_sizegroup = gtk_size_group_new(GTK_SIZE_GROUP_HORIZONTAL); GtkSizeGroup *value_sizegroup = gtk_size_group_new(GTK_SIZE_GROUP_HORIZONTAL); - + /* The outgoing server widgets: */ priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (self); priv->entry_outgoingserver = hildon_entry_new (MODEST_EDITABLE_SIZE); @@ -909,26 +1009,26 @@ create_page_custom_outgoing (ModestEasysetupWizardDialog *self) smtp_caption_label, TRUE, priv->entry_outgoingserver); g_free (smtp_caption_label); gtk_widget_show (priv->entry_outgoingserver); - gtk_box_pack_start (GTK_BOX (box), caption, FALSE, FALSE, MODEST_MARGIN_HALF); + gtk_box_pack_start (GTK_BOX (box), caption, FALSE, FALSE, 0); gtk_widget_show (caption); set_default_custom_servernames (self); /* The secure connection widgets. These are only valid for - protocols with security */ - priv->outgoing_security = + protocols with security */ + priv->outgoing_security = modest_maemo_security_options_view_new (MODEST_SECURITY_OPTIONS_OUTGOING, FALSE, title_sizegroup, value_sizegroup); - gtk_box_pack_start (GTK_BOX (box), priv->outgoing_security, - FALSE, FALSE, MODEST_MARGIN_HALF); + gtk_box_pack_start (GTK_BOX (box), priv->outgoing_security, + FALSE, FALSE, 0); gtk_widget_show (priv->outgoing_security); GtkWidget *separator = gtk_hseparator_new (); - gtk_box_pack_start (GTK_BOX (box), separator, FALSE, FALSE, MODEST_MARGIN_HALF); + gtk_box_pack_start (GTK_BOX (box), separator, FALSE, FALSE, 0); gtk_widget_show (separator); /* connection-specific checkbox: */ priv->checkbox_outgoing_smtp_specific = hildon_check_button_new (HILDON_SIZE_FINGER_HEIGHT); - hildon_check_button_set_active (HILDON_CHECK_BUTTON (priv->checkbox_outgoing_smtp_specific), + hildon_check_button_set_active (HILDON_CHECK_BUTTON (priv->checkbox_outgoing_smtp_specific), FALSE); gtk_button_set_label (GTK_BUTTON (priv->checkbox_outgoing_smtp_specific), _("mcen_fi_advsetup_connection_smtp")); @@ -939,18 +1039,18 @@ create_page_custom_outgoing (ModestEasysetupWizardDialog *self) gtk_widget_show (priv->checkbox_outgoing_smtp_specific); gtk_box_pack_start (GTK_BOX (box), priv->checkbox_outgoing_smtp_specific, - FALSE, FALSE, MODEST_MARGIN_HALF); + FALSE, FALSE, 0); /* Connection-specific SMTP-Severs Edit button: */ priv->button_outgoing_smtp_servers = gtk_button_new_with_label (_("mcen_bd_advsetup_optional_smtp")); - hildon_gtk_widget_set_theme_size (priv->button_outgoing_smtp_servers, - HILDON_SIZE_FINGER_HEIGHT | HILDON_SIZE_AUTO_WIDTH); + hildon_gtk_widget_set_theme_size (priv->button_outgoing_smtp_servers, + HILDON_SIZE_FINGER_HEIGHT | HILDON_SIZE_AUTO_WIDTH); gtk_widget_show (priv->button_outgoing_smtp_servers); - gtk_box_pack_start (GTK_BOX (box), priv->button_outgoing_smtp_servers, - FALSE, FALSE, MODEST_MARGIN_HALF); + gtk_box_pack_start (GTK_BOX (box), priv->button_outgoing_smtp_servers, + FALSE, FALSE, 0); /* Only enable the button when the checkbox is checked: */ - enable_widget_for_checkbutton (priv->button_outgoing_smtp_servers, + enable_widget_for_checkbutton (priv->button_outgoing_smtp_servers, HILDON_CHECK_BUTTON (priv->checkbox_outgoing_smtp_specific)); g_signal_connect (G_OBJECT (priv->button_outgoing_smtp_servers), "clicked", @@ -958,12 +1058,21 @@ create_page_custom_outgoing (ModestEasysetupWizardDialog *self) g_signal_connect (G_OBJECT (priv->entry_outgoingserver), "changed", G_CALLBACK (on_entry_outgoing_servername_changed), self); + + + 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); + gtk_widget_show (align); + gtk_container_add (GTK_CONTAINER (align), box); + + hildon_pannable_area_add_with_viewport (HILDON_PANNABLE_AREA (pannable), align); gtk_widget_show (GTK_WIDGET (box)); + gtk_widget_show (pannable); g_object_unref (title_sizegroup); g_object_unref (value_sizegroup); - - return GTK_WIDGET (box); + + return GTK_WIDGET (pannable); } static gboolean @@ -972,7 +1081,7 @@ show_advanced_edit(gpointer user_data) ModestEasysetupWizardDialog * self = MODEST_EASYSETUP_WIZARD_DIALOG (user_data); ModestEasysetupWizardDialogPrivate *priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (self); gint response; - + /* Show the Account Settings window: */ ModestAccountSettingsDialog *dialog = modest_default_account_settings_dialog_new (); if (priv->pending_load_settings) { @@ -980,13 +1089,13 @@ show_advanced_edit(gpointer user_data) priv->pending_load_settings = FALSE; } modest_account_settings_dialog_load_settings (dialog, priv->settings); - + modest_window_mgr_set_modal (modest_runtime_get_window_mgr (), GTK_WINDOW (dialog), GTK_WINDOW (self)); - + response = gtk_dialog_run (GTK_DIALOG (dialog)); gtk_widget_destroy (GTK_WIDGET (dialog)); - + return FALSE; /* Do not call this timeout callback again. */ } @@ -994,44 +1103,55 @@ static void on_button_edit_advanced_settings (GtkButton *button, gpointer user_data) { ModestEasysetupWizardDialog * self = MODEST_EASYSETUP_WIZARD_DIALOG (user_data); - + /* Show the Account Settings window: */ show_advanced_edit(self); } -static GtkWidget* +static GtkWidget* create_page_complete_custom (ModestEasysetupWizardDialog *self) { - GtkWidget *box = gtk_vbox_new (FALSE, MODEST_MARGIN_NONE); + GtkWidget *align; + GtkWidget *box = gtk_vbox_new (FALSE, MODEST_MARGIN_DEFAULT); GtkWidget *label = gtk_label_new(_("mcen_ia_emailsetup_setup_complete")); GtkWidget *button_edit = gtk_button_new_with_label (_("mcen_fi_advanced_settings")); hildon_gtk_widget_set_theme_size (button_edit, HILDON_SIZE_FINGER_HEIGHT | HILDON_SIZE_AUTO_WIDTH); gtk_misc_set_alignment (GTK_MISC (label), 0, 0.5); gtk_widget_set_size_request (label, LABELS_WIDTH, -1); gtk_label_set_line_wrap (GTK_LABEL (label), TRUE); + gtk_misc_set_padding (GTK_MISC (label), MODEST_MARGIN_DOUBLE, MODEST_MARGIN_DOUBLE); + gtk_misc_set_alignment (GTK_MISC (label), 0.0, 0.0); gtk_box_pack_start (GTK_BOX (box), label, FALSE, FALSE, 0); gtk_widget_show (label); - + label = gtk_label_new (_("mcen_ia_customsetup_complete")); gtk_misc_set_alignment (GTK_MISC (label), 0.0, 0.0); gtk_widget_set_size_request (label, LABELS_WIDTH, -1); gtk_label_set_line_wrap (GTK_LABEL (label), TRUE); + gtk_misc_set_padding (GTK_MISC (label), MODEST_MARGIN_DOUBLE, MODEST_MARGIN_DOUBLE); + gtk_misc_set_alignment (GTK_MISC (label), 0.0, 0.5); gtk_box_pack_start (GTK_BOX (box), label, TRUE, TRUE, 0); gtk_widget_show (label); - + gtk_widget_show (button_edit); - gtk_box_pack_start (GTK_BOX (box), button_edit, FALSE, FALSE, MODEST_MARGIN_HALF); - - g_signal_connect (G_OBJECT (button_edit), "clicked", + gtk_box_pack_start (GTK_BOX (box), button_edit, FALSE, FALSE, 0); + + g_signal_connect (G_OBJECT (button_edit), "clicked", G_CALLBACK (on_button_edit_advanced_settings), self); - + gtk_widget_show (GTK_WIDGET (box)); - return GTK_WIDGET (box); + + align = gtk_alignment_new (0.0, 0.0, 1.0, 1.0); + gtk_alignment_set_padding (GTK_ALIGNMENT (align), 0, MODEST_MARGIN_DOUBLE, MODEST_MARGIN_DOUBLE, 0); + gtk_widget_show (align); + gtk_container_add (GTK_CONTAINER (align), box); + + return GTK_WIDGET (align); } /* */ -static void +static void on_response (ModestWizardDialog *wizard_dialog, gint response_id, gpointer user_data) @@ -1041,7 +1161,7 @@ on_response (ModestWizardDialog *wizard_dialog, invoke_enable_buttons_vfunc (self); } -static void +static void on_response_before (ModestWizardDialog *wizard_dialog, gint response_id, gpointer user_data) @@ -1052,7 +1172,7 @@ on_response_before (ModestWizardDialog *wizard_dialog, /* This is mostly copied from * src/maemo/modest-account-settings-dialog.c */ if (priv->dirty) { - GtkDialog *dialog = GTK_DIALOG (hildon_note_new_confirmation (GTK_WINDOW (self), + GtkDialog *dialog = GTK_DIALOG (hildon_note_new_confirmation (GTK_WINDOW (self), _("imum_nc_wizard_confirm_lose_changes"))); /* TODO: These button names will be ambiguous, and not * specified in the UI specification. */ @@ -1068,24 +1188,11 @@ on_response_before (ModestWizardDialog *wizard_dialog, } } -typedef struct IdleData { - ModestEasysetupWizardDialog *dialog; - ModestPresets *presets; -} IdleData; - -static gboolean -presets_idle (gpointer userdata) +static void +fill_providers (ModestEasysetupWizardDialog *self) { - IdleData *idle_data = (IdleData *) userdata; - ModestEasysetupWizardDialog *self = MODEST_EASYSETUP_WIZARD_DIALOG (idle_data->dialog); ModestEasysetupWizardDialogPrivate *priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (self); - g_assert (idle_data->presets); - - gdk_threads_enter (); - - priv->presets = idle_data->presets; - if (MODEST_IS_COUNTRY_PICKER (priv->account_country_picker)) { /* gint mcc = get_default_country_code(); */ gint mcc; @@ -1093,11 +1200,11 @@ presets_idle (gpointer userdata) modest_country_picker_load_data( MODEST_COUNTRY_PICKER (priv->account_country_picker)); /* connect to country picker's changed signal, so we can fill the provider picker: */ - g_signal_connect (G_OBJECT (hildon_picker_button_get_selector + g_signal_connect (G_OBJECT (hildon_picker_button_get_selector (HILDON_PICKER_BUTTON (priv->account_country_picker))), "changed", G_CALLBACK (on_account_country_selector_changed), self); - + modest_country_picker_set_active_country_locale ( MODEST_COUNTRY_PICKER (priv->account_country_picker)); mcc = modest_country_picker_get_active_country_mcc ( @@ -1106,16 +1213,39 @@ presets_idle (gpointer userdata) MODEST_PROVIDER_PICKER (priv->account_serviceprovider_picker), priv->presets, mcc); /* connect to providers picker's changed signal, so we can fill the email address: */ - g_signal_connect (G_OBJECT (hildon_picker_button_get_selector + g_signal_connect (G_OBJECT (hildon_picker_button_get_selector (HILDON_PICKER_BUTTON (priv->account_serviceprovider_picker))), "changed", G_CALLBACK (on_account_serviceprovider_selector_changed), self); - + modest_provider_picker_set_others_provider (MODEST_PROVIDER_PICKER (priv->account_serviceprovider_picker)); } priv->dirty = FALSE; +} + +typedef struct IdleData { + ModestEasysetupWizardDialog *dialog; + ModestPresets *presets; +} IdleData; + +static gboolean +presets_idle (gpointer userdata) +{ + IdleData *idle_data = (IdleData *) userdata; + ModestEasysetupWizardDialog *self = MODEST_EASYSETUP_WIZARD_DIALOG (idle_data->dialog); + ModestEasysetupWizardDialogPrivate *priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (self); + + g_assert (idle_data->presets); + + gdk_threads_enter (); + + priv->presets = idle_data->presets; + + if (priv->check_support_done) + fill_providers (self); + g_object_unref (idle_data->dialog); g_free (idle_data); @@ -1133,9 +1263,9 @@ presets_loader (gpointer userdata) const gchar* path = NULL; const gchar* path1 = MODEST_PROVIDER_DATA_FILE; - const gchar* path2 = MODEST_MAEMO_PROVIDER_DATA_FILE; - - if (access(path1, R_OK) == 0) + const gchar* path2 = MODEST_FALLBACK_PROVIDER_DATA_FILE; + + if (access(path1, R_OK) == 0) path = path1; else if (access(path2, R_OK) == 0) path = path2; @@ -1150,12 +1280,12 @@ presets_loader (gpointer userdata) g_warning ("%s: failed to parse '%s'", __FUNCTION__, path); return NULL; } - + idle_data = g_new0 (IdleData, 1); idle_data->dialog = self; idle_data->presets = presets; - - g_idle_add (presets_idle, idle_data); + + g_idle_add (presets_idle, idle_data); return NULL; } @@ -1209,62 +1339,68 @@ static void modest_easysetup_wizard_dialog_init (ModestEasysetupWizardDialog *self) { gtk_container_set_border_width (GTK_CONTAINER (self), MODEST_MARGIN_HALF); - + /* Create the notebook to be used by the ModestWizardDialog base class: * Each page of the notebook will be a page of the wizard: */ GtkNotebook *notebook = GTK_NOTEBOOK (gtk_notebook_new()); - gtk_widget_set_size_request (GTK_WIDGET (notebook), DIALOG_WIDTH, -1); - + gtk_widget_set_size_request (GTK_WIDGET (notebook), -1, MODEST_DIALOG_WINDOW_MAX_HEIGHT); + /* Set the notebook used by the ModestWizardDialog base class: */ g_object_set (G_OBJECT(self), "wizard-notebook", notebook, NULL); - + /* Set the wizard title: * The actual window title will be a combination of this and the page's tab label title. */ g_object_set (G_OBJECT(self), "wizard-name", _("mcen_ti_emailsetup"), NULL); /* Read in the information about known service providers: */ ModestEasysetupWizardDialogPrivate *priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (self); - + /* The server fields did not have been manually changed yet */ priv->server_changes = 0; priv->pending_load_settings = TRUE; - /* Get the account manager object, + /* Get the account manager object, * so we can check for existing accounts, * and create new accounts: */ priv->account_manager = modest_runtime_get_account_mgr (); g_object_ref (priv->account_manager); - + /* Initialize fields */ + priv->check_support_done = FALSE; + priv->check_support_show_progress_id = 0; + priv->check_support_progress_pulse_id = 0; + priv->pending_check_support = 0; + priv->destroyed = FALSE; priv->page_welcome = create_page_welcome (self); + gtk_widget_show (priv->page_welcome); priv->page_account_details = create_page_account_details (self); init_user_page (priv); init_incoming_page (priv); init_outgoing_page (priv); - priv->page_complete_easysetup = NULL; + priv->page_complete_easysetup = NULL; priv->page_complete_customsetup = NULL; priv->last_plugin_protocol_selected = MODEST_PROTOCOL_REGISTRY_TYPE_INVALID; priv->missing_data_signals = NULL; /* Add the common pages */ - modest_easysetup_wizard_dialog_append_page (notebook, priv->page_welcome, + modest_easysetup_wizard_dialog_append_page (notebook, priv->page_welcome, _("mcen_ti_emailsetup_welcome")); - modest_easysetup_wizard_dialog_append_page (notebook, priv->page_account_details, + modest_easysetup_wizard_dialog_append_page (notebook, priv->page_account_details, _("mcen_ti_accountdetails")); - - /* Connect to the dialog's response signal so we can enable/disable buttons + + /* Connect to the dialog's response signal so we can enable/disable buttons * for the newly-selected page, because the prev/next buttons cause response to be emitted. * Note that we use g_signal_connect_after() instead of g_signal_connect() - * so that we can be enable/disable after ModestWizardDialog has done its own + * so that we can be enable/disable after ModestWizardDialog has done its own * enabling/disabling of buttons. - * - * HOWEVER, this doesn't work because ModestWizardDialog's response signal handler + * + * HOWEVER, this doesn't work because ModestWizardDialog's response signal handler * does g_signal_stop_emission_by_name(), stopping our signal handler from running. - * - * It's not enough to connect to the notebook's switch-page signal, because - * ModestWizardDialog's "response" signal handler enables the buttons itself, + * + * It's not enough to connect to the notebook's switch-page signal, because + * ModestWizardDialog's "response" signal handler enables the buttons itself, * _after_ switching the page (understandably). * (Note that if we had, if we used g_signal_connect() instead of g_signal_connect_after() * then gtk_notebook_get_current_page() would return an incorrect value.) @@ -1281,27 +1417,29 @@ modest_easysetup_wizard_dialog_init (ModestEasysetupWizardDialog *self) /* Reset dirty, because there was no user input until now */ priv->dirty = FALSE; - - /* When this window is shown, hibernation should not be possible, + + /* When this window is shown, hibernation should not be possible, * because there is no sensible way to save the state: */ modest_window_mgr_prevent_hibernation_while_window_is_shown ( - modest_runtime_get_window_mgr (), GTK_WINDOW (self)); + modest_runtime_get_window_mgr (), GTK_WINDOW (self)); /* Load provider presets */ g_object_ref (self); g_thread_create (presets_loader, self, FALSE, NULL); priv->settings = modest_account_settings_new (); + + check_support_of_protocols (self); } ModestEasysetupWizardDialog* modest_easysetup_wizard_dialog_new (void) -{ - +{ + return g_object_new (MODEST_TYPE_EASYSETUP_WIZARD_DIALOG, NULL); } -static void +static void create_subsequent_customsetup_pages (ModestEasysetupWizardDialog *self) { ModestEasysetupWizardDialogPrivate *priv; @@ -1314,18 +1452,18 @@ create_subsequent_customsetup_pages (ModestEasysetupWizardDialog *self) if (!priv->page_user_details) { priv->page_user_details = create_page_user_details (self); - } + } /* Create the custom pages: */ if(!(priv->page_custom_incoming)) { priv->page_custom_incoming = create_page_custom_incoming (self); } - + /* TODO: only if needed */ if(!(priv->page_custom_outgoing)) { priv->page_custom_outgoing = create_page_custom_outgoing (self); } - + if(!(priv->page_complete_customsetup)) { priv->page_complete_customsetup = create_page_complete_custom (self); } @@ -1335,22 +1473,22 @@ create_subsequent_customsetup_pages (ModestEasysetupWizardDialog *self) _("mcen_ti_emailsetup_userdetails")); if (!gtk_widget_get_parent (GTK_WIDGET (priv->page_custom_incoming))) - modest_easysetup_wizard_dialog_append_page (notebook, priv->page_custom_incoming, + modest_easysetup_wizard_dialog_append_page (notebook, priv->page_custom_incoming, _("mcen_ti_emailsetup_incomingdetails")); - - if (!gtk_widget_get_parent (GTK_WIDGET (priv->page_custom_outgoing))) - modest_easysetup_wizard_dialog_append_page (notebook, priv->page_custom_outgoing, + + if (!gtk_widget_get_parent (GTK_WIDGET (priv->page_custom_outgoing))) + modest_easysetup_wizard_dialog_append_page (notebook, priv->page_custom_outgoing, _("mcen_ti_emailsetup_outgoingdetails")); - + if (!gtk_widget_get_parent (GTK_WIDGET (priv->page_complete_customsetup))) - modest_easysetup_wizard_dialog_append_page (notebook, priv->page_complete_customsetup, + modest_easysetup_wizard_dialog_append_page (notebook, priv->page_complete_customsetup, _("mcen_ti_emailsetup_complete")); - + /* This is unnecessary with GTK+ 2.10: */ modest_wizard_dialog_force_title_update (MODEST_WIZARD_DIALOG(self)); } - -static void + +static void create_subsequent_easysetup_pages (ModestEasysetupWizardDialog *self) { ModestEasysetupWizardDialogPrivate *priv; @@ -1358,24 +1496,24 @@ create_subsequent_easysetup_pages (ModestEasysetupWizardDialog *self) g_object_get (self, "wizard-notebook", ¬ebook, NULL); g_assert(notebook); - + /* Create the easysetup-specific pages: */ priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (self); if(!priv->page_complete_easysetup) priv->page_complete_easysetup = create_page_complete_easysetup (self); if (!gtk_widget_get_parent (GTK_WIDGET (priv->page_complete_easysetup))) - modest_easysetup_wizard_dialog_append_page (notebook, priv->page_complete_easysetup, + modest_easysetup_wizard_dialog_append_page (notebook, priv->page_complete_easysetup, _("mcen_ti_emailsetup_complete")); - + /* This is unnecessary with GTK+ 2.10: */ modest_wizard_dialog_force_title_update (MODEST_WIZARD_DIALOG(self)); } /* */ static void -remove_non_common_tabs (GtkNotebook *notebook, - gboolean remove_user_details) +remove_non_common_tabs (GtkNotebook *notebook, + gboolean remove_user_details) { gint starting_tab; /* The first 2 tabs are the common ones (welcome tab and the @@ -1383,8 +1521,8 @@ remove_non_common_tabs (GtkNotebook *notebook, end */ starting_tab = (remove_user_details) ? 2 : 3; - while (gtk_notebook_get_n_pages (notebook) > starting_tab) - gtk_notebook_remove_page (notebook, -1); + while (gtk_notebook_get_n_pages (notebook) > starting_tab) + gtk_notebook_remove_page (notebook, -1); } static void @@ -1396,10 +1534,10 @@ on_missing_mandatory_data (ModestAccountProtocol *protocol, } /* After the user details page, - * the following pages depend on whether "Other" was chosen + * the following pages depend on whether "Other" was chosen * in the provider combobox on the account page */ -static void +static void create_subsequent_pages (ModestEasysetupWizardDialog *self) { ModestEasysetupWizardDialogPrivate *priv; @@ -1411,6 +1549,9 @@ create_subsequent_pages (ModestEasysetupWizardDialog *self) picker = MODEST_PROVIDER_PICKER (priv->account_serviceprovider_picker); id_type = modest_provider_picker_get_active_id_type (picker); g_object_get (self, "wizard-notebook", ¬ebook, NULL); + modest_wizard_dialog_set_response_override_handler (MODEST_WIZARD_DIALOG (self), + NULL); + if (id_type == MODEST_PROVIDER_PICKER_ID_OTHER) { /* "Other..." was selected: */ @@ -1419,8 +1560,8 @@ create_subsequent_pages (ModestEasysetupWizardDialog *self) if (priv->page_complete_easysetup) { remove_non_common_tabs (notebook, FALSE); priv->page_complete_easysetup = NULL; - } - + } + /* If we come from a rollbacked plugin protocol setup */ if (priv->last_plugin_protocol_selected != MODEST_PROTOCOL_REGISTRY_TYPE_INVALID) { remove_non_common_tabs (notebook, TRUE); @@ -1447,16 +1588,16 @@ create_subsequent_pages (ModestEasysetupWizardDialog *self) gchar *proto_name; ModestProtocolType proto_type; - + /* If we come from a rollbacked easy setup */ - if (priv->last_plugin_protocol_selected == + if (priv->last_plugin_protocol_selected == MODEST_PROTOCOL_REGISTRY_TYPE_INVALID && priv->page_complete_easysetup) { remove_non_common_tabs (notebook, TRUE); init_user_page (priv); priv->page_complete_easysetup = NULL; } - + proto_name = modest_provider_picker_get_active_provider_id (picker); protocol = modest_protocol_registry_get_protocol_by_name (modest_runtime_get_protocol_registry (), MODEST_PROTOCOL_REGISTRY_PROVIDER_PROTOCOLS, @@ -1467,26 +1608,39 @@ create_subsequent_pages (ModestEasysetupWizardDialog *self) proto_type != priv->last_plugin_protocol_selected) { ModestPairList *tabs; GSList *tmp; + gboolean first_page = TRUE; + ModestWizardDialogResponseOverrideFunc response_override; /* Remember the last selected plugin protocol */ priv->last_plugin_protocol_selected = proto_type; /* Get tabs */ tabs = modest_account_protocol_get_easysetupwizard_tabs (MODEST_ACCOUNT_PROTOCOL (protocol)); + response_override = modest_account_protocol_get_wizard_response_override + (MODEST_ACCOUNT_PROTOCOL (protocol)); + modest_wizard_dialog_set_response_override_handler (MODEST_WIZARD_DIALOG (self), + response_override); tmp = (GSList *) tabs; while (tmp) { ModestPair *pair = (ModestPair *) tmp->data; modest_easysetup_wizard_dialog_append_page (notebook, GTK_WIDGET (pair->second), (const gchar *) pair->first); + if (first_page) { + gtk_container_set_focus_child (GTK_CONTAINER (notebook), + GTK_WIDGET (pair->second)); + first_page = FALSE; + } /* Connect signals */ - priv->missing_data_signals = - modest_signal_mgr_connect (priv->missing_data_signals, - G_OBJECT (pair->second), - "missing-mandatory-data", - G_CALLBACK (on_missing_mandatory_data), - self); + if (MODEST_IS_EASYSETUP_WIZARD_PAGE (pair->second)) { + priv->missing_data_signals = + modest_signal_mgr_connect (priv->missing_data_signals, + G_OBJECT (pair->second), + "missing-mandatory-data", + G_CALLBACK (on_missing_mandatory_data), + self); + } g_free (pair->first); tmp = g_slist_next (tmp); @@ -1497,7 +1651,7 @@ create_subsequent_pages (ModestEasysetupWizardDialog *self) } g_free (proto_name); } else { - if (priv->last_plugin_protocol_selected != + if (priv->last_plugin_protocol_selected != MODEST_PROTOCOL_REGISTRY_TYPE_INVALID) { remove_non_common_tabs (notebook, TRUE); init_user_page (priv); @@ -1508,12 +1662,12 @@ create_subsequent_pages (ModestEasysetupWizardDialog *self) } if (!priv->page_user_details) { priv->page_user_details = create_page_user_details (self); - modest_easysetup_wizard_dialog_append_page (notebook, + modest_easysetup_wizard_dialog_append_page (notebook, priv->page_user_details, _("mcen_ti_emailsetup_userdetails")); } } - + /* Create the easysetup pages: */ create_subsequent_easysetup_pages (self); } @@ -1531,37 +1685,37 @@ util_get_default_servername_from_email_address (const gchar* email_address, Mode if (!email_address) return NULL; - + at = g_utf8_strchr (email_address, -1, '@'); if (!at || (g_utf8_strlen (at, -1) < 2)) return NULL; - + domain = g_utf8_next_char (at); if(!domain) return NULL; protocol_registry = modest_runtime_get_protocol_registry (); protocol = modest_protocol_registry_get_protocol_by_type (protocol_registry, protocol_type); - + if (modest_protocol_registry_protocol_type_has_tag (protocol_registry, protocol_type, MODEST_PROTOCOL_REGISTRY_REMOTE_STORE_PROTOCOLS) || modest_protocol_registry_protocol_type_has_tag (protocol_registry, protocol_type, MODEST_PROTOCOL_REGISTRY_TRANSPORT_PROTOCOLS)) { hostname = modest_protocol_get_name (protocol); } - + if (!hostname) return NULL; - + return g_strdup_printf ("%s.%s", hostname, domain); } -static void +static void set_default_custom_servernames (ModestEasysetupWizardDialog *self) { ModestEasysetupWizardDialogPrivate *priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE(self); if (!priv->entry_incomingserver) return; - + /* Set a default domain for the server, based on the email address, * if no server name was already specified. */ @@ -1572,29 +1726,29 @@ set_default_custom_servernames (ModestEasysetupWizardDialog *self) /* This could happen when the combo box has still no active iter */ if (protocol_type != MODEST_PROTOCOL_REGISTRY_TYPE_INVALID) { - const gchar* email_address = hildon_entry_get_text (HILDON_ENTRY(priv->entry_user_email)); - gchar* servername = util_get_default_servername_from_email_address (email_address, + const gchar* email_address = hildon_entry_get_text (HILDON_ENTRY(priv->entry_user_email)); + gchar* servername = util_get_default_servername_from_email_address (email_address, protocol_type); /* Do not set the INCOMING_CHANGED flag because of this edit */ g_signal_handlers_block_by_func (G_OBJECT (priv->entry_incomingserver), G_CALLBACK (on_entry_incoming_servername_changed), self); hildon_entry_set_text (HILDON_ENTRY (priv->entry_incomingserver), servername); g_signal_handlers_unblock_by_func (G_OBJECT (priv->entry_incomingserver), G_CALLBACK (on_entry_incoming_servername_changed), self); - + g_free (servername); } } - + /* Set a default domain for the server, based on the email address, * if no server name was already specified. */ if (!priv->entry_outgoingserver) return; - + if (priv->entry_user_email && ((priv->server_changes & MODEST_EASYSETUP_WIZARD_DIALOG_OUTGOING_CHANGED) == 0)) { const gchar* email_address = hildon_entry_get_text (HILDON_ENTRY(priv->entry_user_email)); - + gchar* servername = util_get_default_servername_from_email_address (email_address, MODEST_PROTOCOLS_TRANSPORT_SMTP); /* Do not set the OUTGOING_CHANGED flag because of this edit */ @@ -1621,15 +1775,15 @@ get_entered_account_title (ModestEasysetupWizardDialog *self) /* Strip it of whitespace at the start and end: */ gchar *result = g_strdup (account_title); result = g_strstrip (result); - + if (!result) return NULL; - + if (g_utf8_strlen (result, -1) == 0) { g_free (result); - return NULL; + return NULL; } - + return result; } } @@ -1647,17 +1801,17 @@ on_before_next (ModestWizardDialog *dialog, GtkWidget *current_page, GtkWidget * * this wizard */ if (next_page != NULL) priv->pending_load_settings = TRUE; - + /* Do extra validation that couldn't be done for every key press, * either because it was too slow, * or because it requires interaction: */ - if (current_page == priv->page_account_details) { + if (current_page == priv->page_account_details) { /* Check that the title is not already in use: */ gchar* account_title = get_entered_account_title (self); if (!account_title) return FALSE; - + /* Aavoid a clash with an existing display name: */ const gboolean name_in_use = modest_account_mgr_account_with_display_name_exists ( priv->account_manager, account_title); @@ -1666,7 +1820,7 @@ on_before_next (ModestWizardDialog *dialog, GtkWidget *current_page, GtkWidget * if (name_in_use) { /* Warn the user via a dialog: */ hildon_banner_show_information(NULL, NULL, _("mail_ib_account_name_already_existing")); - + return FALSE; } @@ -1678,11 +1832,11 @@ on_before_next (ModestWizardDialog *dialog, GtkWidget *current_page, GtkWidget * const gchar* email_address = hildon_entry_get_text (HILDON_ENTRY (priv->entry_user_email)); if ((!email_address) || (g_utf8_strlen (email_address, -1) == 0)) return FALSE; - + if (!modest_text_utils_validate_email_address (email_address, NULL)) { /* Warn the user via a dialog: */ hildon_banner_show_information (NULL, NULL, _("mcen_ib_invalid_email")); - + /* Return focus to the email address entry: */ gtk_widget_grab_focus (priv->entry_user_email); gtk_editable_select_region (GTK_EDITABLE (priv->entry_user_email), 0, -1); @@ -1690,7 +1844,7 @@ on_before_next (ModestWizardDialog *dialog, GtkWidget *current_page, GtkWidget * return FALSE; } } - + if (next_page == priv->page_custom_incoming) { set_default_custom_servernames (self); } else if (next_page == priv->page_custom_outgoing) { @@ -1702,11 +1856,11 @@ on_before_next (ModestWizardDialog *dialog, GtkWidget *current_page, GtkWidget * return FALSE; gtk_widget_show (priv->outgoing_security); } - - /* If this is the last page, and this is a click on Finish, + + /* If this is the last page, and this is a click on Finish, * then attempt to create the dialog. */ - if(!next_page && + if(!next_page && current_page != priv->page_account_details) /* This is NULL when this is a click on Finish. */ { if (priv->pending_load_settings) { @@ -1721,8 +1875,8 @@ on_before_next (ModestWizardDialog *dialog, GtkWidget *current_page, GtkWidget * modest_account_mgr_add_account_from_settings (priv->account_manager, priv->settings); } - - + + return TRUE; } @@ -1730,7 +1884,7 @@ static gboolean entry_is_empty (GtkWidget *entry) { if (!entry) return FALSE; - + const gchar* text = hildon_entry_get_text (HILDON_ENTRY (entry)); if ((!text) || (g_utf8_strlen (text, -1) == 0)) return TRUE; @@ -1738,12 +1892,12 @@ static gboolean entry_is_empty (GtkWidget *entry) /* Strip it of whitespace at the start and end: */ gchar *stripped = g_strdup (text); stripped = g_strstrip (stripped); - + if (!stripped) return TRUE; - + const gboolean result = (g_utf8_strlen (stripped, -1) == 0); - + g_free (stripped); return result; } @@ -1751,7 +1905,7 @@ static gboolean entry_is_empty (GtkWidget *entry) static void real_enable_buttons (ModestWizardDialog *dialog, gboolean enable_next) -{ +{ GtkNotebook *notebook = NULL; gboolean is_finish_tab; GtkWidget *current; @@ -1760,7 +1914,7 @@ real_enable_buttons (ModestWizardDialog *dialog, gboolean enable_next) /* Get data */ priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (dialog); g_object_get (dialog, "wizard-notebook", ¬ebook, NULL); - + /* Disable the Finish button until we are on the last page, * because HildonWizardDialog enables this for all but the * first page */ @@ -1777,8 +1931,12 @@ real_enable_buttons (ModestWizardDialog *dialog, gboolean enable_next) gtk_dialog_set_response_sensitive (GTK_DIALOG (dialog), MODEST_WIZARD_DIALOG_FINISH, FALSE); + + /* If the check support is not done then do not enable + the wizard to continue */ + enable_next = enable_next && priv->check_support_done; } - + gtk_dialog_set_response_sensitive (GTK_DIALOG (dialog), MODEST_WIZARD_DIALOG_NEXT, enable_next); @@ -1789,7 +1947,7 @@ on_enable_buttons (ModestWizardDialog *dialog, GtkWidget *current_page) { ModestEasysetupWizardDialog *self = MODEST_EASYSETUP_WIZARD_DIALOG (dialog); ModestEasysetupWizardDialogPrivate *priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (self); - + gboolean enable_next = TRUE; if (current_page == priv->page_welcome) { enable_next = TRUE; @@ -1797,11 +1955,11 @@ on_enable_buttons (ModestWizardDialog *dialog, GtkWidget *current_page) /* The account details title is mandatory: */ if (entry_is_empty(priv->entry_account_title)) enable_next = FALSE; - } else if (current_page == priv->page_user_details) { + } else if (current_page == priv->page_user_details) { /* The user details username is mandatory: */ if (entry_is_empty(priv->entry_user_username)) enable_next = FALSE; - + /* The user details email address is mandatory: */ if (enable_next && entry_is_empty (priv->entry_user_email)) enable_next = FALSE; @@ -1813,8 +1971,8 @@ on_enable_buttons (ModestWizardDialog *dialog, GtkWidget *current_page) enable_next = !modest_easysetup_wizard_page_validate ( MODEST_EASYSETUP_WIZARD_PAGE (current_page)); } - - /* Enable/disable buttons */ + + /* Enable/disable buttons */ real_enable_buttons (dialog, enable_next); } @@ -1827,13 +1985,15 @@ modest_easysetup_wizard_dialog_class_init (ModestEasysetupWizardDialogClass *kla object_class->dispose = modest_easysetup_wizard_dialog_dispose; object_class->finalize = modest_easysetup_wizard_dialog_finalize; - - /* Provide a vfunc implementation so we can decide + + /* Provide a vfunc implementation so we can decide * when to enable/disable the prev/next buttons. */ ModestWizardDialogClass *base_klass = (ModestWizardDialogClass*)(klass); base_klass->before_next = on_before_next; base_klass->enable_buttons = on_enable_buttons; + base_klass->update_model = on_update_model; + base_klass->save = on_save; } /** @@ -1868,7 +2028,7 @@ save_to_settings (ModestEasysetupWizardDialog *self) /* Get details from the specified presets: */ id_type = modest_provider_picker_get_active_id_type (picker); provider_id = modest_provider_picker_get_active_provider_id (picker); - + /* Let the plugin save the settings. We do a return in order to save an indentation level */ if (id_type == MODEST_PROVIDER_PICKER_ID_PLUGIN_PROTOCOL) { @@ -1912,7 +2072,7 @@ save_to_settings (ModestEasysetupWizardDialog *self) transport_settings = modest_account_settings_get_transport_settings (priv->settings); /* Incoming server: */ - /* Note: We need something as default for the transport store protocol values, + /* Note: We need something as default for the transport store protocol values, * or modest_account_mgr_add_server_account will fail. */ store_port = 0; store_protocol = MODEST_PROTOCOL_REGISTRY_TYPE_INVALID; @@ -1923,20 +2083,20 @@ save_to_settings (ModestEasysetupWizardDialog *self) ModestProtocolType store_provider_server_type; gboolean store_provider_use_alternate_port; /* Use presets: */ - store_hostname = modest_presets_get_server (priv->presets, provider_id, + store_hostname = modest_presets_get_server (priv->presets, provider_id, TRUE /* store */); - + store_provider_server_type = modest_presets_get_info_server_type (priv->presets, - provider_id, + provider_id, TRUE /* store */); store_security = modest_presets_get_info_server_security (priv->presets, - provider_id, - TRUE /* store */); + provider_id, + TRUE /* store */); store_auth_protocol = modest_presets_get_info_server_auth (priv->presets, - provider_id, - TRUE /* store */); + provider_id, + TRUE /* store */); store_provider_use_alternate_port = modest_presets_get_info_server_use_alternate_port (priv->presets, - provider_id, + provider_id, TRUE /* store */); /* We don't check for SMTP here as that is impossible for an incoming server. */ @@ -1957,17 +2117,17 @@ save_to_settings (ModestEasysetupWizardDialog *self) store_port = get_port_from_protocol(store_provider_server_type, use_alternate_port); } - modest_server_account_settings_set_security_protocol (store_settings, + modest_server_account_settings_set_security_protocol (store_settings, store_security); - modest_server_account_settings_set_auth_protocol (store_settings, + modest_server_account_settings_set_auth_protocol (store_settings, store_auth_protocol); if (store_port != 0) modest_server_account_settings_set_port (store_settings, store_port); } else { /* Use custom pages because no preset was specified: */ - store_hostname = g_strdup (hildon_entry_get_text (HILDON_ENTRY (priv->entry_incomingserver) )); + store_hostname = g_strdup (hildon_entry_get_text (HILDON_ENTRY (priv->entry_incomingserver) )); store_protocol = modest_servertype_picker_get_active_servertype ( - MODEST_SERVERTYPE_PICKER (priv->incoming_servertype_picker)); + MODEST_SERVERTYPE_PICKER (priv->incoming_servertype_picker)); modest_security_options_view_save_settings ( MODEST_SECURITY_OPTIONS_VIEW (priv->incoming_security), @@ -1982,27 +2142,27 @@ save_to_settings (ModestEasysetupWizardDialog *self) g_object_unref (store_settings); g_free (store_hostname); - + /* Outgoing server: */ transport_hostname = NULL; transport_protocol = MODEST_PROTOCOL_REGISTRY_TYPE_INVALID; transport_security = MODEST_PROTOCOLS_CONNECTION_NONE; transport_auth_protocol = MODEST_PROTOCOLS_AUTH_NONE; transport_port = 0; - + if (provider_id) { ModestProtocolType transport_provider_server_type; ModestProtocolType transport_provider_security; /* Use presets */ - transport_hostname = modest_presets_get_server (priv->presets, provider_id, + transport_hostname = modest_presets_get_server (priv->presets, provider_id, FALSE /* transport */); - + transport_provider_server_type = modest_presets_get_info_server_type (priv->presets, - provider_id, - FALSE /* transport */); - transport_provider_security = modest_presets_get_info_server_security (priv->presets, - provider_id, + provider_id, + FALSE /* transport */); + transport_provider_security = modest_presets_get_info_server_security (priv->presets, + provider_id, FALSE /* transport */); /* Note: We need something as default, or modest_account_mgr_add_server_account will fail. */ @@ -2015,7 +2175,7 @@ save_to_settings (ModestEasysetupWizardDialog *self) FALSE /* transport */); if (special_port != 0) transport_port = special_port; - else + else transport_port = 465; transport_auth_protocol = MODEST_PROTOCOLS_AUTH_PASSWORD; } else { @@ -2023,12 +2183,12 @@ save_to_settings (ModestEasysetupWizardDialog *self) transport_auth_protocol = MODEST_PROTOCOLS_AUTH_NONE; } - modest_server_account_settings_set_security_protocol (transport_settings, + modest_server_account_settings_set_security_protocol (transport_settings, transport_security); - modest_server_account_settings_set_auth_protocol (transport_settings, + modest_server_account_settings_set_auth_protocol (transport_settings, transport_auth_protocol); if (transport_port != 0) - modest_server_account_settings_set_port (transport_settings, + modest_server_account_settings_set_port (transport_settings, transport_port); } else { ModestProtocolRegistry *registry; @@ -2038,7 +2198,7 @@ save_to_settings (ModestEasysetupWizardDialog *self) /* Use custom pages because no preset was specified: */ transport_hostname = g_strdup (hildon_entry_get_text (HILDON_ENTRY (priv->entry_outgoingserver) )); - store_proto = modest_protocol_registry_get_protocol_by_type (registry, + store_proto = modest_protocol_registry_get_protocol_by_type (registry, store_protocol); if (transport_protocol == MODEST_PROTOCOL_REGISTRY_TYPE_INVALID) { @@ -2062,18 +2222,20 @@ save_to_settings (ModestEasysetupWizardDialog *self) g_object_unref (transport_settings); g_free (transport_hostname); - + fullname = hildon_entry_get_text (HILDON_ENTRY (priv->entry_user_name)); email_address = hildon_entry_get_text (HILDON_ENTRY (priv->entry_user_email)); modest_account_settings_set_fullname (priv->settings, fullname); modest_account_settings_set_email_address (priv->settings, email_address); /* we don't set retrieve type to preserve advanced settings if any. By default account settings are set to headers only */ - + /* Save the connection-specific SMTP server accounts. */ - modest_account_settings_set_use_connection_specific_smtp - (priv->settings, - hildon_check_button_get_active(HILDON_CHECK_BUTTON(priv->checkbox_outgoing_smtp_specific))); + if (priv->checkbox_outgoing_smtp_specific) { + modest_account_settings_set_use_connection_specific_smtp + (priv->settings, + hildon_check_button_get_active(HILDON_CHECK_BUTTON(priv->checkbox_outgoing_smtp_specific))); + } display_name = get_entered_account_title (self); modest_account_settings_set_display_name (priv->settings, display_name); @@ -2081,6 +2243,33 @@ save_to_settings (ModestEasysetupWizardDialog *self) g_free (provider_id); } + +static void +on_update_model (ModestWizardDialog *dialog) +{ + ModestEasysetupWizardDialog *self = MODEST_EASYSETUP_WIZARD_DIALOG (dialog); + save_to_settings (self); + +} + +static gboolean +on_save (ModestWizardDialog *dialog) +{ + ModestEasysetupWizardDialog *self = MODEST_EASYSETUP_WIZARD_DIALOG (dialog); + ModestEasysetupWizardDialogPrivate *priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (self); + + save_to_settings (self); + + if (modest_account_mgr_check_already_configured_account (priv->account_manager, priv->settings)) { + modest_platform_information_banner (NULL, NULL, _("mail_ib_setting_failed")); + return FALSE; + } + + return modest_account_mgr_add_account_from_settings (priv->account_manager, priv->settings); + +} + + static GList* check_for_supported_auth_methods (ModestEasysetupWizardDialog* self) { @@ -2093,7 +2282,7 @@ check_for_supported_auth_methods (ModestEasysetupWizardDialog* self) int port_num; GList *list_auth_methods; ModestEasysetupWizardDialogPrivate *priv; - + priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (self); registry = modest_runtime_get_protocol_registry (); protocol_type = modest_servertype_picker_get_active_servertype ( @@ -2137,7 +2326,7 @@ check_for_supported_auth_methods (ModestEasysetupWizardDialog* self) return NULL; } -static gboolean +static gboolean check_has_supported_auth_methods(ModestEasysetupWizardDialog* self) { GList* methods = check_for_supported_auth_methods(self); @@ -2150,3 +2339,119 @@ check_has_supported_auth_methods(ModestEasysetupWizardDialog* self) return TRUE; } +static gboolean +check_support_progress_pulse (gpointer userdata) +{ + ModestEasysetupWizardDialog *self = (ModestEasysetupWizardDialog *) userdata; + ModestEasysetupWizardDialogPrivate *priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (self); + + if (priv->destroyed) { + priv->check_support_progress_pulse_id = 0; + return FALSE; + } + + gtk_progress_bar_pulse (GTK_PROGRESS_BAR (priv->check_support_progress)); + + return TRUE; +} + +static gboolean +check_support_show_progress (gpointer userdata) +{ + ModestEasysetupWizardDialog *self = (ModestEasysetupWizardDialog *) userdata; + ModestEasysetupWizardDialogPrivate *priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (self); + + priv->check_support_show_progress_id = 0; + + if (priv->destroyed) + return FALSE; + + gtk_widget_show (priv->check_support_progress); + gtk_progress_bar_pulse (GTK_PROGRESS_BAR (priv->check_support_progress)); + + priv->check_support_progress_pulse_id = g_timeout_add (200, check_support_progress_pulse, self); + + return FALSE; +} + +static void +check_support_callback (ModestAccountProtocol *protocol, + gboolean supported, + gpointer userdata) +{ + ModestEasysetupWizardDialog *self = (ModestEasysetupWizardDialog *) userdata; + ModestEasysetupWizardDialogPrivate *priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (self); + + priv->pending_check_support --; + + if (priv->check_support_show_progress_id > 0) { + g_source_remove (priv->check_support_show_progress_id); + priv->check_support_show_progress_id = 0; + } + + if (priv->check_support_progress_pulse_id > 0) { + g_source_remove (priv->check_support_progress_pulse_id); + priv->check_support_progress_pulse_id = 0; + } + + if (priv->pending_check_support == 0) { + priv->check_support_done = TRUE; + + if (!priv->destroyed) { + if (priv->presets) + fill_providers (self); + gtk_widget_hide (priv->check_support_progress); + invoke_enable_buttons_vfunc (self); + } + } + g_object_unref (self); +} + + +static void +check_support_of_protocols (ModestEasysetupWizardDialog *self) +{ + ModestProtocolRegistry *registry; + GSList *provider_protos, *node; + ModestEasysetupWizardDialogPrivate *priv = MODEST_EASYSETUP_WIZARD_DIALOG_GET_PRIVATE (self); + + registry = modest_runtime_get_protocol_registry (); + provider_protos = modest_protocol_registry_get_by_tag (registry, + MODEST_PROTOCOL_REGISTRY_PROVIDER_PROTOCOLS); + + for (node = provider_protos; node != NULL; node = g_slist_next (node)) { + ModestProtocol *proto = MODEST_PROTOCOL (node->data); + + if (!modest_protocol_registry_protocol_type_has_tag (registry, + modest_protocol_get_type_id (proto), + MODEST_PROTOCOL_REGISTRY_STORE_PROTOCOLS)) + continue; + + if (modest_protocol_registry_protocol_type_has_tag + (registry, + modest_protocol_get_type_id (proto), + MODEST_PROTOCOL_REGISTRY_SINGLETON_PROVIDER_PROTOCOLS)) { + /* Check if there's already an account configured with this account type */ + if (modest_account_mgr_singleton_protocol_exists (modest_runtime_get_account_mgr (), + modest_protocol_get_type_id (proto))) + continue; + } + + if (MODEST_ACCOUNT_PROTOCOL (proto)) { + priv->pending_check_support ++; + modest_account_protocol_check_support (MODEST_ACCOUNT_PROTOCOL (proto), + check_support_callback, + g_object_ref (self)); + } + } + g_slist_free (provider_protos); + + if (priv->pending_check_support > 0) { + priv->check_support_show_progress_id = g_timeout_add_full (G_PRIORITY_DEFAULT, 1000, + check_support_show_progress, + g_object_ref (self), g_object_unref); + } else { + priv->check_support_done = TRUE; + } + invoke_enable_buttons_vfunc (self); +}