aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMatthew Barnes <mbarnes@redhat.com>2011-03-25 04:47:53 +0800
committerMatthew Barnes <mbarnes@redhat.com>2011-03-25 04:50:19 +0800
commita09a95559ef640f23da644aef845607774c79f38 (patch)
treea1409cab397c54b6e8581178dfcb9b5cd01379f8
parentd5d651cf7885a2d61cb542a990f00489537224c2 (diff)
downloadgsoc2013-evolution-a09a95559ef640f23da644aef845607774c79f38.tar
gsoc2013-evolution-a09a95559ef640f23da644aef845607774c79f38.tar.gz
gsoc2013-evolution-a09a95559ef640f23da644aef845607774c79f38.tar.bz2
gsoc2013-evolution-a09a95559ef640f23da644aef845607774c79f38.tar.lz
gsoc2013-evolution-a09a95559ef640f23da644aef845607774c79f38.tar.xz
gsoc2013-evolution-a09a95559ef640f23da644aef845607774c79f38.tar.zst
gsoc2013-evolution-a09a95559ef640f23da644aef845607774c79f38.zip
Fix alignment of extra widgets in EAlertDialogs.
Add e_alert_dialog_get_content_area(), which returns the GtkVBox containing the primary and secondary labels. Use this instead of gtk_dialog_get_content_area() to maintain the dialog's left margin beneath the image.
-rw-r--r--calendar/gui/dialogs/send-comp.c2
-rw-r--r--e-util/e-alert-dialog.c23
-rw-r--r--e-util/e-alert-dialog.h1
-rw-r--r--mail/e-mail-reader-utils.c10
-rw-r--r--mail/e-mail-reader.c79
-rw-r--r--mail/em-utils.c18
-rw-r--r--plugins/attachment-reminder/attachment-reminder.c23
-rw-r--r--plugins/attachment-reminder/org-gnome-attachment-reminder.error.xml2
-rw-r--r--plugins/backup-restore/backup-restore.c17
9 files changed, 108 insertions, 67 deletions
diff --git a/calendar/gui/dialogs/send-comp.c b/calendar/gui/dialogs/send-comp.c
index ab80844b32..eab2c2c818 100644
--- a/calendar/gui/dialogs/send-comp.c
+++ b/calendar/gui/dialogs/send-comp.c
@@ -186,7 +186,7 @@ send_component_dialog (GtkWindow *parent, ECal *client, ECalComponent *comp, gbo
}
dialog = e_alert_dialog_new_for_args (parent, id, NULL);
- content_area = gtk_dialog_get_content_area (GTK_DIALOG (dialog));
+ content_area = e_alert_dialog_get_content_area (E_ALERT_DIALOG (dialog));
if (strip_alarms)
sa_checkbox = add_checkbox (GTK_BOX (content_area), _("Send my alarms with this event"));
diff --git a/e-util/e-alert-dialog.c b/e-util/e-alert-dialog.c
index 6d24dcde65..83ef05c68e 100644
--- a/e-util/e-alert-dialog.c
+++ b/e-util/e-alert-dialog.c
@@ -26,7 +26,7 @@
#include "e-util.h"
struct _EAlertDialogPrivate {
- GtkWindow *parent;
+ GtkWidget *content_area; /* not referenced */
EAlert *alert;
};
@@ -184,6 +184,7 @@ alert_dialog_constructed (GObject *object)
widget = gtk_vbox_new (FALSE, 12);
gtk_box_pack_start (GTK_BOX (container), widget, FALSE, FALSE, 0);
+ dialog->priv->content_area = widget;
gtk_widget_show (widget);
container = widget;
@@ -334,7 +335,7 @@ e_alert_run_dialog_for_args (GtkWindow *parent,
/**
* e_alert_dialog_get_alert:
- * @dialog: a #EAlertDialog
+ * @dialog: an #EAlertDialog
*
* Returns the #EAlert associated with @dialog.
*
@@ -347,3 +348,21 @@ e_alert_dialog_get_alert (EAlertDialog *dialog)
return dialog->priv->alert;
}
+
+/**
+ * e_alert_dialog_get_content_area:
+ * @dialog: an #EAlertDialog
+ *
+ * Returns the vertical box containing the primary and secondary labels.
+ * Use this to pack additional widgets into the dialog with the proper
+ * horizontal alignment (maintaining the left margin below the image).
+ *
+ * Returns: the content area #GtkBox
+ **/
+GtkWidget *
+e_alert_dialog_get_content_area (EAlertDialog *dialog)
+{
+ g_return_val_if_fail (E_IS_ALERT_DIALOG (dialog), NULL);
+
+ return dialog->priv->content_area;
+}
diff --git a/e-util/e-alert-dialog.h b/e-util/e-alert-dialog.h
index 5a23d72c4f..a8d1b1d6d5 100644
--- a/e-util/e-alert-dialog.h
+++ b/e-util/e-alert-dialog.h
@@ -73,6 +73,7 @@ gint e_alert_run_dialog_for_args (GtkWindow *parent,
const gchar *tag,
...) G_GNUC_NULL_TERMINATED;
EAlert * e_alert_dialog_get_alert (EAlertDialog *dialog);
+GtkWidget * e_alert_dialog_get_content_area (EAlertDialog *dialog);
G_END_DECLS
diff --git a/mail/e-mail-reader-utils.c b/mail/e-mail-reader-utils.c
index a049e6eb84..c7b651b942 100644
--- a/mail/e-mail-reader-utils.c
+++ b/mail/e-mail-reader-utils.c
@@ -70,7 +70,7 @@ e_mail_reader_confirm_delete (EMailReader *reader)
CamelFolder *folder;
CamelStore *parent_store;
GtkWidget *check_button;
- GtkWidget *content_area;
+ GtkWidget *container;
GtkWidget *dialog;
GtkWindow *window;
const gchar *label;
@@ -104,15 +104,11 @@ e_mail_reader_confirm_delete (EMailReader *reader)
window, "mail:ask-delete-vfolder-msg",
camel_folder_get_full_name (folder), NULL);
- /* XXX e-error should provide a widget layout and API suitable
- * for packing additional widgets to the right of the alert
- * icon. But for now, screw it. */
+ container = e_alert_dialog_get_content_area (E_ALERT_DIALOG (dialog));
label = _("Do not ask me again");
- content_area = gtk_dialog_get_content_area (GTK_DIALOG (dialog));
check_button = gtk_check_button_new_with_label (label);
- gtk_box_pack_start (
- GTK_BOX (content_area), check_button, TRUE, TRUE, 6);
+ gtk_box_pack_start (GTK_BOX (container), check_button, TRUE, TRUE, 6);
gtk_widget_show (check_button);
response = gtk_dialog_run (GTK_DIALOG (dialog));
diff --git a/mail/e-mail-reader.c b/mail/e-mail-reader.c
index b70bebb7b9..79439d3f08 100644
--- a/mail/e-mail-reader.c
+++ b/mail/e-mail-reader.c
@@ -658,7 +658,8 @@ check_close_browser_reader (EMailReader *reader)
if (!parent)
parent = e_mail_reader_get_window (reader);
- dialog = e_alert_dialog_new_for_args (parent, "mail:ask-reply-close-browser", NULL);
+ dialog = e_alert_dialog_new_for_args (
+ parent, "mail:ask-reply-close-browser", NULL);
response = gtk_dialog_run (GTK_DIALOG (dialog));
gtk_widget_destroy (dialog);
@@ -1254,18 +1255,22 @@ action_mail_reply_all_check (CamelFolder *folder,
if (recip_count >= 15) {
GtkWidget *dialog;
- GtkWidget *content_area, *check;
+ GtkWidget *check;
+ GtkWidget *container;
gint response;
dialog = e_alert_dialog_new_for_args (
e_mail_reader_get_window (reader),
"mail:ask-reply-many-recips", NULL);
+ container = e_alert_dialog_get_content_area (
+ E_ALERT_DIALOG (dialog));
+
/* Check buttons */
- check = gtk_check_button_new_with_mnemonic (_("_Do not ask me again."));
- gtk_container_set_border_width (GTK_CONTAINER (check), 12);
- content_area = gtk_dialog_get_content_area (GTK_DIALOG (dialog));
- gtk_box_pack_start (GTK_BOX (content_area), check, FALSE, FALSE, 0);
+ check = gtk_check_button_new_with_mnemonic (
+ _("_Do not ask me again."));
+ gtk_box_pack_start (
+ GTK_BOX (container), check, FALSE, FALSE, 0);
gtk_widget_show (check);
response = gtk_dialog_run (GTK_DIALOG (dialog));
@@ -1282,11 +1287,16 @@ action_mail_reply_all_check (CamelFolder *folder,
gtk_widget_destroy (dialog);
- if (response == GTK_RESPONSE_NO)
- type = E_MAIL_REPLY_TO_SENDER;
- else if (response == GTK_RESPONSE_CANCEL || response == GTK_RESPONSE_DELETE_EVENT) {
- g_object_unref (message);
- return;
+ switch (response) {
+ case GTK_RESPONSE_NO:
+ type = E_MAIL_REPLY_TO_SENDER;
+ break;
+ case GTK_RESPONSE_CANCEL:
+ case GTK_RESPONSE_DELETE_EVENT:
+ g_object_unref (message);
+ return;
+ default:
+ break;
}
}
@@ -1411,20 +1421,22 @@ action_mail_reply_sender_check (CamelFolder *folder,
it's a Reply-To: munged list message... unless we're ignoring munging */
if (ask_ignore_list_reply_to || !munged_list_message) {
GtkWidget *dialog;
- GtkWidget *content_area, *check;
+ GtkWidget *check;
+ GtkWidget *container;
gint response;
dialog = e_alert_dialog_new_for_args (
e_mail_reader_get_window (reader),
"mail:ask-list-private-reply", NULL);
+ container = e_alert_dialog_get_content_area (
+ E_ALERT_DIALOG (dialog));
+
/* Check buttons */
check = gtk_check_button_new_with_mnemonic (
_("_Do not ask me again."));
- gtk_container_set_border_width ((GtkContainer *)check, 12);
- content_area = gtk_dialog_get_content_area (GTK_DIALOG (dialog));
gtk_box_pack_start (
- GTK_BOX (content_area), check, FALSE, FALSE, 0);
+ GTK_BOX (container), check, FALSE, FALSE, 0);
gtk_widget_show (check);
response = gtk_dialog_run (GTK_DIALOG (dialog));
@@ -1446,8 +1458,7 @@ action_mail_reply_sender_check (CamelFolder *folder,
} else if (ask_list_reply_to) {
GtkWidget *dialog;
- GtkWidget *content_area;
- GtkWidget *vbox;
+ GtkWidget *container;
GtkWidget *check_again;
GtkWidget *check_always_ignore;
gint response;
@@ -1456,22 +1467,20 @@ action_mail_reply_sender_check (CamelFolder *folder,
e_mail_reader_get_window (reader),
"mail:ask-list-honour-reply-to", NULL);
- /*Check buttons*/
- vbox = gtk_vbox_new (FALSE, 0);
- content_area = gtk_dialog_get_content_area (GTK_DIALOG (dialog));
- gtk_container_set_border_width (GTK_CONTAINER (vbox), 12);
- gtk_box_pack_start (GTK_BOX (content_area), vbox, FALSE, FALSE, 0);
- gtk_widget_show (vbox);
+ container = e_alert_dialog_get_content_area (
+ E_ALERT_DIALOG (dialog));
check_again = gtk_check_button_new_with_mnemonic (
_("_Do not ask me again."));
- gtk_box_pack_start (GTK_BOX (vbox), check_again, FALSE, FALSE, 0);
+ gtk_box_pack_start (
+ GTK_BOX (container), check_again, FALSE, FALSE, 0);
gtk_widget_show (check_again);
check_always_ignore = gtk_check_button_new_with_mnemonic (
_("_Always ignore Reply-To: for mailing lists."));
gtk_box_pack_start (
- GTK_BOX (vbox), check_always_ignore, FALSE, FALSE, 0);
+ GTK_BOX (container), check_always_ignore,
+ FALSE, FALSE, 0);
gtk_widget_show (check_always_ignore);
response = gtk_dialog_run (GTK_DIALOG (dialog));
@@ -1487,13 +1496,19 @@ action_mail_reply_sender_check (CamelFolder *folder,
gtk_widget_destroy (dialog);
- if (response == GTK_RESPONSE_NO)
- type = E_MAIL_REPLY_TO_FROM;
- else if (response == GTK_RESPONSE_OK)
- type = E_MAIL_REPLY_TO_LIST;
- else if (response == GTK_RESPONSE_CANCEL || response == GTK_RESPONSE_DELETE_EVENT) {
- g_object_unref (message);
- goto exit;
+ switch (response) {
+ case GTK_RESPONSE_NO:
+ type = E_MAIL_REPLY_TO_FROM;
+ break;
+ case GTK_RESPONSE_OK:
+ type = E_MAIL_REPLY_TO_LIST;
+ break;
+ case GTK_RESPONSE_CANCEL:
+ case GTK_RESPONSE_DELETE_EVENT:
+ g_object_unref (message);
+ goto exit;
+ default:
+ break;
}
}
diff --git a/mail/em-utils.c b/mail/em-utils.c
index 15008abd51..e674c61528 100644
--- a/mail/em-utils.c
+++ b/mail/em-utils.c
@@ -128,7 +128,8 @@ em_utils_prompt_user (GtkWindow *parent,
const gchar *tag,
...)
{
- GtkWidget *mbox, *check = NULL;
+ GtkWidget *dialog;
+ GtkWidget *check = NULL;
GtkWidget *container;
va_list ap;
gint button;
@@ -146,24 +147,27 @@ em_utils_prompt_user (GtkWindow *parent,
alert = e_alert_new_valist (tag, ap);
va_end (ap);
- mbox = e_alert_dialog_new (parent, alert);
+ dialog = e_alert_dialog_new (parent, alert);
g_object_unref (alert);
+ container = e_alert_dialog_get_content_area (E_ALERT_DIALOG (dialog));
+
if (promptkey) {
- check = gtk_check_button_new_with_mnemonic (_("_Do not show this message again"));
- container = gtk_dialog_get_content_area (GTK_DIALOG (mbox));
- gtk_box_pack_start (GTK_BOX (container), check, FALSE, FALSE, 0);
+ check = gtk_check_button_new_with_mnemonic (
+ _("_Do not show this message again"));
+ gtk_box_pack_start (
+ GTK_BOX (container), check, FALSE, FALSE, 0);
gtk_widget_show (check);
}
- button = gtk_dialog_run ((GtkDialog *) mbox);
+ button = gtk_dialog_run (GTK_DIALOG (dialog));
if (promptkey)
gconf_client_set_bool (
client, promptkey,
!gtk_toggle_button_get_active (
GTK_TOGGLE_BUTTON (check)), NULL);
- gtk_widget_destroy (mbox);
+ gtk_widget_destroy (dialog);
g_object_unref (client);
diff --git a/plugins/attachment-reminder/attachment-reminder.c b/plugins/attachment-reminder/attachment-reminder.c
index 723e8cfe5d..113424bd78 100644
--- a/plugins/attachment-reminder/attachment-reminder.c
+++ b/plugins/attachment-reminder/attachment-reminder.c
@@ -105,26 +105,29 @@ org_gnome_evolution_attachment_reminder (EPlugin *ep, EMEventTargetComposer *t)
static gboolean
ask_for_missing_attachment (EPlugin *ep, GtkWindow *window)
{
- GtkWidget *check = NULL;
- GtkDialog *dialog = NULL;
- GtkWidget *content_area;
+ GtkWidget *check;
+ GtkWidget *dialog;
+ GtkWidget *container;
gint response;
- dialog = (GtkDialog*) e_alert_dialog_new_for_args (
- window, "org.gnome.evolution.plugins.attachment_reminder:attachment-reminder", NULL);
+ dialog = e_alert_dialog_new_for_args (
+ window, "org.gnome.evolution.plugins.attachment_reminder:"
+ "attachment-reminder", NULL);
+
+ container = e_alert_dialog_get_content_area (E_ALERT_DIALOG (dialog));
/*Check buttons*/
- check = gtk_check_button_new_with_mnemonic (_("_Do not show this message again."));
- content_area = gtk_dialog_get_content_area (dialog);
- gtk_box_pack_start (GTK_BOX (content_area), check, FALSE, FALSE, 0);
+ check = gtk_check_button_new_with_mnemonic (
+ _("_Do not show this message again."));
+ gtk_box_pack_start (GTK_BOX (container), check, FALSE, FALSE, 0);
gtk_widget_show (check);
- response = gtk_dialog_run ((GtkDialog *) dialog);
+ response = gtk_dialog_run (GTK_DIALOG (dialog));
if (gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (check)))
e_plugin_enable (ep, FALSE);
- gtk_widget_destroy ((GtkWidget *)dialog);
+ gtk_widget_destroy (dialog);
if (response == GTK_RESPONSE_OK)
gtk_action_activate (E_COMPOSER_ACTION_ATTACH (window));
diff --git a/plugins/attachment-reminder/org-gnome-attachment-reminder.error.xml b/plugins/attachment-reminder/org-gnome-attachment-reminder.error.xml
index 4d77f4475b..8728c98747 100644
--- a/plugins/attachment-reminder/org-gnome-attachment-reminder.error.xml
+++ b/plugins/attachment-reminder/org-gnome-attachment-reminder.error.xml
@@ -4,7 +4,7 @@
<error id="attachment-reminder" type="info">
<_primary>Message has no attachments</_primary>
<_secondary>Evolution has found some keywords that suggest that this message should contain an attachment, but cannot find one.</_secondary>
- <button response="GTK_RESPONSE_OK" _label="_Add attachment..."/>
+ <button response="GTK_RESPONSE_OK" _label="_Add Attachment..."/>
<button stock="gtk-cancel" response="GTK_RESPONSE_CANCEL" _label="_Edit Message"/>
<button response="GTK_RESPONSE_YES" _label="_Send"></button>
</error>
diff --git a/plugins/backup-restore/backup-restore.c b/plugins/backup-restore/backup-restore.c
index 2ff0ff1bc9..aa0693ece3 100644
--- a/plugins/backup-restore/backup-restore.c
+++ b/plugins/backup-restore/backup-restore.c
@@ -113,7 +113,9 @@ sanity_check (const gchar *filename)
static guint32
dialog_prompt_user (GtkWindow *parent, const gchar *string, const gchar *tag, ...)
{
- GtkWidget *mbox, *check = NULL;
+ GtkWidget *dialog;
+ GtkWidget *check = NULL;
+ GtkWidget *container;
va_list ap;
gint button;
guint32 mask = 0;
@@ -123,24 +125,25 @@ dialog_prompt_user (GtkWindow *parent, const gchar *string, const gchar *tag, ..
alert = e_alert_new_valist (tag, ap);
va_end (ap);
- mbox = e_alert_dialog_new (parent, alert);
+ dialog = e_alert_dialog_new (parent, alert);
g_object_unref (alert);
+ container = e_alert_dialog_get_content_area (E_ALERT_DIALOG (dialog));
+
check = gtk_check_button_new_with_mnemonic (string);
/* We should hardcode this to true */
- gtk_toggle_button_set_active ((GtkToggleButton *)check, TRUE);
- gtk_container_set_border_width ((GtkContainer *)check, 12);
- gtk_box_pack_start ((GtkBox *)gtk_dialog_get_content_area ((GtkDialog *) mbox), check, TRUE, TRUE, 0);
+ gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (check), TRUE);
+ gtk_box_pack_start (GTK_BOX (container), check, FALSE, FALSE, 0);
gtk_widget_show (check);
- button = gtk_dialog_run ((GtkDialog *) mbox);
+ button = gtk_dialog_run (GTK_DIALOG (dialog));
if (button == GTK_RESPONSE_YES)
mask |= BR_OK;
if (gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (check)))
mask |= BR_START;
- gtk_widget_destroy (mbox);
+ gtk_widget_destroy (dialog);
return mask;
}