BUild fixes
[modest] / src / hildon2 / modest-connection-specific-smtp-edit-window.c
index 70e7ac1..2f5a6ec 100644 (file)
 #include "modest-hildon-includes.h"
 #include "modest-runtime.h"
 
-#include "modest-serversecurity-picker.h"
-#include "modest-secureauth-picker.h"
 #include "widgets/modest-validating-entry.h"
-#include <hildon/hildon-pannable-area.h>
+#include <modest-scrollable.h>
+#include <hildon/hildon-entry.h>
+#include <modest-ui-constants.h>
 #include <gtk/gtkbutton.h>
 #include <gtk/gtkhbox.h>
 #include <gtk/gtkvbox.h>
 #include <gtk/gtkstock.h>
 #include "modest-text-utils.h"
 #include "modest-maemo-utils.h"
+#include <modest-toolkit-factory.h>
 
 #include <glib/gi18n.h>
 
@@ -62,10 +63,10 @@ typedef struct _ModestConnectionSpecificSmtpEditWindowPrivate ModestConnectionSp
 struct _ModestConnectionSpecificSmtpEditWindowPrivate
 {
        GtkWidget *entry_outgoingserver;
-       GtkWidget *outgoing_auth_picker;
+       GtkWidget *outgoing_auth_selector;
        GtkWidget *entry_user_username;
        GtkWidget *entry_user_password;
-       GtkWidget *outgoing_security_picker;
+       GtkWidget *outgoing_security_selector;
        GtkWidget *entry_port;
        
        GtkWidget *button_ok;
@@ -148,6 +149,35 @@ on_change(GtkWidget* widget, ModestConnectionSpecificSmtpEditWindow *self)
 }
 
 static void
+on_mandatory_entry_changed (GtkWidget* widget, ModestConnectionSpecificSmtpEditWindow *self)
+{
+       const gchar *text;
+       gboolean sensitive = TRUE;
+       ModestConnectionSpecificSmtpEditWindowPrivate *priv;
+       ModestProtocolType auth_proto;
+
+       priv = CONNECTION_SPECIFIC_SMTP_EDIT_WINDOW_GET_PRIVATE (self);
+
+       /* Check all mandatory entries */
+       on_change (widget, self);
+
+       auth_proto = modest_secureauth_selector_get_active_secureauth (priv->outgoing_auth_selector);
+       if (modest_protocol_registry_protocol_type_is_secure (modest_runtime_get_protocol_registry (),
+                                                             auth_proto)) {
+               text = modest_entry_get_text (priv->entry_user_username);
+               if (!text || (strlen(text) == 0))
+                       sensitive = FALSE;
+       }
+
+       text = modest_entry_get_text (priv->entry_outgoingserver);
+       if (!text || (strlen(text) == 0))
+               sensitive = FALSE;
+
+       gtk_dialog_set_response_sensitive (GTK_DIALOG (self), GTK_RESPONSE_OK, sensitive);
+}
+
+
+static void
 on_value_changed(GtkWidget* widget, GValue* value, ModestConnectionSpecificSmtpEditWindow *self)
 {
        ModestConnectionSpecificSmtpEditWindowPrivate *priv =
@@ -158,7 +188,7 @@ on_value_changed(GtkWidget* widget, GValue* value, ModestConnectionSpecificSmtpE
 }
 
 static gboolean
-on_range_error (GtkWidget *widget, HildonNumberEditorErrorType type, gpointer user_data)
+on_range_error (GtkWidget *widget, ModestNumberEditorErrorType type, gpointer user_data)
 {
        gchar *msg;
        ModestConnectionSpecificSmtpEditWindow *self = user_data;
@@ -170,10 +200,10 @@ on_range_error (GtkWidget *widget, HildonNumberEditorErrorType type, gpointer us
         * remember that such an error occured. */
        priv->range_error_occured = TRUE;
 
-       if (type == HILDON_NUMBER_EDITOR_ERROR_MAXIMUM_VALUE_EXCEED) {
-               msg = g_strdup (_HL("ckct_ib_maximum_value"));
-       } else if (type == HILDON_NUMBER_EDITOR_ERROR_MINIMUM_VALUE_EXCEED) {
-               msg = g_strdup (_HL("ckct_ib_minimum_value"));
+       if (type == MODEST_NUMBER_EDITOR_ERROR_MAXIMUM_VALUE_EXCEED) {
+               msg = g_strdup_printf (dgettext ("hildon-libs", "ckct_ib_maximum_value"), 65535);
+       } else if (type == MODEST_NUMBER_EDITOR_ERROR_MINIMUM_VALUE_EXCEED) {
+               msg = g_strdup_printf (dgettext ("hildon-libs", "ckct_ib_minimum_value"), 1);
        } else {
                msg = g_strdup_printf (_HL("ckct_ib_set_a_value_within_range"), PORT_RANGE_MIN, PORT_RANGE_MAX);
        }
@@ -184,6 +214,21 @@ on_range_error (GtkWidget *widget, HildonNumberEditorErrorType type, gpointer us
        return TRUE;
 }
 
+static gboolean
+on_delete_event (GtkWidget *widget,
+                 GdkEvent  *event,
+                 gpointer   user_data)
+{
+       ModestConnectionSpecificSmtpEditWindow *self = user_data;
+       ModestConnectionSpecificSmtpEditWindowPrivate *priv =
+                       CONNECTION_SPECIFIC_SMTP_EDIT_WINDOW_GET_PRIVATE (self);
+
+       /* If it's dirty, them the response method already asked the
+          user, because it's always executed before (see GtkDialog
+          code). If it's not then simply close */
+       return priv->is_dirty;
+}
+
 static void
 on_response (GtkDialog *dialog, int response_id, gpointer user_data)
 {
@@ -192,7 +237,7 @@ on_response (GtkDialog *dialog, int response_id, gpointer user_data)
        ModestConnectionSpecificSmtpEditWindowPrivate *priv =
                        CONNECTION_SPECIFIC_SMTP_EDIT_WINDOW_GET_PRIVATE (self);
 
-       hostname = gtk_entry_get_text (GTK_ENTRY (priv->entry_outgoingserver));
+       hostname = modest_entry_get_text (priv->entry_outgoingserver);
 
        /* Don't close the dialog if a range error occured */
        if(response_id == GTK_RESPONSE_OK && priv->range_error_occured)
@@ -204,11 +249,11 @@ on_response (GtkDialog *dialog, int response_id, gpointer user_data)
        }
 
        /* Don't close the dialog if a range error occured */
-       if(response_id == GTK_RESPONSE_OK) {
+       if (response_id == GTK_RESPONSE_OK) {
                if (hostname && (hostname[0] != '\0') &&
                    (!modest_text_utils_validate_domain_name (hostname))) { 
                        g_signal_stop_emission_by_name (dialog, "response");
-                       hildon_banner_show_information (NULL, NULL, _("mcen_ib_invalid_servername"));
+                       modest_platform_system_banner (NULL, NULL, _("mcen_ib_invalid_servername"));
                        gtk_widget_grab_focus (priv->entry_outgoingserver);
                        gtk_editable_select_region (GTK_EDITABLE (priv->entry_outgoingserver), 0, -1);
                        return;
@@ -219,13 +264,15 @@ on_response (GtkDialog *dialog, int response_id, gpointer user_data)
                        gint response;
                        response = modest_platform_run_confirmation_dialog (GTK_WINDOW (user_data), 
                                                                            _("imum_nc_wizard_confirm_lose_changes"));
+
                        if (response != GTK_RESPONSE_OK)
                                g_signal_stop_emission_by_name (dialog, "response");
                }
        }
 }
 
-static void on_set_focus_child (GtkContainer *container, GtkWidget *widget, gpointer user_data)
+static void 
+on_set_focus_child (GtkContainer *container, GtkWidget *widget, gpointer user_data)
 {
        ModestConnectionSpecificSmtpEditWindow *self = user_data;
        ModestConnectionSpecificSmtpEditWindowPrivate *priv =
@@ -238,23 +285,65 @@ static void on_set_focus_child (GtkContainer *container, GtkWidget *widget, gpoi
 }
 
 static void
-on_security_picker_changed (HildonPickerButton *widget, gpointer user_data)
+security_selector_set_port (ModestConnectionSpecificSmtpEditWindowPrivate *priv)
+{
+       const gint port_number = 
+               modest_serversecurity_selector_get_active_serversecurity_port (priv->outgoing_security_selector);
+
+       if(port_number != 0) {
+               modest_number_entry_set_value (
+                       priv->entry_port, port_number);
+       }
+}
+
+static void
+on_security_selector_changed (GtkWidget *widget, gpointer user_data)
 {
        ModestConnectionSpecificSmtpEditWindow *self = 
                MODEST_CONNECTION_SPECIFIC_SMTP_EDIT_WINDOW (user_data);
        ModestConnectionSpecificSmtpEditWindowPrivate *priv = 
                CONNECTION_SPECIFIC_SMTP_EDIT_WINDOW_GET_PRIVATE (self);
-       
+
        on_change(GTK_WIDGET(widget), self);
-       
-       const gint port_number = 
-               modest_serversecurity_picker_get_active_serversecurity_port (
-                       MODEST_SERVERSECURITY_PICKER (priv->outgoing_security_picker));
 
-       if(port_number != 0) {
-               hildon_number_editor_set_value (
-                       HILDON_NUMBER_EDITOR (priv->entry_port), port_number);
-       }               
+       /* Enable/disable username and password fields */
+       security_selector_set_port (priv);
+}
+
+static void
+auth_selector_set_sensitive (ModestConnectionSpecificSmtpEditWindowPrivate *priv)
+{
+       ModestProtocolType auth_proto;
+
+       auth_proto =
+               modest_secureauth_selector_get_active_secureauth (priv->outgoing_auth_selector);
+
+       if (auth_proto == modest_protocol_registry_get_none_auth_type_id ()) {
+               gtk_widget_set_sensitive (priv->entry_user_username, FALSE);
+               gtk_widget_set_sensitive (priv->entry_user_password, FALSE);
+       } else {
+               gtk_widget_set_sensitive (priv->entry_user_username, TRUE);
+               gtk_widget_set_sensitive (priv->entry_user_password, TRUE);
+       }
+
+}
+
+static void
+on_auth_selector_changed (GtkWidget *widget, gpointer user_data)
+{
+       ModestConnectionSpecificSmtpEditWindow *self;
+       ModestConnectionSpecificSmtpEditWindowPrivate *priv;
+
+       self = MODEST_CONNECTION_SPECIFIC_SMTP_EDIT_WINDOW (user_data);
+       priv = CONNECTION_SPECIFIC_SMTP_EDIT_WINDOW_GET_PRIVATE (self);
+
+       on_change (GTK_WIDGET(widget), self);
+
+       /* Enable/disable username and password fields */
+       auth_selector_set_sensitive (priv);
+
+       /* Check missing mandatory data */
+       on_mandatory_entry_changed (priv->entry_user_username, self);
 }
 
 static void
@@ -262,7 +351,8 @@ modest_connection_specific_smtp_edit_window_init (ModestConnectionSpecificSmtpEd
 {
        ModestConnectionSpecificSmtpEditWindowPrivate *priv; 
        GtkWidget *dialog_box;
-       GtkWidget *pannable, *vbox;
+       GtkWidget *scrollable, *vbox, *align;
+       gchar *server_label;
 
        /* The title of this dialog is quite long, so make the window wide enough */
        gtk_widget_set_size_request (GTK_WIDGET (self), 600, 320);
@@ -270,9 +360,10 @@ modest_connection_specific_smtp_edit_window_init (ModestConnectionSpecificSmtpEd
        priv = CONNECTION_SPECIFIC_SMTP_EDIT_WINDOW_GET_PRIVATE (self);
        dialog_box = GTK_DIALOG(self)->vbox; /* gtk_vbox_new (FALSE, MODEST_MARGIN_HALF); */
        gtk_box_set_spacing (GTK_BOX (dialog_box), MODEST_MARGIN_NONE);
-       gtk_container_set_border_width (GTK_CONTAINER (dialog_box), MODEST_MARGIN_HALF);
 
        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);
        
        /* 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.
@@ -282,42 +373,52 @@ modest_connection_specific_smtp_edit_window_init (ModestConnectionSpecificSmtpEd
         
        /* The outgoing server widgets: */
        if (!priv->entry_outgoingserver)
-               priv->entry_outgoingserver = gtk_entry_new ();
+               priv->entry_outgoingserver = modest_toolkit_factory_create_entry (modest_runtime_get_toolkit_factory ());
        /* Auto-capitalization is the default, so let's turn it off: */
        hildon_gtk_entry_set_input_mode (GTK_ENTRY (priv->entry_outgoingserver), HILDON_GTK_INPUT_MODE_FULL);
-       g_signal_connect(G_OBJECT(priv->entry_outgoingserver), "changed", G_CALLBACK(on_change), self);
+       g_signal_connect(G_OBJECT(priv->entry_outgoingserver), "changed", G_CALLBACK(on_mandatory_entry_changed), self);
+
+       server_label = g_strconcat (_("mcen_li_emailsetup_smtp"), "\n<small>(SMTP)</small>", NULL);
        
        GtkWidget *captioned = 
          modest_maemo_utils_create_captioned (title_sizegroup, value_sizegroup,
-                                              _("mcen_li_emailsetup_smtp"), 
+                                              server_label, TRUE,
                                               priv->entry_outgoingserver);
+       g_free (server_label);
        gtk_widget_show (priv->entry_outgoingserver);
-       gtk_box_pack_start (GTK_BOX (vbox), captioned, FALSE, FALSE, MODEST_MARGIN_HALF);
+       gtk_box_pack_start (GTK_BOX (vbox), captioned, FALSE, FALSE, 0);
        gtk_widget_show (captioned);
        
        /* The secure authentication widgets: */
-       if (!priv->outgoing_auth_picker) {
-               priv->outgoing_auth_picker = 
-                       GTK_WIDGET (modest_secureauth_picker_new (MODEST_EDITABLE_SIZE,
-                                                                 HILDON_BUTTON_ARRANGEMENT_HORIZONTAL));
+       if (!priv->outgoing_auth_selector) {
+               priv->outgoing_auth_selector = 
+                       modest_toolkit_factory_create_secureauth_selector (modest_runtime_get_toolkit_factory ());
+       }
+       if (GTK_IS_COMBO_BOX (priv->outgoing_auth_selector)) {
+               GtkWidget *captioned;
+               captioned = modest_maemo_utils_create_captioned (title_sizegroup, value_sizegroup,
+                                                                _("mcen_li_emailsetup_secure_authentication"), FALSE,
+                                                                priv->outgoing_auth_selector);
+               gtk_widget_show (captioned);
+               gtk_box_pack_start (GTK_BOX (vbox), captioned, FALSE, FALSE, 0);
+       } else {
+               modest_maemo_utils_set_hbutton_layout (title_sizegroup, value_sizegroup,
+                                                      _("mcen_li_emailsetup_secure_authentication"),
+                                                      priv->outgoing_auth_selector);
+               gtk_box_pack_start (GTK_BOX (vbox), priv->outgoing_auth_selector, FALSE, FALSE, 0);
        }
-       modest_maemo_utils_set_hbutton_layout (title_sizegroup, value_sizegroup,
-                                              _("mcen_li_emailsetup_secure_authentication"),
-                                              priv->outgoing_auth_picker);
-       g_signal_connect (G_OBJECT (priv->outgoing_auth_picker), "value-changed", G_CALLBACK(on_change), self);
-       gtk_widget_show (priv->outgoing_auth_picker);
-       gtk_box_pack_start (GTK_BOX (vbox), priv->outgoing_auth_picker, FALSE, FALSE, MODEST_MARGIN_HALF);
+       gtk_widget_show (priv->outgoing_auth_selector);
        
        /* 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_INPUT_MODE_FULL);
        captioned = modest_maemo_utils_create_captioned (title_sizegroup, value_sizegroup,
-                                                        _("mail_fi_username"), 
+                                                        _("mail_fi_username"), FALSE,
                                                         priv->entry_user_username);
-       g_signal_connect(G_OBJECT(priv->entry_user_username), "changed", G_CALLBACK(on_change), self);
+       g_signal_connect(G_OBJECT(priv->entry_user_username), "changed", G_CALLBACK(on_mandatory_entry_changed), self);
        gtk_widget_show (priv->entry_user_username);
-       gtk_box_pack_start (GTK_BOX (vbox), captioned, FALSE, FALSE, MODEST_MARGIN_HALF);
+       gtk_box_pack_start (GTK_BOX (vbox), captioned, FALSE, FALSE, 0);
        gtk_widget_show (captioned);
        
        /* Prevent the use of some characters in the username, 
@@ -330,79 +431,116 @@ modest_connection_specific_smtp_edit_window_init (ModestConnectionSpecificSmtpEd
        gtk_entry_set_max_length (GTK_ENTRY (priv->entry_user_username), 64);
        
        /* The password widgets: */     
-       priv->entry_user_password = gtk_entry_new ();
+       priv->entry_user_password = modest_toolkit_factory_create_entry (modest_runtime_get_toolkit_factory ());
        /* 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_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), '*'); */
        captioned = modest_maemo_utils_create_captioned (title_sizegroup, value_sizegroup,
-               _("mail_fi_password"), priv->entry_user_password);
+                                                        _("mail_fi_password"), FALSE, priv->entry_user_password);
        g_signal_connect(G_OBJECT(priv->entry_user_password), "changed", G_CALLBACK(on_change), self);
        gtk_widget_show (priv->entry_user_password);
-       gtk_box_pack_start (GTK_BOX (vbox), captioned, FALSE, FALSE, MODEST_MARGIN_HALF);
+       gtk_box_pack_start (GTK_BOX (vbox), captioned, FALSE, FALSE, 0);
        gtk_widget_show (captioned);
        
        /* The secure connection widgets: */    
-       if (!priv->outgoing_security_picker)
-               priv->outgoing_security_picker = 
-                       GTK_WIDGET (modest_serversecurity_picker_new (MODEST_EDITABLE_SIZE,
-                                                                     HILDON_BUTTON_ARRANGEMENT_HORIZONTAL));
-       modest_serversecurity_picker_fill (
-               MODEST_SERVERSECURITY_PICKER (priv->outgoing_security_picker), MODEST_PROTOCOLS_TRANSPORT_SMTP);
-       modest_serversecurity_picker_set_active_serversecurity (
-               MODEST_SERVERSECURITY_PICKER (priv->outgoing_security_picker), MODEST_PROTOCOLS_CONNECTION_NONE);
-       modest_maemo_utils_set_hbutton_layout (title_sizegroup, value_sizegroup,
-                                              _("mcen_li_emailsetup_secure_connection"), 
-                                              priv->outgoing_security_picker);
-       gtk_widget_show (priv->outgoing_security_picker);
-       gtk_box_pack_start (GTK_BOX (vbox), priv->outgoing_security_picker, FALSE, FALSE, MODEST_MARGIN_HALF);
+       if (!priv->outgoing_security_selector)
+               priv->outgoing_security_selector = 
+                       modest_toolkit_factory_create_serversecurity_selector (modest_runtime_get_toolkit_factory ());
+       modest_serversecurity_selector_fill (priv->outgoing_security_selector, MODEST_PROTOCOLS_TRANSPORT_SMTP);
+       modest_serversecurity_selector_set_active_serversecurity (priv->outgoing_security_selector, MODEST_PROTOCOLS_CONNECTION_NONE);
+       if (GTK_IS_COMBO_BOX (priv->outgoing_security_selector)) {
+               GtkWidget *captioned;
+
+               captioned = modest_maemo_utils_create_captioned (title_sizegroup, value_sizegroup,
+                                                                _("mcen_li_emailsetup_secure_connection"), FALSE,
+                                                                priv->outgoing_security_selector);
+               gtk_box_pack_start (GTK_BOX (vbox), captioned, FALSE, FALSE, MODEST_MARGIN_HALF);
+               gtk_widget_show (captioned);
+       } else {
+               modest_maemo_utils_set_hbutton_layout (title_sizegroup, value_sizegroup,
+                                                      _("mcen_li_emailsetup_secure_connection"), 
+                                                      priv->outgoing_security_selector);
+               gtk_box_pack_start (GTK_BOX (vbox), priv->outgoing_security_selector, FALSE, FALSE, 0);
+       }
+       gtk_widget_show (priv->outgoing_security_selector);
        
        /* The port number widgets: */
        if (!priv->entry_port)
-               priv->entry_port = GTK_WIDGET (hildon_number_editor_new (PORT_RANGE_MIN, PORT_RANGE_MAX));
+               priv->entry_port = modest_toolkit_factory_create_number_entry (modest_runtime_get_toolkit_factory (),
+                                                                              PORT_RANGE_MIN, PORT_RANGE_MAX);
        captioned = modest_maemo_utils_create_captioned (title_sizegroup, value_sizegroup,
-                                                        _("mcen_fi_emailsetup_port"), priv->entry_port);
+                                                        _("mcen_fi_emailsetup_port"), FALSE, priv->entry_port);
        gtk_widget_add_events(GTK_WIDGET(priv->entry_port), GDK_FOCUS_CHANGE_MASK);
-       g_signal_connect(G_OBJECT(priv->entry_port), "range-error", G_CALLBACK(on_range_error), self);
-       g_signal_connect(G_OBJECT(priv->entry_port), "notify::value", G_CALLBACK(on_value_changed), self);
        gtk_widget_show (priv->entry_port);
-       gtk_box_pack_start (GTK_BOX (vbox), captioned, FALSE, FALSE, MODEST_MARGIN_HALF);
+       gtk_box_pack_start (GTK_BOX (vbox), captioned, FALSE, FALSE, 0);
        gtk_widget_show (captioned);
-       
-       /* Show a default port number when the security method changes, as per the UI spec: */
-       g_signal_connect (G_OBJECT (priv->outgoing_security_picker), "value-changed", (GCallback)on_security_picker_changed, self);
-       on_security_picker_changed (HILDON_PICKER_BUTTON (priv->outgoing_security_picker), self);
-       
-       /* Add the buttons: */
+
+       /* Add the button. Disabled by default */
        gtk_dialog_add_button (GTK_DIALOG (self), _HL("wdgt_bd_save"), GTK_RESPONSE_OK);
-       
+       gtk_dialog_set_response_sensitive (GTK_DIALOG (self), GTK_RESPONSE_OK, FALSE);
+
        priv->is_dirty = FALSE;
        priv->range_error_occured = FALSE;
        g_signal_connect(G_OBJECT(self), "response", G_CALLBACK(on_response), self);
+       g_signal_connect(G_OBJECT(self), "delete-event", G_CALLBACK(on_delete_event), self);
        g_signal_connect(G_OBJECT(vbox), "set-focus-child", G_CALLBACK(on_set_focus_child), self);
 
        priv->account_name = NULL;
 
-       pannable = hildon_pannable_area_new ();
-       g_object_set (G_OBJECT (pannable), "initial-hint", TRUE, NULL);
-       hildon_pannable_area_add_with_viewport (HILDON_PANNABLE_AREA (pannable), vbox);
-       gtk_box_pack_start (GTK_BOX (dialog_box), pannable, TRUE, TRUE, 0);
-       
+       scrollable = modest_toolkit_factory_create_scrollable (modest_runtime_get_toolkit_factory ());
+       gtk_container_add (GTK_CONTAINER (align), vbox);
+       modest_scrollable_add_with_viewport (MODEST_SCROLLABLE (scrollable), align);
+       gtk_box_pack_start (GTK_BOX (dialog_box), scrollable, TRUE, TRUE, 0);
+
        gtk_widget_show_all (dialog_box);
-       gtk_window_set_default_size (GTK_WINDOW (self), -1, 220);
+       gtk_window_set_default_size (GTK_WINDOW (self), -1, MODEST_DIALOG_WINDOW_MAX_HEIGHT);
 
        g_object_unref (title_sizegroup);
        g_object_unref (value_sizegroup);
-       
+
        /* 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)); 
-       
-       hildon_help_dialog_help_enable (GTK_DIALOG(self),
-                                       "applications_email_connectionspecificsmtpconf",
-                                       modest_maemo_utils_get_osso_context());
+
+       /* Refresh view with current settings */
+       auth_selector_set_sensitive (priv);
+       security_selector_set_port (priv);
+
+       /* Connect signals to track changes */
+       if (GTK_IS_COMBO_BOX (priv->outgoing_security_selector)) {
+               g_signal_connect (G_OBJECT (priv->outgoing_security_selector),
+                                 "changed",
+                                 (GCallback) on_security_selector_changed,
+                                 self);
+       } else {
+               g_signal_connect (G_OBJECT (priv->outgoing_security_selector),
+                                 "value-changed",
+                                 (GCallback) on_security_selector_changed,
+                                 self);
+       }
+       if (GTK_IS_COMBO_BOX (priv->outgoing_auth_selector)) {
+               g_signal_connect (G_OBJECT (priv->outgoing_auth_selector),
+                                 "changed",
+                                 (GCallback) on_auth_selector_changed,
+                                 self);
+       } else {
+               g_signal_connect (G_OBJECT (priv->outgoing_auth_selector),
+                                 "value-changed",
+                                 (GCallback) on_auth_selector_changed,
+                                 self);
+       }
+       g_signal_connect(G_OBJECT(priv->entry_port),
+                        "range-error",
+                        G_CALLBACK(on_range_error),
+                        self);
+       g_signal_connect(G_OBJECT(priv->entry_port),
+                        "notify::value",
+                        G_CALLBACK(on_value_changed),
+                        self);
+
 }
 
 ModestConnectionSpecificSmtpEditWindow*
@@ -430,24 +568,23 @@ modest_connection_specific_smtp_edit_window_set_connection (
                if (priv->account_name)
                        g_free (priv->account_name);
                priv->account_name = g_strdup (modest_server_account_settings_get_account_name (server_settings));
-               gtk_entry_set_text (GTK_ENTRY (priv->entry_outgoingserver), 
-                                   modest_server_account_settings_get_hostname (server_settings));
-               gtk_entry_set_text (GTK_ENTRY (priv->entry_user_username),
-                                   modest_server_account_settings_get_username (server_settings));     
-               gtk_entry_set_text (GTK_ENTRY (priv->entry_user_password), 
-                                   modest_server_account_settings_get_password (server_settings));
+               modest_entry_set_text (priv->entry_outgoingserver,
+                                      modest_server_account_settings_get_hostname (server_settings));
+               modest_entry_set_text (priv->entry_user_username,
+                                      modest_server_account_settings_get_username (server_settings));  
+               modest_entry_set_text (priv->entry_user_password,
+                                      modest_server_account_settings_get_password (server_settings));
        
-               modest_serversecurity_picker_set_active_serversecurity (
-               MODEST_SERVERSECURITY_PICKER (priv->outgoing_security_picker), 
-               modest_server_account_settings_get_security_protocol (server_settings));
+               modest_serversecurity_selector_set_active_serversecurity 
+                       (priv->outgoing_security_selector, 
+                        modest_server_account_settings_get_security_protocol (server_settings));
        
-               modest_secureauth_picker_set_active_secureauth (
-               MODEST_SECUREAUTH_PICKER (priv->outgoing_auth_picker), 
-               modest_server_account_settings_get_auth_protocol (server_settings));
+               modest_secureauth_selector_set_active_secureauth (priv->outgoing_auth_selector,
+                                                                 modest_server_account_settings_get_auth_protocol (server_settings));
                
                /* port: */
-               hildon_number_editor_set_value (
-                       HILDON_NUMBER_EDITOR (priv->entry_port), 
+               modest_number_entry_set_value (
+                       priv->entry_port, 
                        modest_server_account_settings_get_port (server_settings));
                
                
@@ -464,7 +601,7 @@ modest_connection_specific_smtp_edit_window_get_settings (ModestConnectionSpecif
        const gchar *outgoing_server = NULL;
 
        priv =  CONNECTION_SPECIFIC_SMTP_EDIT_WINDOW_GET_PRIVATE (window);
-       outgoing_server = gtk_entry_get_text (GTK_ENTRY (priv->entry_outgoingserver));
+       outgoing_server = modest_entry_get_text (priv->entry_outgoingserver);
 
        /* If the outgoing server is NULL, we are removing the connection specific
         * settings */
@@ -475,26 +612,26 @@ modest_connection_specific_smtp_edit_window_get_settings (ModestConnectionSpecif
        server_settings = modest_server_account_settings_new ();
        
        modest_server_account_settings_set_hostname (server_settings, 
-                                                    gtk_entry_get_text (GTK_ENTRY (priv->entry_outgoingserver)));
+                                                    modest_entry_get_text (priv->entry_outgoingserver));
        modest_server_account_settings_set_protocol (server_settings,
                                                     MODEST_PROTOCOLS_TRANSPORT_SMTP);
        modest_server_account_settings_set_username (server_settings,
-                                                    gtk_entry_get_text (GTK_ENTRY (priv->entry_user_username)));
+                                                    modest_entry_get_text (priv->entry_user_username));
        modest_server_account_settings_set_password (server_settings,
-                                                    gtk_entry_get_text (GTK_ENTRY (priv->entry_user_password)));
+                                                    modest_entry_get_text (priv->entry_user_password));
        
        modest_server_account_settings_set_security_protocol (server_settings, 
-                                                    modest_serversecurity_picker_get_active_serversecurity (
-                                                    MODEST_SERVERSECURITY_PICKER (priv->outgoing_security_picker)));
-       modest_server_account_settings_set_auth_protocol (server_settings,
-                                                         modest_secureauth_picker_get_active_secureauth (
-                                                         MODEST_SECUREAUTH_PICKER (priv->outgoing_auth_picker)));
+                                                             modest_serversecurity_selector_get_active_serversecurity (
+                                                                     priv->outgoing_security_selector));
+       modest_server_account_settings_set_auth_protocol 
+               (server_settings,
+                modest_secureauth_selector_get_active_secureauth (priv->outgoing_auth_selector));
        modest_server_account_settings_set_account_name (server_settings,
                                                         priv->account_name);
        
        /* port: */
        modest_server_account_settings_set_port (server_settings,
-                                                hildon_number_editor_get_value (HILDON_NUMBER_EDITOR (priv->entry_port)));
+                                                modest_number_entry_get_value (priv->entry_port));
                        
        return server_settings;
 }