From: Alberto Garcia <agarcia@igalia.com>
Date: Tue, 21 Aug 2007 12:00:58 +0000 (+0000)
Subject:    * src/modest-ui-actions.c
X-Git-Tag: git_migration_finished~2560
X-Git-Url: http://git.maemo.org/git/?p=modest;a=commitdiff_plain;h=5619278be034c3032ce89fc8ce0a2d52ef351e7b;hp=0b80adcdaddd0d45f9503018a83c3cf661904793

   * src/modest-ui-actions.c
   (modest_ui_actions_on_send):
   Don't ask confirmation if the subject is empty as it can't be empty
   anymore

* src/maemo/modest-msg-edit-window.c
   (update_send_dimming):
   New function to update the dimming of the send button and menu
   entry, based on the old recpt_field_changed()
   (recpt_field_changed):
   (subject_field_changed):
   Call update_send_dimming()
   (send_insensitive_press):
   Check whether the subject is empty or not in order to show the proper
   notification

   Fixes: NB#66112

pmo-trunk-r3031
---

diff --git a/src/maemo/modest-msg-edit-window.c b/src/maemo/modest-msg-edit-window.c
index 393d892..bab1cb5 100644
--- a/src/maemo/modest-msg-edit-window.c
+++ b/src/maemo/modest-msg-edit-window.c
@@ -141,6 +141,7 @@ static void update_paste_dimming (ModestMsgEditWindow *window);
 static void update_copy_cut_dimming (ModestMsgEditWindow *window);
 static void update_select_all_dimming (ModestMsgEditWindow *window);
 static void update_zoom_dimming (ModestMsgEditWindow *window);
+static void update_send_dimming (ModestMsgEditWindow *window);
 
 /* Find toolbar */
 static void modest_msg_edit_window_find_toolbar_search (GtkWidget *widget,
@@ -2643,30 +2644,19 @@ static void
 recpt_field_changed (GtkTextBuffer *buffer,
 		  ModestMsgEditWindow *editor)
 {
-	ModestWindowPrivate *parent_priv = MODEST_WINDOW_GET_PRIVATE (editor);
-	ModestMsgEditWindowPrivate *priv = MODEST_MSG_EDIT_WINDOW_GET_PRIVATE (editor);
-	GtkTextBuffer *to_buffer, *cc_buffer, *bcc_buffer;
-	gboolean dim = FALSE;
-	GtkAction *action;
-
-	to_buffer = modest_recpt_editor_get_buffer (MODEST_RECPT_EDITOR (priv->to_field));
-	cc_buffer = modest_recpt_editor_get_buffer (MODEST_RECPT_EDITOR (priv->cc_field));
-	bcc_buffer = modest_recpt_editor_get_buffer (MODEST_RECPT_EDITOR (priv->bcc_field));
-	
-	dim = ((gtk_text_buffer_get_char_count (to_buffer) + 
-		gtk_text_buffer_get_char_count (cc_buffer) +
-		gtk_text_buffer_get_char_count (bcc_buffer)) == 0);
-			
-	action = gtk_ui_manager_get_action (parent_priv->ui_manager, "/ToolBar/ToolbarSend");
-	gtk_action_set_sensitive (action, !dim);
-	action = gtk_ui_manager_get_action (parent_priv->ui_manager, "/MenuBar/EmailMenu/SendMenu");
-	gtk_action_set_sensitive (action, !dim);
+        update_send_dimming (editor);
 }
 
 static void  
 send_insensitive_press (GtkWidget *widget, ModestMsgEditWindow *editor)
 {
-	hildon_banner_show_information (NULL, NULL, _("mcen_ib_add_recipients_first"));
+	ModestMsgEditWindowPrivate *priv = MODEST_MSG_EDIT_WINDOW_GET_PRIVATE (editor);
+        const gchar *subject = gtk_entry_get_text (GTK_ENTRY (priv->subject_field));
+        if (subject == NULL || subject[0] == '\0') {
+                hildon_banner_show_information (NULL, NULL, _("mcen_ib_subject_or_body_not_modified"));
+        } else {
+                hildon_banner_show_information (NULL, NULL, _("mcen_ib_add_recipients_first"));
+        }
 }
 
 static void
@@ -2815,6 +2805,7 @@ subject_field_changed (GtkEditable *editable,
 	ModestMsgEditWindowPrivate *priv = MODEST_MSG_EDIT_WINDOW_GET_PRIVATE (window);
 	update_window_title (window);
 	gtk_text_buffer_set_modified (priv->text_buffer, TRUE);
+        update_send_dimming (window);
 }
 
 static void  
@@ -3117,6 +3108,33 @@ update_zoom_dimming (ModestMsgEditWindow *window)
 }
 
 static void
+update_send_dimming (ModestMsgEditWindow *window)
+{
+        ModestWindowPrivate *parent_priv = MODEST_WINDOW_GET_PRIVATE (window);
+        ModestMsgEditWindowPrivate *priv = MODEST_MSG_EDIT_WINDOW_GET_PRIVATE (window);
+        GtkTextBuffer *to_buffer, *cc_buffer, *bcc_buffer;
+        const gchar *subject;
+	gboolean dim = FALSE;
+	GtkAction *action;
+
+	to_buffer = modest_recpt_editor_get_buffer (MODEST_RECPT_EDITOR (priv->to_field));
+	cc_buffer = modest_recpt_editor_get_buffer (MODEST_RECPT_EDITOR (priv->cc_field));
+	bcc_buffer = modest_recpt_editor_get_buffer (MODEST_RECPT_EDITOR (priv->bcc_field));
+        subject = gtk_entry_get_text (GTK_ENTRY (priv->subject_field));
+
+	dim = ((gtk_text_buffer_get_char_count (to_buffer) +
+		gtk_text_buffer_get_char_count (cc_buffer) +
+		gtk_text_buffer_get_char_count (bcc_buffer)) == 0);
+
+        dim = dim || (subject == NULL || subject[0] == '\0');
+
+	action = gtk_ui_manager_get_action (parent_priv->ui_manager, "/ToolBar/ToolbarSend");
+	gtk_action_set_sensitive (action, !dim);
+	action = gtk_ui_manager_get_action (parent_priv->ui_manager, "/MenuBar/EmailMenu/SendMenu");
+	gtk_action_set_sensitive (action, !dim);
+}
+
+static void
 edit_menu_activated (GtkAction *action,
 		     gpointer userdata)
 {
diff --git a/src/modest-ui-actions.c b/src/modest-ui-actions.c
index 8ad5deb..d69a796 100644
--- a/src/modest-ui-actions.c
+++ b/src/modest-ui-actions.c
@@ -2090,17 +2090,6 @@ modest_ui_actions_on_send (GtkWidget *widget, ModestMsgEditWindow *edit_window)
 	
 	gchar *from = modest_account_mgr_get_from_string (account_mgr, account_name);
 
-	/* mail content checks and dialogs */
-	if (data->subject == NULL || data->subject[0] == '\0') {
-		GtkResponseType response;
-		response = modest_platform_run_confirmation_dialog (GTK_WINDOW (edit_window),
-								    _("mcen_nc_subject_is_empty_send"));
-		if (response == GTK_RESPONSE_CANCEL) {
-			g_free (account_name);
-			return;
-		}
-	}
-
 	if (data->plain_body == NULL || data->plain_body[0] == '\0') {
 		GtkResponseType response;
 		gchar *note_message;